non invasive rerendering through old board state safe and reuse for a…

…fter board manipulation
mklemmingen · Jan 13, 2024 · 28c8c4d · 28c8c4d
1 parent c961ed5
commit 28c8c4d
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 3 deletions.
diff --git a/core/src/com/boomchess/game/BoomChess.java b/core/src/com/boomchess/game/BoomChess.java
@@ -458,7 +458,7 @@ public class BoomChess extends ApplicationAdapter {
 	// ------------------------------------------------------
 
 	// if on desktop, false, android if true, sets the resizing by tileSize and UI size
-	public static boolean publisher = false;
+	public static boolean publisher = true;
 
 	// ------------------------------------------------------
 

diff --git a/core/src/com/boomchess/game/frontend/stage/GameStage.java b/core/src/com/boomchess/game/frontend/stage/GameStage.java
@@ -407,8 +407,8 @@ public void dragStart(InputEvent event, float x, float y, int pointer) {
                             String tileTeamColor = gameBoard[finalI][finalJ].getTeamColor();
 
                             // If it's not the current team's turn, cancel the drag and return
-                            if ((currentState == BoomChess.GameState.RED_TURN && !tileTeamColor.equals("red")) ||
-                                    (currentState == BoomChess.GameState.GREEN_TURN && !tileTeamColor.equals("green"))) {
+                            if ((currentState == BoomChess.GameState.RED_TURN && tileTeamColor.equals("green")) ||
+                                    (currentState == BoomChess.GameState.GREEN_TURN && tileTeamColor.equals("red"))) {
                                 event.cancel();
 
                                 // adds a logo on screen that says that the movement was not allowed yet
@@ -417,6 +417,19 @@ public void dragStart(InputEvent event, float x, float y, int pointer) {
                                 //plays a brick sound
                                 indi.makeSound();
 
+                                nonInvasiveReRender = true;
+                                BoomChess.reRenderGame();
+                                return;
+                            } else if (currentState == BoomChess.GameState.RED_TURN && !tileTeamColor.equals("red") ||
+                                    currentState == BoomChess.GameState.GREEN_TURN && !tileTeamColor.equals("green")) {
+                                // player tried moving a piece that is of an empty soldier,
+                                // cancel event, do not add indicator, do not make a sound and non invasive rerender
+
+                                // explanation: there is red, green and EMPTY
+
+                                event.cancel();
+
+                                nonInvasiveReRender = true;
                                 BoomChess.reRenderGame();
                                 return;
                             }
@@ -655,6 +668,7 @@ public void changed(ChangeEvent event, Actor actor) {
         intervalsButton.addListener(new ChangeListener() {
             @Override
             public void changed(ChangeEvent event, Actor actor) {
+                nonInvasiveReRender = true;
                 BoomChess.showIntervals = !BoomChess.showIntervals;
                 reRenderGame();
             }
@@ -671,6 +685,7 @@ public void changed(ChangeEvent event, Actor actor) {
             @Override
             public void changed(ChangeEvent event, Actor actor) {
                 showHealth = !showHealth;
+                nonInvasiveReRender = true;
                 reRenderGame();
             }
         });