Merge branch 'master' into 'master'

Don't add a null card to target in animateTgtAi

See merge request core-developers/forge!3272
This commit is contained in:
Michael Kamensky
2020-10-15 09:53:34 +00:00

View File

@@ -340,17 +340,19 @@ public class AnimateAi extends SpellAbilityAi {
// select the worst of the best
final Card worst = ComputerUtilCard.getWorstAI(maxList);
if (worst != null && worst.isLand()) {
// e.g. Clan Guildmage, make sure we're not using the same land we want to animate to activate the ability
this.holdAnimatedTillMain2(ai, worst);
if (!ComputerUtilMana.canPayManaCost(sa, ai, 0)) {
this.releaseHeldTillMain2(ai, worst);
return false;
if (worst != null) {
if (worst.isLand()) {
// e.g. Clan Guildmage, make sure we're not using the same land we want to animate to activate the ability
this.holdAnimatedTillMain2(ai, worst);
if (!ComputerUtilMana.canPayManaCost(sa, ai, 0)) {
this.releaseHeldTillMain2(ai, worst);
return false;
}
}
this.rememberAnimatedThisTurn(ai, worst);
sa.getTargets().add(worst);
}
this.rememberAnimatedThisTurn(ai, worst);
sa.getTargets().add(worst);
return true;
return true;
}
// This is reasonable for now. Kamahl, Fist of Krosa and a sorcery or