From 71e521e7888a9241d1adc7e96be895fcb45ac4f6 Mon Sep 17 00:00:00 2001 From: Northmoc Date: Fri, 18 Sep 2020 12:11:15 -0400 Subject: [PATCH] CountersPutEffect Stringbuilder tweak for 0 targets --- .../java/forge/game/ability/effects/CountersPutEffect.java | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/forge-game/src/main/java/forge/game/ability/effects/CountersPutEffect.java b/forge-game/src/main/java/forge/game/ability/effects/CountersPutEffect.java index 61d58891566..d00896a5d05 100644 --- a/forge-game/src/main/java/forge/game/ability/effects/CountersPutEffect.java +++ b/forge-game/src/main/java/forge/game/ability/effects/CountersPutEffect.java @@ -49,6 +49,13 @@ public class CountersPutEffect extends SpellAbilityEffect { final boolean dividedAsYouChoose = spellAbility.hasParam("DividedAsYouChoose"); final int amount = AbilityUtils.calculateAmount(card, spellAbility.getParamOrDefault("CounterNum", "1"), spellAbility); + //skip the StringBuilder if no targets are chosen ("up to" scenario) + if (spellAbility.usesTargeting()) { + final List targetCards = SpellAbilityEffect.getTargetCards(spellAbility); + if (targetCards.size() == 0) { + return stringBuilder.toString(); + } + } if (spellAbility.hasParam("Bolster")) { stringBuilder.append("Bolster ").append(amount); return stringBuilder.toString();