Pass isHidden to createUser function and send to the hook as well.

Author: cdujeu

core/src/core/classes/class.AuthService.php

@@ -772,13 +772,14 @@ public static function updatePassword($userId, $userPass)
      * @param $userId
      * @param $userPass
      * @param bool $isAdmin
+     * @param bool $isHidden
      * @return null
      * @todo the minlength check is probably causing problem with the bridges
      */
-    public static function createUser($userId, $userPass, $isAdmin=false)
+    public static function createUser($userId, $userPass, $isAdmin=false, $isHidden=false)
     {
         $userId = self::filterUserSensitivity($userId);
-        AJXP_Controller::applyHook("user.before_create", array($userId, $userPass, $isAdmin));
+        AJXP_Controller::applyHook("user.before_create", array($userId, $userPass, $isAdmin, $isHidden));
         if (!ConfService::getCoreConf("ALLOW_GUEST_BROWSING", "auth") && $userId == "guest") {
             throw new Exception("Reserved user id");
         }
@@ -791,18 +792,18 @@ public static function createUser($userId, $userPass, $isAdmin=false)
         $authDriver = ConfService::getAuthDriverImpl();
         $confDriver = ConfService::getConfStorageImpl();
         $authDriver->createUser($userId, $userPass);
-        $user = null;
+        $user = $confDriver->createUserObject($userId);
         if ($isAdmin) {
-            $user = $confDriver->createUserObject($userId);
             $user->setAdmin(true);
             $user->save("superuser");
         }
+        if($isHidden){
+            $user->setHidden(true);
+            $user->save("superuser");
+        }
         if ($authDriver->getOptionAsBool("TRANSMIT_CLEAR_PASS")) {
             $realm = ConfService::getCoreConf("WEBDAV_DIGESTREALM");
             $ha1 = md5("{$userId}:{$realm}:{$userPass}");
-            if (!isSet($user)) {
-                $user = $confDriver->createUserObject($userId);
-            }
             $wData = $user->getPref("AJXP_WEBDAV_DATA");
             if(!is_array($wData)) $wData = array();
             $wData["HA1"] = $ha1;

core/src/plugins/action.share/class.ShareCenter.php

@@ -1922,7 +1922,7 @@ public function createSharedRepository($httpVars, $repository, $accessDriver, $u
                 }
                 if(!isSet($httpVars["minisite"])){
                     // This is an explicit user creation - check possible limits
-                    AJXP_Controller::applyHook("user.before_create", array($userName));
+                    AJXP_Controller::applyHook("user.before_create", array($userName, null, false, false));
                     $limit = $loggedUser->personalRole->filterParameterValue("core.conf", "USER_SHARED_USERS_LIMIT", AJXP_REPO_SCOPE_ALL, "");
                     if (!empty($limit) && intval($limit) > 0) {
                         $count = count(ConfService::getConfStorageImpl()->getUserChildren($loggedUser->getId()));
@@ -1932,7 +1932,7 @@ public function createSharedRepository($httpVars, $repository, $accessDriver, $u
                         }
                     }
                 }
-                AuthService::createUser($userName, $pass);
+                AuthService::createUser($userName, $pass, false, isSet($httpVars["minisite"]));
                 $userObject = $confDriver->createUserObject($userName);
                 $userObject->personalRole->clearAcls();
                 $userObject->setParent($loggedUser->id);