Fixed layer issues

This commit is contained in:
jmharmon 2020-09-06 13:32:05 -07:00 committed by GitHub
parent 394e88f041
commit 93a8e176de
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -17,9 +17,7 @@ import mage.filter.predicate.Predicates;
import mage.filter.predicate.permanent.TokenPredicate; import mage.filter.predicate.permanent.TokenPredicate;
import mage.game.Game; import mage.game.Game;
import mage.game.permanent.Permanent; import mage.game.permanent.Permanent;
import mage.players.Player;
import java.util.List;
import java.util.UUID; import java.util.UUID;
/** /**
@ -66,6 +64,7 @@ class AshayaSoulOfTheWildEffect extends ContinuousEffectImpl {
public AshayaSoulOfTheWildEffect() { public AshayaSoulOfTheWildEffect() {
super(Duration.WhileOnBattlefield, Outcome.Neutral); super(Duration.WhileOnBattlefield, Outcome.Neutral);
staticText = "Nontoken creatures you control are Forest lands in addition to their other types"; staticText = "Nontoken creatures you control are Forest lands in addition to their other types";
this.dependencyTypes.add(DependencyType.BecomeForest);
} }
public AshayaSoulOfTheWildEffect(final AshayaSoulOfTheWildEffect effect) { public AshayaSoulOfTheWildEffect(final AshayaSoulOfTheWildEffect effect) {
@ -79,35 +78,22 @@ class AshayaSoulOfTheWildEffect extends ContinuousEffectImpl {
@Override @Override
public boolean apply(Layer layer, SubLayer subLayer, Ability source, Game game) { public boolean apply(Layer layer, SubLayer subLayer, Ability source, Game game) {
Player you = game.getPlayer(source.getControllerId()); for (Permanent land : game.getBattlefield().getAllActivePermanents(filter, source.getControllerId(), game)) {
List<Permanent> creatures = game.getBattlefield().getActivePermanents(filter, source.getControllerId(), game);
if (you != null) {
for (Permanent creature : creatures) {
if (creature != null) {
switch (layer) { switch (layer) {
case TypeChangingEffects_4: case TypeChangingEffects_4:
creature.addCardType(CardType.LAND); // land abilities are intrinsic, so add them here, not in layer 6
creature.getSubtype(game).add(SubType.FOREST); if (!land.hasSubtype(SubType.FOREST, game)) {
land.getSubtype(game).add(SubType.FOREST);
if (!land.getAbilities(game).containsClass(GreenManaAbility.class)) {
land.addAbility(new GreenManaAbility(), source.getSourceId(), game);
}
}
land.addCardType(CardType.LAND);
break; break;
case AbilityAddingRemovingEffects_6:
boolean flag = false;
for (Ability ability : creature.getAbilities(game)) {
if (ability instanceof GreenManaAbility) {
flag = true;
break;
}
}
if (!flag) {
creature.addAbility(new GreenManaAbility(), source.getSourceId(), game);
}
break;
}
} }
} }
return true; return true;
} }
return false;
}
@Override @Override
public boolean apply(Game game, Ability source) { public boolean apply(Game game, Ability source) {
@ -116,6 +102,6 @@ class AshayaSoulOfTheWildEffect extends ContinuousEffectImpl {
@Override @Override
public boolean hasLayer(Layer layer) { public boolean hasLayer(Layer layer) {
return layer == Layer.TypeChangingEffects_4 || layer == Layer.AbilityAddingRemovingEffects_6; return layer == Layer.TypeChangingEffects_4;
} }
} }