From cbd59a32411a22e9e5fb5c79fd76da96184ea46f Mon Sep 17 00:00:00 2001 From: Sloth Date: Mon, 3 Dec 2012 08:39:09 +0000 Subject: [PATCH] - Fixed Liliana of the Veil. - Fixed AI choosing logic of AF TwoPiles. --- res/cardsfolder/l/liliana_of_the_veil.txt | 2 +- .../java/forge/card/abilityfactory/effects/TwoPilesEffect.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/res/cardsfolder/l/liliana_of_the_veil.txt b/res/cardsfolder/l/liliana_of_the_veil.txt index f70bec8706f..a800736c1e5 100644 --- a/res/cardsfolder/l/liliana_of_the_veil.txt +++ b/res/cardsfolder/l/liliana_of_the_veil.txt @@ -6,7 +6,7 @@ Loyalty:3 A:AB$ Discard | Cost$ AddCounter<1/LOYALTY> | NumCards$ 1 | Mode$ TgtChoose | Defined$ Each | Planeswalker$ True | SpellDescription$ Each player discards a card. A:AB$ Sacrifice | Cost$ SubCounter<2/LOYALTY> | ValidTgts$ Player | SacValid$ Creature | SacMessage$ Creature | Planeswalker$ True | SpellDescription$ Target player sacrifices a creature. A:AB$ TwoPiles | Cost$ SubCounter<6/LOYALTY> | ValidTgts$ Player | TgtPrompt$ Select target player | Separator$ You | ChosenPile$ DBSacAll | ValidCards$ Permanent | Zone$ Battlefield | Planeswalker$ True | Ultimate$ True | SpellDescription$ Separate all permanents target player controls into two piles. That player sacrifices all permanents in the pile of his or her choice. -SVar:DBSacAll:DB$ SacrificeAll | ValidCards$ Permanent | Defined$ Remembered | SubAbility$ Cleanup +SVar:DBSacAll:DB$ SacrificeAll | ValidCards$ Permanent.IsRemembered | SubAbility$ Cleanup SVar:Cleanup:DB$ Cleanup | ClearRemembered$ True SVar:RemAIDeck:True SVar:Rarity:Mythic diff --git a/src/main/java/forge/card/abilityfactory/effects/TwoPilesEffect.java b/src/main/java/forge/card/abilityfactory/effects/TwoPilesEffect.java index c5a658d3513..9f5eef364b3 100644 --- a/src/main/java/forge/card/abilityfactory/effects/TwoPilesEffect.java +++ b/src/main/java/forge/card/abilityfactory/effects/TwoPilesEffect.java @@ -233,7 +233,7 @@ public class TwoPilesEffect extends SpellEffect { // for now, this assumes that the outcome will be bad // TODO: This should really have a ChooseLogic param to // figure this out - pile1WasChosen = cmc1 >= cmc2; + pile1WasChosen = cmc1 <= cmc2; if (pile1WasChosen) { JOptionPane.showMessageDialog(null, "Computer chooses the Pile 1", "", JOptionPane.INFORMATION_MESSAGE);