Scarab commit: svn commit: r10213 - branches/release/b21/src/java/org/tigris/scarab/actions/ModifyIssue.java

Author: dabbous
Date: 2006-07-22 02:37:43-0700
New Revision: 10213

Modified:
   branches/release/b21/src/java/org/tigris/scarab/actions/ModifyIssue.java

Log:
- removed duplicate notification when save history as comment


Modified: branches/release/b21/src/java/org/tigris/scarab/actions/ModifyIssue.java
Url: http://scarab.tigris.org/source/browse/scarab/branches/release/b21/src/java/org/tigris/scarab/actions/ModifyIssue.java?view=diff&rev=10213&p1=branches/release/b21/src/java/org/tigris/scarab/actions/ModifyIssue.java&p2=branches/release/b21/src/java/org/tigris/scarab/actions/ModifyIssue.java&r1=10212&r2=10213
==============================================================================
--- branches/release/b21/src/java/org/tigris/scarab/actions/ModifyIssue.java	(original)
+++ branches/release/b21/src/java/org/tigris/scarab/actions/ModifyIssue.java	2006-07-22 02:37:43-0700
@@ -403,14 +403,18 @@
             final ActivitySet activitySet = issue.setAttributeValues(null, 
                     newAttVals, attachment, user);
             // save reason as a comment as well?
-            if( saveAsComment ){
+            if( saveAsComment )
+            {
                 issue.addComment(activitySet, attachment, user);
             }
+            else
+            {
+                NotificationManagerFactory.getInstance().addActivityNotification(
+                                ActivityType.ATTRIBUTE_CHANGED,
+                                activitySet, issue, user);
+            }
             intake.removeAll();
             scarabR.setConfirmMessage(L10NKeySet.ChangesSaved);
-            NotificationManagerFactory.getInstance().addActivityNotification(
-                            ActivityType.ATTRIBUTE_CHANGED,
-                            activitySet, issue, user);
         }
         catch (Exception se)
         {