Merge pull request #25879 from nextcloud/fix/user-list-infinite-loading

Fix user list infinite loading state in user settings

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()
+					}
 				})
 		},
 

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)) {