Issue #196 has been updated by Sébastien Bahloul.

Fixed in r683. This will probably require some backport to 1.2 branch.

Interesting diff is :
-                                       msgBuffer.append(ln.get(0));            
                            
-                                       msgBuffer.append("\ndeleteoldrdn: 
1\nnewsuperior: ");               
-                                       msgBuffer.append(ln.getSuffix(1));
+                                       msgBuffer.append(ln.get(ln.size()-1));  
                                            
+                                       msgBuffer.append("\ndeleteoldrdn: 
1\nnewsuperior: ");                               
+                                       if(ln.size() <= 1) {                    
                                            
+                                               msgBuffer.append(baseUrl);      
                                            
+                                       } else {                                
                                            
+                                               
msgBuffer.append(ln.getPrefix(ln.size()-1) + "," + baseUrl);                
+                                       }

This also fix a minor issue about missing naming context in newsuperior value.
----------------------------------------
Bug #196: Ldif layout bug for modrdn LDAP operation
http://tools.lsc-project.org/issues/show/196

Author: Sébastien Bahloul
Status: Assigned
Priority: Normal
Assigned to: Sébastien Bahloul
Category: Core
Target version: 1.3 branch


LDIF layout for modrdn LDAP operation is incorrect. For example when running 
ldap2ldap tests, format is 
<pre>
dn: cn=CommonName0002,ou=ldap2script2TestTaskDst,ou=Test 
Data,dc=lsc-project,dc=org
changetype: modrdn
newrdn: ou=Test Data
deleteoldrdn: 1
newsuperior: cn=CN0002,ou=ldap2script2TestTaskDst,dc=lsc-project,dc=org
</pre>

It must be :
<pre>
dn: cn=CommonName0002,ou=ldap2script2TestTaskDst,ou=Test 
Data,dc=lsc-project,dc=org
changetype: modrdn
newrdn: cn=CN0002
deleteoldrdn: 1
newsuperior: ou=ldap2script2TestTaskDst,ou=Test Data,dc=lsc-project,dc=org
</pre>


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://tools.lsc-project.org/my/account
_______________________________________________________________
Ldap Synchronization Connector (LSC) - http://lsc-project.org

lsc-dev mailing list
[email protected]
http://lists.lsc-project.org/listinfo/lsc-dev

Reply via email to