From 1e49c001d9075b2873797e42b187cb303b9739eb Mon Sep 17 00:00:00 2001 From: "olav%bkor.dhs.org" Date: Mon, 5 Feb 2007 21:34:20 +0000 Subject: [PATCH] Bug 366187: Fix regression; ensure dependent changed bugmail will be sent out Patch by Olav Vitters r=LpSolit a=LpSolit --- webtools/bugzilla/Bugzilla/BugMail.pm | 1 + webtools/bugzilla/Bugzilla/User.pm | 9 ++++++++- 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/webtools/bugzilla/Bugzilla/BugMail.pm b/webtools/bugzilla/Bugzilla/BugMail.pm index a4462b7ba59..7a35d9f9e0d 100644 --- a/webtools/bugzilla/Bugzilla/BugMail.pm +++ b/webtools/bugzilla/Bugzilla/BugMail.pm @@ -438,6 +438,7 @@ sub Send { $relationship, $diffs, $comments{$lang}, + $deptext, $changer, !$start)) { diff --git a/webtools/bugzilla/Bugzilla/User.pm b/webtools/bugzilla/Bugzilla/User.pm index c6fd0b524e5..3e952e56df9 100644 --- a/webtools/bugzilla/Bugzilla/User.pm +++ b/webtools/bugzilla/Bugzilla/User.pm @@ -1338,7 +1338,8 @@ our %names_to_events = ( # Note: the "+" signs before the constants suppress bareword quoting. sub wants_bug_mail { my $self = shift; - my ($bug_id, $relationship, $fieldDiffs, $commentField, $changer, $bug_is_new) = @_; + my ($bug_id, $relationship, $fieldDiffs, $commentField, $dependencyText, + $changer, $bug_is_new) = @_; # Make a list of the events which have happened during this bug change, # from the point of view of this user. @@ -1390,6 +1391,12 @@ sub wants_bug_mail { $events{+EVT_COMMENT} = 1; } + # Dependent changed bugmails must have an event to ensure the bugmail is + # emailed. + if ($dependencyText ne '') { + $events{+EVT_DEPEND_BLOCK} = 1; + } + my @event_list = keys %events; my $wants_mail = $self->wants_mail(\@event_list, $relationship);