* Ludevic, Necro-Alchemist - Fixed tooltip text.

This commit is contained in:
LevelX2 2018-09-23 18:03:48 +02:00
parent 3eea43e475
commit 6e9810696f
3 changed files with 10 additions and 8 deletions

View file

@ -1,4 +1,3 @@
package mage.cards.l;
import java.util.Objects;
@ -33,7 +32,8 @@ public final class LudevicNecroAlchemist extends CardImpl {
this.toughness = new MageInt(4);
// At the beginning of each player's end step, that player may draw a card if a player other than you lost life this turn.
this.addAbility(new BeginningOfEndStepTriggeredAbility(Zone.BATTLEFIELD, new LudevicNecroAlchemistEffect(), TargetController.ANY, new LudevicNecroAlchemistCondition(), false));
this.addAbility(new BeginningOfEndStepTriggeredAbility(Zone.BATTLEFIELD,
new LudevicNecroAlchemistEffect(), TargetController.EACH_PLAYER, new LudevicNecroAlchemistCondition(), false));
// Partner
this.addAbility(PartnerAbility.getInstance());
@ -54,11 +54,10 @@ class LudevicNecroAlchemistCondition implements Condition {
@Override
public boolean apply(Game game, Ability source) {
PlayerLostLifeWatcher watcher = (PlayerLostLifeWatcher) game.getState().getWatchers().get(PlayerLostLifeWatcher.class.getSimpleName());
UUID player = game.getActivePlayerId();
PlayerList playerList = game.getState().getPlayerList().copy();
Player currentPlayer = null;
Player currentPlayer;
UUID sourcePlayerId = source.getControllerId();
Player firstPlayer = null;
Player firstPlayer;
if (playerList == null) {
return false;
}
@ -77,6 +76,7 @@ class LudevicNecroAlchemistCondition implements Condition {
return false;
}
@Override
public String toString() {
return "if a player other than you lost life this turn";
}
@ -86,7 +86,7 @@ class LudevicNecroAlchemistEffect extends OneShotEffect {
public LudevicNecroAlchemistEffect() {
super(Outcome.DrawCard);
staticText = "that player may draw a card if a player other than you lost life this turn";
staticText = "that player may draw a card";
}
public LudevicNecroAlchemistEffect(final LudevicNecroAlchemistEffect effect) {

View file

@ -1,4 +1,3 @@
package mage.abilities.common;
import java.util.Locale;
@ -68,6 +67,7 @@ public class BeginningOfEndStepTriggeredAbility extends TriggeredAbilityImpl {
}
break;
case ANY:
case EACH_PLAYER:
case NEXT:
if (getTargets().isEmpty()) {
for (Effect effect : this.getEffects()) {
@ -123,6 +123,8 @@ public class BeginningOfEndStepTriggeredAbility extends TriggeredAbilityImpl {
return sb.insert(0, generateConditionString()).insert(0, abilityWordRule + "At the beginning of each opponent's end step, ").toString();
case ANY:
return sb.insert(0, generateConditionString()).insert(0, abilityWordRule + "At the beginning of each end step, ").toString();
case EACH_PLAYER:
return sb.insert(0, generateConditionString()).insert(0, abilityWordRule + "At the beginning of each player's end step, ").toString();
case CONTROLLER_ATTACHED_TO:
return sb.insert(0, generateConditionString()).insert(0, abilityWordRule + "At the beginning of the end step of enchanted permanent's controller, ").toString();
}

View file

@ -6,5 +6,5 @@ package mage.constants;
*/
public enum TargetController {
ACTIVE, ANY, YOU, NOT_YOU, OPPONENT, TEAM, OWNER, CONTROLLER_ATTACHED_TO, NEXT
ACTIVE, ANY, YOU, NOT_YOU, OPPONENT, TEAM, OWNER, CONTROLLER_ATTACHED_TO, NEXT, EACH_PLAYER
}