From aad8ad0871c13fe987ff5de72662a3f6a8c8aba2 Mon Sep 17 00:00:00 2001 From: LevelX2 Date: Mon, 14 Jul 2014 14:19:18 +0200 Subject: [PATCH] * Not targeted selections - Fixed a but where not targeted selections did trigger the target events (e.g. Kira, Great Glass-Spinner counter ability was triggered by taping creatures for convoke ability). --- .../mage/sets/betrayersofkamigawa/KiraGreatGlassSpinner.java | 4 ++-- Mage.Sets/src/mage/sets/magic2010/Fireball.java | 2 +- Mage/src/mage/target/TargetImpl.java | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/Mage.Sets/src/mage/sets/betrayersofkamigawa/KiraGreatGlassSpinner.java b/Mage.Sets/src/mage/sets/betrayersofkamigawa/KiraGreatGlassSpinner.java index b3f036c501f..1fc55da4b3f 100644 --- a/Mage.Sets/src/mage/sets/betrayersofkamigawa/KiraGreatGlassSpinner.java +++ b/Mage.Sets/src/mage/sets/betrayersofkamigawa/KiraGreatGlassSpinner.java @@ -67,7 +67,7 @@ public class KiraGreatGlassSpinner extends CardImpl { this.addAbility(FlyingAbility.getInstance()); // Creatures you control have "Whenever this creature becomes the target of a spell or ability for the first time in a turn, counter that spell or ability." - this.addAbility(new SimpleStaticAbility(Zone.BATTLEFIELD, new GainAbilityControlledEffect(new KiraGreatGlassSpinnerAbility(), Duration.WhileOnBattlefield, new FilterCreaturePermanent()))); + this.addAbility(new SimpleStaticAbility(Zone.BATTLEFIELD, new GainAbilityControlledEffect(new KiraGreatGlassSpinnerAbility(), Duration.WhileOnBattlefield, new FilterCreaturePermanent("Creatures you control")))); } @@ -86,7 +86,7 @@ public class KiraGreatGlassSpinner extends CardImpl { class KiraGreatGlassSpinnerAbility extends TriggeredAbilityImpl { - protected Map turnUsed = new HashMap(); + protected Map turnUsed = new HashMap<>(); public KiraGreatGlassSpinnerAbility() { super(Zone.BATTLEFIELD, new CounterTargetEffect(), false); diff --git a/Mage.Sets/src/mage/sets/magic2010/Fireball.java b/Mage.Sets/src/mage/sets/magic2010/Fireball.java index 36bcf3b7836..13d291bc100 100644 --- a/Mage.Sets/src/mage/sets/magic2010/Fireball.java +++ b/Mage.Sets/src/mage/sets/magic2010/Fireball.java @@ -141,7 +141,7 @@ class FireballTargetCreatureOrPlayer extends TargetCreatureOrPlayer { @Override public List getTargetOptions(Ability source, Game game) { - List options = new ArrayList(); + List options = new ArrayList<>(); int xVal = source.getManaCostsToPay().getX(); if (xVal < 1) { diff --git a/Mage/src/mage/target/TargetImpl.java b/Mage/src/mage/target/TargetImpl.java index 0fa8de6a45d..785a8aa538b 100644 --- a/Mage/src/mage/target/TargetImpl.java +++ b/Mage/src/mage/target/TargetImpl.java @@ -219,7 +219,7 @@ public abstract class TargetImpl implements Target { @Override public void addTarget(UUID id, Ability source, Game game) { - addTarget(id, source, game, false); + addTarget(id, source, game, notTarget); } @Override