From 4b92f2749b4c44d60f8ae7b06e2c250a3202e2b7 Mon Sep 17 00:00:00 2001
From: "lpsolit%gmail.com" <>
Date: Sat, 19 Aug 2006 00:32:11 +0000
Subject: 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
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 process_bug.cgi | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

(limited to 'process_bug.cgi')

diff --git a/process_bug.cgi b/process_bug.cgi
index b26352d22..156f75620 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;
             }
         }
     }
-- 
cgit v1.2.3-24-g4f1b