github/mage
1
0
Fork 0
mirror of https://github.com/correl/mage.git synced 2024-12-25 11:11:16 +00:00
Code Issues Projects Releases Packages Wiki Activity

Adding check to Flagbearer requirement that only checks for spells that are cast and abilities that are activated -- choosing targets from triggered abilities is exempt.

Browse source
This commit is contained in:
Clint Herron 2017-03-16 03:26:48 -04:00
parent 4824b96640
commit 3eb9be1b18
1 changed files with 13 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
Mage/src/main/java/mage/abilities/effects/common/ruleModifying/TargetsHaveToTargetPermanentIfAbleEffect.java
View file

@ -9,6 +9,8 @@ import java.util.List;
import java.util.UUID; import java.util.UUID;
import mage.MageObject; import mage.MageObject;
import mage.abilities.Ability; import mage.abilities.Ability;
import mage.abilities.ActivatedAbility;
import mage.abilities.SpellAbility;
import mage.abilities.effects.ContinuousRuleModifyingEffectImpl; import mage.abilities.effects.ContinuousRuleModifyingEffectImpl;
import mage.constants.Duration; import mage.constants.Duration;
import mage.constants.Outcome; import mage.constants.Outcome;
@ -24,6 +26,11 @@ import mage.target.Target;
* 6/8/2016 If a spell or ability’s targets are changed, or if a copy of a spell * 6/8/2016 If a spell or ability’s targets are changed, or if a copy of a spell
* or ability is put onto the stack and has new targets chosen, it doesn’t have * or ability is put onto the stack and has new targets chosen, it doesn’t have
* to target a Flagbearer. * to target a Flagbearer.
*
* 3/16/2017 A Flagbearer only requires targeting of itself when choosing targets
* as a result of casting a spell or activating an ability. Notably, triggered
* abilities are exempt from this targeting restriction (in addition to the note
* about choosing targets for a copy from 6/8/2016).
* *
* @author LevelX2 * @author LevelX2
*/ */
@ -73,6 +80,12 @@ public class TargetsHaveToTargetPermanentIfAbleEffect extends ContinuousRuleModi
if (stackObject.isCopy()) { if (stackObject.isCopy()) {
return false; return false;
} }
Ability stackAbility = stackObject.getStackAbility();
// Ensure that this ability is activated or a cast spell, because Flag Bearer effects don't require triggered abilities to choose a Standard Bearer
if (!(stackAbility instanceof ActivatedAbility) &&
!(stackAbility instanceof SpellAbility)) {
return false;
}
Ability ability = (Ability) getValue("targetAbility"); Ability ability = (Ability) getValue("targetAbility");
if (ability != null) { if (ability != null) {
// Get all the allowed permanents on the battlefield in range of the abilities controller // Get all the allowed permanents on the battlefield in range of the abilities controller