aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorCôme Chilliet <come.chilliet@nextcloud.com>2025-03-06 16:36:47 +0100
committerCôme Chilliet <come.chilliet@nextcloud.com>2025-03-31 15:39:40 +0200
commit3d33ac2d34a526a6307993890e8a42aac29a2436 (patch)
tree85f03530270238e5df6175a51b664a6659f65298
parent107c18dff2ef4ca9b3cd871493dd8c52b8b0000b (diff)
downloadnextcloud-server-backport/51108/stable30.tar.gz
nextcloud-server-backport/51108/stable30.zip
chore: Move magic number into a documented constbackport/51108/stable30
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
-rw-r--r--apps/settings/lib/Controller/UsersController.php4
1 files changed, 3 insertions, 1 deletions
diff --git a/apps/settings/lib/Controller/UsersController.php b/apps/settings/lib/Controller/UsersController.php
index fbf0789694d..a7841787013 100644
--- a/apps/settings/lib/Controller/UsersController.php
+++ b/apps/settings/lib/Controller/UsersController.php
@@ -51,6 +51,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,
@@ -148,7 +150,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) {