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

[email protected] Mon, 19 Apr 2010 08:21:06 -0700 (PDT)
Newsgroups gmane.comp.java.scarab.cvs
Message-ID <[email protected]>
Author: jhoech
Date: 2010-04-19 08:21:06-0700
New Revision: 10997

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

Log:
FIX - Fix for duplicated references of dependencies in history tab.

Modified: branches/release/0.22/src/java/org/tigris/scarab/actions/ModifyIssue.java
Url: http://scarab.tigris.org/source/browse/scarab/branches/release/0.22/src/java/org/tigris/scarab/actions/ModifyIssue.java?view=diff&pathrev=10997&r1=10996&r2=10997
==============================================================================
--- branches/release/0.22/src/java/org/tigris/scarab/actions/ModifyIssue.java	(original)
+++ branches/release/0.22/src/java/org/tigris/scarab/actions/ModifyIssue.java	2010-04-19 08:21:06-0700
@@ -1310,14 +1310,17 @@
             ActivitySet activitySet = null;
             try
             {
-                Issue workingIssue = issue;
                 if (needRoleSwitch)
                 {
                     depend.exchangeRoles();
-                    workingIssue = childIssue;
+                    activitySet = issue
+                    .doAddDependency(activitySet, depend, childIssue, user);
                 }
-                activitySet = issue
-                    .doAddDependency(activitySet, depend, workingIssue, user);
+                else{
+                    activitySet = childIssue
+                    .doAddDependency(activitySet, depend, issue, user);
+                }
+
             }
             catch (ScarabException se)
             {
@@ -1414,10 +1417,10 @@
                     newDepend.exchangeRoles();
                 }
                 
-                // make the changes
-                activitySet = 
-                    workingIssue.doChangeDependencyType(activitySet, oldDepend,
-                                                 newDepend, user);
+                 // make the changes
+                  activitySet = 
+                  workingIssue.doChangeDependencyType(activitySet, oldDepend,
+                                                      newDepend, user);
             }
             intake.remove(group);
         }

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