summaryrefslogtreecommitdiffstats
path: root/extensions/Needinfo
diff options
context:
space:
mode:
authorByron Jones <bjones@mozilla.com>2012-11-21 18:52:07 +0100
committerByron Jones <bjones@mozilla.com>2012-11-21 18:52:07 +0100
commitc20a547c86471041956f364cf73980c92f4434cd (patch)
treeae735ddd5d2539f53e88a8f9e81b41d37a8452ca /extensions/Needinfo
parent7528bd9d3e339c656070d721eacc87005579d1b0 (diff)
downloadbugzilla-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.pm19
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];
- }
}
}