Merge pull request #17366 from nextcloud/fix/usersIngroup

Fix DB usersInGroups fetching
pull/17363/head
Roeland Jago Douma 2019-10-02 13:25:49 +07:00 committed by GitHub
commit 9bf55c33e1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 3 deletions

@ -322,7 +322,7 @@ class Database extends ABackend
* @param int $offset
* @return array an array of user ids
*/
public function usersInGroup($gid, $search = '', $limit = null, $offset = null) {
public function usersInGroup($gid, $search = '', $limit = -1, $offset = 0) {
$this->fixDI();
$query = $this->dbConn->getQueryBuilder();
@ -337,8 +337,13 @@ class Database extends ABackend
)));
}
$query->setMaxResults($limit)
->setFirstResult($offset);
if ($limit !== -1) {
$query->setMaxResults($limit);
}
if ($offset !== 0) {
$query->setFirstResult($offset);
}
$result = $query->execute();
$users = [];