Bug 289012: Can't use an undefined value as a HASH reference at userprefs.cgi line...

Patch By Tomas Kopal <Tomas.Kopal@altap.cz> r=mkanat, a=justdave

diff --git a/Bugzilla/User/Setting.pm b/Bugzilla/User/Setting.pm
index a8e1cbaed9806de16276636913b03dc81a2d0d01..2319b8a55700ddaf1ec1538a706c7d32e625e289 100644 (file)
--- a/Bugzilla/User/Setting.pm
+++ b/Bugzilla/User/Setting.pm
@@ -127,7 +127,7 @@ sub add_setting {
 
 sub get_all_settings {
     my ($user_id) = @_;
-    my $settings;
+    my $settings = {};
     my $dbh = Bugzilla->dbh;
 
     my $sth = $dbh->prepare(
@@ -162,7 +162,7 @@ sub get_all_settings {
 
 sub get_defaults {
     my $dbh = Bugzilla->dbh;
-    my $default_settings;
+    my $default_settings = {};
 
     my $sth = $dbh->prepare(q{SELECT name, default_value, is_enabled
                                 FROM setting

diff --git a/checksetup.pl b/checksetup.pl
index 09e8dc4baa4fc9725b13a9b68c5fd4cdcdb3593d..565fa1e70226fb3a0b070ef250b15f3e34d97fcc 100755 (executable)
--- a/checksetup.pl
+++ b/checksetup.pl
@@ -84,6 +84,7 @@
 #     add more database-related checks                 --DATABASE--
 #     change table definitions                         --TABLE--
 #     add more groups                                  --GROUPS--
+#     add user-adjustable sttings                      --SETTINGS--
 #     create initial administrator account             --ADMIN--
 #
 # Note: sometimes those special comments occur more than once. For
@@ -3772,9 +3773,6 @@ if($dbh->bz_column_info('bugs', 'lastdiffed')->{NOTNULL}) {
     $dbh->bz_alter_column('bugs', 'lastdiffed', {TYPE => 'DATETIME'});
 }
 
-# 2005-03-03 travis@sedsystems.ca -- Bug 41972
-add_setting ("display_quips", {"on" => 1, "off" => 2 }, "on" );
-
 # 2005-03-09 qa_contact should be NULL instead of 0, bug 285534
 if ($dbh->bz_column_info('bugs', 'qa_contact')->{NOTNULL}) {
     $dbh->bz_alter_column('bugs', 'qa_contact', {TYPE => 'INT3'});
@@ -3923,13 +3921,6 @@ if ( $dbh->isa('Bugzilla::DB::Mysql') ) {
     }
 }
 
-# 2005-03-10 travis@sedsystems.ca -- Bug 199048
-add_setting ("comment_sort_order", {"oldest_to_newest" => 1,
-                                    "newest_to_oldest" => 2,
-                                    "newest_to_oldest_desc_first" => 3}, 
-             "oldest_to_newest" );
-
-
 # 2005-04-07 - alt@sonic.net, bug 289455
 # make classification_id field type be consistent with DB:Schema
 $dbh->bz_alter_column('products', 'classification_id',
@@ -4010,6 +4001,19 @@ if (!GroupDoesExist('bz_canusewhines')) {
              GROUP_MEMBERSHIP . ")") unless $group_exists;
 }
 
+###########################################################################
+# Create --SETTINGS-- users can adjust
+###########################################################################
+
+# 2005-03-03 travis@sedsystems.ca -- Bug 41972
+add_setting ("display_quips", {"on" => 1, "off" => 2 }, "on" );
+
+# 2005-03-10 travis@sedsystems.ca -- Bug 199048
+add_setting ("comment_sort_order", {"oldest_to_newest" => 1,
+                                    "newest_to_oldest" => 2,
+                                    "newest_to_oldest_desc_first" => 3}, 
+             "oldest_to_newest" );
+
 ###########################################################################
 # Create Administrator  --ADMIN--
 ###########################################################################