From 2fe49f40feabcabebfd6604765d897cfaddd222f Mon Sep 17 00:00:00 2001 From: "lpsolit%gmail.com" <> Date: Sat, 13 Aug 2005 21:04:53 +0000 Subject: Bug 303824: BugMail should ignore deleted or renamed user accounts - Patch by Frédéric Buclin r=wurblzap a=justdave MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- Bugzilla/BugMail.pm | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) (limited to 'Bugzilla') diff --git a/Bugzilla/BugMail.pm b/Bugzilla/BugMail.pm index 35026b48e..e51f0c83b 100644 --- a/Bugzilla/BugMail.pm +++ b/Bugzilla/BugMail.pm @@ -358,17 +358,20 @@ sub ProcessOneBug { if ($old) { # You can't stop being the reporter, and mail isn't sent if you # remove your vote. + # Ignore people whose user account has been deleted or renamed. if ($what eq "CC") { foreach my $cc_user (split(/[\s,]+/, $old)) { - push(@{$recipients{&::DBNameToIdAndCheck($cc_user)}}, - REL_CC); + my $uid = login_to_id($cc_user); + push(@{$recipients{$uid}}, REL_CC) if $uid; } } elsif ($what eq "QAContact") { - push(@{$recipients{&::DBNameToIdAndCheck($old)}}, REL_QA); + my $uid = login_to_id($old); + push(@{$recipients{$uid}}, REL_QA) if $uid; } elsif ($what eq "AssignedTo") { - push(@{$recipients{&::DBNameToIdAndCheck($old)}}, REL_ASSIGNEE); + my $uid = login_to_id($old); + push(@{$recipients{$uid}}, REL_ASSIGNEE) if $uid; } } } -- cgit v1.2.3-24-g4f1b