Skip to content

[stable30] fix(settings): Fix infinitely loading account management page with pagination of groups #51782

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
susnux merged 28 commits into from
Apr 3, 2025
Merged

[stable30] fix(settings): Fix infinitely loading account management page with pagination of groups #51782

Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
ff76f4a
perf(settings): Remove computation of all groups
Pytal Mar 25, 2025
8f10477
fix(settings): Fix infinitely loading account management page with pa…
Pytal Mar 25, 2025
5167b56
feat(provisioning_api): Add endpoint for fetching user groups with de…
Pytal Mar 25, 2025
f664eb8
perf(settings): Cancel request on new search
Pytal Mar 25, 2025
86e521d
fix(settings): Allow searching for groups in user row
Pytal Mar 25, 2025
328bb10
fix(settings): Allow searching for groups in new account dialog
Pytal Mar 25, 2025
c3dba3b
perf(settings): Make scrolling smooth when a large number of groups a…
Pytal Mar 25, 2025
1fbf26a
refactor(settings): Consolidate group formatting
Pytal Mar 25, 2025
537d68d
chore(settings): Add note on groups sorting
Pytal Mar 25, 2025
82cd9a2
fix(settings): Fix loaded groups being undefined
Pytal Mar 25, 2025
74b26f7
fix(settings): Prevent selection of invalid groups that are not fully…
Pytal Mar 25, 2025
ae390d2
fix(settings): Fix erroneous hiding of group admin column with pagina…
Pytal Mar 25, 2025
2cc7ebe
feat(provisioning_api): Add endpoint for fetching user subadmin group…
Pytal Mar 25, 2025
d3b8171
fix(settings): Fix editing groups and subadmin groups of user
Pytal Mar 25, 2025
21dbef7
fix(settings): Only change usercount if group can be found
Pytal Mar 25, 2025
91af8d1
fix(settings): Fix group creation when editing users
Pytal Mar 25, 2025
c022d2f
fix(settings): Fix group creation in new account dialog
Pytal Mar 25, 2025
42b7fda
fix(settings): Fix duplicated group options when editing account
Pytal Mar 25, 2025
a5e2c7d
fix(settings): Fix duplicated group options in new account dialog
Pytal Mar 25, 2025
75708f5
fix(settings): Natural order groups
Pytal Mar 25, 2025
368fcc3
fix(settings): Preserve system groups on reset
Pytal Mar 25, 2025
f1ed995
fix(settings): Fix initialization of store
Pytal Mar 25, 2025
3402b7c
fix(settings): Separate subadmin options
Pytal Mar 25, 2025
06af6de
test(settings): Wait until groups list has loaded
Pytal Mar 25, 2025
85dc4b4
test(settings): Correctly find group in select
Pytal Mar 25, 2025
58fc803
test(settings): Fix group items not being found
Pytal Mar 25, 2025
9cac821
chore(assets): Recompile assets
nextcloud-command Mar 28, 2025
fcd5da1
chore: Backport fixes
artonge Apr 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
perf(settings): Remove computation of all groups 
Signed-off-by: Christopher Ng <[email protected]>
  • Loading branch information
@Pytal @backportbot
Pytal authored and backportbot[bot] committed Apr 1, 2025
commit ff76f4ad599d34e7e08b7719f0bc9d423aca1be7
13 changes: 10 additions & 3 deletions apps/settings/lib/Controller/UsersController.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -130,8 +130,15 @@ public function usersList(): TemplateResponse {
$this->userSession
);

$groupsInfo->setSorting($sortGroupsBy);
[$adminGroup, $groups] = $groupsInfo->get();
$adminGroup = $this->groupManager->get('admin');
$adminGroupData = [
'id' => $adminGroup->getGID(),
'name' => $adminGroup->getDisplayName(),
'usercount' => $sortGroupsBy === MetaData::SORT_USERCOUNT ? $adminGroup->count() : 0,
'disabled' => $adminGroup->countDisabled(),
'canAdd' => $adminGroup->canAddUser(),
'canRemove' => $adminGroup->canRemoveUser(),
];

if (!$isLDAPUsed && $this->appManager->isEnabledForUser('user_ldap')) {
$isLDAPUsed = (bool)array_reduce($this->userManager->getBackends(), function ($ldapFound, $backend) {
Expand Down Expand Up @@ -192,7 +199,7 @@ public function usersList(): TemplateResponse {
/* FINAL DATA */
$serverData = [];
// groups
$serverData['groups'] = array_merge_recursive($adminGroup, [$recentUsersGroup, $disabledUsersGroup], $groups);
$serverData['systemGroups'] = [$adminGroupData, $recentUsersGroup, $disabledUsersGroup];
// Various data
$serverData['isAdmin'] = $isAdmin;
$serverData['isDelegatedAdmin'] = $isDelegatedAdmin;
Expand Down