Yes. Really makes sense. Thanks for reporting.

Moved shadow attribute writing some lines down to be not overwritten. Closes #810

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

diff --git a/gosa-core/include/functions.inc b/gosa-core/include/functions.inc
index 1234a6d44cf6090ad67c60779b3c0da574ae019b..d7921871bec4b6ff74053fbdc5b294c326fde589 100644 (file)
--- a/gosa-core/include/functions.inc
+++ b/gosa-core/include/functions.inc
@@ -2977,13 +2977,12 @@ function change_password ($dn, $password, $mode=0, $hash= "")
 
     // Not for groups
     if ($mode == 0){
+      // Create SMB Password
+      $attrs= generate_smb_nt_hash($password);
 
       if ($shadow != 0){
         $attrs['shadowLastChange']= $shadow;
       }
-
-      // Create SMB Password
-      $attrs= generate_smb_nt_hash($password);
     }
 
     $attrs['userPassword']= array();