Scarab commit: svn commit: r11022 - trunk/src/java/org/tigris/scarab/om/NotificationRuleManager.java

Author: dabbous
Date: 2010-10-26 07:32:30-0700
New Revision: 11022

Modified:
   trunk/src/java/org/tigris/scarab/om/NotificationRuleManager.java

Log:
Avoid to send Notifications when a sealed issue (typically an issue in state "closed") is modified (by an administrator).
Note that the modifier (administrator) will always be notified (even if the issue is sealed) if he/she has enabled 
"email to self modifications"

Modified: trunk/src/java/org/tigris/scarab/om/NotificationRuleManager.java
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/java/org/tigris/scarab/om/NotificationRuleManager.java?view=diff&pathrev=11022&r1=11021&r2=11022
==============================================================================
--- trunk/src/java/org/tigris/scarab/om/NotificationRuleManager.java	(original)
+++ trunk/src/java/org/tigris/scarab/om/NotificationRuleManager.java	2010-10-26 07:32:30-0700
@@ -118,7 +118,14 @@
         }
         else
         {
-            isEnabled = rule.getFilterState();
+            if(issue.isSealed())
+            {
+                isEnabled = false; // Do not send notifications to other users when Issue is "closed"
+            }
+            else
+            {
+                isEnabled = rule.getFilterState();
+            }
         }
         
         if(isEnabled)

------------------------------------------------------
http://scarab.tigris.org/ds/viewMessage.do?dsForumId=3577&dsMessageId=2675727