- Even though the AI doesn't do anything with AF RearrangeTopOfLibrary, it should at least not miss mandatory triggers so they actually go on stack.

This commit is contained in:
Agetian
2017-01-15 10:37:56 +00:00
parent 126eb33e45
commit 0fda15d033

View File

@@ -12,7 +12,7 @@ public class RearrangeTopOfLibraryAi extends SpellAbilityAi {
*/ */
@Override @Override
protected boolean canPlayAI(Player aiPlayer, SpellAbility sa) { protected boolean canPlayAI(Player aiPlayer, SpellAbility sa) {
return false; return sa.isMandatory(); // AI doesn't do anything with this SA yet, but at least it shouldn't miss mandatory triggers
} }
/* (non-Javadoc) /* (non-Javadoc)
@@ -39,6 +39,9 @@ public class RearrangeTopOfLibraryAi extends SpellAbilityAi {
// if it's just defined, no big deal // if it's just defined, no big deal
} }
return false; // TODO: the AI currently doesn't do anything with this ability, consider improving.
// For now, "true" is returned (without any action) if the SA is mandatory in order
// not to miss triggers.
return sa.isMandatory();
} }
} }