From 9300d07160176267989758e8f40f854164dff36d Mon Sep 17 00:00:00 2001 From: jeffwadsworth Date: Sun, 11 Jul 2021 10:05:55 -0500 Subject: [PATCH] - addSubType() already checks for duplicates --- .../BecomesAllBasicsControlledEffect.java | 21 ++++++------------- 1 file changed, 6 insertions(+), 15 deletions(-) diff --git a/Mage/src/main/java/mage/abilities/effects/common/continuous/BecomesAllBasicsControlledEffect.java b/Mage/src/main/java/mage/abilities/effects/common/continuous/BecomesAllBasicsControlledEffect.java index 9023fce669a..65b3eefe695 100644 --- a/Mage/src/main/java/mage/abilities/effects/common/continuous/BecomesAllBasicsControlledEffect.java +++ b/Mage/src/main/java/mage/abilities/effects/common/continuous/BecomesAllBasicsControlledEffect.java @@ -36,21 +36,12 @@ public class BecomesAllBasicsControlledEffect extends ContinuousEffectImpl { public boolean apply(Game game, Ability source) { for (Permanent permanent : game.getBattlefield().getActivePermanents( StaticFilters.FILTER_CONTROLLED_PERMANENT_LAND, source.getControllerId(), game)) { - if (!permanent.getSubtype(game).contains(SubType.PLAINS)) { - permanent.addSubType(SubType.PLAINS); - } - if (!permanent.getSubtype(game).contains(SubType.ISLAND)) { - permanent.addSubType(SubType.ISLAND); - } - if (!permanent.getSubtype(game).contains(SubType.SWAMP)) { - permanent.addSubType(SubType.SWAMP); - } - if (!permanent.getSubtype(game).contains(SubType.MOUNTAIN)) { - permanent.addSubType(SubType.MOUNTAIN); - } - if (!permanent.getSubtype(game).contains(SubType.FOREST)) { - permanent.addSubType(SubType.FOREST); - } + permanent.addSubType(SubType.PLAINS); + permanent.addSubType(SubType.ISLAND); + permanent.addSubType(SubType.SWAMP); + permanent.addSubType(SubType.MOUNTAIN); + permanent.addSubType(SubType.FOREST); + if (!permanent.getAbilities().containsRule(new WhiteManaAbility())) { permanent.addAbility(new WhiteManaAbility(), source.getSourceId(), game); }