who messed up?

diff --git a/public/alert.js b/public/alert.js
index b83146567663a071e8cde9bba7656578177596c5..e4901cefd31c3cc09d609632586f555a8ed8838c 100644 (file)
--- a/public/alert.js
+++ b/public/alert.js
@@ -220,7 +220,7 @@
         for (const key of options.choices) {
             [].push.call(alertSelect.options, new Option(key, key));
         }
-        alertSelect.value = defaultValue;
+        alertSelect.value = alertInput.value = defaultValue;
 
         divAlertContent.textContent = msg;
         divAlertInput.style.display = "";

diff --git a/public/trivabble.js b/public/trivabble.js
index 0914a0cff459c4054e8bfc9fcce3590ad959abc7..7e0ede41e033f994cdd735a1d3f45f7abaa81215 100644 (file)
--- a/public/trivabble.js
+++ b/public/trivabble.js
@@ -723,6 +723,14 @@
     }
 
     function setCell(index, letter, highlight) {
+        if (lastPlayer) {
+            if (letter) {
+                currentTilePlayed[index] = letter;
+            } else {
+                delete currentTilePlayed[index];
+            }
+        }
+
         setTileParent(boardCells[index].getElementsByClassName("tile-placeholder")[0], letter, highlight);
     }
 
@@ -914,7 +922,6 @@
             tablePlayers[currentPlayer].classList.remove("current-player");
         }
 
-        currentTilePlayed = {};
         currentPlayer = player;
         refreshCurrentPlayer();
     }
@@ -1049,6 +1056,15 @@
         refreshCurrentPlayer();
     }
 
+    function handleLastPlayer(data) {
+        if (data.player) {
+            if (lastPlayer !== data.player) {
+                currentTilePlayed = {};
+            }
+            lastPlayer = data.player;
+        }
+    }
+
     function applyAction(data) {
         switch (data.action) {
             case "pushBag": //TODO
@@ -1065,9 +1081,12 @@
                 sendCmds([{cmd: "hello"}]);
 
                 lastPlayer = null;
+                currentTilePlayed = {};
                 break;
 
             case "moveLetter":
+                handleLastPlayer(data);
+
                 if (data.from === "board") {
                     setCell(data.indexFrom, "");
                 } else if (data.from === "rack") {
@@ -1075,27 +1094,32 @@
                 }
 
                 if (data.to === "board") {
-                    setCell(data.indexTo, data.letter, Object.prototype.hasOwnProperty.call(data, "player") && data.player !== getSetting("PlayerName"));
+                    setCell(
+                        data.indexTo,
+                        data.letter,
+                        data.player && data.player !== getSetting("PlayerName")
+                    );
                 } else if (data.to === "rack") {
                     setRackCell(data.indexTo, data.letter);
                 }
 
-                if (data.player) {
-                    lastPlayer = data.player;
-                }
-
                 break;
+
             case "setCell":
-                setCell(data.indexTo, data.letter, Object.prototype.hasOwnProperty.call(data, "player") && data.player !== getSetting("PlayerName"));
+                handleLastPlayer(data);
 
-                if (data.player) {
-                    lastPlayer = data.player;
-                }
+                setCell(
+                    data.indexTo,
+                    data.letter,
+                    data.player && data.player !== getSetting("PlayerName")
+                );
 
                 if ((data.letter !== "") && (currentTilePlayed[data.indexTo] === "-")) {
                     currentTilePlayed[data.indexTo] = data.letter;
                 }
+
                 break;
+
             case "setRackCell":
                 setRackCell(data.indexTo, data.letter);
         }
@@ -1906,6 +1930,8 @@
                     return;
                 }
 
+                currentTilePlayed = {};
+
                 sendCmds([{
                     cmd: "score",
                     player: name,