You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@jackrabbit.apache.org by mr...@apache.org on 2007/02/21 14:06:53 UTC

svn commit: r509994 - /jackrabbit/trunk/contrib/spi/spi2dav/src/main/java/org/apache/jackrabbit/spi2dav/IdFactoryImpl.java

Author: mreutegg
Date: Wed Feb 21 05:06:52 2007
New Revision: 509994

URL: http://svn.apache.org/viewvc?view=rev&rev=509994
Log:
ItemId.equals does not check path part if unique id is null

Modified:
    jackrabbit/trunk/contrib/spi/spi2dav/src/main/java/org/apache/jackrabbit/spi2dav/IdFactoryImpl.java

Modified: jackrabbit/trunk/contrib/spi/spi2dav/src/main/java/org/apache/jackrabbit/spi2dav/IdFactoryImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/contrib/spi/spi2dav/src/main/java/org/apache/jackrabbit/spi2dav/IdFactoryImpl.java?view=diff&rev=509994&r1=509993&r2=509994
==============================================================================
--- jackrabbit/trunk/contrib/spi/spi2dav/src/main/java/org/apache/jackrabbit/spi2dav/IdFactoryImpl.java (original)
+++ jackrabbit/trunk/contrib/spi/spi2dav/src/main/java/org/apache/jackrabbit/spi2dav/IdFactoryImpl.java Wed Feb 21 05:06:52 2007
@@ -122,8 +122,8 @@
         }
 
         boolean equals(ItemId other) {
-            return (uniqueID == null) ? other.getUniqueID() == null : uniqueID.equals(other.getUniqueID())
-                && ((path == null) ? other.getPath() == null : path.equals(other.getPath()));
+            return (uniqueID == null ? other.getUniqueID() == null : uniqueID.equals(other.getUniqueID()))
+                && (path == null ? other.getPath() == null : path.equals(other.getPath()));
         }
 
         /**