You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2005/10/26 23:36:52 UTC

svn commit: r328745 - /directory/shared/ldap/branches/shared-ldap-utf8/common/src/java/org/apache/ldap/common/message/ModifyRequestImpl.java

Author: elecharny
Date: Wed Oct 26 14:36:48 2005
New Revision: 328745

URL: http://svn.apache.org/viewcvs?rev=328745&view=rev
Log:
Fixed the update of the Hashmap used to compare attribute's values

Modified:
    directory/shared/ldap/branches/shared-ldap-utf8/common/src/java/org/apache/ldap/common/message/ModifyRequestImpl.java

Modified: directory/shared/ldap/branches/shared-ldap-utf8/common/src/java/org/apache/ldap/common/message/ModifyRequestImpl.java
URL: http://svn.apache.org/viewcvs/directory/shared/ldap/branches/shared-ldap-utf8/common/src/java/org/apache/ldap/common/message/ModifyRequestImpl.java?rev=328745&r1=328744&r2=328745&view=diff
==============================================================================
--- directory/shared/ldap/branches/shared-ldap-utf8/common/src/java/org/apache/ldap/common/message/ModifyRequestImpl.java (original)
+++ directory/shared/ldap/branches/shared-ldap-utf8/common/src/java/org/apache/ldap/common/message/ModifyRequestImpl.java Wed Oct 26 14:36:48 2005
@@ -271,12 +271,15 @@
 	        	{
                     if (value instanceof byte[] )
                     {
-                        if ( attrHash0.contains( StringUtils.toUtf8( (byte[])value ) ) == false )
+                        String sValue = StringUtils.toUtf8( (byte[])value );
+                        
+                        if ( attrHash0.contains( sValue ) == false )
                         {
                             return false;
                         }
                         else
                         {
+                            attrHash0.remove( sValue );
                             continue;
                         }
                     }
@@ -291,7 +294,7 @@
 	        	}
 	        }
 	        
-	        return ( attrHash0.size() != 0 );
+	        return ( attrHash0.size() == 0 );
         }
         catch ( NamingException ne )
         {