diff --git a/forge-game/src/main/java/forge/game/ability/effects/UntapAllEffect.java b/forge-game/src/main/java/forge/game/ability/effects/UntapAllEffect.java index 6a6f4c83471..c79d87dae2f 100644 --- a/forge-game/src/main/java/forge/game/ability/effects/UntapAllEffect.java +++ b/forge-game/src/main/java/forge/game/ability/effects/UntapAllEffect.java @@ -45,8 +45,8 @@ public class UntapAllEffect extends SpellAbilityEffect { untapper = c.getController(); } if (c.untap(true)) { - untapMap.computeIfAbsent(untapper, i -> new CardCollection()).add(c); - if (sa.hasParam("RememberUntapped")) card.addRemembered(c); + untapMap.computeIfAbsent(untapper, i -> new CardCollection()).add(c); + if (sa.hasParam("RememberUntapped")) card.addRemembered(c); } } diff --git a/forge-game/src/main/java/forge/game/card/Card.java b/forge-game/src/main/java/forge/game/card/Card.java index 6a3633348c9..69aeafa08d9 100644 --- a/forge-game/src/main/java/forge/game/card/Card.java +++ b/forge-game/src/main/java/forge/game/card/Card.java @@ -1290,7 +1290,6 @@ public class Card extends GameEntity implements Comparable, IHasSVars { craft.setExiledBy(cause.getActivatingPlayer()); } } - } public final List getStoredRolls() { diff --git a/forge-game/src/main/java/forge/game/zone/MagicStack.java b/forge-game/src/main/java/forge/game/zone/MagicStack.java index 409b7c5606b..4e6092fd82e 100644 --- a/forge-game/src/main/java/forge/game/zone/MagicStack.java +++ b/forge-game/src/main/java/forge/game/zone/MagicStack.java @@ -244,7 +244,7 @@ public class MagicStack /* extends MyObservable */ implements Iterable otherCardViewsToSelect, ITriggerEvent triggerEvent); + boolean selectCard(CardView cardView, List otherCardViewsToSelect, ITriggerEvent triggerEvent); void selectAbility(SpellAbilityView sa); diff --git a/forge-gui/src/main/java/forge/player/HumanCostDecision.java b/forge-gui/src/main/java/forge/player/HumanCostDecision.java index 78933606020..c09d9dea41b 100644 --- a/forge-gui/src/main/java/forge/player/HumanCostDecision.java +++ b/forge-gui/src/main/java/forge/player/HumanCostDecision.java @@ -413,7 +413,6 @@ public class HumanCostDecision extends CostDecisionMakerBase { final CardCollection exiled = new CardCollection(); final String required = sharedType ? " (must share a card type)" : ""; - final List chosen = controller.chooseCardsForZoneChange(ZoneType.Exile, origin, ability, typeList, mandatory ? nNeeded : 0, nNeeded, null, cost.toString(nNeeded) + required, null);