From 556440471a882cd47721777c59881ee3aae69c11 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 123fa73144..3566abf0a0 100644 --- a/lib/private/Collaboration/Collaborators/UserPlugin.php +++ b/lib/private/Collaboration/Collaborators/UserPlugin.php @@ -79,7 +79,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 {