From d5b23705c756f40ba0d7d06ce029a1adbe505c3e Mon Sep 17 00:00:00 2001 From: xenohedron Date: Fri, 13 Sep 2024 20:23:03 -0400 Subject: [PATCH] refactor: use getter method when setting paw prints, rather than direct access --- Mage.Sets/src/mage/cards/s/SeasonOfGathering.java | 4 ++-- Mage.Sets/src/mage/cards/s/SeasonOfLoss.java | 7 +++---- Mage.Sets/src/mage/cards/s/SeasonOfTheBold.java | 2 +- Mage.Sets/src/mage/cards/s/SeasonOfTheBurrow.java | 2 +- Mage.Sets/src/mage/cards/s/SeasonOfWeaving.java | 2 +- 5 files changed, 8 insertions(+), 9 deletions(-) diff --git a/Mage.Sets/src/mage/cards/s/SeasonOfGathering.java b/Mage.Sets/src/mage/cards/s/SeasonOfGathering.java index 96b73cd7a23..628ee0dad99 100644 --- a/Mage.Sets/src/mage/cards/s/SeasonOfGathering.java +++ b/Mage.Sets/src/mage/cards/s/SeasonOfGathering.java @@ -44,7 +44,7 @@ public final class SeasonOfGathering extends CardImpl { // {P} -- Put a +1/+1 counter on a creature you control. It gains vigilance and trample until end of turn. this.getSpellAbility().addEffect(new SeasonOfGatheringCounterEffect()); - this.spellAbility.getModes().getMode().withPawPrintValue(1); + this.getSpellAbility().getModes().getMode().withPawPrintValue(1); // {P}{P} -- Choose artifact or enchantment. Destroy all permanents of the chosen type. Mode mode2 = new Mode(new SeasonOfGatheringRemovalEffect()); @@ -159,4 +159,4 @@ class SeasonOfGatheringRemovalEffect extends OneShotEffect { game.informPlayers(controller.getLogName() + " has chosen " + choiceImpl.getChoiceKey()); return new DestroyAllEffect(filter).apply(game, source); } -} \ No newline at end of file +} diff --git a/Mage.Sets/src/mage/cards/s/SeasonOfLoss.java b/Mage.Sets/src/mage/cards/s/SeasonOfLoss.java index 119e67a245d..581db0f199d 100644 --- a/Mage.Sets/src/mage/cards/s/SeasonOfLoss.java +++ b/Mage.Sets/src/mage/cards/s/SeasonOfLoss.java @@ -1,12 +1,9 @@ package mage.cards.s; -import java.util.UUID; - import mage.abilities.Ability; import mage.abilities.Mode; import mage.abilities.dynamicvalue.DynamicValue; import mage.abilities.dynamicvalue.common.CardsInControllerGraveyardCount; -import mage.abilities.dynamicvalue.common.GreatestPowerAmongControlledCreaturesValue; import mage.abilities.effects.Effect; import mage.abilities.effects.common.DrawCardSourceControllerEffect; import mage.abilities.effects.common.LoseLifeOpponentsEffect; @@ -18,6 +15,8 @@ import mage.filter.StaticFilters; import mage.game.Game; import mage.watchers.common.CreaturesDiedWatcher; +import java.util.UUID; + /** * * @author jimga150 @@ -35,7 +34,7 @@ public final class SeasonOfLoss extends CardImpl { // {P} -- Each player sacrifices a creature. this.getSpellAbility().addEffect(new SacrificeAllEffect(1, StaticFilters.FILTER_PERMANENT_CREATURE)); - this.spellAbility.getModes().getMode().withPawPrintValue(1); + this.getSpellAbility().getModes().getMode().withPawPrintValue(1); // {P}{P} -- Draw a card for each creature you controlled that died this turn. Mode mode2 = new Mode(new DrawCardSourceControllerEffect(SeasonOfLossValue.instance)); diff --git a/Mage.Sets/src/mage/cards/s/SeasonOfTheBold.java b/Mage.Sets/src/mage/cards/s/SeasonOfTheBold.java index 1e4cca5e2cd..ba56af51cb7 100644 --- a/Mage.Sets/src/mage/cards/s/SeasonOfTheBold.java +++ b/Mage.Sets/src/mage/cards/s/SeasonOfTheBold.java @@ -34,7 +34,7 @@ public final class SeasonOfTheBold extends CardImpl { // {P} -- Create a tapped Treasure token. this.getSpellAbility().addEffect(new CreateTokenEffect(new TreasureToken(), 1, true)); - this.spellAbility.getModes().getMode().withPawPrintValue(1); + this.getSpellAbility().getModes().getMode().withPawPrintValue(1); // {P}{P} -- Exile the top two cards of your library. Until the end of your next turn, you may play them. Mode mode2 = new Mode(new ExileTopXMayPlayUntilEffect(2, Duration.UntilEndOfYourNextTurn)); diff --git a/Mage.Sets/src/mage/cards/s/SeasonOfTheBurrow.java b/Mage.Sets/src/mage/cards/s/SeasonOfTheBurrow.java index 529371b0a3d..3966d11d8f4 100644 --- a/Mage.Sets/src/mage/cards/s/SeasonOfTheBurrow.java +++ b/Mage.Sets/src/mage/cards/s/SeasonOfTheBurrow.java @@ -40,7 +40,7 @@ public final class SeasonOfTheBurrow extends CardImpl { // {P} -- Create a 1/1 white Rabbit creature token. this.getSpellAbility().addEffect(new CreateTokenEffect(new RabbitToken())); - this.spellAbility.getModes().getMode().withPawPrintValue(1); + this.getSpellAbility().getModes().getMode().withPawPrintValue(1); // {P}{P} -- Exile target nonland permanent. Its controller draws a card. Mode mode2 = new Mode(new ExileTargetEffect()); diff --git a/Mage.Sets/src/mage/cards/s/SeasonOfWeaving.java b/Mage.Sets/src/mage/cards/s/SeasonOfWeaving.java index 26f62b14ef5..b5999376830 100644 --- a/Mage.Sets/src/mage/cards/s/SeasonOfWeaving.java +++ b/Mage.Sets/src/mage/cards/s/SeasonOfWeaving.java @@ -44,7 +44,7 @@ public final class SeasonOfWeaving extends CardImpl { // {P} -- Draw a card. this.getSpellAbility().addEffect(new DrawCardSourceControllerEffect(1)); - this.spellAbility.getModes().getMode().withPawPrintValue(1); + this.getSpellAbility().getModes().getMode().withPawPrintValue(1); // {P}{P} -- Choose an artifact or creature you control. Create a token that's a copy of it. Mode mode2 = new Mode(new SeasonOfWeavingEffect());