From a5e2c7dd3876e7e582ef7de2a0ec3a9864d54269 Mon Sep 17 00:00:00 2001
From: Christopher Ng <chrng8@gmail.com>
Date: Tue, 25 Mar 2025 14:31:45 -0700
Subject: [PATCH] fix(settings): Fix duplicated group options in new account
 dialog

Signed-off-by: Christopher Ng <chrng8@gmail.com>
---
 .../src/components/Users/NewUserDialog.vue    | 27 +++++++++++--------
 1 file changed, 16 insertions(+), 11 deletions(-)

diff --git a/apps/settings/src/components/Users/NewUserDialog.vue b/apps/settings/src/components/Users/NewUserDialog.vue
index 6d0c4455dc7..8b345ea748a 100644
--- a/apps/settings/src/components/Users/NewUserDialog.vue
+++ b/apps/settings/src/components/Users/NewUserDialog.vue
@@ -71,8 +71,7 @@
 					:multiple="true"
 					:taggable="true"
 					:required="!settings.isAdmin && !settings.isDelegatedAdmin"
-					:create-option="(value) => ({ name: value })"
-					@input="handleGroupInput"
+					:create-option="(value) => ({ id: value, name: value, isCreating: true })"
 					@search="searchGroups"
 					@option:created="createGroup" />
 					<!-- If user is not admin, he is a subadmin.
@@ -271,15 +270,6 @@ export default {
 			}
 		},
 
-		handleGroupInput(groups) {
-			/**
-			 * Filter out groups with no id to prevent duplicate selected options
-			 *
-			 * Created groups are added programmatically by `createGroup()`
-			 */
-			 this.newUser.groups = groups.filter(group => Boolean(group.id))
-		},
-
 		async searchGroups(query, toggleLoading) {
 			if (this.promise) {
 				this.promise.cancel()
@@ -318,6 +308,21 @@ export default {
 			this.loading.groups = false
 		},
 
+		/**
+		 * Add user to group
+		 *
+		 * @param {object} group Group object
+		 */
+		async addGroup(group) {
+			if (group.isCreating) {
+				return
+			}
+			if (group.canAdd === false) {
+				return
+			}
+			this.newUser.groups.push(group)
+		},
+
 		/**
 		 * Validate quota string to make sure it's a valid human file size
 		 *