From 2b2b4e9e96a3bd07dc099f1500f0854527585b0e Mon Sep 17 00:00:00 2001 From: Maksim Sukharev Date: Sat, 28 Sep 2024 18:46:16 +0200 Subject: [PATCH] fix: correctly count disabled users for subadmins Signed-off-by: Maksim Sukharev --- apps/settings/lib/Controller/UsersController.php | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/apps/settings/lib/Controller/UsersController.php b/apps/settings/lib/Controller/UsersController.php index 71f256f57d69d..88a2a6ba6eb34 100644 --- a/apps/settings/lib/Controller/UsersController.php +++ b/apps/settings/lib/Controller/UsersController.php @@ -147,13 +147,13 @@ public function usersList(): TemplateResponse { }, 0); } else { // User is subadmin ! - // Map group list to names to retrieve the countDisabledUsersOfGroups + // Map group list to ids to retrieve the countDisabledUsersOfGroups $userGroups = $this->groupManager->getUserGroups($user); - $groupsNames = []; + $groupsIds = []; foreach ($groups as $key => $group) { // $userCount += (int)$group['usercount']; - $groupsNames[] = $group['name']; + $groupsIds[] = $group['id']; // we prevent subadmins from looking up themselves // so we lower the count of the groups he belongs to if (array_key_exists($group['id'], $userGroups)) { @@ -163,7 +163,7 @@ public function usersList(): TemplateResponse { } $userCount += $this->userManager->countUsersOfGroups($groupsInfo->getGroups()); - $disabledUsers = $this->userManager->countDisabledUsersOfGroups($groupsNames); + $disabledUsers = $this->userManager->countDisabledUsersOfGroups($groupsIds); } $userCount -= $disabledUsers;