spjspj - Fix for Paradoxical Outcome (KLD)

Mage.Sets/src/mage/sets/kaladesh/ParadoxicalOutcome.java

@@ -28,18 +28,27 @@
 package mage.sets.kaladesh;
 
 import java.util.UUID;
+import mage.abilities.Ability;
 import mage.abilities.dynamicvalue.DynamicValue;
-import mage.abilities.dynamicvalue.common.SweepNumber;
+import mage.abilities.effects.Effect;
+import mage.abilities.effects.OneShotEffect;
 import mage.abilities.effects.common.DrawCardSourceControllerEffect;
-import mage.abilities.effects.keyword.SweepEffect;
+import mage.cards.Card;
 import mage.cards.CardImpl;
+import mage.cards.Cards;
+import mage.cards.CardsImpl;
 import mage.constants.CardType;
+import mage.constants.Outcome;
 import mage.constants.Rarity;
-import mage.filter.FilterPermanent;
+import mage.constants.Zone;
 import mage.filter.common.FilterControlledPermanent;
 import mage.filter.predicate.Predicates;
 import mage.filter.predicate.mageobject.CardTypePredicate;
 import mage.filter.predicate.permanent.TokenPredicate;
+import mage.game.Game;
+import mage.players.Player;
+import mage.target.common.TargetControlledPermanent;
+import mage.util.CardUtil;
 
 /**
  *
@@ -47,17 +56,21 @@ import mage.filter.predicate.permanent.TokenPredicate;
  */
 public class ParadoxicalOutcome extends CardImpl {
 
+    private static FilterControlledPermanent filter = new FilterControlledPermanent(new StringBuilder("any number of of target nonland, nontoken permanents you control").toString());
+
+    static {
+        filter.add(Predicates.not(new CardTypePredicate(CardType.LAND)));
+        filter.add(Predicates.not(new TokenPredicate()));
+    }
+
     public ParadoxicalOutcome(UUID ownerId) {
         super(ownerId, 60, "Paradoxical Outcome", Rarity.RARE, new CardType[]{CardType.INSTANT}, "{3}{U}");
         this.expansionSetCode = "KLD";
 
         // Return any number of target nonland, nontoken permanents you control to their owners' hands. Draw a card for each card returned to your hand this way.
-        FilterPermanent filter = new FilterControlledPermanent(new StringBuilder("any number of of target nonland, nontoken permanents you control").toString());            
-        filter.add(Predicates.not(new CardTypePredicate(CardType.LAND)));
-        filter.add(Predicates.not(new TokenPredicate()));
-
-        this.getSpellAbility().addEffect(new SweepEffect(filter, "nonland, nontoken permanents ", false));
-        DynamicValue paradoxicalOutcomeValue = new SweepNumber("nonland, nontoken permanents ", false);
+        this.getSpellAbility().addEffect(new ParadoxicalOutcomeEffect());
+        this.getSpellAbility().addTarget(new TargetControlledPermanent(0, Integer.MAX_VALUE, filter, false));
+        DynamicValue paradoxicalOutcomeValue = new ParadoxicalOutcomeNumber(false);
         this.getSpellAbility().addEffect(new DrawCardSourceControllerEffect(paradoxicalOutcomeValue));
     }
 
@@ -70,3 +83,76 @@ public class ParadoxicalOutcome extends CardImpl {
         return new ParadoxicalOutcome(this);
     }
 }
+
+class ParadoxicalOutcomeEffect extends OneShotEffect {
+
+    ParadoxicalOutcomeEffect() {
+        super(Outcome.Benefit);
+        this.staticText = "Return any number of target artifact cards from target player's graveyard to the top of his or her library in any order";
+    }
+
+    ParadoxicalOutcomeEffect(final ParadoxicalOutcomeEffect effect) {
+        super(effect);
+    }
+
+    @Override
+    public ParadoxicalOutcomeEffect copy() {
+        return new ParadoxicalOutcomeEffect(this);
+    }
+
+    @Override
+    public boolean apply(Game game, Ability source) {
+        Player controller = game.getPlayer(source.getControllerId());
+        if (controller != null) {
+            Cards cards = new CardsImpl(source.getTargets().get(0).getTargets());
+            game.getState().setValue(CardUtil.getCardZoneString("ParadoxicalOutcomeEffect", source.getSourceId(), game), cards.size());
+            controller.moveCards(new CardsImpl(source.getTargets().get(0).getTargets()), Zone.HAND, source, game);
+            return true;
+        }
+        return false;
+    }
+}
+
+class ParadoxicalOutcomeNumber implements DynamicValue {
+
+    private int zoneChangeCounter = 0;
+    private final boolean previousZone;
+
+    public ParadoxicalOutcomeNumber(boolean previousZone) {
+        this.previousZone = previousZone;
+    }
+
+    @Override
+    public int calculate(Game game, Ability source, Effect effect) {
+        if (zoneChangeCounter == 0) {
+            Card card = game.getCard(source.getSourceId());
+            if (card != null) {
+                zoneChangeCounter = card.getZoneChangeCounter(game);
+                if (previousZone) {
+                    zoneChangeCounter--;
+                }
+            }
+        }
+        int number = 0;
+        Integer sweepNumber = (Integer) game.getState().getValue(new StringBuilder("ParadoxicalOutcomeEffect").append(source.getSourceId()).append(zoneChangeCounter).toString());
+        if (sweepNumber != null) {
+            number = sweepNumber;
+        }
+        return number;
+    }
+
+    @Override
+    public ParadoxicalOutcomeNumber copy() {
+        return new ParadoxicalOutcomeNumber(previousZone);
+    }
+
+    @Override
+    public String toString() {
+        return "X";
+    }
+
+    @Override
+    public String getMessage() {
+        return "the number of permanents returned this way";
+    }
+}