mirror of
https://github.com/correl/mage.git
synced 2025-04-10 01:01:05 -09:00
fixed various cards which don't add an ability
This commit is contained in:
parent
a5afba5d5a
commit
d0b90379f6
8 changed files with 12 additions and 8 deletions
Mage.Sets/src/mage/cards
g
l
s
t
u
w
Mage.Verify/src/test/java/mage/verify
|
@ -42,6 +42,7 @@ public final class GeistlightSnare extends CardImpl {
|
||||||
).setRuleAtTheTop(true);
|
).setRuleAtTheTop(true);
|
||||||
ability.addEffect(new SpellCostReductionSourceEffect(1, condition2).setCanWorksOnStackOnly(true)
|
ability.addEffect(new SpellCostReductionSourceEffect(1, condition2).setCanWorksOnStackOnly(true)
|
||||||
.setText("It also costs {1} less to cast if you control an enchantment"));
|
.setText("It also costs {1} less to cast if you control an enchantment"));
|
||||||
|
this.addAbility(ability);
|
||||||
|
|
||||||
// Counter target spell unless its controller pays {3}.
|
// Counter target spell unless its controller pays {3}.
|
||||||
this.getSpellAbility().addEffect(new CounterUnlessPaysEffect(new GenericManaCost(3)));
|
this.getSpellAbility().addEffect(new CounterUnlessPaysEffect(new GenericManaCost(3)));
|
||||||
|
|
|
@ -27,6 +27,7 @@ public final class GorgonFlail extends CardImpl {
|
||||||
ability.addEffect(new GainAbilityAttachedEffect(
|
ability.addEffect(new GainAbilityAttachedEffect(
|
||||||
DeathtouchAbility.getInstance(), AttachmentType.EQUIPMENT
|
DeathtouchAbility.getInstance(), AttachmentType.EQUIPMENT
|
||||||
).setText("and has deathtouch"));
|
).setText("and has deathtouch"));
|
||||||
|
this.addAbility(ability);
|
||||||
|
|
||||||
this.addAbility(new EquipAbility(2));
|
this.addAbility(new EquipAbility(2));
|
||||||
}
|
}
|
||||||
|
|
|
@ -40,6 +40,7 @@ public final class LandscaperColos extends CardImpl {
|
||||||
false, "put target card from an opponent's graveyard on the bottom of their library"
|
false, "put target card from an opponent's graveyard on the bottom of their library"
|
||||||
));
|
));
|
||||||
ability.addTarget(new TargetCardInGraveyard());
|
ability.addTarget(new TargetCardInGraveyard());
|
||||||
|
this.addAbility(ability);
|
||||||
|
|
||||||
// Basic landcycling {1}{W}
|
// Basic landcycling {1}{W}
|
||||||
this.addAbility(new BasicLandcyclingAbility(new ManaCostsImpl<>("{1}{W}")));
|
this.addAbility(new BasicLandcyclingAbility(new ManaCostsImpl<>("{1}{W}")));
|
||||||
|
|
|
@ -33,6 +33,7 @@ public final class ShieldOfTheRighteous extends CardImpl {
|
||||||
ability.addEffect(new GainAbilityAttachedEffect(
|
ability.addEffect(new GainAbilityAttachedEffect(
|
||||||
VigilanceAbility.getInstance(), AttachmentType.EQUIPMENT
|
VigilanceAbility.getInstance(), AttachmentType.EQUIPMENT
|
||||||
).setText("and has vigilance"));
|
).setText("and has vigilance"));
|
||||||
|
this.addAbility(ability);
|
||||||
|
|
||||||
// Whenever equipped creature blocks a creature, that creature doesn't untap during its controller's next untap step.
|
// Whenever equipped creature blocks a creature, that creature doesn't untap during its controller's next untap step.
|
||||||
this.addAbility(new BlocksAttachedTriggeredAbility(
|
this.addAbility(new BlocksAttachedTriggeredAbility(
|
||||||
|
|
|
@ -58,6 +58,7 @@ public final class TovolarsPackleader extends CardImpl {
|
||||||
), new ManaCostsImpl<>("{2}{G}{G}"));
|
), new ManaCostsImpl<>("{2}{G}{G}"));
|
||||||
ability.addTarget(new TargetPermanent(filter));
|
ability.addTarget(new TargetPermanent(filter));
|
||||||
ability.addTarget(new TargetPermanent(StaticFilters.FILTER_CREATURE_YOU_DONT_CONTROL));
|
ability.addTarget(new TargetPermanent(StaticFilters.FILTER_CREATURE_YOU_DONT_CONTROL));
|
||||||
|
this.addAbility(ability);
|
||||||
|
|
||||||
// Nightbound
|
// Nightbound
|
||||||
this.addAbility(new NightboundAbility());
|
this.addAbility(new NightboundAbility());
|
||||||
|
|
|
@ -39,6 +39,7 @@ public final class UnscytheKillerOfKings extends CardImpl {
|
||||||
ability.addEffect(new GainAbilityAttachedEffect(
|
ability.addEffect(new GainAbilityAttachedEffect(
|
||||||
FirstStrikeAbility.getInstance(), AttachmentType.EQUIPMENT
|
FirstStrikeAbility.getInstance(), AttachmentType.EQUIPMENT
|
||||||
).setText("and has first strike"));
|
).setText("and has first strike"));
|
||||||
|
this.addAbility(ability);
|
||||||
|
|
||||||
// Whenever a creature dealt damage by equipped creature this turn dies, you may exile that card. If you do, create a 2/2 black Zombie creature token.
|
// Whenever a creature dealt damage by equipped creature this turn dies, you may exile that card. If you do, create a 2/2 black Zombie creature token.
|
||||||
this.addAbility(new UnscytheKillerOfKingsTriggeredAbility(new UnscytheEffect()));
|
this.addAbility(new UnscytheKillerOfKingsTriggeredAbility(new UnscytheEffect()));
|
||||||
|
|
|
@ -36,6 +36,7 @@ public final class WebspinnerCuff extends CardImpl {
|
||||||
ability.addEffect(new GainAbilityAttachedEffect(
|
ability.addEffect(new GainAbilityAttachedEffect(
|
||||||
ReachAbility.getInstance(), AttachmentType.EQUIPMENT
|
ReachAbility.getInstance(), AttachmentType.EQUIPMENT
|
||||||
).setText("and has reach"));
|
).setText("and has reach"));
|
||||||
|
this.addAbility(ability);
|
||||||
|
|
||||||
// Reconfigure {4}
|
// Reconfigure {4}
|
||||||
this.addAbility(new ReconfigureAbility("{4}"));
|
this.addAbility(new ReconfigureAbility("{4}"));
|
||||||
|
|
|
@ -1140,7 +1140,7 @@ public class VerifyCardDataTest {
|
||||||
Token token = (Token) createNewObject(tokenClass);
|
Token token = (Token) createNewObject(tokenClass);
|
||||||
if (token == null) {
|
if (token == null) {
|
||||||
errorsList.add("Error: token must have default constructor with zero params: " + tokenClass.getName());
|
errorsList.add("Error: token must have default constructor with zero params: " + tokenClass.getName());
|
||||||
} else if (tokDataNamesIndex.getOrDefault(token.getName().replace(" Token",""), "").isEmpty()) {
|
} else if (tokDataNamesIndex.getOrDefault(token.getName().replace(" Token", ""), "").isEmpty()) {
|
||||||
errorsList.add("Error: can't find data in card-pictures-tok.txt for token: " + tokenClass.getName() + " -> " + token.getName());
|
errorsList.add("Error: can't find data in card-pictures-tok.txt for token: " + tokenClass.getName() + " -> " + token.getName());
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -1594,13 +1594,9 @@ public class VerifyCardDataTest {
|
||||||
}
|
}
|
||||||
}*/
|
}*/
|
||||||
private static final boolean compareText(String cardText, String refText, String name) {
|
private static final boolean compareText(String cardText, String refText, String name) {
|
||||||
if (cardText.equals(refText)) {
|
return cardText.equals(refText)
|
||||||
return true;
|
|| cardText.replace(name, name.split(", ")[0]).equals(refText)
|
||||||
}
|
|| cardText.replace(name, name.split(" ")[0]).equals(refText);
|
||||||
if (cardText.replace(name, name.split(", ")[0]).equals(refText)) {
|
|
||||||
return true;
|
|
||||||
}
|
|
||||||
return cardText.replace(name, name.split(" ")[0]).equals(refText);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
private static final boolean checkForEffect(Card card, Class<? extends Effect> effectClazz) {
|
private static final boolean checkForEffect(Card card, Class<? extends Effect> effectClazz) {
|
||||||
|
@ -1711,6 +1707,7 @@ public class VerifyCardDataTest {
|
||||||
for (int j = 0; j <= refRules.length - 1; j++) {
|
for (int j = 0; j <= refRules.length - 1; j++) {
|
||||||
String refRule = refRules[j];
|
String refRule = refRules[j];
|
||||||
if (!refRule.startsWith("+ ")) {
|
if (!refRule.startsWith("+ ")) {
|
||||||
|
isFine = false;
|
||||||
refRules[j] = "- " + refRules[j];
|
refRules[j] = "- " + refRules[j];
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Add table
Reference in a new issue