From b446585c0098992e13fd3407e1b769aedc1f5e9a Mon Sep 17 00:00:00 2001
From: North <robyter@gmail>
Date: Mon, 19 Mar 2012 00:28:36 +0200
Subject: [PATCH] [DKA] Helvault fixed ReturnFromExileForSourceEffect

---
 .../src/mage/sets/darkascension/Helvault.java | 85 +++++++++++++++++++
 .../ReturnFromExileForSourceEffect.java       |  4 +-
 2 files changed, 88 insertions(+), 1 deletion(-)
 create mode 100644 Mage.Sets/src/mage/sets/darkascension/Helvault.java

diff --git a/Mage.Sets/src/mage/sets/darkascension/Helvault.java b/Mage.Sets/src/mage/sets/darkascension/Helvault.java
new file mode 100644
index 0000000000..62768cf5c8
--- /dev/null
+++ b/Mage.Sets/src/mage/sets/darkascension/Helvault.java
@@ -0,0 +1,85 @@
+/*
+ *  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.darkascension;
+
+import java.util.UUID;
+import mage.Constants.CardType;
+import mage.Constants.Rarity;
+import mage.Constants.TargetController;
+import mage.Constants.Zone;
+import mage.abilities.common.DiesTriggeredAbility;
+import mage.abilities.common.SimpleActivatedAbility;
+import mage.abilities.costs.common.TapSourceCost;
+import mage.abilities.costs.mana.GenericManaCost;
+import mage.abilities.effects.common.ExileTargetForSourceEffect;
+import mage.abilities.effects.common.ReturnFromExileForSourceEffect;
+import mage.cards.CardImpl;
+import mage.filter.common.FilterCreaturePermanent;
+import mage.target.common.TargetControlledCreaturePermanent;
+import mage.target.common.TargetCreaturePermanent;
+
+/**
+ *
+ * @author North
+ */
+public class Helvault extends CardImpl<Helvault> {
+
+    private static final FilterCreaturePermanent filter = new FilterCreaturePermanent("creature you don't control");
+
+    static {
+        filter.setTargetController(TargetController.NOT_YOU);
+    }
+
+    public Helvault(UUID ownerId) {
+        super(ownerId, 151, "Helvault", Rarity.MYTHIC, new CardType[]{CardType.ARTIFACT}, "{3}");
+        this.expansionSetCode = "DKA";
+        this.supertype.add("Legendary");
+
+        // {1}, {tap}: Exile target creature you control.
+        SimpleActivatedAbility ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, new ExileTargetForSourceEffect("Helvault exile"), new GenericManaCost(1));
+        ability.addCost(new TapSourceCost());
+        ability.addTarget(new TargetControlledCreaturePermanent());
+        this.addAbility(ability);
+        // {7}, {tap}: Exile target creature you don't control.
+        ability = new SimpleActivatedAbility(Zone.BATTLEFIELD, new ExileTargetForSourceEffect("Helvault exile"), new GenericManaCost(7));
+        ability.addCost(new TapSourceCost());
+        ability.addTarget(new TargetCreaturePermanent(filter));
+        this.addAbility(ability);
+        // When Helvault is put into a graveyard from the battlefield, return all cards exiled with it to the battlefield under their owners' control.
+        this.addAbility(new DiesTriggeredAbility(new ReturnFromExileForSourceEffect(Zone.BATTLEFIELD)));
+    }
+
+    public Helvault(final Helvault card) {
+        super(card);
+    }
+
+    @Override
+    public Helvault copy() {
+        return new Helvault(this);
+    }
+}
diff --git a/Mage/src/mage/abilities/effects/common/ReturnFromExileForSourceEffect.java b/Mage/src/mage/abilities/effects/common/ReturnFromExileForSourceEffect.java
index bddae590e5..c9c3c6a7f4 100644
--- a/Mage/src/mage/abilities/effects/common/ReturnFromExileForSourceEffect.java
+++ b/Mage/src/mage/abilities/effects/common/ReturnFromExileForSourceEffect.java
@@ -28,6 +28,7 @@
 
 package mage.abilities.effects.common;
 
+import java.util.LinkedList;
 import java.util.UUID;
 import mage.Constants.Outcome;
 import mage.Constants.Zone;
@@ -73,7 +74,8 @@ public class ReturnFromExileForSourceEffect extends OneShotEffect<ReturnFromExil
 		UUID exileId = source.getSourceId();
 		ExileZone exile = game.getExile().getExileZone(exileId);
 		if (exile != null) {
-			for (UUID cardId: exile) {
+            LinkedList<UUID> cards = new LinkedList<UUID>(exile);
+			for (UUID cardId: cards) {
 				Card card = game.getCard(cardId);
 				card.moveToZone(zone, source.getId(), game, tapped);
 			}