[ICE] Implement Kjeldoran Guard (#11184)

closes #8593

Mage.Sets/src/mage/cards/k/KjeldoranGuard.java

@@ -0,0 +1,155 @@
+package mage.cards.k;
+
+import mage.MageInt;
+import mage.MageObjectReference;
+import mage.abilities.Ability;
+import mage.abilities.DelayedTriggeredAbility;
+import mage.abilities.condition.CompoundCondition;
+import mage.abilities.condition.InvertCondition;
+import mage.abilities.condition.common.DefendingPlayerControlsNoSourceCondition;
+import mage.abilities.condition.common.IsPhaseCondition;
+import mage.abilities.costs.common.TapSourceCost;
+import mage.abilities.decorator.ConditionalActivatedAbility;
+import mage.abilities.effects.OneShotEffect;
+import mage.abilities.effects.common.SacrificeTargetEffect;
+import mage.abilities.effects.common.continuous.BoostTargetEffect;
+import mage.cards.CardImpl;
+import mage.cards.CardSetInfo;
+import mage.constants.*;
+import mage.filter.common.FilterLandPermanent;
+import mage.game.Game;
+import mage.game.events.GameEvent;
+import mage.game.permanent.Permanent;
+import mage.target.common.TargetCreaturePermanent;
+import mage.target.targetpointer.FixedTarget;
+
+import java.util.UUID;
+
+/**
+ * @author Alex-Vasile, Susucr
+ */
+public final class KjeldoranGuard extends CardImpl {
+
+    private static final FilterLandPermanent filter = new FilterLandPermanent();
+
+    static {
+        filter.add(SuperType.SNOW.getPredicate());
+    }
+
+    public KjeldoranGuard(UUID ownerId, CardSetInfo setInfo) {
+        super(ownerId, setInfo, new CardType[]{CardType.CREATURE}, "{1}{W}");
+
+        this.subtype.add(SubType.HUMAN);
+        this.subtype.add(SubType.SOLDIER);
+        this.power = new MageInt(1);
+        this.toughness = new MageInt(1);
+
+        // {T}: Target creature gets +1/+1 until end of turn. When that creature leaves the battlefield this turn, sacrifice Kjeldoran Guard. Activate only during combat and only if defending player controls no snow lands.
+        CompoundCondition condition = new CompoundCondition(
+                new IsPhaseCondition(TurnPhase.COMBAT, false), // Only during combat
+                new InvertCondition(new DefendingPlayerControlsNoSourceCondition(filter)) // Only if defending player controls no snow land
+        );
+
+        Ability ability = new ConditionalActivatedAbility(
+                Zone.BATTLEFIELD,
+                new KjeldoranGuardEffect(),
+                new TapSourceCost(),
+                condition,
+                "{T}: Target creature gets +1/+1 until end of turn. "
+                        + "When that creature leaves the battlefield this turn, sacrifice {this}. "
+                        + "Activate only during combat and only if defending player controls no snow lands."
+        );
+        ability.addTarget(new TargetCreaturePermanent());
+        this.addAbility(ability);
+    }
+
+    private KjeldoranGuard(final KjeldoranGuard card) {
+        super(card);
+    }
+
+    @Override
+    public KjeldoranGuard copy() {
+        return new KjeldoranGuard(this);
+    }
+}
+
+class KjeldoranGuardEffect extends OneShotEffect {
+
+    KjeldoranGuardEffect() {
+        super(Outcome.BoostCreature);
+        staticText = "Target creature gets +1/+1 until end of turn. "
+                + "When that creature leaves the battlefield this turn, sacrifice {this}.";
+    }
+
+    private KjeldoranGuardEffect(KjeldoranGuardEffect effect) {
+        super(effect);
+    }
+
+    @Override
+    public KjeldoranGuardEffect copy() {
+        return new KjeldoranGuardEffect(this);
+    }
+
+    @Override
+    public boolean apply(Game game, Ability source) {
+        Permanent targetPermanent = game.getPermanent(source.getFirstTarget());
+        if (targetPermanent == null) {
+            return false;
+        }
+
+        // Target creature gets +1/+1 until end of turn.
+        BoostTargetEffect buffEffect = new BoostTargetEffect(1, 1, Duration.EndOfTurn);
+        buffEffect.setTargetPointer(new FixedTarget(targetPermanent, game));
+        game.addEffect(buffEffect, source);
+
+        // When that creature leaves the battlefield this turn, sacrifice Kjeldoran Guard.
+        DelayedTriggeredAbility delayed = new KjeldoranGuardDelayedTriggeredAbility(
+                new MageObjectReference(targetPermanent, game)
+        );
+
+        // Locking in the Kjeldoran Guard (and its zcc), to be sacrificed later.
+        Permanent guard = source.getSourcePermanentIfItStillExists(game);
+        if(guard != null) {
+            delayed.getEffects().setTargetPointer(new FixedTarget(guard, game));
+        }
+        game.addDelayedTriggeredAbility(delayed, source);
+
+        return true;
+    }
+}
+
+class KjeldoranGuardDelayedTriggeredAbility extends DelayedTriggeredAbility {
+
+    private final MageObjectReference mor;
+
+    KjeldoranGuardDelayedTriggeredAbility(MageObjectReference mor) {
+        super(new SacrificeTargetEffect(), Duration.EndOfTurn, true);
+        this.mor = mor;
+    }
+
+    private KjeldoranGuardDelayedTriggeredAbility(final KjeldoranGuardDelayedTriggeredAbility ability) {
+        super(ability);
+        this.mor = ability.mor;
+    }
+
+    @Override
+    public KjeldoranGuardDelayedTriggeredAbility copy() {
+        return new KjeldoranGuardDelayedTriggeredAbility(this);
+    }
+
+    @Override
+    public boolean checkEventType(GameEvent event, Game game) {
+        return event.getType() == GameEvent.EventType.ZONE_CHANGE;
+    }
+
+    @Override
+    public boolean checkTrigger(GameEvent event, Game game) {
+        Permanent permanentLeaving = game.getPermanentOrLKIBattlefield(event.getTargetId());
+        return permanentLeaving != null && mor.equals(new MageObjectReference(permanentLeaving, game));
+    }
+
+    @Override
+    public String getRule() {
+        return "When that creature leaves the battlefield, Sacrifice {this}";
+    }
+}

Mage.Sets/src/mage/sets/IceAge.java

@@ -199,6 +199,7 @@ public final class IceAge extends ExpansionSet {
         cards.add(new SetCardInfo("Kjeldoran Dead", 137, Rarity.COMMON, mage.cards.k.KjeldoranDead.class));
         cards.add(new SetCardInfo("Kjeldoran Elite Guard", 34, Rarity.UNCOMMON, mage.cards.k.KjeldoranEliteGuard.class));
         cards.add(new SetCardInfo("Kjeldoran Frostbeast", 296, Rarity.UNCOMMON, mage.cards.k.KjeldoranFrostbeast.class));
+        cards.add(new SetCardInfo("Kjeldoran Guard", 35, Rarity.COMMON, mage.cards.k.KjeldoranGuard.class));
         cards.add(new SetCardInfo("Kjeldoran Knight", 36, Rarity.RARE, mage.cards.k.KjeldoranKnight.class));
         cards.add(new SetCardInfo("Kjeldoran Phalanx", 37, Rarity.RARE, mage.cards.k.KjeldoranPhalanx.class));
         cards.add(new SetCardInfo("Kjeldoran Royal Guard", 38, Rarity.RARE, mage.cards.k.KjeldoranRoyalGuard.class));

Mage/src/main/java/mage/abilities/condition/common/DefendingPlayerControlsNoSourceCondition.java

@@ -10,6 +10,10 @@ import mage.game.Game;
 /**
  * This condition always returns false outside of the combat phase.
  *
+ * The name is bad, but "No Source" is refering to the fact this is to be used outside
+ * a "Whenever ~ attacks," or similar, where the "defending player" takes another
+ * meaning. {@link DefendingPlayerControlsNoSourcePredicate} for more info.
+ *
  * @author Susucr
  */
 public class DefendingPlayerControlsNoSourceCondition implements Condition {