From bfb29249664bbc8b11ddf545e1c3c5f74ebd4058 Mon Sep 17 00:00:00 2001 From: LevelX2 Date: Sun, 6 Nov 2016 22:06:35 +0100 Subject: [PATCH] * Cabal Conditioning - Fixed that only the first target player had to discard cards. --- .../effects/common/discard/DiscardTargetEffect.java | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/Mage/src/main/java/mage/abilities/effects/common/discard/DiscardTargetEffect.java b/Mage/src/main/java/mage/abilities/effects/common/discard/DiscardTargetEffect.java index 0b7b1eea3dd..95f92e6ab20 100644 --- a/Mage/src/main/java/mage/abilities/effects/common/discard/DiscardTargetEffect.java +++ b/Mage/src/main/java/mage/abilities/effects/common/discard/DiscardTargetEffect.java @@ -27,6 +27,7 @@ */ package mage.abilities.effects.common.discard; +import java.util.UUID; import mage.abilities.Ability; import mage.abilities.Mode; import mage.abilities.dynamicvalue.DynamicValue; @@ -85,9 +86,11 @@ public class DiscardTargetEffect extends OneShotEffect { @Override public boolean apply(Game game, Ability source) { - Player player = game.getPlayer(targetPointer.getFirst(game, source)); - if (player != null) { - player.discard(amount.calculate(game, source, this), randomDiscard, source, game); + for (UUID targetPlayerId : targetPointer.getTargets(game, source)) { + Player player = game.getPlayer(targetPlayerId); + if (player != null) { + player.discard(amount.calculate(game, source, this), randomDiscard, source, game); + } return true; } return false; @@ -99,7 +102,7 @@ public class DiscardTargetEffect extends OneShotEffect { return staticText; } StringBuilder sb = new StringBuilder(); - if(mode.getTargets().isEmpty()){ + if (mode.getTargets().isEmpty()) { sb.append("that player"); } else { sb.append("target ").append(mode.getTargets().get(0).getTargetName());