From 962a9716438109875b46bebb4bb73d4f6a387a3c 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 b677147b162..42e42f52b9f 100644 --- a/apps/settings/lib/Controller/UsersController.php +++ b/apps/settings/lib/Controller/UsersController.php @@ -147,13 +147,13 @@ class UsersController extends Controller { }, 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 @@ class UsersController extends Controller { } $userCount += $this->userManager->countUsersOfGroups($groupsInfo->getGroups()); - $disabledUsers = $this->userManager->countDisabledUsersOfGroups($groupsNames); + $disabledUsers = $this->userManager->countDisabledUsersOfGroups($groupsIds); } $userCount -= $disabledUsers; -- 2.39.5