diff options
| author |   lpsolit%gmail.com <> | 2009-01-08 00:06:49 +0000 |
|---|---|---|
| committer | lpsolit%gmail.com <> | 2009-01-08 00:06:49 +0000 |
| commit | 11fa9ce3596363a4d4badd7960ae73e8dc535050 (patch) | |
| tree | 3dac3780376ee6d73ed6a2fd0e2801878ef8412d /userprefs.cgi | |
| parent | Bug 450403: Add ability to view comments via the web service (Bug.comments) (diff) | |
| download | bugzilla-11fa9ce3596363a4d4badd7960ae73e8dc535050.tar.gz bugzilla-11fa9ce3596363a4d4badd7960ae73e8dc535050.tar.bz2 bugzilla-11fa9ce3596363a4d4badd7960ae73e8dc535050.zip | |
Bug 472549: Ignore undefined values when committing user pref changes - Patch by Frédéric Buclin <LpSolit@gmail.com> r/a=mkanat
Diffstat (limited to 'userprefs.cgi')
| -rwxr-xr-x | userprefs.cgi | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/userprefs.cgi b/userprefs.cgi index 1bf2d7f5f..085372bac 100755 --- a/userprefs.cgi +++ b/userprefs.cgi @@ -127,7 +127,6 @@ sub SaveAccount { $cgi->param('Bugzilla_password') || ThrowUserError("old_password_required"); - use Bugzilla::Token; # Block multiple email changes for the same user. if (Bugzilla::Token::HasEmailChangeToken($user->id)) { ThrowUserError("email_change_in_progress"); @@ -183,6 +182,7 @@ sub SaveSettings { foreach my $name (@setting_list) { next if ! ($settings->{$name}->{'is_enabled'}); my $value = $cgi->param($name); + next unless defined $value; my $setting = new Bugzilla::User::Setting($name); if ($value eq "${name}-isdefault" ) { |