From 2adcc754137e0b31518693c3e823eb40548403d7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julius=20H=C3=A4rtl?= <jus@bitgrid.net>
Date: Fri, 11 Oct 2019 16:52:29 +0200
Subject: [PATCH] Fetch groups on the fly
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Julius Härtl <jus@bitgrid.net>
---
 .../components/Checks/RequestUserGroup.vue    | 88 +++++++++++++------
 1 file changed, 59 insertions(+), 29 deletions(-)

diff --git a/apps/workflowengine/src/components/Checks/RequestUserGroup.vue b/apps/workflowengine/src/components/Checks/RequestUserGroup.vue
index 0a548a83ba..f254a5185f 100644
--- a/apps/workflowengine/src/components/Checks/RequestUserGroup.vue
+++ b/apps/workflowengine/src/components/Checks/RequestUserGroup.vue
@@ -22,61 +22,91 @@
 
 <template>
 	<div>
-		<Multiselect v-model="newValue"
-			:class="{'icon-loading-small': groups.length === 0}"
+		<Multiselect :value="currentValue"
+			:loading="status.isLoading && groups.length === 0"
 			:options="groups"
 			:multiple="false"
 			label="displayname"
 			track-by="id"
-			@input="setValue" />
+			@search-change="searchAsync"
+			@input="(value) => $emit('input', value.id)" />
 	</div>
 </template>
 
 <script>
 import { Multiselect } from 'nextcloud-vue/dist/Components/Multiselect'
-import valueMixin from '../../mixins/valueMixin'
 import axios from '@nextcloud/axios'
+
+const groups = []
+const status = {
+	isLoading: false
+}
+
 export default {
 	name: 'RequestUserGroup',
 	components: {
 		Multiselect
 	},
-	mixins: [
-		valueMixin
-	],
-	data() {
-		return {
-			groups: []
+	props: {
+		value: {
+			type: String,
+			default: ''
+		},
+		check: {
+			type: Object,
+			default: () => { return {} }
 		}
 	},
-	beforeMount() {
-		axios.get(OC.linkToOCS('cloud', 2) + 'groups').then((response) => {
-			this.groups = response.data.ocs.data.groups.reduce((obj, item) => {
-				obj.push({
-					id: item,
-					displayname: item
-				})
-				return obj
-			}, [])
-			this.updateInternalValue(this.value)
-		}, (error) => {
-			console.error('Error while loading group list', error.response)
-		})
+	data() {
+		return {
+			groups: groups,
+			status: status
+		}
+	},
+	computed: {
+		currentValue() {
+			return this.groups.find(group => group.id === this.value) || null
+		}
+	},
+	async mounted() {
+		if (this.groups.length === 0) {
+			await this.searchAsync('')
+		}
+		if (this.currentValue === null) {
+			await this.searchAsync(this.value)
+		}
 	},
 	methods: {
-		updateInternalValue() {
-			this.newValue = this.groups.find(group => group.id === this.value) || null
+		searchAsync(searchQuery) {
+			if (this.status.isLoading) {
+				return
+			}
+
+			this.status.isLoading = true
+			return axios.get(OC.linkToOCS('cloud', 2) + 'groups?limit=20&search=' + encodeURI(searchQuery)).then((response) => {
+				response.data.ocs.data.groups.reduce((obj, item) => {
+					obj.push({
+						id: item,
+						displayname: item
+					})
+					return obj
+				}, []).forEach((group) => this.addGroup(group))
+				this.status.isLoading = false
+			}, (error) => {
+				console.error('Error while loading group list', error.response)
+			})
 		},
-		setValue(value) {
-			if (value !== null) {
-				this.$emit('input', this.newValue.id)
+		addGroup(group) {
+			const index = this.groups.findIndex((item) => item.id === group.id)
+			if (index === -1) {
+				this.groups.push(group)
 			}
 		}
 	}
 }
 </script>
 <style scoped>
-	.multiselect, input[type='text'] {
+	.multiselect {
 		width: 100%;
 	}
 </style>