Some refactoring in variants logic: more robust FEN handling (untested)

[vchess.git] / public / javascripts / variants / Loser.js

diff --git a/public/javascripts/variants/Loser.js b/public/javascripts/variants/Loser.js
index 2d60d6e..5f15612 100644 (file)
--- a/public/javascripts/variants/Loser.js
+++ b/public/javascripts/variants/Loser.js
@@ -1,20 +1,15 @@
 class LoserRules extends ChessRules
 {
-       initVariables(fen)
-       {
-               const epSq = this.moves.length > 0 ? this.getEpSquare(this.lastMove) : undefined;
-               this.epSquares = [ epSq ];
-       }
-
-       static IsGoodFlags(flags)
-       {
-               return true; //anything is good: no flags
-       }
+       static get HasFlags() { return false; }
 
-       setFlags(fenflags)
+       setOtherVariables(fen)
        {
-               // No castling, hence no flags; but flags defined for compatibility
-               this.castleFlags = { "w":[false,false], "b":[false,false] };
+               const parsedFen = V.ParseFen(fen);
+               const epSq = parsedFen.enpassant != "-"
+                       ? V.SquareToCoords(parsedFen.enpassant)
+                       : undefined;
+               this.epSquares = [ epSq ];
+               this.scanKingsRooks(fen);
        }
 
        getPotentialPawnMoves([x,y])
@@ -48,6 +43,7 @@ class LoserRules extends ChessRules
 
        getPotentialKingMoves(sq)
        {
+               // No castle:
                return this.getSlideNJumpMoves(sq,
                        V.steps[V.ROOK].concat(V.steps[V.BISHOP]), "oneStep");
        }
@@ -111,22 +107,19 @@ class LoserRules extends ChessRules
                return [];
        }
 
-       // Unused:
+       // No variables update because no castling
        updateVariables(move) { }
        unupdateVariables(move) { }
 
-       getFlagsFen()
-       {
-               return "-";
-       }
-
        checkGameEnd()
        {
                // No valid move: you win!
                return this.turn == "w" ? "1-0" : "0-1";
        }
 
-       static get VALUES() { //experimental...
+       static get VALUES()
+       {
+               // Experimental...
                return {
                        'p': 1,
                        'r': 7,
@@ -197,6 +190,6 @@ class LoserRules extends ChessRules
                return pieces["b"].join("") +
                        "/pppppppp/8/8/8/8/PPPPPPPP/" +
                        pieces["w"].join("").toUpperCase() +
-                       " 0000 w"; //add flags (TODO?!)
+                       " w -"; //no en-passant
        }
 }