From 452a5251943fe8dd86dbea1540ca7df898883231 Mon Sep 17 00:00:00 2001 From: Evan Kranzler Date: Mon, 23 Jul 2018 17:42:03 -0400 Subject: [PATCH] fixed Silverclad Ferocidons (and likely other cards) allowing players to sacrifice creatures they don't control (fixes #5166) --- .../abilities/effects/common/SacrificeOpponentsEffect.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/Mage/src/main/java/mage/abilities/effects/common/SacrificeOpponentsEffect.java b/Mage/src/main/java/mage/abilities/effects/common/SacrificeOpponentsEffect.java index 308264e35de..243c524d5c8 100644 --- a/Mage/src/main/java/mage/abilities/effects/common/SacrificeOpponentsEffect.java +++ b/Mage/src/main/java/mage/abilities/effects/common/SacrificeOpponentsEffect.java @@ -8,7 +8,9 @@ import mage.abilities.dynamicvalue.DynamicValue; import mage.abilities.dynamicvalue.common.StaticValue; import mage.abilities.effects.OneShotEffect; import mage.constants.Outcome; +import mage.constants.TargetController; import mage.filter.FilterPermanent; +import mage.filter.predicate.permanent.ControllerPredicate; import mage.game.Game; import mage.game.permanent.Permanent; import mage.players.Player; @@ -36,7 +38,8 @@ public class SacrificeOpponentsEffect extends OneShotEffect { public SacrificeOpponentsEffect(DynamicValue amount, FilterPermanent filter) { super(Outcome.Sacrifice); this.amount = amount; - this.filter = filter; + this.filter = filter.copy(); + this.filter.add(new ControllerPredicate(TargetController.YOU)); setText(); }