Skip to content

Commit d86877b

Browse files
authored
Merge pull request #28701 from nextcloud/fix/stable22-user-list-infinite-loading
[stable22] Fix user list infinite loading state in user settings
2 parents 3e3ce19 + d2c6fa0 commit d86877b

File tree

6 files changed

+21
-16
lines changed

6 files changed

+21
-16
lines changed

apps/settings/js/vue-settings-apps-users-management.js

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

apps/settings/js/vue-settings-apps-users-management.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

apps/settings/js/vue-settings-users.js

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

apps/settings/js/vue-settings-users.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

apps/settings/src/components/UserList.vue

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -461,8 +461,13 @@ export default {
461461
group: this.selectedGroup !== 'disabled' ? this.selectedGroup : '',
462462
search: this.searchQuery,
463463
})
464-
.then((response) => {
465-
response ? $state.loaded() : $state.complete()
464+
.then((usersCount) => {
465+
if (usersCount > 0) {
466+
$state.loaded()
467+
}
468+
if (usersCount < this.usersLimit) {
469+
$state.complete()
470+
}
466471
})
467472
},
468473

apps/settings/src/store/users.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -224,11 +224,11 @@ const actions = {
224224
cancelToken: searchRequestCancelSource.token,
225225
})
226226
.then((response) => {
227-
if (Object.keys(response.data.ocs.data.users).length > 0) {
227+
const usersCount = Object.keys(response.data.ocs.data.users).length
228+
if (usersCount > 0) {
228229
context.commit('appendUsers', response.data.ocs.data.users)
229-
return Object.keys(response.data.ocs.data.users).length >= limit
230230
}
231-
return false
231+
return usersCount
232232
})
233233
.catch((error) => {
234234
if (!axios.isCancel(error)) {
@@ -241,11 +241,11 @@ const actions = {
241241
cancelToken: searchRequestCancelSource.token,
242242
})
243243
.then((response) => {
244-
if (Object.keys(response.data.ocs.data.users).length > 0) {
244+
const usersCount = Object.keys(response.data.ocs.data.users).length
245+
if (usersCount > 0) {
245246
context.commit('appendUsers', response.data.ocs.data.users)
246-
return Object.keys(response.data.ocs.data.users).length >= limit
247247
}
248-
return false
248+
return usersCount
249249
})
250250
.catch((error) => {
251251
if (!axios.isCancel(error)) {

0 commit comments

Comments
 (0)