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 2006/11/07 22:27:18 UTC

svn commit: r472281 - /directory/trunks/apacheds/mitosis/src/main/java/org/apache/directory/mitosis/operation/OperationFactory.java

Author: akarasulu
Date: Tue Nov  7 13:27:18 2006
New Revision: 472281

URL: http://svn.apache.org/viewvc?view=rev&rev=472281
Log:
correcting improper use of getSuffix()

Modified:
    directory/trunks/apacheds/mitosis/src/main/java/org/apache/directory/mitosis/operation/OperationFactory.java

Modified: directory/trunks/apacheds/mitosis/src/main/java/org/apache/directory/mitosis/operation/OperationFactory.java
URL: http://svn.apache.org/viewvc/directory/trunks/apacheds/mitosis/src/main/java/org/apache/directory/mitosis/operation/OperationFactory.java?view=diff&rev=472281&r1=472280&r2=472281
==============================================================================
--- directory/trunks/apacheds/mitosis/src/main/java/org/apache/directory/mitosis/operation/OperationFactory.java (original)
+++ directory/trunks/apacheds/mitosis/src/main/java/org/apache/directory/mitosis/operation/OperationFactory.java Tue Nov  7 13:27:18 2006
@@ -172,7 +172,10 @@
 
     public Operation newModifyRn( LdapDN oldName, String newRdn, boolean deleteOldRn ) throws NamingException
     {
-        return newMove( oldName, ( LdapDN ) oldName.getSuffix( 1 ), newRdn, deleteOldRn );
+        LdapDN newParentName = ( LdapDN ) oldName.clone();
+        newParentName.remove( oldName.size() - 1 );
+        
+        return newMove( oldName, newParentName, newRdn, deleteOldRn );
     }