From 4b8a2c21d1ce8590053049b894d5f60c79bdf2ec Mon Sep 17 00:00:00 2001 From: Evan Kranzler Date: Fri, 7 Jun 2019 13:58:55 -0400 Subject: [PATCH] fixed Seasoned Pyromancer requiring controller to choose cards in hand when there aren't enough --- .../src/mage/cards/s/SeasonedPyromancer.java | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/Mage.Sets/src/mage/cards/s/SeasonedPyromancer.java b/Mage.Sets/src/mage/cards/s/SeasonedPyromancer.java index 7a1dcd9cd02..31910a5c0b2 100644 --- a/Mage.Sets/src/mage/cards/s/SeasonedPyromancer.java +++ b/Mage.Sets/src/mage/cards/s/SeasonedPyromancer.java @@ -82,13 +82,16 @@ class SeasonedPyromancerEffect extends OneShotEffect { if (player == null) { return false; } - TargetCard target = new TargetCardInHand(2, StaticFilters.FILTER_CARD); int nonlands = 0; - if (player.choose(outcome, player.getHand(), target, game)) { - Cards cards = new CardsImpl(target.getTargets()); - for (Card card : cards.getCards(game)) { - if (player.discard(card, source, game) && !card.isLand()) { - nonlands++; + int toDiscard = Math.max(player.getHand().size(), 2); + if (toDiscard > 0) { + TargetCard target = new TargetCardInHand(toDiscard, StaticFilters.FILTER_CARD); + if (player.choose(outcome, player.getHand(), target, game)) { + Cards cards = new CardsImpl(target.getTargets()); + for (Card card : cards.getCards(game)) { + if (player.discard(card, source, game) && !card.isLand()) { + nonlands++; + } } } }