Fix play comp move in ComputerGame component

[vchess.git] / client / src / components / BaseGame.vue

diff --git a/client/src/components/BaseGame.vue b/client/src/components/BaseGame.vue
index ceadcd3..8285c0c 100644 (file)
--- a/client/src/components/BaseGame.vue
+++ b/client/src/components/BaseGame.vue
@@ -7,7 +7,7 @@
         label.modal-close(for="modalEog")
         h3#eogMessage.section {{ endgameMessage }}
     Board(:vr="vr" :last-move="lastMove" :analyze="analyze" :user-color="mycolor"
-      :orientation="orientation" :vname="variant.name" @play-move="play")
+      :orientation="orientation" :vname="vname" @play-move="play")
     .button-group
       button(@click="play") Play
       button(@click="undo") Undo
@@ -37,18 +37,16 @@ export default {
     Board,
     //MoveList,
   },
-  props: ["variant","analyze","players"],
+  // "vr": VariantRules object, describing the game state + rules
+  props: ["vname","analyze","vr","fenStart","players","mycolor"],
   data: function() {
     return {
       st: store.state,
-      vr: null, //VariantRules object, describing the game state + rules
+      // NOTE: all following variables must be reset at the beginning of a game
       endgameMessage: "",
       orientation: "w",
       score: "*", //'*' means 'unfinished'
-      // userColor: given by gameId, or fen in problems mode (if no game Id)...
-      mycolor: "w",
-      fenStart: "",
-      moves: [], //all moves played in current game
+      moves: [],
       cursor: -1, //index of the move just played
       lastMove: null,
     };
@@ -59,7 +57,7 @@ export default {
       //return window.innerWidth >= 768;
     },
     showFen: function() {
-      return this.variant.name != "Dark" || this.score != "*";
+      return this.vname != "Dark" || this.score != "*";
     },
   },
   methods: {
@@ -93,7 +91,7 @@ export default {
     getPgn: function() {
       let pgn = "";
       pgn += '[Site "vchess.club"]\n';
-      pgn += '[Variant "' + this.variant.name + '"]\n';
+      pgn += '[Variant "' + this.vname + '"]\n';
       pgn += '[Date "' + getDate(new Date()) + '"]\n';
       pgn += '[White "' + this.players[0] + '"]\n';
       pgn += '[Black "' + this.players[1] + '"]\n';
@@ -187,8 +185,6 @@ export default {
         move.fen = this.vr.getFen();
       if (this.st.settings.sound == 2)
         new Audio("/sounds/move.mp3").play().catch(err => {});
-      // Send the move to web worker (including his own moves)
-      this.compWorker.postMessage(["newmove",move]);
       if (!navigate && (this.score == "*" || this.analyze))
       {
         // Stack move on movesList at current cursor
@@ -207,15 +203,7 @@ export default {
         else //just show score on screen (allow undo)
           this.showScoreMsg(score);
       }
-      // subTurn condition for Marseille (and Avalanche) rules
-      else if ((this.mode == "computer" && (!this.vr.subTurn || this.vr.subTurn <= 1))
-        && (this.subMode == "auto" || this.vr.turn != this.mycolor))
-      {
-        this.playComputerMove();
-      }
-      // https://vuejs.org/v2/guide/list.html#Caveats (also for undo)
-      //if (navigate)
-      //  this.$children[0].$forceUpdate(); //TODO!?
+      this.$emit("newmove", move); //post-processing (e.g. computer play)
     },
     undo: function(move) {
       let navigate = !move;
@@ -231,9 +219,7 @@ export default {
       if (this.st.settings.sound == 2)
         new Audio("/sounds/undo.mp3").play().catch(err => {});
       this.incheck = this.vr.getCheckSquares(this.vr.turn);
-      if (navigate)
-        this.$children[0].$forceUpdate(); //TODO!?
-      else if (this.analyze) //TODO: can this happen?
+      if (this.analyze) //TODO: can this happen?
         this.moves.pop();
     },
     gotoMove: function(index) {