external-repos/forge
1
0
Fork 0
mirror of https://github.com/Card-Forge/forge.git synced 2025-11-19 04:08:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

Countering from abilities will have a 100% chance to get used, unless it costs cards to activate, otherwise normal counter settings are used.

Browse Source 
This ensures the AI never wastes an opportunity to counter a spell if it does not cost him a card - holding a "tap to counter target spell" ability for later use is generally bad as the AI cannot determine if they need the mana later or not - but in most cases they don't.

Required for cards Null Brooch and Ertai Wizard Adept in Tempest quest world.
This commit is contained in:
Seravy
2018-02-13 13:56:06 +01:00
parent 36917c0d18
commit b5dbb7da90
1 changed files with 14 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
forge-ai/src/main/java/forge/ai/ability/CounterAi.java
View File

@@ -7,6 +7,9 @@ import forge.game.ability.ApiType;
import forge.game.card.Card;
import forge.game.card.CardFactoryUtil;
import forge.game.cost.Cost;
import forge.game.cost.CostDiscard;
import forge.game.cost.CostExile;
import forge.game.cost.CostSacrifice;
import forge.game.player.Player;
import forge.game.spellability.SpellAbility;
import forge.game.spellability.SpellAbilityStackInstance;
@@ -201,6 +204,17 @@ public class CounterAi extends SpellAbilityAi {
}
}
// Should ALWAYS counter if it doesn't spend a card, otherwise it wastes an opportunity
// to gain card advantage
if (sa.isAbility()
&& (!sa.getPayCosts().hasSpecificCostType(CostDiscard.class))
&& (!sa.getPayCosts().hasSpecificCostType(CostSacrifice.class))
&& (!sa.getPayCosts().hasSpecificCostType(CostExile.class))
// maybe also disallow CostPayLife?
) {
dontCounter = false;
}
if (dontCounter) {
return false;
}