From eccc367920fc06ab4ff3c8f2c507caa59924bf91 Mon Sep 17 00:00:00 2001 From: LevelX2 Date: Sun, 2 Aug 2015 10:59:03 +0200 Subject: [PATCH] Some minor changes. --- Mage.Sets/src/mage/sets/futuresight/YixlidJailer.java | 4 ++-- Mage.Sets/src/mage/sets/mastersedition/RainbowVale.java | 2 +- .../src/test/java/org/mage/test/utils/ManaOptionsTest.java | 2 ++ 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/Mage.Sets/src/mage/sets/futuresight/YixlidJailer.java b/Mage.Sets/src/mage/sets/futuresight/YixlidJailer.java index 28f969c85bc..4e35a0f736c 100644 --- a/Mage.Sets/src/mage/sets/futuresight/YixlidJailer.java +++ b/Mage.Sets/src/mage/sets/futuresight/YixlidJailer.java @@ -100,7 +100,7 @@ class YixlidJailerEffect extends ContinuousEffectImpl { if (player != null) { for (Card card : player.getGraveyard().getCards(game)) { if (card != null) { - card.getAbilities().clear(); // Will the abilities ever come back???? + card.getAbilities(game).clear(); // Will the abilities ever come back???? // TODO: Fix that (LevelX2) // game.getContinuousEffects().removeGainedEffectsForSource(card.getId()); // game.getState().resetTriggersForSourceId(card.getId()); @@ -127,4 +127,4 @@ class YixlidJailerEffect extends ContinuousEffectImpl { public boolean hasLayer(Layer layer) { return layer == Layer.AbilityAddingRemovingEffects_6; } -} \ No newline at end of file +} diff --git a/Mage.Sets/src/mage/sets/mastersedition/RainbowVale.java b/Mage.Sets/src/mage/sets/mastersedition/RainbowVale.java index 9e02da1308b..4cc33020179 100644 --- a/Mage.Sets/src/mage/sets/mastersedition/RainbowVale.java +++ b/Mage.Sets/src/mage/sets/mastersedition/RainbowVale.java @@ -134,7 +134,7 @@ class OpponentGainControlEffect extends ContinuousEffectImpl { } Player targetOpponent = game.getPlayer(opponentId); if (targetOpponent != null && permanent != null) { - game.informPlayers(permanent.getName() + " is now controlled by " + targetOpponent.getLogName()); + game.informPlayers(permanent.getLogName() + " is now controlled by " + targetOpponent.getLogName()); } else { discard(); } diff --git a/Mage.Tests/src/test/java/org/mage/test/utils/ManaOptionsTest.java b/Mage.Tests/src/test/java/org/mage/test/utils/ManaOptionsTest.java index 2f0af102cbf..9d528310d3c 100644 --- a/Mage.Tests/src/test/java/org/mage/test/utils/ManaOptionsTest.java +++ b/Mage.Tests/src/test/java/org/mage/test/utils/ManaOptionsTest.java @@ -346,6 +346,8 @@ public class ManaOptionsTest extends CardTestPlayerBase { Assert.assertEquals("{B}{B}", getManaOption(0, manaOptions)); } + // TODO + // Test Calciform Pools combination mana lands private String getManaOption(int index, ManaOptions manaOptions) { if (manaOptions.size() < index + 1) { return "";