mirror of
https://github.com/correl/mage.git
synced 2024-12-25 03:00:15 +00:00
Ability picker is shown now for only one ability if preference setting is on and ability has no mana costs or is no play land ability.
This commit is contained in:
parent
1ae85dc1b4
commit
2b59a6faae
1 changed files with 3 additions and 8 deletions
|
@ -48,6 +48,7 @@ import mage.abilities.TriggeredAbility;
|
||||||
import mage.abilities.costs.Cost;
|
import mage.abilities.costs.Cost;
|
||||||
import mage.abilities.costs.VariableCost;
|
import mage.abilities.costs.VariableCost;
|
||||||
import mage.abilities.costs.common.PayLifeCost;
|
import mage.abilities.costs.common.PayLifeCost;
|
||||||
|
import mage.abilities.costs.common.RemoveCounterCost;
|
||||||
import mage.abilities.costs.common.SacrificeSourceCost;
|
import mage.abilities.costs.common.SacrificeSourceCost;
|
||||||
import mage.abilities.costs.common.TapSourceCost;
|
import mage.abilities.costs.common.TapSourceCost;
|
||||||
import mage.abilities.costs.mana.ManaCost;
|
import mage.abilities.costs.mana.ManaCost;
|
||||||
|
@ -877,14 +878,8 @@ public class HumanPlayer extends PlayerImpl<HumanPlayer> {
|
||||||
if (ability instanceof ManaAbility) {
|
if (ability instanceof ManaAbility) {
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
for(Cost cost : ability.getCosts() ) {
|
// if ability has no mana costs you have to pick it from ability picker
|
||||||
if (!(cost instanceof TapSourceCost) && !(cost instanceof PayLifeCost)) {
|
return !ability.getManaCostsToPay().isEmpty();
|
||||||
// if cost exists that have to be paid, pick ability dialog can be suppressed
|
|
||||||
return true;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
return false;
|
|
||||||
|
|
||||||
}
|
}
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue