Added Dwarven Landslide.

Mage.Sets/src/mage/sets/apocalypse/DwarvenLandslide.java

@@ -0,0 +1,80 @@
+/*
+ *  Copyright 2010 BetaSteward_at_googlemail.com. All rights reserved.
+ *
+ *  Redistribution and use in source and binary forms, with or without modification, are
+ *  permitted provided that the following conditions are met:
+ *
+ *     1. Redistributions of source code must retain the above copyright notice, this list of
+ *        conditions and the following disclaimer.
+ *
+ *     2. Redistributions in binary form must reproduce the above copyright notice, this list
+ *        of conditions and the following disclaimer in the documentation and/or other materials
+ *        provided with the distribution.
+ *
+ *  THIS SOFTWARE IS PROVIDED BY BetaSteward_at_googlemail.com ``AS IS'' AND ANY EXPRESS OR IMPLIED
+ *  WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND
+ *  FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL BetaSteward_at_googlemail.com OR
+ *  CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ *  CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+ *  SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON
+ *  ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+ *  NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
+ *  ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ *  The views and conclusions contained in the software and documentation are those of the
+ *  authors and should not be interpreted as representing official policies, either expressed
+ *  or implied, of BetaSteward_at_googlemail.com.
+ */
+package mage.sets.apocalypse;
+
+import java.util.UUID;
+import mage.abilities.Ability;
+import mage.abilities.condition.common.KickedCondition;
+import mage.abilities.costs.Cost;
+import mage.abilities.costs.Costs;
+import mage.abilities.costs.CostsImpl;
+import mage.abilities.costs.common.SacrificeTargetCost;
+import mage.abilities.costs.mana.ManaCostsImpl;
+import mage.abilities.effects.common.DestroyTargetEffect;
+import mage.abilities.keyword.KickerAbility;
+import mage.cards.CardImpl;
+import mage.constants.CardType;
+import mage.constants.Rarity;
+import mage.filter.common.FilterControlledLandPermanent;
+import mage.game.Game;
+import mage.target.common.TargetControlledPermanent;
+import mage.target.common.TargetLandPermanent;
+
+/**
+ *
+ * @author LevelX2
+ */
+public class DwarvenLandslide extends CardImpl {
+
+    public DwarvenLandslide(UUID ownerId) {
+        super(ownerId, 60, "Dwarven Landslide", Rarity.COMMON, new CardType[]{CardType.SORCERY}, "{3}{R}");
+        this.expansionSetCode = "APC";
+
+        // Kicker-{2}{R}, Sacrifice a land.
+        Costs<Cost> kickerCosts = new CostsImpl<>();
+        kickerCosts.add(new ManaCostsImpl<>("{2}{R}"));
+        kickerCosts.add(new SacrificeTargetCost(new TargetControlledPermanent(new FilterControlledLandPermanent("a land"))));
+        this.addAbility(new KickerAbility(kickerCosts));
+        // Destroy target land. If Dwarven Landslide was kicked, destroy another target land.
+        getSpellAbility().addEffect(new DestroyTargetEffect("Destroy target land. If {this} was kicked, destroy another target land"));
+    }
+
+    public DwarvenLandslide(final DwarvenLandslide card) {
+        super(card);
+    }
+
+    @Override
+    public void adjustTargets(Ability ability, Game game) {
+        ability.addTarget(new TargetLandPermanent(KickedCondition.getInstance().apply(game, ability) ? 2 : 1));
+    }
+
+    @Override
+    public DwarvenLandslide copy() {
+        return new DwarvenLandslide(this);
+    }
+}

Mage/src/mage/abilities/costs/OptionalAdditionalCost.java

@@ -25,13 +25,12 @@
  *  authors and should not be interpreted as representing official policies, either expressed
  *  or implied, of BetaSteward_at_googlemail.com.
  */
-
 package mage.abilities.costs;
 
 /**
  * @author LevelX2
  */
-public interface OptionalAdditionalCost extends Cost {
+public interface OptionalAdditionalCost extends Costs {
 
     String getName();
 
@@ -52,15 +51,15 @@ public interface OptionalAdditionalCost extends Cost {
     String getReminderText();
 
     /**
-     * Returns a text suffix for the game log, that can be added to
+     * Returns a text suffix for the game log, that can be added to the cast
-     * the cast message.
+     * message.
      *
-     * @param position - if there are multiple costs, it's the postion the cost is set (starting with 0)
+     * @param position - if there are multiple costs, it's the postion the cost
+     * is set (starting with 0)
      * @return
      */
     String getCastSuffixMessage(int position);
 
-
     /**
      * If the player intends to pay the cost, the cost will be activated
      *
@@ -96,6 +95,7 @@ public interface OptionalAdditionalCost extends Cost {
 
     /**
      * Returns the number of times the cost was activated
+     *
      * @return
      */
     int getActivateCount();

Mage/src/mage/abilities/keyword/KickerAbility.java

@@ -37,6 +37,7 @@ import mage.abilities.SpellAbility;
 import mage.abilities.StaticAbility;
 import mage.abilities.costs.Cost;
 import mage.abilities.costs.Costs;
+import mage.abilities.costs.CostsImpl;
 import mage.abilities.costs.OptionalAdditionalCost;
 import mage.abilities.costs.OptionalAdditionalCostImpl;
 import mage.abilities.costs.OptionalAdditionalSourceCosts;
@@ -217,26 +218,16 @@ public class KickerAbility extends StaticAbility implements OptionalAdditionalSo
                         if (kickerCost.canPay(ability, sourceId, controllerId, game)
                                 && player.chooseUse(Outcome.Benefit, "Pay " + times + kickerCost.getText(false) + " ?", ability, game)) {
                             this.activateKicker(kickerCost, ability, game);
-                            for (Iterator it = ((Costs) kickerCost).iterator(); it.hasNext();) {
+                            for (Iterator itKickerCost = kickerCost.iterator(); itKickerCost.hasNext();) {
-                                Cost cost = (Cost) it.next();
+                                Object kickerCostObject = itKickerCost.next();
-                                if (cost instanceof ManaCostsImpl) {
+                                if ((kickerCostObject instanceof Costs) || (kickerCostObject instanceof CostsImpl)) {
-                                    List<VariableManaCost> varCosts = ((ManaCostsImpl) cost).getVariableCosts();
+                                    for (@SuppressWarnings("unchecked") Iterator<Cost> itDetails = ((Costs) kickerCostObject).iterator(); itDetails.hasNext();) {
-                                    if (!varCosts.isEmpty()) {
+                                        addKickerCostsToAbility(itDetails.next(), ability, game);
-                                        // use only first variable cost
-                                        xManaValue = game.getPlayer(this.controllerId).announceXMana(varCosts.get(0).getMinX(), Integer.MAX_VALUE, "Announce kicker value for " + varCosts.get(0).getText(), game, this);
-                                        // kicker variable X costs handled internally as multikicker with {1} cost (no multikicker on card)
-                                        if (!game.isSimulation()) {
-                                            game.informPlayers(game.getPlayer(this.controllerId).getLogName() + " announced a value of " + xManaValue + " for " + " kicker X ");
-                                        }
-                                        ability.getManaCostsToPay().add(new GenericManaCost(xManaValue));
-                                    } else {
-                                        ability.getManaCostsToPay().add((ManaCostsImpl) cost.copy());
                                     }
                                 } else {
-                                    ability.getCosts().add(cost.copy());
+                                    addKickerCostsToAbility((Cost) kickerCostObject, ability, game);
                                 }
                             }
-
                             again = kickerCost.isRepeatable();
                         } else {
                             again = false;
@@ -247,6 +238,26 @@ public class KickerAbility extends StaticAbility implements OptionalAdditionalSo
         }
     }
 
+    private void addKickerCostsToAbility(Cost cost, Ability ability, Game game) {
+        if (cost instanceof ManaCostsImpl) {
+            @SuppressWarnings("unchecked")
+            List<VariableManaCost> varCosts = ((ManaCostsImpl) cost).getVariableCosts();
+            if (!varCosts.isEmpty()) {
+                // use only first variable cost
+                xManaValue = game.getPlayer(this.controllerId).announceXMana(varCosts.get(0).getMinX(), Integer.MAX_VALUE, "Announce kicker value for " + varCosts.get(0).getText(), game, this);
+                // kicker variable X costs handled internally as multikicker with {1} cost (no multikicker on card)
+                if (!game.isSimulation()) {
+                    game.informPlayers(game.getPlayer(this.controllerId).getLogName() + " announced a value of " + xManaValue + " for " + " kicker X ");
+                }
+                ability.getManaCostsToPay().add(new GenericManaCost(xManaValue));
+            } else {
+                ability.getManaCostsToPay().add((ManaCostsImpl) cost.copy());
+            }
+        } else {
+            ability.getCosts().add(cost.copy());
+        }
+    }
+
     @Override
     public String getRule() {
         StringBuilder sb = new StringBuilder();