Added fix for

Trying to overwrite cn=****,ou=people,...,dc=de, which already exists.
View file comments for description.

git-svn-id: https://oss.gonicus.de/repositories/gosa/trunk@1270 594d385d-05f5-0310-b6e9-bd551577e9d8

diff --git a/plugins/admin/users/tabs_user.inc b/plugins/admin/users/tabs_user.inc
index 451d9f973f2f1b84184179214f38b627134d963b..f19f5b856bdadb550657760e9bdbcd172cf186d2 100644 (file)
--- a/plugins/admin/users/tabs_user.inc
+++ b/plugins/admin/users/tabs_user.inc
@@ -84,7 +84,20 @@ class usertabs extends tabs
         $new_dn= "cn=".$baseobject->givenName." ".$baseobject->sn.','.
           get_people_ou().$baseobject->base;
       }
-
+      
+      /* Hickert: 31.08.2005 
+       * fixed error = Trying to overwrite cn=***  which already exists.
+       * If you changed one of the "Name" / "Givenname" attributes to lower or upper case, like "test" to "Test", (in DNMODE=cn)
+       *  the old_dn and new dn were different. cn=test ... cn=Test ...
+       * The following code decided to move the entry from old_dn to new_dn. (Because they differ)
+       * But the ldap functions are not case sensitive and so the function plugin::move() quits with 
+       *  an error, because source and destination dn are both the same (for ldap, but not for php).
+       * This fix allow you to change the givenname or name attributes.
+       */
+      if(strtolower($this->dn)==strtolower($new_dn)){
+        $this->dn=$new_dn;
+      }
+  
       if ($this->dn != $new_dn){
 
         /* Write entry on new 'dn' */
@@ -99,6 +112,7 @@ class usertabs extends tabs
         $this->dn= $new_dn;
       }
     }
+    print_a($this);
     return tabs::save();
   }