From 94e0c178f03f512a84c2e75304b5ebaa8caafe6c Mon Sep 17 00:00:00 2001
From: MaxJa4 <74194322+MaxJa4@users.noreply.github.com>
Date: Tue, 16 Jan 2024 20:54:49 +0100
Subject: [PATCH] Implement edit-player. Optimizations.

---
 .gitignore                          |   1 +
 controllers/player_controller.go    |   6 +--
 isc_data.db                         | Bin 16384 -> 0 bytes
 main.go                             |   1 +
 models/player.go                    |   1 +
 static/index.js                     |   8 ++-
 templates/index.html                |   2 +
 templates/modals/add_clan.html      |   7 +--
 templates/modals/add_player.html    |   5 +-
 templates/modals/delete_clan.html   |   7 +--
 templates/modals/delete_player.html |   7 +--
 templates/modals/edit_clan.html     |  13 ++---
 templates/modals/edit_player.html   |  75 ++++++++++++++++++++++++++++
 templates/player_list_item.html     |   2 +-
 14 files changed, 100 insertions(+), 35 deletions(-)
 delete mode 100644 isc_data.db
 create mode 100644 templates/modals/edit_player.html

diff --git a/.gitignore b/.gitignore
index e4194d8..2713e6a 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1 +1,2 @@
 /isc_rest.log
+/isc_data.db
diff --git a/controllers/player_controller.go b/controllers/player_controller.go
index eb0f95e..8940b1f 100644
--- a/controllers/player_controller.go
+++ b/controllers/player_controller.go
@@ -46,7 +46,7 @@ func GetPlayersByClanHTML(c *gin.Context) {
 
 	var htmlOptions string
 	for _, player := range players {
-		htmlOptions += fmt.Sprintf(playerItem, player.Name, player.ID)
+		htmlOptions += fmt.Sprintf(playerItem, player.Name, player.ID, player.ID)
 	}
 
 	c.Header("Content-Type", "text/html")
@@ -124,13 +124,13 @@ func UpdatePlayerByID(c *gin.Context) {
 		}
 	}
 
