diff options
author | Côme Chilliet <come.chilliet@nextcloud.com> | 2025-03-06 16:36:47 +0100 |
---|---|---|
committer | Côme Chilliet <come.chilliet@nextcloud.com> | 2025-03-31 15:42:29 +0200 |
commit | 0710cfb800981b76727ba656943a58a0f49e4414 (patch) | |
tree | dddde041d2af3caaa85879a87b92ba144a1ae53d | |
parent | 874dcbf8df3edccd1649866bd87cb17fe94b8c32 (diff) | |
download | nextcloud-server-backport/51108/stable29.tar.gz nextcloud-server-backport/51108/stable29.zip |
chore: Move magic number into a documented constbackport/51108/stable29
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
-rw-r--r-- | apps/settings/lib/Controller/UsersController.php | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/apps/settings/lib/Controller/UsersController.php b/apps/settings/lib/Controller/UsersController.php index d117067cf84..01670a817a4 100644 --- a/apps/settings/lib/Controller/UsersController.php +++ b/apps/settings/lib/Controller/UsersController.php @@ -71,6 +71,8 @@ use function in_array; #[OpenAPI(scope: OpenAPI::SCOPE_IGNORE)] class UsersController extends Controller { + /** Limit for counting users for subadmins, to avoid spending too much time */ + private const COUNT_LIMIT_FOR_SUBADMINS = 999; public function __construct( string $appName, @@ -168,7 +170,7 @@ class UsersController extends Controller { }, 0); } else { // User is subadmin ! - [$userCount,$disabledUsers] = $this->userManager->countUsersAndDisabledUsersOfGroups($groupsInfo->getGroups(), 999); + [$userCount,$disabledUsers] = $this->userManager->countUsersAndDisabledUsersOfGroups($groupsInfo->getGroups(), self::COUNT_LIMIT_FOR_SUBADMINS); } if ($disabledUsers > 0) { |