external-repos/forge
1
0
Fork 0
mirror of https://github.com/Card-Forge/forge.git synced 2025-11-18 11:48:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Prevent AI taking over if exception thrown while AI controller temporarily set

Browse Source
This commit is contained in:
drdev
2013-12-04 01:33:42 +00:00
parent 4fb59b6071
commit 624365cda3
2 changed files with 20 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
forge-gui/src/main/java/forge/game/player/Player.java
View File

@@ -2943,16 +2943,27 @@ public class Player extends GameEntity implements Comparable<Player> {
} }
/** /**
* Run a procedure using a different controller * Run a procedure using an AI controller
* @param proc * @param proc
* @param tempController
*/ */
public void runWithController(Runnable proc, PlayerController tempController) { public void runAsAi(Runnable proc) {
if (PlayerControllerAi.class.isInstance(controller)) {
proc.run(); //can just run with current controller if it's an AI controller
return;
}
PlayerController oldController = controller; PlayerController oldController = controller;
controller = tempController; controller = new PlayerControllerAi(this.game, this, this.getOriginalLobbyPlayer());
try {
proc.run(); proc.run();
controller = oldController; controller = oldController;
} }
catch (Exception ex) {
//ensure controller restored before throwing exception in case user chooses to continue
controller = oldController;
throw ex;
}
}
/** /**
* <p> * <p>

10
forge-gui/src/main/java/forge/gui/input/InputPayMana.java
View File

@@ -19,7 +19,6 @@ import forge.game.card.CardUtil;
import forge.game.mana.ManaCostBeingPaid; import forge.game.mana.ManaCostBeingPaid;
import forge.game.player.HumanPlay; import forge.game.player.HumanPlay;
import forge.game.player.Player; import forge.game.player.Player;
import forge.game.player.PlayerControllerAi;
import forge.game.replacement.ReplacementEffect; import forge.game.replacement.ReplacementEffect;
import forge.game.spellability.AbilityManaPart; import forge.game.spellability.AbilityManaPart;
import forge.game.spellability.SpellAbility; import forge.game.spellability.SpellAbility;
@@ -50,11 +49,6 @@ public abstract class InputPayMana extends InputSyncronizedBase {
this.saPaidFor = saToPayFor; this.saPaidFor = saToPayFor;
} }
public void runAsAi(Runnable proc) {
this.player.runWithController(proc, new PlayerControllerAi(this.game, this.player, this.player.getOriginalLobbyPlayer()));
}
@Override @Override
protected void onCardSelected(final Card card, final MouseEvent triggerEvent) { protected void onCardSelected(final Card card, final MouseEvent triggerEvent) {
if (card.getManaAbility().isEmpty()) { if (card.getManaAbility().isEmpty()) {
@@ -295,7 +289,7 @@ public abstract class InputPayMana extends InputSyncronizedBase {
ComputerUtilMana.payManaCost(manaCost, saPaidFor, player); ComputerUtilMana.payManaCost(manaCost, saPaidFor, player);
} }
}; };
runAsAi(proc); this.player.runAsAi(proc);
this.showMessage(); this.showMessage();
} }
} }
@@ -317,7 +311,7 @@ public abstract class InputPayMana extends InputSyncronizedBase {
return ComputerUtilMana.canPayManaCost(manaCost, saPaidFor, player); return ComputerUtilMana.canPayManaCost(manaCost, saPaidFor, player);
} }
}; };
runAsAi(proc); this.player.runAsAi(proc);
canPayManaCost = proc.getResult(); canPayManaCost = proc.getResult();
} }
if (canPayManaCost) { if (canPayManaCost) {