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 <[email protected]>
nextcloud · Aug 21, 2020 · 51922ca · 51922ca
1 parent 6e4b089
commit 51922ca
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/lib/private/Collaboration/Collaborators/UserPlugin.php b/lib/private/Collaboration/Collaborators/UserPlugin.php
@@ -95,7 +95,15 @@ public function search($search, $limit, $offset, ISearchResult $searchResult) {
 				$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 {