diff options
author | Byron Jones <bjones@mozilla.com> | 2012-11-21 18:52:07 +0100 |
---|---|---|
committer | Byron Jones <bjones@mozilla.com> | 2012-11-21 18:52:07 +0100 |
commit | c20a547c86471041956f364cf73980c92f4434cd (patch) | |
tree | ae735ddd5d2539f53e88a8f9e81b41d37a8452ca /extensions/Needinfo | |
parent | 7528bd9d3e339c656070d721eacc87005579d1b0 (diff) | |
download | bugzilla-c20a547c86471041956f364cf73980c92f4434cd.tar.gz bugzilla-c20a547c86471041956f364cf73980c92f4434cd.tar.xz |
Bug 813861: needinfo is sending duplicate request emails when the flag is cleared
Diffstat (limited to 'extensions/Needinfo')
-rw-r--r-- | extensions/Needinfo/Extension.pm | 19 |
1 files changed, 0 insertions, 19 deletions
diff --git a/extensions/Needinfo/Extension.pm b/extensions/Needinfo/Extension.pm index 7999d6ae7..98644f997 100644 --- a/extensions/Needinfo/Extension.pm +++ b/extensions/Needinfo/Extension.pm @@ -52,8 +52,6 @@ sub bug_start_of_update { my ($self, $args) = @_; my $bug = $args->{bug}; my $old_bug = $args->{old_bug}; - my $timestamp = $args->{timestamp}; - my $changes = $args->{changes}; my $user = Bugzilla->user; my $cgi = Bugzilla->cgi; @@ -151,23 +149,6 @@ sub bug_start_of_update { if (@flags || @new_flags) { $bug->set_flags(\@flags, \@new_flags); - my ($removed, $added) = Bugzilla::Flag->update_flags($bug, $old_bug, $timestamp); - if ($removed || $added) { - my $field = 'flagtypes.name'; - $removed = defined $removed ? $removed : ''; - $added = defined $added ? $added : ''; - - # Do not overwrite other flag changes - if ($changes->{$field}) { - $removed = defined $changes->{$field}->[0] - ? $changes->{$field}->[0] . ", $removed" - : $removed; - $added = defined $changes->{$field}->[1] - ? $changes->{$field}->[1] . ", $added" - : $added; - } - $changes->{$field} = [$removed, $added]; - } } } |