nextcloud
/
server
mirror of https://github.com/nextcloud/server
0
0
Fork 0
Code Issues Releases Wiki Activity

Invert parameter order in getDisabledUserList to be consistent 

This matches what was done in the calls and so fixes getting disabled
 user list when there are several backends returning disabled users.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
This commit is contained in:
Côme Chilliet 2023-10-19 10:35:49 +02:00 committed by Christopher Ng
parent ceca099f29
commit adc43eae9b
3 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
apps/user_ldap/lib/User_LDAP.php
View File

@ -682,7 +682,7 @@ class User_LDAP extends BackendUtility implements IUserBackend, UserInterface, I
return $enabled;
}
public function getDisabledUserList(int $offset = 0, ?int $limit = null): array {
public function getDisabledUserList(?int $limit = null, int $offset = 0): array {
throw new \Exception('This is implemented directly in User_Proxy');
}
}

2
apps/user_ldap/lib/User_Proxy.php
View File

@ -463,7 +463,7 @@ class User_Proxy extends Proxy implements IUserBackend, UserInterface, IUserLDAP
return $this->handleRequest($uid, 'setUserEnabled', [$uid, $enabled, $queryDatabaseValue, $setDatabaseValue]);
}
public function getDisabledUserList(int $offset = 0, ?int $limit = null): array {
public function getDisabledUserList(?int $limit = null, int $offset = 0): array {
return array_map(
fn (OfflineUser $user) => $user->getOCName(),
array_slice(

2
lib/public/User/Backend/IProvideEnabledStateBackend.php
View File

@ -52,5 +52,5 @@ interface IProvideEnabledStateBackend {
*
* @return string[]
*/
public function getDisabledUserList(int $offset = 0, ?int $limit = null): array;
public function getDisabledUserList(?int $limit = null, int $offset = 0): array;
}