From 51922caa5f5c3c4a7bc446b1cfc5348451f0ad62 Mon Sep 17 00:00:00 2001 From: Roeland Jago Douma Date: Fri, 1 May 2020 16:27:44 +0200 Subject: [PATCH] Properly search for users when limittogroups is enabled Searching just for the uid is not enough. This makes sure this done properly again now. Signed-off-by: Roeland Jago Douma --- lib/private/Collaboration/Collaborators/UserPlugin.php | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/lib/private/Collaboration/Collaborators/UserPlugin.php b/lib/private/Collaboration/Collaborators/UserPlugin.php index 72368e5052..0a60344852 100644 --- a/lib/private/Collaboration/Collaborators/UserPlugin.php +++ b/lib/private/Collaboration/Collaborators/UserPlugin.php @@ -95,7 +95,15 @@ class UserPlugin implements ISearchPlugin { $usersInGroup = $this->groupManager->displayNamesInGroup($userGroupId, $search, $limit, $offset); foreach ($usersInGroup as $userId => $displayName) { $userId = (string) $userId; - $users[$userId] = $this->userManager->get($userId); + $user = $this->userManager->get($userId); + if (!$user->isEnabled()) { + // Ignore disabled users + continue; + } + $users[$userId] = $user; + } + if (count($usersInGroup) >= $limit) { + $hasMoreResults = true; } } } else {