From c11689b5b28afb4ad555d137a05be14ad8f0fa86 Mon Sep 17 00:00:00 2001 From: Julien Veyssier Date: Mon, 23 Aug 2021 10:54:13 +0200 Subject: [PATCH] avoid 'No users in here' in user list when there actually are users Signed-off-by: Julien Veyssier --- apps/settings/src/components/UserList.vue | 9 +++++++-- apps/settings/src/store/users.js | 12 ++++++------ 2 files changed, 13 insertions(+), 8 deletions(-) diff --git a/apps/settings/src/components/UserList.vue b/apps/settings/src/components/UserList.vue index afbc1de76f877..68c07d1ab2178 100644 --- a/apps/settings/src/components/UserList.vue +++ b/apps/settings/src/components/UserList.vue @@ -461,8 +461,13 @@ export default { group: this.selectedGroup !== 'disabled' ? this.selectedGroup : '', search: this.searchQuery, }) - .then((response) => { - response ? $state.loaded() : $state.complete() + .then((usersCount) => { + if (usersCount > 0) { + $state.loaded() + } + if (usersCount < this.usersLimit) { + $state.complete() + } }) }, diff --git a/apps/settings/src/store/users.js b/apps/settings/src/store/users.js index 475486d6c402c..ef93f7e40abf1 100644 --- a/apps/settings/src/store/users.js +++ b/apps/settings/src/store/users.js @@ -224,11 +224,11 @@ const actions = { cancelToken: searchRequestCancelSource.token, }) .then((response) => { - if (Object.keys(response.data.ocs.data.users).length > 0) { + const usersCount = Object.keys(response.data.ocs.data.users).length + if (usersCount > 0) { context.commit('appendUsers', response.data.ocs.data.users) - return Object.keys(response.data.ocs.data.users).length >= limit } - return false + return usersCount }) .catch((error) => { if (!axios.isCancel(error)) { @@ -241,11 +241,11 @@ const actions = { cancelToken: searchRequestCancelSource.token, }) .then((response) => { - if (Object.keys(response.data.ocs.data.users).length > 0) { + const usersCount = Object.keys(response.data.ocs.data.users).length + if (usersCount > 0) { context.commit('appendUsers', response.data.ocs.data.users) - return Object.keys(response.data.ocs.data.users).length >= limit } - return false + return usersCount }) .catch((error) => { if (!axios.isCancel(error)) {