From: Benjamin Auder Date: Fri, 20 Mar 2020 22:17:51 +0000 (+0100) Subject: Fix bug in sentry attacked detections when pushing lancers X-Git-Url: https://git.auder.net/doc/html/%7B%7B%20path%28%27mixstore_static_about%27%29%20%7D%7D?a=commitdiff_plain;h=9dca2c935cebcaa3817b4f926f482f9cad4c5b53;p=vchess.git Fix bug in sentry attacked detections when pushing lancers --- diff --git a/client/src/variants/Eightpieces.js b/client/src/variants/Eightpieces.js index 81d69d81..9ca85956 100644 --- a/client/src/variants/Eightpieces.js +++ b/client/src/variants/Eightpieces.js @@ -761,6 +761,7 @@ export class EightpiecesRules extends ChessRules { super.isAttacked(sq, color) || this.isAttackedByLancer(sq, color) || this.isAttackedBySentry(sq, color) + // The jailer doesn't capture. ); } @@ -836,6 +837,7 @@ export class EightpiecesRules extends ChessRules { // Helper to check sentries attacks: selfAttack([x1, y1], [x2, y2]) { const color = this.getColor(x1, y1); + const oppCol = V.GetOppCol(color); const sliderAttack = (allowedSteps, lancer) => { const deltaX = x2 - x1, absDeltaX = Math.abs(deltaX); @@ -854,7 +856,7 @@ export class EightpiecesRules extends ChessRules { if ( // NOTE: no need to check OnBoard in this special case (!lancer && this.board[sq[0]][sq[1]] != V.EMPTY) || - (!!lancer && this.getColor(sq[0], sq[1]) != color) + (!!lancer && this.getColor(sq[0], sq[1]) == oppCol) ) { return false; }