diff options
| author | Byron Jones <bjones@mozilla.com> | 2013-10-15 17:15:48 +0200 |
|---|---|---|
| committer | Byron Jones <bjones@mozilla.com> | 2013-10-15 17:15:48 +0200 |
| commit | 46ab4b8a85d62bc7b06c1d84c5e9553f9fade0ae (patch) | |
| tree | f912fbd8e55fd7884d48edf8e7d2106bf6cc598e /extensions/TrackingFlags | |
| parent | b098a034925adfa887da30f02f4a8576fe5ec07f (diff) | |
| download | bugzilla-46ab4b8a85d62bc7b06c1d84c5e9553f9fade0ae.tar.gz bugzilla-46ab4b8a85d62bc7b06c1d84c5e9553f9fade0ae.tar.xz | |
Bug 926842: unable to change status and tracking flag values once they are set
Diffstat (limited to 'extensions/TrackingFlags')
| -rw-r--r-- | extensions/TrackingFlags/Extension.pm | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/extensions/TrackingFlags/Extension.pm b/extensions/TrackingFlags/Extension.pm index 3628ef8f5..455afdd1f 100644 --- a/extensions/TrackingFlags/Extension.pm +++ b/extensions/TrackingFlags/Extension.pm @@ -492,8 +492,8 @@ sub object_end_of_set_all { foreach my $flag (@$tracking_flags) { my $flag_name = $flag->name; - if (defined $params->{$flag_name}) { - $object->{$flag_name} = $params->{$flag_name}; + if (exists $params->{$flag_name}) { + $object->set($flag_name, $params->{$flag_name}); } } } @@ -516,8 +516,6 @@ sub bug_end_of_update { my $new_value = $bug->$flag_name; my $old_value = $old_bug->$flag_name; - next if $new_value eq $old_value; - if ($new_value ne $old_value) { # Do not allow if the user cannot set the old value or the new value if (!$flag->can_set_value($new_value)) { |