* Leeching Sliver - Fixed that all defending players of combat lost life instead of only the defending player of Leeching Sliver.

This commit is contained in:
LevelX2 2016-03-16 10:12:56 +01:00
parent 32c014f2f8
commit 867fe61007
4 changed files with 32 additions and 78 deletions

View file

@ -1,16 +1,16 @@
/*
* 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
@ -20,12 +20,11 @@
* 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.abilities.effects.common;
import mage.abilities.Ability;
@ -37,20 +36,25 @@ import mage.constants.Outcome;
import mage.game.Game;
import mage.players.Player;
import java.util.UUID;
/**
* @author noxx
*/
public class LoseLifeDefendingPlayerEffect extends OneShotEffect {
protected DynamicValue amount;
protected boolean attackerIsSource;
public LoseLifeDefendingPlayerEffect(int amount) {
this(new StaticValue(amount));
/**
*
* @param amount
* @param attackerIsSource true if the source.getSourceId() contains the
* attacker false if attacker has to be taken from targetPointer
*/
public LoseLifeDefendingPlayerEffect(int amount, boolean attackerIsSource) {
this(new StaticValue(amount), attackerIsSource);
}
public LoseLifeDefendingPlayerEffect(DynamicValue amount) {
public LoseLifeDefendingPlayerEffect(DynamicValue amount, boolean attackerIsSource) {
super(Outcome.Damage);
this.amount = amount;
}
@ -58,6 +62,7 @@ public class LoseLifeDefendingPlayerEffect extends OneShotEffect {
public LoseLifeDefendingPlayerEffect(final LoseLifeDefendingPlayerEffect effect) {
super(effect);
this.amount = effect.amount.copy();
this.attackerIsSource = effect.attackerIsSource;
}
@Override
@ -67,21 +72,21 @@ public class LoseLifeDefendingPlayerEffect extends OneShotEffect {
@Override
public boolean apply(Game game, Ability source) {
for (UUID defenderId : game.getCombat().getDefenders()) {
Player defender = game.getPlayer(defenderId);
if (defender != null) {
defender.loseLife(amount.calculate(game, source, this), game);
}
Player defender = null;
if (attackerIsSource) {
defender = game.getPlayer(game.getCombat().getDefenderId(source.getSourceId()));
} else {
defender = game.getPlayer(game.getCombat().getDefenderId(getTargetPointer().getFirst(game, source)));
}
if (defender != null) {
defender.loseLife(amount.calculate(game, source, this), game);
}
return true;
}
@Override
public String getText(Mode mode) {
StringBuilder sb = new StringBuilder("defending player loses ");
sb.append(amount);
sb.append(" life");
return sb.toString();
return "defending player loses " + amount + " life";
}
}