Fix issue subscribing users to platform admin - refs CT#7899

main/admin/dashboard_add_users_to_user.php

@@ -40,7 +40,9 @@ $userStatus = api_get_user_status($user_id);
 $firstLetterUser = isset($_POST['firstLetterUser']) ? $_POST['firstLetterUser'] : null;
 
 // setting the name of the tool
-if (UserManager::is_admin($user_id)) {
+$isAdmin = UserManager::is_admin($user_id);
+if ($isAdmin) {
+    $userStatus = PLATFORM_ADMIN;
     $tool_name= get_lang('AssignUsersToPlatformAdministrator');
 } else if ($user_info['status'] == SESSIONADMIN) {
     $tool_name= get_lang('AssignUsersToSessionsAdministrator');
@@ -59,7 +61,7 @@ if (!api_is_platform_admin()) {
     api_not_allowed(true);
 }
 
-function search_users($needle,$type)
+function search_users($needle, $type)
 {
     global $tbl_access_url_rel_user,  $tbl_user, $user_anonymous, $current_user_id, $user_id, $userStatus;
 
@@ -70,6 +72,8 @@ function search_users($needle,$type)
 
         switch ($userStatus) {
             case DRH:
+                //no break;
+            case PLATFORM_ADMIN:
                 $assigned_users_to_hrm = UserManager::get_users_followed_by_drh($user_id);
                 break;
             case STUDENT_BOSS:
@@ -284,9 +288,11 @@ if (isset($_POST['formSent']) && intval($_POST['formSent']) == 1) {
 
     switch ($userStatus) {
         case DRH:
+            //no break;
+        case PLATFORM_ADMIN:
             $affected_rows = UserManager::suscribe_users_to_hr_manager($user_id, $user_list);
             break;
-        case STUDENT_BOSS;
+        case STUDENT_BOSS:
             $affected_rows = UserManager::subscribeUsersToBoss($user_id, $user_list);
             break;
         default:
@@ -331,9 +337,11 @@ $assigned_users_to_hrm = array();
 
 switch ($userStatus) {
     case DRH:
+        //no break;
+    case PLATFORM_ADMIN:
         $assigned_users_to_hrm = UserManager::get_users_followed_by_drh($user_id);
         break;
-    case STUDENT_BOSS;
+    case STUDENT_BOSS:
         $assigned_users_to_hrm = UserManager::getUsersFollowedByStudentBoss($user_id);
         break;
 }

main/inc/lib/usermanager.lib.php

@@ -3974,7 +3974,7 @@ class UserManager
         $drhConditions = null;
         $teacherSelect = null;
 
-        switch($status) {
+        switch ($status) {
             case DRH:
                 $drhConditions .= " AND
                     friend_user_id = '$userId' AND
@@ -4057,7 +4057,6 @@ class UserManager
         if ($getSql) {
             return $sql;
         }
-
         if ($getCount) {
             $result = Database::query($sql);
             $row = Database::fetch_array($result);
@@ -4079,7 +4078,6 @@ class UserManager
 
         $sql .= $orderBy;
         $sql .= $limitCondition;
-
         $result = Database::query($sql);
         $users = array();
         if (Database::num_rows($result) > 0) {
@@ -4130,7 +4128,6 @@ class UserManager
                 . "WHERE friend_user_id = $userId "
                 . "AND relation_type = $relationType";
         }
-
         $result = Database::query($sql);
 
         if (Database::num_rows($result) > 0) {