[M21] various text fixes

This commit is contained in:
Evan Kranzler 2020-10-06 21:12:07 -04:00
parent 44b44f4103
commit 524b9fcd44
14 changed files with 19 additions and 16 deletions

View file

@ -29,7 +29,7 @@ import java.util.UUID;
public final class BarrinTolarianArchmage extends CardImpl { public final class BarrinTolarianArchmage extends CardImpl {
private static final FilterCreatureOrPlaneswalkerPermanent filter private static final FilterCreatureOrPlaneswalkerPermanent filter
= new FilterCreatureOrPlaneswalkerPermanent(" other target creature or planeswalker"); = new FilterCreatureOrPlaneswalkerPermanent("other target creature or planeswalker");
static { static {
filter.add(AnotherPredicate.instance); filter.add(AnotherPredicate.instance);

View file

@ -126,6 +126,6 @@ class FelineSovereignTriggeredAbility extends TriggeredAbilityImpl {
@Override @Override
public String getRule() { public String getRule() {
return "Whenever one or more Cats you control deal combat damage to a player, destroy up to one target artifact or enchantment that player controls"; return "Whenever one or more Cats you control deal combat damage to a player, destroy up to one target artifact or enchantment that player controls.";
} }
} }

View file

@ -34,7 +34,7 @@ public final class FurorOfTheBitten extends CardImpl {
// Enchanted creature gets +2/+2 and attacks each turn if able. // Enchanted creature gets +2/+2 and attacks each turn if able.
Ability ability = new SimpleStaticAbility(Zone.BATTLEFIELD, new BoostEnchantedEffect(2, 2, Duration.WhileOnBattlefield)); Ability ability = new SimpleStaticAbility(Zone.BATTLEFIELD, new BoostEnchantedEffect(2, 2, Duration.WhileOnBattlefield));
Effect effect = new AttacksIfAbleAttachedEffect(Duration.WhileOnBattlefield, AttachmentType.AURA); Effect effect = new AttacksIfAbleAttachedEffect(Duration.WhileOnBattlefield, AttachmentType.AURA);
effect.setText("and attacks each turn if able"); effect.setText("and attacks each combat if able");
ability.addEffect(effect); ability.addEffect(effect);
this.addAbility(ability); this.addAbility(ability);
} }

View file

@ -21,7 +21,7 @@ public final class GloriousAnthem extends CardImpl {
super(ownerId, setInfo, new CardType[]{CardType.ENCHANTMENT}, "{1}{W}{W}"); super(ownerId, setInfo, new CardType[]{CardType.ENCHANTMENT}, "{1}{W}{W}");
// Creatures you control get +1/+1. // Creatures you control get +1/+1.
this.addAbility(new SimpleStaticAbility(Zone.BATTLEFIELD, new BoostControlledEffect(1, 1, Duration.WhileOnBattlefield, StaticFilters.FILTER_PERMANENT_CREATURE, false))); this.addAbility(new SimpleStaticAbility(Zone.BATTLEFIELD, new BoostControlledEffect(1, 1, Duration.WhileOnBattlefield, StaticFilters.FILTER_PERMANENT_CREATURES, false)));
} }
public GloriousAnthem(final GloriousAnthem card) { public GloriousAnthem(final GloriousAnthem card) {

View file

@ -37,7 +37,9 @@ public final class HeartfireImmolator extends CardImpl {
// {R}, Sacrifice Heartfire Immolator: It deals damage equal to its power to target creature or planeswalker. // {R}, Sacrifice Heartfire Immolator: It deals damage equal to its power to target creature or planeswalker.
Ability ability = new SimpleActivatedAbility( Ability ability = new SimpleActivatedAbility(
new DamageTargetEffect(xValue, "it"), new ManaCostsImpl("{R}") new DamageTargetEffect(xValue)
.setText("it deals damage equal to its power to target creature or planeswalker"),
new ManaCostsImpl("{R}")
); );
ability.addCost(new SacrificeSourceCost()); ability.addCost(new SacrificeSourceCost());
ability.addTarget(new TargetCreatureOrPlaneswalker()); ability.addTarget(new TargetCreatureOrPlaneswalker());

View file

@ -34,8 +34,8 @@ public final class HistorianOfZhalfir extends CardImpl {
// Whenever Historian of Zhalfir attacks, if you control a Teferi planeswalker, draw a card. // Whenever Historian of Zhalfir attacks, if you control a Teferi planeswalker, draw a card.
this.addAbility(new ConditionalInterveningIfTriggeredAbility( this.addAbility(new ConditionalInterveningIfTriggeredAbility(
new AttacksTriggeredAbility(new DrawCardSourceControllerEffect(1), false), new AttacksTriggeredAbility(new DrawCardSourceControllerEffect(1), false),
condition, "Whenever {this} attacks, if you control a Teferi planeswalker, draw a card") condition, "Whenever {this} attacks, if you control a Teferi planeswalker, draw a card."
); ));
} }
private HistorianOfZhalfir(final HistorianOfZhalfir card) { private HistorianOfZhalfir(final HistorianOfZhalfir card) {

View file

@ -27,7 +27,7 @@ import java.util.UUID;
*/ */
public final class LeafkinAvenger extends CardImpl { public final class LeafkinAvenger extends CardImpl {
private static final FilterPermanent filter = new FilterCreaturePermanent("creature you control with power 4 or greater"); private static final FilterPermanent filter = new FilterCreaturePermanent("creature with power 4 or greater you control");
static { static {
filter.add(new PowerPredicate(ComparisonType.MORE_THAN, 3)); filter.add(new PowerPredicate(ComparisonType.MORE_THAN, 3));

View file

@ -49,9 +49,9 @@ public final class RadhaHeartOfKeld extends CardImpl {
// You may look at the top card of your library any time, and you may play lands from the top of your library. // You may look at the top card of your library any time, and you may play lands from the top of your library.
LookAtTopCardOfLibraryAnyTimeEffect lookEffect = new LookAtTopCardOfLibraryAnyTimeEffect(); LookAtTopCardOfLibraryAnyTimeEffect lookEffect = new LookAtTopCardOfLibraryAnyTimeEffect();
lookEffect.overrideRuleText("You may look at the top card of your library any time, "); lookEffect.overrideRuleText("You may look at the top card of your library any time");
PlayTheTopCardEffect playEffect = new PlayTheTopCardEffect(filter); PlayTheTopCardEffect playEffect = new PlayTheTopCardEffect(filter);
playEffect.overrideRuleText("and you may play lands from the top of your library"); playEffect.overrideRuleText(", and you may play lands from the top of your library");
SimpleStaticAbility lookAndPlayAbility = new SimpleStaticAbility(lookEffect); SimpleStaticAbility lookAndPlayAbility = new SimpleStaticAbility(lookEffect);
lookAndPlayAbility.addEffect(playEffect); lookAndPlayAbility.addEffect(playEffect);

View file

@ -68,7 +68,7 @@ class SnarespinnerTriggeredAbility extends TriggeredAbilityImpl {
@Override @Override
public String getRule() { public String getRule() {
return "Whenever {this} blocks a creature with flying, {this} gets +2/+0 until end of turn"; return "Whenever {this} blocks a creature with flying, {this} gets +2/+0 until end of turn.";
} }
@Override @Override

View file

@ -74,6 +74,6 @@ enum SpeakerOfTheHeavensCondition implements Condition {
@Override @Override
public String toString() { public String toString() {
return "you have at least 7 more life than your starting life total and only any time you could cast a sorcery"; return "you have at least 7 life more than your starting life total and only any time you could cast a sorcery";
} }
} }

View file

@ -26,7 +26,7 @@ import java.util.UUID;
*/ */
public final class ThievesGuildEnforcer extends CardImpl { public final class ThievesGuildEnforcer extends CardImpl {
private static final FilterPermanent filter = new FilterPermanent(SubType.ROGUE, "{this} or another Rogue"); private static final FilterPermanent filter = new FilterPermanent(SubType.ROGUE, "Rogue");
public ThievesGuildEnforcer(UUID ownerId, CardSetInfo setInfo) { public ThievesGuildEnforcer(UUID ownerId, CardSetInfo setInfo) {
super(ownerId, setInfo, new CardType[]{CardType.CREATURE}, "{B}"); super(ownerId, setInfo, new CardType[]{CardType.CREATURE}, "{B}");

View file

@ -52,7 +52,7 @@ public class VerifyCardDataTest {
private static final Logger logger = Logger.getLogger(VerifyCardDataTest.class); private static final Logger logger = Logger.getLogger(VerifyCardDataTest.class);
private static final String FULL_ABILITIES_CHECK_SET_CODE = "IKO"; // check all abilities and output cards with wrong abilities texts; private static final String FULL_ABILITIES_CHECK_SET_CODE = "M21"; // check all abilities and output cards with wrong abilities texts;
private static final boolean AUTO_FIX_SAMPLE_DECKS = false; // debug only: fix sample decks if it contains errors like wrong card numbers private static final boolean AUTO_FIX_SAMPLE_DECKS = false; // debug only: fix sample decks if it contains errors like wrong card numbers
private static final HashMap<String, Set<String>> skipCheckLists = new HashMap<>(); private static final HashMap<String, Set<String>> skipCheckLists = new HashMap<>();

View file

@ -7,6 +7,7 @@ import mage.constants.Duration;
import mage.constants.Layer; import mage.constants.Layer;
import mage.constants.Outcome; import mage.constants.Outcome;
import mage.constants.SubLayer; import mage.constants.SubLayer;
import mage.filter.StaticFilters;
import mage.filter.common.FilterCreaturePermanent; import mage.filter.common.FilterCreaturePermanent;
import mage.game.Game; import mage.game.Game;
import mage.game.permanent.Permanent; import mage.game.permanent.Permanent;
@ -22,7 +23,7 @@ public class BoostOpponentsEffect extends ContinuousEffectImpl {
protected FilterCreaturePermanent filter; protected FilterCreaturePermanent filter;
public BoostOpponentsEffect(int power, int toughness, Duration duration) { public BoostOpponentsEffect(int power, int toughness, Duration duration) {
this(power, toughness, duration, new FilterCreaturePermanent("Creatures")); this(power, toughness, duration, StaticFilters.FILTER_PERMANENT_CREATURES);
} }
public BoostOpponentsEffect(int power, int toughness, Duration duration, FilterCreaturePermanent filter) { public BoostOpponentsEffect(int power, int toughness, Duration duration, FilterCreaturePermanent filter) {

View file

@ -34,7 +34,7 @@ public class SearchLibraryGraveyardPutInHandEffect extends OneShotEffect {
super(Outcome.Benefit); super(Outcome.Benefit);
this.filter = filter; this.filter = filter;
this.forceToSearchBoth = forceToSearchBoth; this.forceToSearchBoth = forceToSearchBoth;
staticText = (youMay ? "you may" : "") + " search your library and" + (forceToSearchBoth ? "" : "/or") + " graveyard for a card named " + filter.getMessage() staticText = (youMay ? "you may " : "") + "search your library and" + (forceToSearchBoth ? "" : "/or") + " graveyard for a card named " + filter.getMessage()
+ ", reveal it, and put it into your hand. " + (forceToSearchBoth ? "Then shuffle your library" : "If you search your library this way, shuffle it"); + ", reveal it, and put it into your hand. " + (forceToSearchBoth ? "Then shuffle your library" : "If you search your library this way, shuffle it");
} }