Scarab commit: svn commit: r11030 - trunk/src/java/org/tigris/scarab/om/ActivitySet.java

Author: dabbous
Date: 2010-11-24 08:06:34-0800
New Revision: 11030

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

Log:
Fix for SCB3055
The check for sealed issues also fails when an Activity only has a new value. the check on the oldValue then fails with another NPE. I changed the string compare such that the NPE does no longer occur.

Modified: trunk/src/java/org/tigris/scarab/om/ActivitySet.java
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/java/org/tigris/scarab/om/ActivitySet.java?view=diff&pathrev=11030&r1=11029&r2=11030
==============================================================================
--- trunk/src/java/org/tigris/scarab/om/ActivitySet.java	(original)
+++ trunk/src/java/org/tigris/scarab/om/ActivitySet.java	2010-11-24 08:06:34-0800
@@ -274,7 +274,7 @@
                                        String oldv = act.getOldValue();
                                        String newv = act.getNewValue();
                                        
-                                       if(oldv.equals(value) || newv.equals(value))
+                                       if(value.equals(oldv) || value.equals(newv))
                                        {
                                            result = true;
                                            break;

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