Bug 366187: Bugmail should consider 'Any field not mentioned above changes' even...

Patch by Olav Vitters <olav@bkor.dhs.org> r=LpSolit a=myk

diff --git a/Bugzilla/User.pm b/Bugzilla/User.pm
index 02c6c4db96026ca10156b172ededf5f31003a7f1..c6fd0b524e5e9a72095ce99635ab1af5cfc39c77 100644 (file)
--- a/Bugzilla/User.pm
+++ b/Bugzilla/User.pm
@@ -1351,11 +1351,7 @@ sub wants_bug_mail {
         }
         else {
             # Catch-all for any change not caught by a more specific event
-            # XXX: Temporary Compatibility Change 2 of 2:
-            # This code is disabled, and replaced with the code a few lines
-            # below, in order to make the behaviour more like the original, 
-            # which only added this event if _all_ changes were of "other" type.
-            # $events{+EVT_OTHER} = 1;            
+            $events{+EVT_OTHER} = 1;            
         }
 
         # If the user is in a particular role and the value of that role
@@ -1396,12 +1392,6 @@ sub wants_bug_mail {
     
     my @event_list = keys %events;
     
-    # XXX Temporary Compatibility Change 2 of 2:
-    # See above comment.
-    if (!scalar(@event_list)) {
-      @event_list = (EVT_OTHER);
-    }
-    
     my $wants_mail = $self->wants_mail(\@event_list, $relationship);
 
     # The negative events are handled separately - they can't be incorporated