owncloud/core — pull request #8216 by blizzz (2014-04-15 16:04:14+00:00)
Title:  	Remove limit and offset manipulation when getting users or groups, be… 

…cause it does not work when more than one user or group backend. Fixing it would be too costly performance wise, so we switch back to the model used in OC 5: limit and offset are effective per backend, and not a general constraint.

What happens currently when fetching users: users from the first registered backend are read first correctly. Then the users are read from the second registered backend, but with an incorrectly computed offset. As a consquence not every user is loaded.

To fix it to work as intentend we would need to compute all users in the backends that are already progressed. With the search string in mind it can be costly process, so we refrain.

What happens with this PR when fetchung users: users from all backends are fetched the same time. Limit/Offset does not need to be manipulated any more. I.e. more than 30 users (if existent) are loaded upon first opening the users page.

As discussed with @icewind1991 priorly on IRC.

Reviewers please @DeepDiver1975 @bantu or others.