Chakart: almost there
[vchess.git] / client / src / components / BaseGame.vue
index 70a304b..79f089e 100644 (file)
@@ -121,13 +121,15 @@ export default {
     // TODO: is it OK to pass "computed" as properties?
     // Also, some are seemingly not recomputed when vr is initialized.
     showMoves: function() {
-      return this.game.score != "*"
-        ? "all"
-        : (!!this.vr ? this.vr.showMoves : "none");
+      return (
+        !!this.game.score && this.game.score != "*"
+          ? "all"
+          : (!!this.vr ? this.vr.showMoves : "none")
+      );
     },
     showTurn: function() {
       return (
-        this.game.score == '*' &&
+        !!this.game.score && this.game.score == '*' &&
         !!this.vr &&
         (
           this.vr.showMoves != "all" ||
@@ -138,7 +140,7 @@ export default {
     },
     canAnalyze: function() {
       return (
-        this.game.mode != "analyze" &&
+        !!this.game.mode && this.game.mode != "analyze" &&
         !!this.vr && this.vr.canAnalyze
       );
     },
@@ -147,8 +149,8 @@ export default {
     },
     allowDownloadPGN: function() {
       return (
-        this.game.score != "*" ||
-        (!!this.vr && this.vr.showMoves == "all")
+        (!!this.game.score && this.game.score != "*") ||
+        (!!this.vr && !this.vr.someHiddenMoves)
       );
     }
   },
@@ -277,12 +279,18 @@ export default {
     },
     toggleAnalyze: function() {
       // Freeze while choices are shown (and autoplay has priority)
-      if (this.$refs["board"].choices.length > 0 || this.autoplay) return;
+      if (
+        this.inPlay ||
+        this.$refs["board"].choices.length > 0 ||
+        this.autoplay
+      ) {
+        return;
+      }
       if (this.mode != "analyze") {
         // Enter analyze mode:
+        this.mode = "analyze";
         if (this.inMultimove) this.cancelCurrentMultimove();
         this.gameMode = this.mode; //was not 'analyze'
-        this.mode = "analyze";
         this.gameCursor = this.cursor;
         this.gameMoves = JSON.parse(JSON.stringify(this.moves));
         document.getElementById("analyzeBtn").classList.add("active");
@@ -461,7 +469,7 @@ export default {
           // To play a received move, cursor must be at the end of the game:
           this.gotoEnd();
       }
-      // The board may show some the possible moves: (TODO: bad solution)
+      // The board may show some possible moves: (TODO: bad solution)
       this.$refs["board"].resetCurrentAttempt();
       const playSubmove = (smove) => {
         smove.notation = this.vr.getNotation(smove);