-	msg := "Updating player '" + player.Name + "'#" + strconv.FormatUint(uint64(player.ID), 10)
+	msg := "Updating player '" + player.Name + "' #" + strconv.FormatUint(uint64(player.ID), 10)
 	if player.Name != input.Name {
 		msg += " (new: '" + input.Name + "')"
 	}
 	msg += " with clan #" + utils.UintToString(player.ClanID)
 	if player.ClanID != input.ClanID {
-		msg += " (new: ä" + utils.UintToString(input.ClanID) + ")"
+		msg += " (new: #" + utils.UintToString(input.ClanID) + ")"
 	}
 	_, err := fmt.Fprintf(utils.GinWriter, msg+"\n")
 	if err != nil {
diff --git a/isc_data.db b/isc_data.db
deleted file mode 100644
index ce19e86199a826b2198cd5b65b44a341120eeee7..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 16384
zcmeI2&u<$=6vt=mwY{$Kj-6!LE}N+bwV`zzs(9Cq6ix)Ns){zWsS(O4-g<YVKw^^Y
zMlFI%{sZnkaOfQ_h#SZDjKm3)0|yQqxNt!9$a`bE#8uTIq)JqJ>)oAS@6G$Z&-k*M
zxpni}b{4DT(}UeGQvs_n&e_{aF~;t}W56@Ha!@(h=97VEl=s->kAJ9wLEid?iO)q7
z3c82@5g-CYfCvx)B0vO)01+SpM1Tl9kHFo0$?-gXbR`RK@5FmM;b-wdIxdWN*So8|
zuIjD6bFHiT<KDiy(BF>wYI~T)Tk(Oqv3~XX>iULyzq_GUKkR*Qbq(CEch`E0{b9Hp
z_f;0(%@+HEop88$!moKFS9C62;u|_}y1xTo&6{bwe;5x3@e|Gbl#nL|FJP2K@N6FC
zc*$`bel*Z?V#HLxa#Q|VO8Otu$il5jMxVs--sa(66lO8t7#{A%2it=tew)KbKNAnc
z18eQy{hqf-G)@GF01+SpM1Tko0U|&IhyW2F0xy8TnfU?>Tr*6gBn_h^iISEdv;w~s
zVCe^QUV(X1kB4b8F|+Ed<Sm$WSzxYff*ClB>>1<V!bqA#Xghgu>iN?Jb{0mUsw1Ug
z8bY`z4bckxzBoU#48dFznjJrALpR*wnBRmLQL+-ID^a?#6eeQ8j$LU6?GCEecbHf|
zv$sH~o*pM134mVP^H9zxt`1#o6_{=7oKVZ$Qh`aZ!8!=ihEXEkV0G7qAU?<d9jSG>
zz-C?ZPP>J|q7+oTCmk&|!1=*5u@@mn$6W4!HZU1w;0Ivp%Gdu%{jt2-MNs0INv9p)
zoM2yT7MP<E_-&XH)<GOIx6!~jFpTP88K8`kne#y3GXo#}@PYLL<FiNs4Mzkpoyr18
zpT&eBat#*Ev&}3{GgvY#Y*k<l01!r_Wrsv?e)zYYtDe)SvUqndn*qnhW5>WpSN#6Z
ziQky`6JB%?0U|&IhyW2F0z`la5CI}U1c(3;AObHnfl4m#@$r#}Mw#awu8$flSH&O9
zweEBAwfIVWDUL)Yw!}x`iuhIhBp!+-;fn9YBT*M+!L7fn->qM)AFYShch>!v8V89d
z0z`la5CI}U1c(3;AOb{y2oQn)AAx$qU<({}w(*87@>Za%GX`t$(SZectBN$3i*2w`
zmHBa_-C0(rJcBtn((b78YX-A*U74=IX0fV@rwt}}AdAkpd=*Py+Or0Glj|M(2zKIO
z&mOkaLut=EwcDzqpd{YE6T>NhL|U&*w<{IY1fy`v%Bd5LYT00t>oW_m(J!m!bhD(J
yFeM%bkk#T;vp9!ZTQaQI5fSd^r?I{>S*~HS*d;3##v2_TP_Vuw$$Le6l;+<e`+dCt

diff --git a/main.go b/main.go
index fc7b885..2538605 100644
--- a/main.go
+++ b/main.go
@@ -40,6 +40,7 @@ func main() {
 			"./templates/modals/edit_clan.html",
 			"./templates/modals/add_player.html",
 			"./templates/modals/delete_player.html",
+			"./templates/modals/edit_player.html",
 			"./templates/components/header.html",
 		}
 		tmpl, err := template.ParseFiles(files...)
diff --git a/models/player.go b/models/player.go
index 4f65b5c..e042481 100644
--- a/models/player.go
+++ b/models/player.go
@@ -4,4 +4,5 @@ type Player struct {
 	ID     uint   `json:"id" gorm:"primary_key"`
 	Name   string `json:"name"`
 	ClanID uint   `json:"clan_id"`
+	Clan   Clan   `gorm:"references:ID"`
 }
diff --git a/static/index.js b/static/index.js
index 075d833..d4bdf3f 100644
--- a/static/index.js
+++ b/static/index.js
@@ -44,7 +44,9 @@ function getClanLists(event) {
 
 function getPlayerLists(event) {
     const button = event.relatedTarget;
-    const playerListId = button.getAttribute('data-bs-list');
+    let playerListId = button.getAttribute('data-bs-list');
+    if (playerListId === null)
+        playerListId = '#' + button.closest('ul').parentElement.parentElement.id;
     const playerList = document.querySelector(playerListId);
 
     let otherPlayerListId;
@@ -59,7 +61,9 @@ function getPlayerLists(event) {
 
 function getSelectedClan(event) {
     const button = event.relatedTarget;
-    const playerListId = button.getAttribute('data-bs-list');
+    let playerListId = button.getAttribute('data-bs-list');
+    if (playerListId === null)
+        playerListId = '#' + button.closest('ul').parentElement.parentElement.id;
 
     let clanListId;
     if (playerListId === '#home-player-list')
diff --git a/templates/index.html b/templates/index.html
index 7be7381..564cd1a 100644
--- a/templates/index.html
+++ b/templates/index.html
@@ -42,6 +42,8 @@
 {{ template "add_player" . }}
 <!-- Delete Player Modal -->
 {{ template "delete_player" . }}
+<!-- Edit Player Modal -->
+{{ template "edit_player" . }}
 
 <script lang="javascript">
     document.addEventListener('DOMContentLoaded', function() {
diff --git a/templates/modals/add_clan.html b/templates/modals/add_clan.html
index 000c109..db26367 100644
--- a/templates/modals/add_clan.html
+++ b/templates/modals/add_clan.html
@@ -36,10 +36,7 @@
         if (addClanModal) {
             addClanModal.addEventListener('shown.bs.modal', event => {
                 const submitButton = addClanModal.querySelector('button[name="submit"]');
-                submitButton.addEventListener('click', function (e) {
-                    e.preventDefault();
-                    submitButton.onclick = function () {}
-
+                submitButton.addEventListener('click', function () {
                     const [clanList, otherClanList] = getClanLists(event);
 
                     const clanName = addClanModal.querySelector('#clanName');
@@ -89,7 +86,7 @@
                         .catch((error) => {
                             throw new Error(error)
                         });
-                })
+                }, { once: true });
             });
         }
     });
diff --git a/templates/modals/add_player.html b/templates/modals/add_player.html
index 08a87c2..56df7c4 100644
--- a/templates/modals/add_player.html
+++ b/templates/modals/add_player.html
@@ -41,10 +41,7 @@
                 const oppClanListIndex = document.getElementById('opponent-clan').selectedIndex;
 
                 const submitButton = addPlayerModal.querySelector('button[name="submit"]');
-                submitButton.addEventListener('click', function (e) {
-                    e.preventDefault();
-                    submitButton.onclick = function () {}
-
+                submitButton.addEventListener('click', function () {
                     fetch("/player", {
                         method: "POST",
                         body: JSON.stringify({
diff --git a/templates/modals/delete_clan.html b/templates/modals/delete_clan.html
index bd0782d..14820ef 100644
--- a/templates/modals/delete_clan.html
+++ b/templates/modals/delete_clan.html
@@ -33,10 +33,7 @@
                 modalBodyInput.innerText = selectedClan;
 
                 const submitButton = deleteClanModal.querySelector('button[name="submit"]');
-                submitButton.addEventListener('click', function (e) {
-                    e.preventDefault();
-                    submitButton.onclick = function () {}
-
+                submitButton.addEventListener('click', function () {
                     const clanId = parseInt(clanList.value);
 
                     fetch("/clan/" + clanId, {
@@ -69,7 +66,7 @@
                         .catch((error) => {
                             throw new Error(error)
                         });
-                })
+                }, { once: true });
             });
         }
     });
diff --git a/templates/modals/delete_player.html b/templates/modals/delete_player.html
index 88eec07..c24c963 100644
--- a/templates/modals/delete_player.html
+++ b/templates/modals/delete_player.html
@@ -38,10 +38,7 @@
                 const oppClanListIndex = document.getElementById('opponent-clan').selectedIndex;
 
                 const submitButton = deletePlayerModal.querySelector('button[name="submit"]');
-                submitButton.addEventListener('click', function (e) {
-                    e.preventDefault();
-                    submitButton.onclick = function () {}
-
+                submitButton.addEventListener('click', function () {
                     fetch("/player/" + playerId, {
                         method: "DELETE",
                         headers: {
@@ -60,7 +57,7 @@
                         .catch((error) => {
                             throw new Error(error)
                         });
-                })
+                }, { once: true });
             });
         }
     });
diff --git a/templates/modals/edit_clan.html b/templates/modals/edit_clan.html
index e09b920..993a204 100644
--- a/templates/modals/edit_clan.html
+++ b/templates/modals/edit_clan.html
@@ -4,7 +4,7 @@
     <div class="modal-dialog modal-dialog-centered">
         <div class="modal-content">
             <div class="modal-header">
-                <h1 class="modal-title fs-3 text-success fw-bold" id="editClanModalLabel">Clan bearbeiten</h1>
+                <h1 class="modal-title fs-3 text-primary fw-bold" id="editClanModalLabel">Clan bearbeiten</h1>
                 <button type="button" class="btn-close" data-bs-dismiss="modal"></button>
             </div>
             <div class="modal-body">
@@ -32,17 +32,12 @@
 
 <script lang="javascript">
     document.addEventListener('DOMContentLoaded', function() {
-
         const editClanModal = document.getElementById('editClanModal');
         const editClanModalBS = new bootstrap.Modal('#editClanModal');
         if (editClanModal) {
             editClanModal.addEventListener('show.bs.modal', event => {
                 const [clanList, otherClanList] = getClanLists(event);
 
-                const selectedClanIndex = clanList.selectedIndex;
-                const selectedClanItem = clanList.options[selectedClanIndex];
-                const otherSelClanItem = otherClanList.options[selectedClanIndex];
-
                 const clanName = editClanModal.querySelector('#editClanName');
                 const clanTag = editClanModal.querySelector('#editClanTag');
                 const keepUpdated = editClanModal.querySelector('#editKeepUpdated');
@@ -67,9 +62,7 @@
                         throw new Error(error);
                     });
 
-                submitButton.addEventListener('click', function (e) {
-                    e.preventDefault();
-                    submitButton.onclick = function () {}
+                submitButton.addEventListener('click', function () {
                     fetch("/clan/" + clanId, {
                         method: "PATCH",
                         body: JSON.stringify({
@@ -111,7 +104,7 @@
                         .catch((error) => {
                             throw new Error(error)
                         });
-                })
+                }, { once: true });
             });
         }
     });
diff --git a/templates/modals/edit_player.html b/templates/modals/edit_player.html
new file mode 100644
index 0000000..a408862
--- /dev/null
+++ b/templates/modals/edit_player.html
@@ -0,0 +1,75 @@
+{{ define "edit_player" }}
+
+<div class="modal fade" id="editPlayerModal" tabindex="-1">
+    <div class="modal-dialog modal-dialog-centered">
+        <div class="modal-content">
+            <div class="modal-header">
+                <h1 class="modal-title fs-3 text-primary fw-bold" id="editPlayerModalLabel">Spieler bearbeiten</h1>
+                <button type="button" class="btn-close" data-bs-dismiss="modal"></button>
+            </div>
+            <div class="modal-body">
+                <div class="form-floating mb-3">
+                    <input type="text" class="form-control form-control-lg" id="editPlayerName" placeholder="Spieler-Name">
+                    <label for="editPlayerName">Spieler-Name</label>
+                </div>
+                <input type="hidden" id="editPlayerId" value="">
+            </div>
+            <div class="modal-footer">
+                <button type="submit" name="submit" class="btn btn-lg btn-primary">Speichern</button>
+                <button type="button" class="btn btn-lg btn-secondary" data-bs-dismiss="modal">Abbrechen</button>
+            </div>
+        </div>
+    </div>
+</div>
+
+<script lang="javascript">
+    document.addEventListener('DOMContentLoaded', function() {
+        const editPlayerModal = document.getElementById('editPlayerModal');
+        const editPlayerModalBS = new bootstrap.Modal('#editPlayerModal');
+        if (editPlayerModal) {
+            editPlayerModal.addEventListener('show.bs.modal', event => {
+                const [playerList, otherPlayerList] = getPlayerLists(event);
+                const selectedClan = getSelectedClan(event);
+                const playerName = editPlayerModal.querySelector('#editPlayerName');
+                const clanId = parseInt(selectedClan.value);
+                const homeClanListIndex = document.getElementById('home-clan').selectedIndex;
+                const oppClanListIndex = document.getElementById('opponent-clan').selectedIndex;
+
+                const button = event.relatedTarget;
+                const playerId = parseInt(button.getAttribute('data-bs-id'));
+                playerName.value = button.closest('.input-group').querySelector('span').innerText;
+
+                const submitButton = editPlayerModal.querySelector('button[name="submit"]');
+                submitButton.addEventListener('click', function (e) {
+                    e.preventDefault();
+                    submitButton.onclick = function () {}
+
+                    fetch("/player/" + playerId, {
+                        method: "PATCH",
+                        body: JSON.stringify({
+                            name: playerName.value,
+                            clan_id: clanId
+                        }),
+                        headers: {
+                            "Content-type": "application/json; charset=UTF-8"
+                        }
+                    })
+                        .then(() => {
+                            const sameClan = homeClanListIndex === oppClanListIndex;
+                            if (playerList.id === 'home-player-list' || sameClan)
+                                htmx.ajax('GET', '/players_html', {target: '#home-player-list', values: {"clan_id": getSelectedClanId("home-clan")}});
+                            if (playerList.id === 'opponent-player-list' || sameClan)
+                                htmx.ajax('GET', '/players_html', {target: '#opponent-player-list', values: {"clan_id": getSelectedClanId("opponent-clan")}});
+
+                            editPlayerModalBS.hide();
+                            playerName.value = "";
+                        }).catch((error) => {
+                            throw new Error(error)
+                        });
+                }, { once: true });
+            });
+        }
+    });
+</script>
+
+{{ end }}
\ No newline at end of file
diff --git a/templates/player_list_item.html b/templates/player_list_item.html
index b06d9b3..d4db368 100644
--- a/templates/player_list_item.html
+++ b/templates/player_list_item.html
@@ -5,7 +5,7 @@
     <span class="form-control py-1 px-2">%s</span>
     <button class="btn btn-outline-secondary text-secondary-emphasis dropdown-toggle py-1" type="button" data-bs-toggle="dropdown"></button>
     <ul class="dropdown-menu dropdown-menu-end">
-        <li><a class="dropdown-item text-primary fs-5" href="#">
+        <li><a class="dropdown-item text-primary fs-5" href="#" data-bs-toggle="modal" data-bs-id="%d" data-bs-target="#editPlayerModal">
                 <i class="bi bi-person-gear fs-4 me-2"></i>Bearbeiten
         </a></li>
         <li><a class="dropdown-item text-danger fs-5" href="#" data-bs-toggle="modal" data-bs-id="%d" data-bs-target="#deletePlayerModal">