From: lpsolit%gmail.com <>
Date: Sat, 19 Aug 2006 00:32:11 +0000 (+0000)
Subject: Bug 343050: process_bug.cgi tries to send emails to too many bugs - Patch by Frédéric... 
X-Git-Tag: bugzilla-2.23.3~145
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=4b92f2749b4c44d60f8ae7b06e2c250a3202e2b7;p=thirdparty%2Fbugzilla.git

Bug 343050: process_bug.cgi tries to send emails to too many bugs - Patch by Frédéric Buclin <LpSolit@gmail.com> r=mkanat a=myk
---

diff --git a/process_bug.cgi b/process_bug.cgi
index b26352d227..156f756209 100755
--- a/process_bug.cgi
+++ b/process_bug.cgi
@@ -1726,12 +1726,16 @@ foreach my $id (@idlist) {
         my @oldlist = @{$dbh->selectcol_arrayref("SELECT $target FROM dependencies
                                                   WHERE $me = ? ORDER BY $target",
                                                   undef, $id)};
-        @dependencychanged{@oldlist} = 1;
+
+        # Only bugs depending on the current one should get notification.
+        # Bugs blocking the current one never get notification, unless they
+        # are added or removed from the dependency list. This case is treated
+        # below.
+        @dependencychanged{@oldlist} = 1 if ($me eq 'dependson');
 
         if (defined $cgi->param($target)) {
             my %snapshot;
             my @newlist = sort {$a <=> $b} @{$deps{$target}};
-            @dependencychanged{@newlist} = 1;
 
             while (0 < @oldlist || 0 < @newlist) {
                 if (@oldlist == 0 || (@newlist > 0 &&
@@ -1768,6 +1772,9 @@ foreach my $id (@idlist) {
                 }
                 LogDependencyActivity($id, $oldsnap, $target, $me, $timestamp);
                 $check_dep_bugs = 1;
+                # All bugs added or removed from the dependency list
+                # must be notified.
+                @dependencychanged{@keys} = 1;
             }
         }
     }