You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by ak...@apache.org on 2005/11/12 17:43:33 UTC

svn commit: r332797 - /directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/schema/SchemaService.java

Author: akarasulu
Date: Sat Nov 12 08:43:28 2005
New Revision: 332797

URL: http://svn.apache.org/viewcvs?rev=332797&view=rev
Log:
fix from DIREVE-306 thanks to Claus Turkalj

Modified:
    directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/schema/SchemaService.java

Modified: directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/schema/SchemaService.java
URL: http://svn.apache.org/viewcvs/directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/schema/SchemaService.java?rev=332797&r1=332796&r2=332797&view=diff
==============================================================================
--- directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/schema/SchemaService.java (original)
+++ directory/apacheds/trunk/core/src/main/java/org/apache/ldap/server/schema/SchemaService.java Sat Nov 12 08:43:28 2005
@@ -452,7 +452,7 @@
      */
     private boolean isCompleteRemoval( Attribute change, Attributes entry ) throws NamingException
     {
-        // if change size is 0 then all values are deleted then we're screwed
+        // if change size is 0 then all values are deleted then we're in trouble
         if ( change.size() == 0 )
         {
             return true;
@@ -462,7 +462,7 @@
         // values in the modify request may not be in the entry.  we need to
         // remove the values from a cloned version of the attribute and see
         // if nothing is left.
-        Attribute changedEntryAttr = entry.get( change.getID() );
+        Attribute changedEntryAttr = ( Attribute ) entry.get( change.getID() ).clone();
         for ( int jj = 0; jj < change.size(); jj++ )
         {
             changedEntryAttr.remove( change.get( jj ) );