Attempt to fix the non-transmitted moves issue

[vchess.git] / public / javascripts / components / game.js

diff --git a/public/javascripts/components/game.js b/public/javascripts/components/game.js
index 81ced5b..42e6507 100644 (file)
--- a/public/javascripts/components/game.js
+++ b/public/javascripts/components/game.js
@@ -16,6 +16,7 @@ Vue.component('my-game', {
                        oppConnected: false,
                        seek: false,
                        fenStart: "",
+                       incheck: [],
                };
        },
        render(h) {
@@ -23,6 +24,9 @@ Vue.component('my-game', {
                // Precompute hints squares to facilitate rendering
                let hintSquares = doubleArray(sizeX, sizeY, false);
                this.possibleMoves.forEach(m => { hintSquares[m.end.x][m.end.y] = true; });
+               // Also precompute in-check squares
+               let incheckSq = doubleArray(sizeX, sizeY, false);
+               this.incheck.forEach(sq => { incheckSq[sq[0]][sq[1]] = true; });
                let elementArray = [];
                let square00 = document.getElementById("sq-0-0");
                let squareWidth = !!square00
@@ -35,6 +39,8 @@ Vue.component('my-game', {
                                {
                                        on: {
                                                click: () => {
+                                                       if (this.mode == "human")
+                                                               return; //no newgame while playing
                                                        if (this.seek)
                                                                delete localStorage["newgame"]; //cancel game seek
                                                        else
@@ -55,7 +61,13 @@ Vue.component('my-game', {
                                [h('i', { 'class': { "material-icons": true } }, "accessibility")]),
                        h('button',
                                {
-                                       on: { click: () => this.newGame("computer") },
+                                       on: {
+                                               click: () => {
+                                                       if (this.mode == "human")
+                                                               return; //no newgame while playing
+                                                       this.newGame("computer");
+                                               }
+                                       },
                                        attrs: { "aria-label": 'New game VS computer' },
                                        'class': {
                                                "tooltip":true,
@@ -162,7 +174,7 @@ Vue.component('my-game', {
                                                                );
                                                        }
                                                        const lm = this.vr.lastMove;
-                                                       const highlight = !!lm && _.isMatch(lm.end, {x:ci,y:cj}); //&& _.isMatch(lm.start, {x:ci,y:cj})
+                                                       const highlight = !!lm && _.isMatch(lm.end, {x:ci,y:cj});
                                                        return h(
                                                                'div',
                                                                {
@@ -171,6 +183,7 @@ Vue.component('my-game', {
                                                                                'light-square': !highlight && (i+j)%2==0,
                                                                                'dark-square': !highlight && (i+j)%2==1,
                                                                                'highlight': highlight,
+                                                                               'incheck': incheckSq[ci][cj],
                                                                        },
                                                                        attrs: {
                                                                                id: this.getSquareId({x:ci,y:cj}),
@@ -365,7 +378,7 @@ Vue.component('my-game', {
                        {
                                // New game request has been cancelled on disconnect
                                this.seek = true;
-                               this.newGame("human");
+                               this.newGame("human", "reconnect");
                        }
                };
                const socketMessageListener = msg => {
@@ -453,6 +466,13 @@ Vue.component('my-game', {
                        this.score = "*";
                        if (mode=="human" && !oppId)
                        {
+                               const storageVariant = localStorage.getItem("variant");
+                               if (!!storageVariant && storageVariant !== variant)
+                               {
+                                       // TODO: find a better way to ensure this. Newgame system is currently a mess.
+                                       alert("Finish your " + storageVariant + " game first!");
+                                       return;
+                               }
                                // Send game request and wait..
                                this.clearStorage(); //in case of
                                try {
@@ -460,13 +480,17 @@ Vue.component('my-game', {
                                } catch (INVALID_STATE_ERR) {
                                        return; //nothing achieved
                                }
-                               let modalBox = document.getElementById("modal-control2");
-                               modalBox.checked = true;
-                               setTimeout(() => { modalBox.checked = false; }, 2000);
+                               if (!fenInit || fenInit!="reconnect") //TODO: bad HACK...
+                               {
+                                       let modalBox = document.getElementById("modal-control2");
+                                       modalBox.checked = true;
+                                       setTimeout(() => { modalBox.checked = false; }, 2000);
+                               }
                                return;
                        }
                        this.vr = new VariantRules(fen, moves || []);
                        this.mode = mode;
+                       this.incheck = []; //in case of
                        this.fenStart = continuation
                                ? localStorage.getItem("fenStart")
                                : fen.split(" ")[0]; //Only the position matters
@@ -483,6 +507,14 @@ Vue.component('my-game', {
                                this.oppConnected = true;
                                this.mycolor = color;
                                this.seek = false;
+                               if (!!moves && moves.length > 0) //imply continuation
+                               {
+                                       const oppCol = this.vr.turn;
+                                       const lastMove = moves[moves.length-1];
+                                       this.vr.undo(lastMove, "ingame");
+                                       this.incheck = this.vr.getCheckSquares(lastMove, oppCol);
+                                       this.vr.play(lastMove, "ingame");
+                               }
                                delete localStorage["newgame"];
                                this.setStorage(); //in case of interruptions
                        }
@@ -610,6 +642,8 @@ Vue.component('my-game', {
                                this.animateMove(move);
                                return;
                        }
+                       const oppCol = this.vr.getOppCol(this.vr.turn);
+                       this.incheck = this.vr.getCheckSquares(move, oppCol); //is opponent in check?
                        // Not programmatic, or animation is over
                        if (this.mode == "human" && this.vr.turn == this.mycolor)
                        {