diff --git a/Mage.Sets/src/mage/sets/dragonsmaze/ProgenitorMimic.java b/Mage.Sets/src/mage/sets/dragonsmaze/ProgenitorMimic.java index adbf77737ee..0a5b592bd9e 100644 --- a/Mage.Sets/src/mage/sets/dragonsmaze/ProgenitorMimic.java +++ b/Mage.Sets/src/mage/sets/dragonsmaze/ProgenitorMimic.java @@ -54,7 +54,6 @@ import mage.game.permanent.Permanent; import mage.game.permanent.token.EmptyToken; import mage.util.CardUtil; import mage.util.functions.ApplyToPermanent; -import org.apache.log4j.Logger; /** * diff --git a/Mage.Sets/src/mage/sets/gatecrash/MasterBiomancer.java b/Mage.Sets/src/mage/sets/gatecrash/MasterBiomancer.java index 898b2369dce..9ecba762707 100644 --- a/Mage.Sets/src/mage/sets/gatecrash/MasterBiomancer.java +++ b/Mage.Sets/src/mage/sets/gatecrash/MasterBiomancer.java @@ -46,7 +46,6 @@ import mage.game.events.GameEvent; import mage.game.events.GameEvent.EventType; import mage.game.permanent.Permanent; import mage.target.targetpointer.FixedTarget; -import org.apache.log4j.Logger; /** * diff --git a/Mage/src/mage/abilities/TriggeredAbilities.java b/Mage/src/mage/abilities/TriggeredAbilities.java index c9d00fcbccf..90977d65fdf 100644 --- a/Mage/src/mage/abilities/TriggeredAbilities.java +++ b/Mage/src/mage/abilities/TriggeredAbilities.java @@ -130,7 +130,6 @@ public class TriggeredAbilities extends ConcurrentHashMap uuidList = new LinkedList<>(); uuidList.add(sourceId); @@ -140,7 +139,6 @@ public class TriggeredAbilities extends ConcurrentHashMap extends ArrayList return false; } else if (effect.isDiscarded()) { it.remove(); - } else if (game.getObject(ability.getSourceId()) == null) { - logger.debug("Ability without source object removed: " + ability.getSourceId() + " " + ability.getRule()); + } else if (ability.getSourceId() != null && game.getObject(ability.getSourceId()) == null) { // Commander effects have no sourceId it.remove(); // if the related source object does no longer exist the effect has to be removed } else { switch(effect.getDuration()) { case OneUse: if (effect.isUsed()) { - logger.debug("Ability one use removed: " + ability.getSourceId() + " " + ability.getRule()); it.remove(); } break; case Custom: case UntilYourNextTurn: if (effect.isInactive(ability , game)) { - logger.debug("Ability custom removed: " + ability.getSourceId() + " " + ability.getRule()); it.remove(); } } diff --git a/Mage/src/mage/cards/CardImpl.java b/Mage/src/mage/cards/CardImpl.java index a69b43d715a..287125f3c31 100644 --- a/Mage/src/mage/cards/CardImpl.java +++ b/Mage/src/mage/cards/CardImpl.java @@ -553,8 +553,6 @@ public abstract class CardImpl extends MageObjectImpl implements Card { } updateZoneChangeCounter(); PermanentCard permanent = new PermanentCard(this, event.getPlayerId()); - // reset is done to end continuous effects from previous instances of that permanent (e.g undying) - // game.resetForSourceId(permanent.getId()); // make sure the controller of all continuous effects of this card are switched to the current controller game.getContinuousEffects().setController(objectId, event.getPlayerId()); game.addPermanent(permanent); diff --git a/Mage/src/mage/game/GameCommanderImpl.java b/Mage/src/mage/game/GameCommanderImpl.java index da41b36c238..abb582b5fa1 100644 --- a/Mage/src/mage/game/GameCommanderImpl.java +++ b/Mage/src/mage/game/GameCommanderImpl.java @@ -106,7 +106,7 @@ public abstract class GameCommanderImpl extends GameImpl { } } - this.getState().addAbility(ability, this.getId(), null); + this.getState().addAbility(ability, null); if (startingPlayerSkipsDraw) { state.getTurnMods().add(new TurnMod(startingPlayerId, PhaseStep.DRAW)); } diff --git a/Mage/src/mage/game/GameState.java b/Mage/src/mage/game/GameState.java index fc4575202de..033dc6f2f0d 100644 --- a/Mage/src/mage/game/GameState.java +++ b/Mage/src/mage/game/GameState.java @@ -398,7 +398,6 @@ public class GameState implements Serializable, Copyable { } public void applyEffects(Game game) { - Logger.getLogger(GameState.class).debug("Apply Effects turn: " + game.getTurnNum() + " - Step: " + (game.getStep() == null ? "null":game.getStep().getType().toString())); for (Player player: players.values()) { player.reset(); } @@ -510,7 +509,6 @@ public class GameState implements Serializable, Copyable { eventsToHandle.addAll(simultaneousEvents); simultaneousEvents.clear(); for (GameEvent event:eventsToHandle) { - Logger.getLogger(GameState.class).debug("Handle Simultanous events - type: " + event.getType().toString() + " sourceId " + event.getSourceId() + " targetId " + event.getTargetId()); this.handleEvent(event, game); } } @@ -760,7 +758,7 @@ public class GameState implements Serializable, Copyable { } /** - * Used for diagbostic purposes + * Onl used for diagnostic purposes of tests * @return */ public TriggeredAbilities getTriggers() {