From e6fbf7aa5b230733ffd92fdfe851b9f86001c859 Mon Sep 17 00:00:00 2001
From: Philipp Dormann <philipp.dormann1@gmail.com>
Date: Tue, 12 Jan 2021 21:19:12 +0100
Subject: [PATCH] UserDetail - fixed group updating

ref #12
---
 src/components/UserDetail.svelte | 26 +++++++-------------------
 1 file changed, 7 insertions(+), 19 deletions(-)

diff --git a/src/components/UserDetail.svelte b/src/components/UserDetail.svelte
index a9c77d80..2dc058d0 100644
--- a/src/components/UserDetail.svelte
+++ b/src/components/UserDetail.svelte
@@ -26,6 +26,9 @@
     original_data = Object.assign(original_data, data);
     editable_userdata = data;
     usergroups_array = [];
+    data.groups.forEach((g) => {
+      usergroups_array.push(g.id);
+    });
     allgroups.forEach((g) => {
       allgroups_ids.push(g.id);
     });
@@ -42,26 +45,11 @@
   });
   $: changes_performed = !lodashIsEqual(original_data, editable_userdata);
   function updateGroups() {
-    console.log("updating groups...");
-    console.log(editable_userdata.groups);
-    allgroups.forEach((g) => {
-      const exists_in_currentstate = editable_userdata.groups.some(
-        (obj) => obj.id === g.id
-      );
-      console.log("exists:" + exists_in_currentstate);
-      if (exists_in_currentstate === true) {
-        editable_userdata.groups = editable_userdata.groups.filter(function (
-          obj
-        ) {
-          return obj.id !== g.id;
-        });
-      } else {
-        console.log("add to arr:" + g.id);
-        editable_userdata.groups.push(g);
-      }
+    usergroups_array.forEach((id) => {
+      const group = allgroups.find((e) => e.id === id);
+      editable_userdata.groups.push(group);
     });
-    console.log("new groups:");
-    console.log(editable_userdata.groups);
+    editable_userdata.groups = editable_userdata.groups;
   }
   function submit() {
     if (