From a91f19e72f1a320b10c45d7488ea9ca0ef4af7f6 Mon Sep 17 00:00:00 2001
From: "Yann Esposito (Yogsototh)" <yann.esposito@gmail.com>
Date: Mon, 6 Sep 2021 18:08:32 +0200
Subject: [PATCH] Prevent duplicate entries in state

---
 index.html | 27 ++++++++++++++++++---------
 1 file changed, 18 insertions(+), 9 deletions(-)

diff --git a/index.html b/index.html
index abef636..c80b674 100644
--- a/index.html
+++ b/index.html
@@ -48,7 +48,10 @@
 <script>
   // (function () {
 
-  function debug(args) { console.log(args); }
+  function debug(args) {
+      console.log(args);
+      return null;
+  }
 
   function enabledrag(event) { event.preventDefault(); }
 
@@ -257,10 +260,13 @@
   }
 
   function createAssignment(id, name) {
-          if (! state.assignments[id]) {
-              state.assignments[id]=[];
-          }
+      if (! state.assignments[id]) {
+          state.assignments[id]=[];
+      }
+      if ( state.assignments[id].indexOf(name) == -1 ) {
           state.assignments[id].push(name);
+      }
+      debug("createAssignment");
       debug(state.assignments);
   }
 
@@ -270,7 +276,10 @@
       if (! state.assignments[id]) {
           state.assignments[id]=[];
       }
-      state.assignments[id].push(memberName);
+      if ( state.assignments[id].indexOf(memberName) == -1 ) {
+          state.assignments[id].push(memberName);
+      }
+      debug("moveAssignment");
       debug(state.assignments);
   }
 
@@ -281,18 +290,17 @@
           debug("move: " + name);
           document.getElementById(name).remove();
           var memberName = name.replace(/^[^-]*-/,'').replace(/-cell-.*$/,'');
-          var newdevcard =devcard(memberName,el.id);
+          var newdevcard = devcard(memberName,el.id);
           if (! document.getElementById(newdevcard.id)) {
               el.appendChild(newdevcard);
               moveAssignment(el.id, name);
           }
       } else {
           debug("create: " + name);
-
           var newdevcard =devcard(name,el.id);
           if (! document.getElementById(newdevcard.id)) {
-          el.appendChild(newdevcard);
-          createAssignment(el.id, name);
+              el.appendChild(newdevcard);
+              createAssignment(el.id, name);
           }
       }
       saveState();
@@ -308,6 +316,7 @@
       if (idx !== -1) {
           state.assignments[oldid].splice(idx,1);
       }
+      debug("removeAssignment");
       debug(state.assignments);
   }