Scarab commit: svn commit: r10934 - branches/release/0.22/src/java/org/tigris/scarab/om/Issue.java

Johannes Höchstädter <[email protected]>
Newsgroups gmane.comp.java.scarab.cvs
Message-ID <[email protected]>
Author: jhoech
Date: 2010-03-01 01:13:19-0800
New Revision: 10934

Modified:
   branches/release/0.22/src/java/org/tigris/scarab/om/Issue.java

Log:
FIX - Fix for activity tab. If only the observed Id is investigated for dependencies it might be that the value for otherIssue is the issue itself.

Modified: branches/release/0.22/src/java/org/tigris/scarab/om/Issue.java
Url: http://scarab.tigris.org/source/browse/scarab/branches/release/0.22/src/java/org/tigris/scarab/om/Issue.java?view=diff&pathrev=10934&r1=10933&r2=10934
==============================================================================
--- branches/release/0.22/src/java/org/tigris/scarab/om/Issue.java	(original)
+++ branches/release/0.22/src/java/org/tigris/scarab/om/Issue.java	2010-03-01 01:13:19-0800
@@ -3372,8 +3372,9 @@
         // only change dependency type for non-deleted deps
         if ( isActive && ( rolesHaveSwitched || typeHasChanged ) )
         {
-            final Issue otherIssue = IssueManager
-                            .getInstance(newDepend.getObservedId(), false);
+            final Issue otherIssue = (newDepend.getObservedId().longValue() != this.getIssueId().longValue()
+            		? IssueManager.getInstance(newDepend.getObservedId(), false)
+            		: IssueManager.getInstance(newDepend.getObserverId(), false));
 
             // always delete an old dependency
             oldDepend.setDeleted(true);

------------------------------------------------------
http://scarab.tigris.org/ds/viewMessage.do?dsForumId=3577&dsMessageId=2453317
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.