fix default values and type hints for GroupManager::search

ints really are ints

Signed-off-by: Robin Appelman <robin@icewind.nl>

1 files changed, 2 insertions, 2 deletions

diff --git a/lib/private/Group/Manager.php b/lib/private/Group/Manager.php
index b718afa5168..4dde6152275 100644
--- a/lib/private/Group/Manager.php
+++ b/lib/private/Group/Manager.php
@@ -240,7 +240,7 @@ class Manager extends PublicEmitter implements IGroupManager {
 	 * @param int $offset
 	 * @return \OC\Group\Group[]
 	 */
-	public function search($search, $limit = null, $offset = null) {
+	public function search(string $search, int $limit = -1, int $offset = 0) {
 		$groups = [];
 		foreach ($this->backends as $backend) {
 			$groupIds = $backend->getGroups($search, $limit, $offset);
@@ -252,7 +252,7 @@ class Manager extends PublicEmitter implements IGroupManager {
 					$this->logger->debug('Group "' . $groupId . '" was returned by search but not found through direct access', ['app' => 'core']);
 				}
 			}
-			if (!is_null($limit) and $limit <= 0) {
+			if ($limit === 0) {
 				return array_values($groups);
 			}
 		}