From 4150a5bb12252a947d69b6386c215b51ab2275b8 Mon Sep 17 00:00:00 2001 From: Evan Kranzler Date: Fri, 16 Jul 2021 18:43:09 -0400 Subject: [PATCH] small change to test failure fix --- .../continuous/GainAbilityWithAttachmentEffect.java | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/Mage/src/main/java/mage/abilities/effects/common/continuous/GainAbilityWithAttachmentEffect.java b/Mage/src/main/java/mage/abilities/effects/common/continuous/GainAbilityWithAttachmentEffect.java index 4b5262f6234..8cfe2280bbd 100644 --- a/Mage/src/main/java/mage/abilities/effects/common/continuous/GainAbilityWithAttachmentEffect.java +++ b/Mage/src/main/java/mage/abilities/effects/common/continuous/GainAbilityWithAttachmentEffect.java @@ -19,7 +19,7 @@ import mage.target.Target; import mage.target.Targets; import mage.target.targetpointer.FixedTarget; -import java.util.Objects; +import java.util.Collections; /** * @author TheElk801 @@ -39,14 +39,8 @@ public class GainAbilityWithAttachmentEffect extends ContinuousEffectImpl { super(Duration.WhileOnBattlefield, Layer.AbilityAddingRemovingEffects_6, SubLayer.NA, Outcome.AddAbility); this.staticText = rule; this.effects.addAll(effects); - this.effects.removeIf(Objects::isNull); this.targets.addAll(targets); - this.targets.removeIf(Objects::isNull); - for (Cost cost : costs) { - if (cost != null) { - this.costs.add(cost); - } - } + Collections.addAll(this.costs, costs); this.useAttachedCost = attachedCost; this.generateGainAbilityDependencies(makeAbility(null, null), null); }