Merge pull request #96 from fedecarboni7/dev

fix deletion logic for created players
fedecarboni7 · Oct 24, 2024 · fe88652 · fe88652
2 parents 1b8ae66 + b0ad8e7
commit fe88652
Showing 1 changed file with 10 additions and 7 deletions.
diff --git a/static/js/script-v1.0.7.js b/static/js/script-v1.0.7.js
@@ -364,11 +364,13 @@ function addPlayer() {
     deleteButton.appendChild(trashIcon);
 
     deleteButton.addEventListener("click", function() {
-        deletePlayer(this);
-        container.removeChild(playerDiv);
-        renumerarJugadores();
-        updateSelectedCount();
-        updateToggleButtonText();
+        response = deletePlayer(deleteButton);
+        if (response) {
+            container.removeChild(playerDiv);
+            renumerarJugadores();
+            updateSelectedCount();
+            updateToggleButtonText();
+        }
     });
     playerHeader.appendChild(deleteButton);
 
@@ -500,9 +502,10 @@ function deletePlayer(button) {
                 container = document.getElementById("players-container");
                 container.removeChild(button.parentNode.parentNode);
         });
-
-    updateSelectedCount();
+        updateSelectedCount();
+        return true;
     }
+    return false;
 }
 
 // Actualizar el texto del botón de seleccionar/deseleccionar según el estado actual de los checkboxes