Added additional parameter for effect outcome. Updated choosing permanents for AI.

Mage.Server.Plugins/Mage.Player.AI/src/main/java/mage/player/ai/ComputerPlayer.java

@@ -196,12 +196,16 @@ public class ComputerPlayer<T extends ComputerPlayer<T>> extends PlayerImpl<T> i
 		}
 		if (target instanceof TargetPermanent) {
 			List<Permanent> targets;
-			if (outcome.isGood()) {
+            if (outcome.isCanTargetAll()) {
-				targets = threats(playerId, ((TargetPermanent)target).getFilter(), game, target.getTargets());
+                targets = threats(null, ((TargetPermanent)target).getFilter(), game, target.getTargets());
-			}
+            } else {
-			else {
+                if (outcome.isGood()) {
-				targets = threats(opponentId, ((TargetPermanent)target).getFilter(), game, target.getTargets());
+                    targets = threats(playerId, ((TargetPermanent)target).getFilter(), game, target.getTargets());
-			}
+                }
+                else {
+                    targets = threats(opponentId, ((TargetPermanent)target).getFilter(), game, target.getTargets());
+                }
+            }
 			for (Permanent permanent: targets) {
 				if (((TargetPermanent)target).canTarget(playerId, permanent.getId(), null, game) && !target.getTargets().contains(permanent.getId())) {
 					target.add(permanent.getId(), game);
@@ -1486,7 +1490,10 @@ public class ComputerPlayer<T extends ComputerPlayer<T>> extends PlayerImpl<T> i
 	}
 
 	protected List<Permanent> threats(UUID playerId, FilterPermanent filter, Game game, List<UUID> targets) {
-		List<Permanent> threats = game.getBattlefield().getAllActivePermanents(filter, playerId);
+		List<Permanent> threats = playerId == null ?
+                game.getBattlefield().getAllActivePermanents(filter) :
+                game.getBattlefield().getAllActivePermanents(filter, playerId);
+
         Iterator<Permanent> it = threats.iterator();
         while (it.hasNext()) { // remove permanents already targetted
             Permanent test = it.next();
@@ -1494,6 +1501,7 @@ public class ComputerPlayer<T extends ComputerPlayer<T>> extends PlayerImpl<T> i
                 it.remove();
         }
 		Collections.sort(threats, new PermanentComparator(game));
+        Collections.reverse(threats);
 		return threats;
 	}
 

Mage/src/mage/Constants.java

@@ -341,23 +341,32 @@ public final class Constants {
 		Transform(true),
 		Untap(true),
 		Win(true),
-		Copy(true),
+		Copy(true, true),
 		Benefit(true),
 		Detriment(false),
 		Neutral(true),
 		Removal(true);
 
 		private boolean good;
+        private boolean canTargetAll;
 
 		Outcome(boolean good) {
 			this.good = good;
 		}
 
+        Outcome(boolean good, boolean canTargetAll) {
+			this.good = good;
+            this.canTargetAll = canTargetAll;
+		}
+
 		public boolean isGood() {
 			return good;
 		}
 
-	}
+        public boolean isCanTargetAll() {
+            return canTargetAll;
+        }
+    }
 
 	public enum Zone {
 		HAND, GRAVEYARD, LIBRARY, BATTLEFIELD, STACK, EXILED, ALL, OUTSIDE, PICK, COMMAND;