* Zoetic Cavern - Fixed that if cast with Morph the spell was not handled as spell with card type "Creature" (e.g. causing problem with Possibility Storm).

Mage.Tests/src/test/java/org/mage/test/cards/triggers/PossibilityStormTest.java

@@ -0,0 +1,95 @@
+/*
+ *  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 org.mage.test.cards.triggers;
+
+
+import mage.cards.Card;
+import mage.constants.PhaseStep;
+import mage.constants.Zone;
+import org.junit.Assert;
+import org.junit.Test;
+import org.mage.test.serverside.base.CardTestPlayerBase;
+
+/**
+ *
+ * @author LevelX2
+ */
+
+public class PossibilityStormTest extends CardTestPlayerBase {
+
+    /**
+     * There's currently a bug with Possibility Storm and Zoetic Cavern. The way
+     * it's supposed to work is the P. Storm trigger exiles Zoetic Cavern and
+     * then uses last known information about the spell to determine the type of
+     * card the trigger is looking for(creature in this instance). Instead it's
+     * basing the type solely off what's printed on the card. What happened to
+     * me earlier was the trigger skipped right over an Emrakul and then
+     * revealed a Flooded Strand. I was prompted whether or not I wanted to
+     * "cast" Flooded Strand without paying it's cost. Eventually I clicked yes
+     * and it produced a Game Error that resulted in rollback. I recreated the
+     * error against an AI opponent and copied the code. Can't actually post it
+     * because the filter on this site claims it makes my post look too
+     * "spammy". Here's a screenshot of it instead(in spoiler tag).
+     */
+    
+    @Test
+    public void TestWithZoeticCavern() {
+        addCard(Zone.BATTLEFIELD, playerA, "Swamp", 3);
+        // Whenever a player casts a spell from his or her hand, that player exiles it, then exiles cards from
+        // the top of his or her library until he or she exiles a card that shares a card type with it. That
+        // player may cast that card without paying its mana cost. Then he or she puts all cards exiled with
+        // Possibility Storm on the bottom of his or her library in a random order.        
+        addCard(Zone.BATTLEFIELD, playerA, "Possibility Storm", 2);
+        
+        // {T}: Add {1} to your mana pool.
+        // Morph {2}        
+        addCard(Zone.HAND, playerA, "Zoetic Cavern");
+
+        addCard(Zone.LIBRARY, playerA, "Silvercoat Lion");
+        skipInitShuffling();
+
+        playLand(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Zoetic Cavern");
+        setChoice(playerA, "Yes");
+
+        setStopAt(1, PhaseStep.BEGIN_COMBAT);
+        execute();
+
+        assertPermanentCount(playerA, "Zoetic Cavern", 0);
+        
+        boolean zoeticCavernInLibrary = false;
+        for (Card card: playerA.getLibrary().getCards(currentGame)) {
+            if (card.getName().equals("Zoetic Cavern")) {
+                zoeticCavernInLibrary = true;
+            }
+        }
+        Assert.assertEquals("Zoetic Cavern has to be in the library", true, zoeticCavernInLibrary);
+        
+        assertPermanentCount(playerA, "Silvercoat Lion", 1);
+
+    }
+}

Mage/src/mage/abilities/AbilityImpl.java

@@ -299,7 +299,7 @@ public abstract class AbilityImpl implements Ability {
         // A player can't apply two alternative methods of casting or two alternative costs to a single spell.
         if (!activateAlternateOrAdditionalCosts(sourceObject, noMana, controller, game)){
             if (getAbilityType().equals(AbilityType.SPELL)
-                    && ((SpellAbility) this).getSpellAbilityType().equals(SpellAbilityType.LAND_ALTERNATE)) {
+                    && ((SpellAbility) this).getSpellAbilityType().equals(SpellAbilityType.FACE_DOWN_CREATURE)) {
                 return false;
             }
         }

Mage/src/mage/constants/SpellAbilityType.java

@@ -7,7 +7,7 @@ package mage.constants;
 public enum SpellAbilityType {
     BASE("Basic SpellAbility"),
     BASE_ALTERNATE("Basic SpellAbility Alternate"), // used for Overload, Flashback to know they must be handled as Alternate casting costs
-    LAND_ALTERNATE("Basic SpellAbility Alternate Land"), // used for Lands with Morph to cast as Face Down creature
+    FACE_DOWN_CREATURE("Face down creature"), // used for Lands with Morph to cast as Face Down creature
     SPLIT("Split SpellAbility"),
     SPLIT_FUSED("Split SpellAbility"),
     SPLIT_LEFT("LeftSplit SpellAbility"),

Mage/src/mage/game/stack/Spell.java

@@ -372,6 +372,11 @@ public class Spell extends StackObjImpl implements Card {
 
     @Override
     public List<CardType> getCardType() {
+        if (this.getSpellAbility().getSpellAbilityType().equals(SpellAbilityType.FACE_DOWN_CREATURE)) {
+            List<CardType> cardTypes = new ArrayList<>();
+            cardTypes.add(CardType.CREATURE);
+            return cardTypes;
+        }
         if (this.getSpellAbility() instanceof BestowAbility) {
             List<CardType> cardTypes = new ArrayList<>();
             cardTypes.addAll(card.getCardType());

Mage/src/mage/players/PlayerImpl.java

@@ -989,7 +989,7 @@ public abstract class PlayerImpl implements Player, Serializable {
             }
         }
         if (found) {
-            SpellAbility spellAbility = new SpellAbility(null, "", game.getState().getZone(card.getId()), SpellAbilityType.LAND_ALTERNATE);
+            SpellAbility spellAbility = new SpellAbility(null, "", game.getState().getZone(card.getId()), SpellAbilityType.FACE_DOWN_CREATURE);
             spellAbility.setControllerId(this.getId());
             spellAbility.setSourceId(card.getId());
             if (cast(spellAbility, game, false)) {