From: lpsolit%gmail.com <>
Date: Sat, 23 Sep 2006 06:00:32 +0000 (+0000)
Subject: Bug 349852: "I'm added to or removed from this capacity" email pref doesn't work... 
X-Git-Tag: bugzilla-2.22.1~16
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b1655e76f9019edbbe43772568197fc587139077;p=thirdparty%2Fbugzilla.git

Bug 349852: "I'm added to or removed from this capacity" email pref doesn't work for new bugs - Original patch by Olav Vitters <bugzilla-mozilla@bkor.dhs.org>, slightly modified by Frédéric Buclin <LpSolit@gmail.com> r=LpSolit, r=mkanat a=justdave
---

diff --git a/Bugzilla/BugMail.pm b/Bugzilla/BugMail.pm
index a9a1a3c30d..3fd821be60 100644
--- a/Bugzilla/BugMail.pm
+++ b/Bugzilla/BugMail.pm
@@ -426,7 +426,8 @@ sub ProcessOneBug {
                                           $relationship, 
                                           $diffs, 
                                           $newcomments, 
-                                          $changer))
+                                          $changer,
+                                          !$start))
                 {
                     push(@rels_which_want, $relationship);
                 }
diff --git a/Bugzilla/User.pm b/Bugzilla/User.pm
index c34239b0ff..ca882f3fc3 100644
--- a/Bugzilla/User.pm
+++ b/Bugzilla/User.pm
@@ -1126,7 +1126,7 @@ 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) = @_;
+    my ($bug_id, $relationship, $fieldDiffs, $commentField, $changer, $bug_is_new) = @_;
 
     # Don't send any mail, ever, if account is disabled 
     # XXX Temporary Compatibility Change 1 of 2:
@@ -1171,6 +1171,16 @@ sub wants_bug_mail {
         }
     }
 
+    # You role is new if the bug itself is.
+    # Only makes sense for the assignee, QA contact and the CC list.
+    if ($bug_is_new
+        && ($relationship == REL_ASSIGNEE
+            || $relationship == REL_QA
+            || $relationship == REL_CC))
+    {
+        $events{+EVT_ADDED_REMOVED} = 1;
+    }
+
     if ($commentField =~ /Created an attachment \(/) {
         $events{+EVT_ATTACHMENT} = 1;
     }