diff --git a/forge-ai/src/main/java/forge/ai/ComputerUtilCombat.java b/forge-ai/src/main/java/forge/ai/ComputerUtilCombat.java index 148952c0840..1d13ed4f781 100644 --- a/forge-ai/src/main/java/forge/ai/ComputerUtilCombat.java +++ b/forge-ai/src/main/java/forge/ai/ComputerUtilCombat.java @@ -472,7 +472,6 @@ public class ComputerUtilCombat { * @return a boolean. */ public static boolean wouldLoseLife(final Player ai, final Combat combat) { - return (ComputerUtilCombat.lifeThatWouldRemain(ai, combat) < ai.getLife()); } diff --git a/forge-ai/src/main/java/forge/ai/ComputerUtilCost.java b/forge-ai/src/main/java/forge/ai/ComputerUtilCost.java index 468da12f98d..0f0d72882c4 100644 --- a/forge-ai/src/main/java/forge/ai/ComputerUtilCost.java +++ b/forge-ai/src/main/java/forge/ai/ComputerUtilCost.java @@ -346,6 +346,19 @@ public class ComputerUtilCost { return true; } + /** + * Check sacrifice cost. + * + * @param cost + * the cost + * @param source + * the source + * @return true, if successful + */ + public static boolean checkSacrificeCost(final Player ai, final Cost cost, final Card source, final SpellAbility sourceAbility) { + return checkSacrificeCost(ai, cost, source, sourceAbility, true); + } + public static boolean isSacrificeSelfCost(final Cost cost) { if (cost == null) { return false; @@ -406,19 +419,6 @@ public class ComputerUtilCost { return true; } - /** - * Check sacrifice cost. - * - * @param cost - * the cost - * @param source - * the source - * @return true, if successful - */ - public static boolean checkSacrificeCost(final Player ai, final Cost cost, final Card source, final SpellAbility sourceAbility) { - return checkSacrificeCost(ai, cost, source, sourceAbility, true); - } - /** *

* shouldPayCost. diff --git a/forge-game/src/main/java/forge/game/ability/effects/PlayEffect.java b/forge-game/src/main/java/forge/game/ability/effects/PlayEffect.java index 5ba073cc8ae..3e5a0a4d5fd 100644 --- a/forge-game/src/main/java/forge/game/ability/effects/PlayEffect.java +++ b/forge-game/src/main/java/forge/game/ability/effects/PlayEffect.java @@ -187,7 +187,7 @@ public class PlayEffect extends SpellAbilityEffect { while (!tgtCards.isEmpty() && amount > 0) { activator.getController().tempShowCards(showCards); - Card tgtCard = controller.getController().chooseSingleEntityForEffect(tgtCards, sa, Localizer.getInstance().getMessage("lblSelectCardToPlay"), !singleOption, null); + Card tgtCard = controller.getController().chooseSingleEntityForEffect(tgtCards, sa, Localizer.getInstance().getMessage("lblSelectCardToPlay"), !singleOption && optional, null); activator.getController().endTempShowCards(); if (tgtCard == null) { break; diff --git a/forge-game/src/main/java/forge/game/ability/effects/RollDiceEffect.java b/forge-game/src/main/java/forge/game/ability/effects/RollDiceEffect.java index 4fc0511c5b4..5328f6a9170 100644 --- a/forge-game/src/main/java/forge/game/ability/effects/RollDiceEffect.java +++ b/forge-game/src/main/java/forge/game/ability/effects/RollDiceEffect.java @@ -143,7 +143,7 @@ public class RollDiceEffect extends SpellAbilityEffect { final PlayerCollection playersToRoll = getTargetPlayers(sa); - for(Player player : playersToRoll) { + for (Player player : playersToRoll) { rollDice(sa, player, amount, sides); } }