diff --git a/apps/provisioning_api/lib/Controller/GroupsController.php b/apps/provisioning_api/lib/Controller/GroupsController.php index a1a94b56b4334..62f639b48e4e4 100644 --- a/apps/provisioning_api/lib/Controller/GroupsController.php +++ b/apps/provisioning_api/lib/Controller/GroupsController.php @@ -78,10 +78,10 @@ public function __construct( #[NoAdminRequired] public function getGroups(string $search = '', ?int $limit = null, int $offset = 0): DataResponse { $groups = $this->groupManager->search($search, $limit, $offset); - $groups = array_values(array_map(function ($group) { + $groups = array_map(function ($group) { /** @var IGroup $group */ return $group->getGID(); - }, $groups)); + }, $groups); return new DataResponse(['groups' => $groups]); } @@ -101,7 +101,7 @@ public function getGroups(string $search = '', ?int $limit = null, int $offset = #[AuthorizedAdminSetting(settings: Users::class)] public function getGroupsDetails(string $search = '', ?int $limit = null, int $offset = 0): DataResponse { $groups = $this->groupManager->search($search, $limit, $offset); - $groups = array_values(array_map(function ($group) { + $groups = array_map(function ($group) { /** @var IGroup $group */ return [ 'id' => $group->getGID(), @@ -111,7 +111,7 @@ public function getGroupsDetails(string $search = '', ?int $limit = null, int $o 'canAdd' => $group->canAddUser(), 'canRemove' => $group->canRemoveUser(), ]; - }, $groups)); + }, $groups); return new DataResponse(['groups' => $groups]); } diff --git a/build/psalm-baseline.xml b/build/psalm-baseline.xml index 92479b887aac3..6dea3b11d9c96 100644 --- a/build/psalm-baseline.xml +++ b/build/psalm-baseline.xml @@ -3701,8 +3701,6 @@ - - diff --git a/lib/private/Group/Manager.php b/lib/private/Group/Manager.php index 8feb027d1807f..1dcbcfd7a0c77 100644 --- a/lib/private/Group/Manager.php +++ b/lib/private/Group/Manager.php @@ -264,12 +264,6 @@ public function createGroup($gid) { } } - /** - * @param string $search - * @param ?int $limit - * @param ?int $offset - * @return \OC\Group\Group[] - */ public function search(string $search, ?int $limit = null, ?int $offset = 0) { $groups = []; foreach ($this->backends as $backend) { diff --git a/lib/private/User/Manager.php b/lib/private/User/Manager.php index 028ee87b40bb3..5624a5901af65 100644 --- a/lib/private/User/Manager.php +++ b/lib/private/User/Manager.php @@ -278,14 +278,6 @@ public function search($pattern, $limit = null, $offset = null) { return $users; } - /** - * Search by displayName - * - * @param string $pattern - * @param int $limit - * @param int $offset - * @return IUser[] - */ public function searchDisplayName($pattern, $limit = null, $offset = null) { $users = []; foreach ($this->backends as $backend) { diff --git a/lib/public/IGroupManager.php b/lib/public/IGroupManager.php index ee88990df79ae..eda95187f63da 100644 --- a/lib/public/IGroupManager.php +++ b/lib/public/IGroupManager.php @@ -75,7 +75,7 @@ public function createGroup($gid); * @param string $search * @param ?int $limit * @param ?int $offset - * @return \OCP\IGroup[] + * @return list * @since 8.0.0 */ public function search(string $search, ?int $limit = null, ?int $offset = 0); diff --git a/lib/public/IUserManager.php b/lib/public/IUserManager.php index a4b70327e1e63..48a161390448e 100644 --- a/lib/public/IUserManager.php +++ b/lib/public/IUserManager.php @@ -113,7 +113,7 @@ public function search($pattern, $limit = null, $offset = null); * @param string $pattern * @param int $limit * @param int $offset - * @return \OCP\IUser[] + * @return list * @since 8.0.0 */ public function searchDisplayName($pattern, $limit = null, $offset = null);