External/mage
1
0
Fork 1
mirror of https://github.com/magefree/mage.git synced 2025-12-20 02:30:08 -08:00
Code Issues Projects Releases Packages Wiki Activity Actions

Refactor to use DependencyType.BecomesNonbasicLand to track effects which add land type.

Browse source
This commit is contained in:
Grath 2025-08-22 22:38:22 -04:00
parent b085446d04
commit c7f92a093b
21 changed files with 60 additions and 26 deletions
Download patch file Download diff file Expand all files Collapse all files

1
Mage.Sets/src/mage/cards/a/AshayaSoulOfTheWild.java
View file

@ -64,6 +64,7 @@ class AshayaSoulOfTheWildEffect extends ContinuousEffectImpl {
staticText = "Nontoken creatures you control are Forest lands in addition to their other types";
this.dependendToTypes.add(DependencyType.BecomeCreature);
this.dependencyTypes.add(DependencyType.BecomeForest);
this.dependencyTypes.add(DependencyType.BecomeNonbasicLand);
}
private AshayaSoulOfTheWildEffect(final AshayaSoulOfTheWildEffect effect) {

3
Mage.Sets/src/mage/cards/c/CelestialDawn.java
View file

@ -63,6 +63,7 @@ class CelestialDawnToPlainsEffect extends ContinuousEffectImpl {
CelestialDawnToPlainsEffect() {
super(Duration.WhileOnBattlefield, Outcome.Detriment);
this.staticText = "Lands you control are Plains";
this.dependendToTypes.add(DependencyType.BecomeNonbasicLand);
}
private CelestialDawnToPlainsEffect(final CelestialDawnToPlainsEffect effect) {
@ -126,7 +127,7 @@ class CelestialDawnToWhiteEffect extends ContinuousEffectImpl {
}
}
// Exile
for (Card card : game.getExile().getAllCards(game)) {
for (Card card : game.getExile().getAllCardsByRange(game, controller.getId())) {
if (card.isOwnedBy(controller.getId())) {
setColor(card.getColor(game), game);
}

8
Mage.Sets/src/mage/cards/c/ChromaticLantern.java
View file

@ -3,13 +3,14 @@ package mage.cards.c;
import java.util.UUID;
import mage.abilities.common.SimpleStaticAbility;
import mage.abilities.effects.ContinuousEffect;
import mage.abilities.effects.common.continuous.GainAbilityControlledEffect;
import mage.abilities.mana.AnyColorManaAbility;
import mage.cards.CardImpl;
import mage.cards.CardSetInfo;
import mage.constants.CardType;
import mage.constants.DependencyType;
import mage.constants.Duration;
import mage.constants.Zone;
import mage.filter.StaticFilters;
/**
@ -21,7 +22,10 @@ public final class ChromaticLantern extends CardImpl {
super(ownerId,setInfo,new CardType[]{CardType.ARTIFACT},"{3}");
// Lands you control have "{T}: Add one mana of any color."
this.addAbility(new SimpleStaticAbility(new GainAbilityControlledEffect(new AnyColorManaAbility(), Duration.WhileOnBattlefield, StaticFilters.FILTER_LANDS, false)));
ContinuousEffect effect = new GainAbilityControlledEffect(new AnyColorManaAbility(), Duration.WhileOnBattlefield, StaticFilters.FILTER_LANDS, false);
effect.getDependencyTypes().add(DependencyType.BecomeNonbasicLand);
this.addAbility(new SimpleStaticAbility(effect));
// {T}: Add one mana of any color.
this.addAbility(new AnyColorManaAbility());

8
Mage.Sets/src/mage/cards/d/DuneChanter.java
View file

@ -5,6 +5,7 @@ import mage.abilities.Ability;
import mage.abilities.common.SimpleActivatedAbility;
import mage.abilities.common.SimpleStaticAbility;
import mage.abilities.costs.common.TapSourceCost;
import mage.abilities.effects.ContinuousEffect;
import mage.abilities.effects.ContinuousEffectImpl;
import mage.abilities.effects.OneShotEffect;
import mage.abilities.effects.common.GainLifeEffect;
@ -45,10 +46,12 @@ public final class DuneChanter extends CardImpl {
this.addAbility(new SimpleStaticAbility(new DuneChanterContinuousEffect()));
// Lands you control have "{T}: Add one mana of any color."
this.addAbility(new SimpleStaticAbility(new GainAbilityControlledEffect(
ContinuousEffect effect = new GainAbilityControlledEffect(
new AnyColorManaAbility(), Duration.WhileOnBattlefield,
StaticFilters.FILTER_LANDS, false
)));
);
effect.getDependencyTypes().add(DependencyType.BecomeNonbasicLand);
this.addAbility(new SimpleStaticAbility(effect));
// {T}: Mill two cards. You gain 1 life for each land card milled this way.
this.addAbility(new SimpleActivatedAbility(new DuneChanterEffect(), new TapSourceCost()));
@ -76,6 +79,7 @@ class DuneChanterContinuousEffect extends ContinuousEffectImpl {
public DuneChanterContinuousEffect() {
super(Duration.WhileOnBattlefield, Layer.TypeChangingEffects_4, SubLayer.NA, Outcome.Benefit);
staticText = "Lands you control and land cards you own that aren't on the battlefield are Deserts in addition to their other types";
dependendToTypes.add(DependencyType.BecomeNonbasicLand);
}
private DuneChanterContinuousEffect(final DuneChanterContinuousEffect effect) {

1
Mage.Sets/src/mage/cards/e/EncroachingMycosynth.java
View file

@ -45,6 +45,7 @@ class EncroachingMycosynthEffect extends ContinuousEffectImpl {
super(Duration.WhileOnBattlefield, Layer.TypeChangingEffects_4, SubLayer.NA, Outcome.Benefit);
staticText = "Nonland permanents you control are artifacts in addition to their other types. " +
"The same is true for permanent spells you control and nonland permanent cards you own that aren't on the battlefield";
this.dependendToTypes.add(DependencyType.BecomeNonbasicLand);
this.dependencyTypes.add(DependencyType.ArtifactAddingRemoving); // March of the Machines
}

1
Mage.Sets/src/mage/cards/h/HarbingerOfTheSeas.java
View file

@ -54,6 +54,7 @@ class HarbingerOfTheSeasEffect extends ContinuousEffectImpl {
HarbingerOfTheSeasEffect() {
super(Duration.WhileOnBattlefield, Outcome.Detriment);
this.staticText = "Nonbasic lands are Islands";
dependendToTypes.add(DependencyType.BecomeNonbasicLand);
dependencyTypes.add(DependencyType.BecomeIsland);
}

7
Mage.Sets/src/mage/cards/k/KormusBell.java
View file

@ -27,11 +27,8 @@ public final class KormusBell extends CardImpl {
new CreatureToken(1, 1, "1/1 black creatures").withColor("B"),
"lands", filter,
Duration.WhileOnBattlefield, true);
effect.addDependedToType(DependencyType.BecomeSwamp); // TODO: are these dependencies correct/complete?
effect.addDependedToType(DependencyType.BecomeIsland);
effect.addDependedToType(DependencyType.BecomeForest);
effect.addDependedToType(DependencyType.BecomeMountain);
effect.addDependedToType(DependencyType.BecomePlains);
effect.addDependedToType(DependencyType.BecomeNonbasicLand);
effect.addDependedToType(DependencyType.BecomeSwamp);
this.addAbility(new SimpleStaticAbility(effect));
}

2
Mage.Sets/src/mage/cards/l/LeylineOfSingularity.java
View file

@ -3,7 +3,6 @@ package mage.cards.l;
import java.util.UUID;
import mage.abilities.Ability;
import mage.abilities.Mode;
import mage.abilities.common.SimpleStaticAbility;
import mage.abilities.effects.ContinuousEffectImpl;
import mage.abilities.keyword.LeylineAbility;
@ -47,6 +46,7 @@ class SetSupertypeAllEffect extends ContinuousEffectImpl {
public SetSupertypeAllEffect() {
super(Duration.WhileOnBattlefield, Layer.TypeChangingEffects_4, SubLayer.NA, Outcome.Detriment);
this.staticText = "All nonland permanents are legendary";
this.dependendToTypes.add(DependencyType.BecomeNonbasicLand);
}
private SetSupertypeAllEffect(final SetSupertypeAllEffect effect) {

1
Mage.Sets/src/mage/cards/l/LeylineOfTheGuildpact.java
View file

@ -50,6 +50,7 @@ class LeylineOfTheGuildpactEffect extends ContinuousEffectImpl {
LeylineOfTheGuildpactEffect() {
super(Duration.WhileOnBattlefield, Layer.ColorChangingEffects_5, SubLayer.NA, Outcome.Benefit);
staticText = "each nonland permanent you control is all colors";
dependendToTypes.add(DependencyType.BecomeNonbasicLand);
}
private LeylineOfTheGuildpactEffect(final LeylineOfTheGuildpactEffect effect) {

1
Mage.Sets/src/mage/cards/l/LifeAndLimb.java
View file

@ -50,6 +50,7 @@ class LifeAndLimbEffect extends ContinuousEffectImpl {
super(Duration.WhileOnBattlefield, Outcome.Neutral);
staticText = "All Forests and all Saprolings are 1/1 green Saproling creatures and Forest lands in addition to their other types";
this.dependendToTypes.add(DependencyType.BecomeNonbasicLand);
this.dependendToTypes.add(DependencyType.BecomeForest);
this.dependendToTypes.add(DependencyType.BecomeCreature);
}

4
Mage.Sets/src/mage/cards/l/LivingLands.java
View file

@ -26,7 +26,9 @@ public final class LivingLands extends CardImpl {
ContinuousEffect effect = new BecomesCreatureAllEffect(
new CreatureToken(1, 1, "1/1 creatures"),
"lands", filter, Duration.WhileOnBattlefield, false);
effect.getDependencyTypes().add(DependencyType.BecomeForest); // TODO: are these dependencies correct/complete?
effect.getDependedToTypes().add(DependencyType.BecomeNonbasicLand);
effect.getDependedToTypes().add(DependencyType.BecomeForest);
effect.getDependencyTypes().add(DependencyType.BecomeCreature);
this.addAbility(new SimpleStaticAbility(effect));
}

13
Mage.Sets/src/mage/cards/l/LivingPlane.java
View file

@ -2,13 +2,11 @@ package mage.cards.l;
import java.util.UUID;
import mage.abilities.common.SimpleStaticAbility;
import mage.abilities.effects.ContinuousEffect;
import mage.abilities.effects.common.continuous.BecomesCreatureAllEffect;
import mage.cards.CardImpl;
import mage.cards.CardSetInfo;
import mage.constants.CardType;
import mage.constants.Duration;
import mage.constants.SuperType;
import mage.constants.Zone;
import mage.constants.*;
import mage.filter.common.FilterLandPermanent;
import mage.game.permanent.token.custom.CreatureToken;
@ -25,9 +23,12 @@ public final class LivingPlane extends CardImpl {
this.supertype.add(SuperType.WORLD);
// All lands are 1/1 creatures that are still lands.
this.addAbility(new SimpleStaticAbility(new BecomesCreatureAllEffect(
ContinuousEffect effect = new BecomesCreatureAllEffect(
new CreatureToken(1, 1, "1/1 creatures"),
"lands", filter, Duration.WhileOnBattlefield, false)));
"lands", filter, Duration.WhileOnBattlefield, false);
effect.getDependedToTypes().add(DependencyType.BecomeNonbasicLand);
effect.getDependencyTypes().add(DependencyType.BecomeCreature);
this.addAbility(new SimpleStaticAbility(effect));
}
private LivingPlane(final LivingPlane card) {

1
Mage.Sets/src/mage/cards/m/MagusOfTheMoon.java
View file

@ -54,6 +54,7 @@ class MagusOfTheMoonEffect extends ContinuousEffectImpl {
MagusOfTheMoonEffect() {
super(Duration.WhileOnBattlefield, Outcome.Detriment);
this.staticText = "Nonbasic lands are Mountains";
dependendToTypes.add(DependencyType.BecomeNonbasicLand);
dependencyTypes.add(DependencyType.BecomeMountain);
}

1
Mage.Sets/src/mage/cards/m/Melting.java
View file

@ -39,6 +39,7 @@ class MeltingEffect extends ContinuousEffectImpl {
MeltingEffect() {
super(Duration.WhileOnBattlefield, Layer.TypeChangingEffects_4, SubLayer.NA, Outcome.Detriment);
this.staticText = "All lands are no longer snow";
this.dependendToTypes.add(DependencyType.BecomeNonbasicLand);
}
private MeltingEffect(final MeltingEffect effect) {

10
Mage.Sets/src/mage/cards/n/NaturesRevolt.java
View file

@ -2,12 +2,13 @@ package mage.cards.n;
import java.util.UUID;
import mage.abilities.common.SimpleStaticAbility;
import mage.abilities.effects.ContinuousEffect;
import mage.abilities.effects.common.continuous.BecomesCreatureAllEffect;
import mage.cards.CardImpl;
import mage.cards.CardSetInfo;
import mage.constants.CardType;
import mage.constants.DependencyType;
import mage.constants.Duration;
import mage.constants.Zone;
import mage.filter.common.FilterLandPermanent;
import mage.game.permanent.token.custom.CreatureToken;
@ -24,9 +25,12 @@ public final class NaturesRevolt extends CardImpl {
super(ownerId, setInfo, new CardType[]{CardType.ENCHANTMENT}, "{3}{G}{G}");
// All lands are 2/2 creatures that are still lands.
this.addAbility(new SimpleStaticAbility(new BecomesCreatureAllEffect(
ContinuousEffect effect = new BecomesCreatureAllEffect(
new CreatureToken(2, 2, "2/2 creatures"),
"lands", filter, Duration.WhileOnBattlefield, false)));
"lands", filter, Duration.WhileOnBattlefield, false);
effect.getDependedToTypes().add(DependencyType.BecomeNonbasicLand);
effect.getDependencyTypes().add(DependencyType.BecomeCreature);
this.addAbility(new SimpleStaticAbility(effect));
}
private NaturesRevolt(final NaturesRevolt card) {

6
Mage.Sets/src/mage/cards/r/Realmwright.java
View file

@ -51,6 +51,12 @@ class RealmwrightEffect extends ContinuousEffectImpl {
RealmwrightEffect() {
super(Duration.WhileOnBattlefield, Layer.TypeChangingEffects_4, SubLayer.NA, Outcome.Neutral);
staticText = "Lands you control are the chosen type in addition to their other types";
dependendToTypes.add(DependencyType.BecomeNonbasicLand);
dependencyTypes.add(DependencyType.BecomeMountain);
dependencyTypes.add(DependencyType.BecomeForest);
dependencyTypes.add(DependencyType.BecomeSwamp);
dependencyTypes.add(DependencyType.BecomeIsland);
dependencyTypes.add(DependencyType.BecomePlains);
}
private RealmwrightEffect(final RealmwrightEffect effect) {

1
Mage.Sets/src/mage/cards/r/RootpathPurifier.java
View file

@ -47,6 +47,7 @@ class RootpathPurifierEffect extends ContinuousEffectImpl {
RootpathPurifierEffect() {
super(Duration.WhileOnBattlefield, Layer.TypeChangingEffects_4, SubLayer.NA, Outcome.Benefit);
staticText = "lands you control and land cards in your library are basic";
dependendToTypes.add(DependencyType.BecomeNonbasicLand);
}
private RootpathPurifierEffect(final RootpathPurifierEffect effect) {

2
Mage.Sets/src/mage/cards/t/TophTheFirstMetalbender.java
View file

@ -63,6 +63,8 @@ class TophTheFirstMetalbenderEffect extends ContinuousEffectImpl {
TophTheFirstMetalbenderEffect() {
super(Duration.WhileOnBattlefield, Layer.TypeChangingEffects_4, SubLayer.NA, Outcome.Benefit);
staticText = "nontoken artifacts you control are lands in addition to their other types";
this.dependendToTypes.add(DependencyType.ArtifactAddingRemoving);
this.dependencyTypes.add(DependencyType.BecomeNonbasicLand);
}
private TophTheFirstMetalbenderEffect(final TophTheFirstMetalbenderEffect effect) {

5
Mage.Sets/src/mage/cards/u/UrborgTombOfYawgmoth.java
View file

@ -1,6 +1,7 @@
package mage.cards.u;
import mage.abilities.common.SimpleStaticAbility;
import mage.abilities.effects.ContinuousEffect;
import mage.abilities.effects.common.continuous.AddBasicLandTypeAllLandsEffect;
import mage.cards.CardImpl;
import mage.cards.CardSetInfo;
@ -18,7 +19,9 @@ public final class UrborgTombOfYawgmoth extends CardImpl {
this.supertype.add(SuperType.LEGENDARY);
// Each land is a Swamp in addition to its other land types.
this.addAbility(new SimpleStaticAbility(new AddBasicLandTypeAllLandsEffect(SubType.SWAMP)));
ContinuousEffect effect = new AddBasicLandTypeAllLandsEffect(SubType.SWAMP);
effect.getDependedToTypes().add(DependencyType.BecomeNonbasicLand);
this.addAbility(new SimpleStaticAbility(effect));
}
private UrborgTombOfYawgmoth(final UrborgTombOfYawgmoth card) {

6
Mage.Sets/src/mage/cards/y/YavimayaCradleOfGrowth.java
View file

@ -1,10 +1,12 @@
package mage.cards.y;
import mage.abilities.common.SimpleStaticAbility;
import mage.abilities.effects.ContinuousEffect;
import mage.abilities.effects.common.continuous.AddBasicLandTypeAllLandsEffect;
import mage.cards.CardImpl;
import mage.cards.CardSetInfo;
import mage.constants.CardType;
import mage.constants.DependencyType;
import mage.constants.SubType;
import mage.constants.SuperType;
@ -21,7 +23,9 @@ public final class YavimayaCradleOfGrowth extends CardImpl {
this.supertype.add(SuperType.LEGENDARY);
// Each land is a Forest in addition to its other land types.
this.addAbility(new SimpleStaticAbility(new AddBasicLandTypeAllLandsEffect(SubType.FOREST)));
ContinuousEffect effect = new AddBasicLandTypeAllLandsEffect(SubType.FOREST);
effect.getDependedToTypes().add(DependencyType.BecomeNonbasicLand);
this.addAbility(new SimpleStaticAbility(effect));
}
private YavimayaCradleOfGrowth(final YavimayaCradleOfGrowth card) {

4
Mage/src/main/java/mage/abilities/effects/common/continuous/BecomesAllBasicsControlledEffect.java
View file

@ -8,9 +8,6 @@ import mage.filter.StaticFilters;
import mage.game.Game;
import mage.game.permanent.Permanent;
import java.util.ArrayList;
import java.util.List;
/**
* @author TheElk801
*/
@ -27,6 +24,7 @@ public class BecomesAllBasicsControlledEffect extends ContinuousEffectImpl {
public BecomesAllBasicsControlledEffect() {
super(Duration.WhileOnBattlefield, Layer.TypeChangingEffects_4, SubLayer.NA, Outcome.Detriment);
this.staticText = "Lands you control are every basic land type in addition to their other types";
dependendToTypes.add(DependencyType.BecomeNonbasicLand);
dependencyTypes.add(DependencyType.BecomeMountain);
dependencyTypes.add(DependencyType.BecomeForest);
dependencyTypes.add(DependencyType.BecomeSwamp);