Move back to IGroupManager::displayNamesInGroup()

The problem is that despite it's name IGroup::searchDisplayName()
only searches by userid and this is less fixable than changing back to this method here

Signed-off-by: Joas Schilling <coding@schilljs.com>

diff --git a/lib/private/Collaboration/Collaborators/UserPlugin.php b/lib/private/Collaboration/Collaborators/UserPlugin.php
index 5fb278fc5ffd38161067dc59449c9a99d099d1d2..c955b244f48239b30d653093c6df35c7900955fb 100644 (file)
--- a/lib/private/Collaboration/Collaborators/UserPlugin.php
+++ b/lib/private/Collaboration/Collaborators/UserPlugin.php
@@ -68,14 +68,14 @@ class UserPlugin implements ISearchPlugin {
                $users = [];
                $hasMoreResults = false;
 
-               $userGroups = [];
+               $currentUserGroups = $this->groupManager->getUserGroupIds($this->userSession->getUser());
                if ($this->shareWithGroupOnly) {
                        // Search in all the groups this user is part of
-                       $userGroups = $this->groupManager->getUserGroups($this->userSession->getUser());
-                       foreach ($userGroups as $userGroup) {
-                               $usersInGroup = $userGroup->searchDisplayName($search, $limit, $offset);
-                               foreach ($usersInGroup as $user) {
-                                       $users[$user->getUID()] = $user;
+                       foreach ($currentUserGroups as $userGroupId) {
+                               $usersInGroup = $this->groupManager->displayNamesInGroup($userGroupId, $search, $limit, $offset);
+                               foreach ($usersInGroup as $userId => $displayName) {
+                                       $userId = (string) $userId;
+                                       $users[$userId] = $this->userManager->get($userId);
                                }
                        }
                } else {
@@ -136,10 +136,7 @@ class UserPlugin implements ISearchPlugin {
 
                                if ($this->shareWithGroupOnly) {
                                        // Only add, if we have a common group
-                                       $userGroupIds = array_map(function(IGroup $group) {
-                                               return $group->getGID();
-                                       }, $userGroups);
-                                       $commonGroups = array_intersect($userGroupIds, $this->groupManager->getUserGroupIds($user));
+                                       $commonGroups = array_intersect($currentUserGroups, $this->groupManager->getUserGroupIds($user));
                                        $addUser = !empty($commonGroups);
                                }