From 1ecd09881f33f0d8a60dd4b5cc75f5065a7e3dff Mon Sep 17 00:00:00 2001
From: Tony <anthony@correa.co>
Date: Thu, 9 Jun 2022 17:19:51 -0500
Subject: [PATCH] fixed bug where dataset didn't update (so threw off copying
 to google sheet format)

---
 benchcoach/static/js/project.js | 35 +++------------------------------
 teamsnap/forms.py               |  2 +-
 2 files changed, 4 insertions(+), 33 deletions(-)

diff --git a/benchcoach/static/js/project.js b/benchcoach/static/js/project.js
index 78f78c0..5d43766 100644
--- a/benchcoach/static/js/project.js
+++ b/benchcoach/static/js/project.js
@@ -1,37 +1,8 @@
 /* Project specific Javascript goes here. */
 function positionSelectChanged(elem) {
-    let row = elem.parentElement.parentElement
-    let original_table = elem.parentElement.parentElement.parentElement
-    let lineup_table = document.getElementById('lineup')
-    let bench_table = document.getElementById('bench')
-    let dhd_table = document.getElementById('dhd')
-    let player_order = row.querySelector('[id^="player-order"]')
-    let player_available = row.querySelector('[id^="player-availability"]')
-
-    console.dir(original_table == bench_table)
-    if (elem.value == "P" && original_table == bench_table ) {
-        dhd_table.appendChild(row)
-        player_order.classList.remove('d-none')
-        // player_available.classList.add('d-none')
-        refresh_lineup_order()
-    }
-    else if (elem.value && original_table == dhd_table ) {
-        dhd_table.appendChild(row)
-        player_order.classList.remove('d-none')
-        // player_available.classList.add('d-none')
-        refresh_lineup_order()
-    }
-    else if (elem.value) {
-        lineup_table.appendChild(row)
-        player_order.classList.remove('d-none')
-        // player_available.classList.add('d-none')
-        refresh_lineup_order()
-    }
-    else {
-        bench_table.prepend(row)
-        // player_order.classList.add('d-none')
-        player_available.classList.remove('d-none')
-    }
+    let row = elem.closest("tr")
+    row.dataset.position=elem.value
+    colorPositions()
 }
 
 function colorPositions() {
diff --git a/teamsnap/forms.py b/teamsnap/forms.py
index d263989..c784108 100644
--- a/teamsnap/forms.py
+++ b/teamsnap/forms.py
@@ -45,7 +45,7 @@ class LineupEntryForm(forms.Form):
             ("DR", "DR"),
             ("EH", "EH"),
         ],
-        widget=forms.Select(attrs={"onchange": "colorPositions();"}),
+        widget=forms.Select(attrs={"onchange": "positionSelectChanged(this);"}),
     )