Failure/foul-magics
1
0
Fork 0
forked from External/mage
Code Pull requests Releases Packages Activity Actions

Attempt to fix test by marking dependencies on other land-type changing effects.

Browse source
This commit is contained in:
Grath 2025-08-23 00:24:35 -04:00
parent 848ea5a903
commit f5b6c31152
4 changed files with 29 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

4
Mage.Sets/src/mage/cards/b/BloodMoon.java
View file

@ -48,6 +48,10 @@ public final class BloodMoon extends CardImpl {
this.staticText = "Nonbasic lands are Mountains"; this.staticText = "Nonbasic lands are Mountains";
this.dependencyTypes.add(DependencyType.BecomeMountain); this.dependencyTypes.add(DependencyType.BecomeMountain);
this.dependendToTypes.add(DependencyType.BecomeNonbasicLand); this.dependendToTypes.add(DependencyType.BecomeNonbasicLand);
this.dependendToTypes.add(DependencyType.BecomeForest);
this.dependendToTypes.add(DependencyType.BecomeIsland);
this.dependendToTypes.add(DependencyType.BecomePlains);
this.dependendToTypes.add(DependencyType.BecomeSwamp);
} }
private BloodMoonEffect(final BloodMoonEffect effect) { private BloodMoonEffect(final BloodMoonEffect effect) {

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

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

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

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

17
Mage/src/main/java/mage/abilities/effects/common/continuous/BecomesBasicLandTargetEffect.java
View file

@ -62,6 +62,23 @@ public class BecomesBasicLandTargetEffect extends ContinuousEffectImpl {
} }
this.chooseLandType = chooseLandType; this.chooseLandType = chooseLandType;
this.loseOther = loseOther; this.loseOther = loseOther;
if (this.loseOther) {
if (!landTypes.contains(SubType.MOUNTAIN)) {
dependendToTypes.add(DependencyType.BecomeMountain);
}
if (!landTypes.contains(SubType.FOREST)) {
dependendToTypes.add(DependencyType.BecomeForest);
}
if (!landTypes.contains(SubType.SWAMP)) {
dependendToTypes.add(DependencyType.BecomeSwamp);
}
if (!landTypes.contains(SubType.ISLAND)) {
dependendToTypes.add(DependencyType.BecomeIsland);
}
if (!landTypes.contains(SubType.PLAINS)) {
dependendToTypes.add(DependencyType.BecomePlains);
}
}
} }
protected BecomesBasicLandTargetEffect(final BecomesBasicLandTargetEffect effect) { protected BecomesBasicLandTargetEffect(final BecomesBasicLandTargetEffect effect) {