From 1ed8937cef78e7d68fa017ccc7d496160a1b2b92 Mon Sep 17 00:00:00 2001 From: Susucre <34709007+Susucre@users.noreply.github.com> Date: Sat, 6 Apr 2024 16:44:53 +0200 Subject: [PATCH] Fixes a few rule texts --- Mage.Sets/src/mage/cards/c/CactusPreserve.java | 3 ++- Mage.Sets/src/mage/cards/c/CataclysmicProspecting.java | 3 ++- Mage.Sets/src/mage/cards/e/EmergentHaunting.java | 2 +- Mage.Sets/src/mage/cards/s/SandScout.java | 2 +- Mage.Sets/src/mage/cards/t/TerritoryForge.java | 2 +- Mage.Verify/src/test/java/mage/verify/VerifyCardDataTest.java | 2 +- 6 files changed, 8 insertions(+), 6 deletions(-) diff --git a/Mage.Sets/src/mage/cards/c/CactusPreserve.java b/Mage.Sets/src/mage/cards/c/CactusPreserve.java index 78feffa7e38..351bec63b10 100644 --- a/Mage.Sets/src/mage/cards/c/CactusPreserve.java +++ b/Mage.Sets/src/mage/cards/c/CactusPreserve.java @@ -51,7 +51,8 @@ class CactusPreserveEffect extends OneShotEffect { CactusPreserveEffect() { super(Outcome.BecomeCreature); - this.staticText = "{this} becomes an X/X green Plant creature with reach, where X is the greatest mana value among your commanders. It's still a land."; + this.staticText = "Until end of turn, {this} becomes an X/X green Plant creature with reach, " + + "where X is the greatest mana value among your commanders. It's still a land."; } private CactusPreserveEffect(final CactusPreserveEffect effect) { diff --git a/Mage.Sets/src/mage/cards/c/CataclysmicProspecting.java b/Mage.Sets/src/mage/cards/c/CataclysmicProspecting.java index 112e17c7f87..f0335fcfaa0 100644 --- a/Mage.Sets/src/mage/cards/c/CataclysmicProspecting.java +++ b/Mage.Sets/src/mage/cards/c/CataclysmicProspecting.java @@ -41,7 +41,8 @@ public final class CataclysmicProspecting extends CardImpl { // Cataclysmic Prospecting deals X damage to each creature. For each mana from a Desert spent to cast this spell, create a tapped Treasure token. this.getSpellAbility().addEffect(new DamageAllEffect(ManacostVariableValue.REGULAR, StaticFilters.FILTER_PERMANENT_CREATURE)); - this.getSpellAbility().addEffect(new CreateTokenEffect(new TreasureToken(), CataclysmicProspectingValue.instance, true, false)); + this.getSpellAbility().addEffect(new CreateTokenEffect(new TreasureToken(), CataclysmicProspectingValue.instance, true, false) + .setText("For each mana from a Desert spent to cast this spell, create a tapped Treasure token.")); this.getSpellAbility().addWatcher(new CataclysmicProspectingWatcher()); this.getSpellAbility().addHint(hint); } diff --git a/Mage.Sets/src/mage/cards/e/EmergentHaunting.java b/Mage.Sets/src/mage/cards/e/EmergentHaunting.java index 0314616274b..c6171eebe19 100644 --- a/Mage.Sets/src/mage/cards/e/EmergentHaunting.java +++ b/Mage.Sets/src/mage/cards/e/EmergentHaunting.java @@ -42,7 +42,7 @@ public final class EmergentHaunting extends CardImpl { null, Duration.WhileOnBattlefield ), TargetController.YOU, condition, false - ).addHint(HaventCastSpellFromHandThisTurnCondition.hint)); + ).withRuleTextReplacement(true).addHint(HaventCastSpellFromHandThisTurnCondition.hint)); // {2}{U}: Surveil 1. this.addAbility(new SimpleActivatedAbility(new SurveilEffect(1), new ManaCostsImpl<>("{2}{U}"))); diff --git a/Mage.Sets/src/mage/cards/s/SandScout.java b/Mage.Sets/src/mage/cards/s/SandScout.java index 0380fa02aae..71d14bd9fa4 100644 --- a/Mage.Sets/src/mage/cards/s/SandScout.java +++ b/Mage.Sets/src/mage/cards/s/SandScout.java @@ -51,7 +51,7 @@ public final class SandScout extends CardImpl { // Whenever one or more land cards are put into your graveyard from anywhere, create a 1/1 red, green, and white Sand Warrior creature token. This ability triggers only once each turn. this.addAbility(new PutCardIntoGraveFromAnywhereAllTriggeredAbility( new CreateTokenEffect(new HazezonTamarSandWarriorToken()), false, StaticFilters.FILTER_CARD_LAND, TargetController.YOU - ).setTriggersOnceEachTurn(true)); + ).setTriggerPhrase("Whenever one or more land cards are put into your graveyard from anywhere, ").setTriggersOnceEachTurn(true)); } private SandScout(final SandScout card) { diff --git a/Mage.Sets/src/mage/cards/t/TerritoryForge.java b/Mage.Sets/src/mage/cards/t/TerritoryForge.java index 2693d8d958d..d0fb868573c 100644 --- a/Mage.Sets/src/mage/cards/t/TerritoryForge.java +++ b/Mage.Sets/src/mage/cards/t/TerritoryForge.java @@ -64,7 +64,7 @@ class TerritoryForgeStaticEffect extends ContinuousEffectImpl { TerritoryForgeStaticEffect() { super(Duration.WhileOnBattlefield, Layer.AbilityAddingRemovingEffects_6, SubLayer.NA, Outcome.Benefit); - staticText = "{this} has all activated ability of the exiled card"; + staticText = "{this} has all activated abilities of the exiled card"; } private TerritoryForgeStaticEffect(final TerritoryForgeStaticEffect effect) { diff --git a/Mage.Verify/src/test/java/mage/verify/VerifyCardDataTest.java b/Mage.Verify/src/test/java/mage/verify/VerifyCardDataTest.java index 571980f1f72..fd99a43db86 100644 --- a/Mage.Verify/src/test/java/mage/verify/VerifyCardDataTest.java +++ b/Mage.Verify/src/test/java/mage/verify/VerifyCardDataTest.java @@ -70,7 +70,7 @@ public class VerifyCardDataTest { private static final Logger logger = Logger.getLogger(VerifyCardDataTest.class); - private static final String FULL_ABILITIES_CHECK_SET_CODES = "OTJ;BIG"; // check ability text due mtgjson, can use multiple sets like MAT;CMD or * for all + private static final String FULL_ABILITIES_CHECK_SET_CODES = "OTJ;BIG;OTC"; // check ability text due mtgjson, can use multiple sets like MAT;CMD or * for all private static final boolean CHECK_ONLY_ABILITIES_TEXT = false; // use when checking text locally, suppresses unnecessary checks and output messages private static final boolean AUTO_FIX_SAMPLE_DECKS = false; // debug only: auto-fix sample decks by test_checkSampleDecks test run