While opening or searching for users within admin user listing, all groups are fetched

[mrow4a]

I have noticed, that with any operation done within user listing, e.g.

we are fetching not only all the groups in the system, but also, sequentialy, for each of them fetching their metadata (https://github.com/owncloud/core/blob/master/lib/private/Group/MetaData.php#L90-L90 with https://github.com/owncloud/core/blob/master/settings/Controller/GroupsController.php#L88-L88).

This means, that having 50k groups, with each manipulation in the user interface, we will first fetch 50k groups from table, and then for each of 50k groups we will count number of users in that group.

@DeepDiver1975 @PVince81

[mrow4a]

@felixheidecke is it possible to keep section with group listing fixed or at least paged (e.g. displaying first 10) ? For now with each refresh of /user/users we execute this line https://github.com/owncloud/core/blob/master/settings/users.php#L68-L68.

[mrow4a]

I actually achieved that by with first refresh, just leaving code unchanged.

Before I started searching, I just commented out lines 68, 110, 112, 115 in https://github.com/owncloud/core/blob/master/settings/users.php

Then I searched for users, and the group pannel was untouched, and no data was fetched from DB. for groups.

@felixheidecke @PVince81 other thing will be to have the group section paged + independent from user search.

[mrow4a]

@DeepDiver1975 for now it might sound trivial, but with central group table(#29107), it will not anymore go to LDAP, it will go to database and become immediately visible for sysadmins. With LDAP it was a bit hidden probably, since it was not hitting DB.

[PVince81]

Possibly related and overdue: #9201 and loading groups with XHR/ajax

[ownclouders]

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 7 days. Feel free to reopen this issue if you deem it appropriate.

(This is an automated comment from GitMate.io.)

[ownclouders]

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 7 days. Feel free to reopen this issue if you deem it appropriate.

(This is an automated comment from GitMate.io.)

[stale]

This issue has been automatically closed.