From: Jesse Weaver <pianohacker@gmail.com>
Date: Wed, 4 Nov 2009 04:27:30 +0000 (-0700)
Subject: Fix handling of pref defaults (#3747, #3588)
X-Git-Tag: v3.02.00-alpha~44^2~5
X-Git-Url: http://git.rot13.org/?a=commitdiff_plain;h=870abd703bf0536d81191651f83146017ebf9134;p=koha.git

Fix handling of pref defaults (#3747, #3588)
---

diff --git a/admin/preferences.pl b/admin/preferences.pl
index c359d32a46..146fe3f8fc 100755
--- a/admin/preferences.pl
+++ b/admin/preferences.pl
@@ -126,9 +126,9 @@ sub TransformPrefsToHTML {
                     my $name = $piece->{'pref'};
 
                     if ( $name ) {
-                        my $row = $dbh->selectrow_hashref( "SELECT value FROM systempreferences WHERE variable = ?", {}, $name );
+                        my $row = $dbh->selectrow_hashref( "SELECT value, type FROM systempreferences WHERE variable = ?", {}, $name );
                         my $value;
-                        if ( !defined( $row ) && defined( $piece->{'default'} ) ) {
+                        if ( ( !defined( $row ) || ( !defined( $row->{'value'} ) && $row->{'type'} ne 'YesNo' ) ) && defined( $piece->{'default'} ) ) {
                             $value = $piece->{'default'};
                         } else {
                             $value = $row->{'value'};
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/cataloguing.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/cataloguing.pref
index a4b2fd8522..5bdc1e657d 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/cataloguing.pref
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/cataloguing.pref
@@ -70,7 +70,7 @@ Cataloging:
                   incremental: generated in the form 1, 2, 3.
                   annual: generated in the form &lt;year&gt;-0001, &lt;year&gt;-0002.
                   hbyymmincr: generated in the form &lt;branchcode&gt;yymm0001.
-                  OFF: not generated automatically.
+                  "OFF": not generated automatically.
     Display:
         -
             - 'Separate multiple displayed authors, series or subjects with '
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/patrons.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/patrons.pref
index c207cd1d6a..3ff3628935 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/patrons.pref
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/patrons.pref
@@ -1,4 +1,4 @@
-Patron:
+Patrons:
      -
          - List
          - pref: AddPatronLists
@@ -13,6 +13,7 @@ Patron:
                no: "Don't send"
          - an email to newly created patrons with their account details at their
          - pref: AutoEmailPrimaryAddress
+           default: "OFF"
            choices:
                email: home
                emailpro: work