You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by tr...@apache.org on 2005/06/23 06:50:08 UTC

svn commit: r193073 - /directory/apacheds/branches/db_refactor/core/src/main/java/org/apache/ldap/server/partition/AbstractContextPartition.java

Author: trustin
Date: Wed Jun 22 21:50:07 2005
New Revision: 193073

URL: http://svn.apache.org/viewcvs?rev=193073&view=rev
Log:
* move(Name,Name,String,boolean) now calls move first to make sure move operation is supported.

Modified:
    directory/apacheds/branches/db_refactor/core/src/main/java/org/apache/ldap/server/partition/AbstractContextPartition.java

Modified: directory/apacheds/branches/db_refactor/core/src/main/java/org/apache/ldap/server/partition/AbstractContextPartition.java
URL: http://svn.apache.org/viewcvs/directory/apacheds/branches/db_refactor/core/src/main/java/org/apache/ldap/server/partition/AbstractContextPartition.java?rev=193073&r1=193072&r2=193073&view=diff
==============================================================================
--- directory/apacheds/branches/db_refactor/core/src/main/java/org/apache/ldap/server/partition/AbstractContextPartition.java (original)
+++ directory/apacheds/branches/db_refactor/core/src/main/java/org/apache/ldap/server/partition/AbstractContextPartition.java Wed Jun 22 21:50:07 2005
@@ -222,15 +222,17 @@
     }
 
     /**
-     * This method calls {@link ContextPartition#modifyRn(Name, String, boolean)}
-     * and {@link ContextPartition#move(Name, Name)} subsequently by default.
-     * Please override this method if there is more effactive way for your
-     * implementation.
+     * This method calls {@link ContextPartition#move(Name, Name)} and
+     * {@link ContextPartition#modifyRn(Name, String, boolean)} subsequently
+     * by default.  Please override this method if there is more effactive
+     * way for your implementation.
      */
     public void move( Name oldName, Name newParentName, String newRn, boolean deleteOldRn ) throws NamingException
     {
-        modifyRn( oldName, newRn, deleteOldRn );
+        Name newName = ( Name ) newParentName.clone();
+        newName.add( newRn );
         move( oldName, newParentName );
+        modifyRn( newName, newRn, deleteOldRn );
     }
 
     /**