Merge branch 'Card-Forge:master' into master

Author: agylesox

docs/Adventure/GAMEPAD.md

@@ -11,57 +11,52 @@ https://github.com/Ryochan7/DS4Windows/releases/
 HidHide latest:
 https://github.com/ViGEm/HidHide/releases
 
-Other XInput Controller should work, XBox or DS4 Preferaably and other similar XInput Controller with comparable button layout. Custom Key Mapping is not yet supported.
+Other XInput Controller should work, XBox or DS4 preferably and other similar XInput Controller with comparable button layout. Custom Key Mapping is not yet supported.
 
 Controls:
 
-UIScenes
-
-DPAD Up/Down/Left/Right - for selecting texboxes and textfield, Scroll Up or Down
-Button A - Ok, Show Context
-Button B - Cancel
-Button X - Increase Difficulty in NewGame Plus/Flip Backside Deck Editor
-Button Y - Decrease Difficulty in NewGame Plus/Zoom or Text Mode in Deck Editor
-Left/Right Shoulder Button - Scroll Up or Scroll Down on some UIScenes
-
-RewardScene
-
-DPAD Left/Right - Selector
-Button A - Confirm/Flip Reward
-Button B - Show Rewards/Done
-Button Y - Show/Hide Zoom Card
-
-TextInput
-
-DPAD Up/Down/Left/Right - Key Selector
-Left Shoulder Button - Shift Keys
-Right Shoulder Button - Backspace
-Button Start - Jump to Ok
-Button A - Confirm
-Button B - Cancel
-
-World/Gamescene
-
-DPAD Up/Down/Left/Right - Character Movement
-Left Analog - Character Movement
-Button A - Menu/Confirm
-Button B - Statistics/Edit
-Button X - Deck Select/Deck Edit
-Button Y - Inventory/Rename
-
-Match/Battle
-
-Left Trigger - Play/Draw/OK (Bottom Left Button)
-Right Trigger - Keep/Mulligan/Cancel/End Turn/Alpha Strike (Bottom Right Button)
-
-(To select cards on the battlefield, close Zone tabs first (Button B), then use DPAD)
-
-DPAD Up/Down/Left/Right - Selector
-Left Shoulder - Player Panel Selector
-Right Shoulder - Zone Selector/Show
-Left Analog Down - Select Player (current selected panel)
-
-Button A - Confirm
-Button B - Cancel/Hide
-Button Y - Show Zoom
-Button Back - Show Menu Tabs
+- UIScenes
+  - DPAD Up/Down/Left/Right - for selecting texboxes and textfield, Scroll Up or Down
+  - Button A - Ok, Show Context
+  - Button B - Cancel
+  - Button X - Increase Difficulty in NewGame Plus/Flip Backside Deck Editor
+  - Button Y - Decrease Difficulty in NewGame Plus/Zoom or Text Mode in Deck Editor
+  - Left/Right Shoulder Button - Scroll Up or Scroll Down on some UIScenes
+
+
+- RewardScene
+  - DPAD Left/Right - Selector
+  - Button A - Confirm/Flip Reward
+  - Button B - Show Rewards/Done
+  - Button Y - Show/Hide Zoom Card
+
+
+- TextInput
+  - DPAD Up/Down/Left/Right - Key Selector
+  - Left Shoulder Button - Shift Keys
+  - Right Shoulder Button - Backspace
+  - Button Start - Jump to Ok
+  - Button A - Confirm
+  - Button B - Cancel
+
+
+- World/Gamescene
+  - DPAD Up/Down/Left/Right - Character Movement
+  - Left Analog - Character Movement
+  - Button A - Menu/Confirm
+  - Button B - Statistics/Edit
+  - Button X - Deck Select/Deck Edit
+  - Button Y - Inventory/Rename
+
+
+- Match/Battle
+  - Left Trigger - Play/Draw/OK (Bottom Left Button)
+  - Right Trigger - Keep/Mulligan/Cancel/End Turn/Alpha Strike (Bottom Right Button)
+  - DPAD Up/Down/Left/Right - Selector (To select cards on the battlefield, close Zone tabs first (Button B))
+  - Left Shoulder - Player Panel Selector
+  - Right Shoulder - Zone Selector/Show
+  - Left Analog Down - Select Player (current selected panel)
+  - Button A - Confirm
+  - Button B - Cancel/Hide
+  - Button Y - Show Zoom
+  - Button Back - Show Menu Tabs

docs/Card-scripting-API/AbilityFactory.md

@@ -318,7 +318,7 @@ This follows our general approach where we try to find a reasonable middle groun
 
 ## Fight
 
-## FlipACoin
+## FlipCoin
 
 ## Fog
 This AF is based on the original *Fog* spell: "Prevent all combat damage that would be dealt this turn." While this could be done with an Effect, the specialized nature of the AI gives it its own AF.

forge-ai/src/main/java/forge/ai/SpellApiToAi.java

@@ -103,7 +103,7 @@ public enum SpellApiToAi {
             .put(ApiType.ExchangeZone, ZoneExchangeAi.class)
             .put(ApiType.Explore, ExploreAi.class)
             .put(ApiType.Fight, FightAi.class)
-            .put(ApiType.FlipACoin, FlipACoinAi.class)
+            .put(ApiType.FlipCoin, FlipCoinAi.class)
             .put(ApiType.FlipOntoBattlefield, FlipOntoBattlefieldAi.class)
             .put(ApiType.Fog, FogAi.class)
             .put(ApiType.GainControl, ControlGainAi.class)

forge-ai/src/main/java/forge/ai/ability/CharmAi.java

@@ -60,7 +60,7 @@ protected AiAbilityDecision checkApiLogic(Player ai, SpellAbility sa) {
              * bonus choice(s) for the AI otherwise it might be too hard to ever fulfil
              * minimum choice requirements with canPlayAi() alone.
              */
-            chosenList = min > 1 ? chooseMultipleOptionsAi(choices, ai, min)
+            chosenList = min > 1 ? chooseMultipleOptionsAi(sa, choices, ai, min)
                     : chooseOptionsAi(sa, choices, ai, timingRight, num, min);
         }
 
@@ -92,12 +92,11 @@ protected AiAbilityDecision checkApiLogic(Player ai, SpellAbility sa) {
     }
 
     private List<AbilitySub> chooseOptionsAi(SpellAbility sa, List<AbilitySub> choices, final Player ai, boolean isTrigger, int num, int min) {
-        List<AbilitySub> chosenList = Lists.newArrayList();
+        List<AbilitySub> chosen = Lists.newArrayList();
         AiController aic = ((PlayerControllerAi) ai.getController()).getAi();
         // TODO unused for now, the AI doesn't know how to effectively handle repeated choices
         boolean allowRepeat = sa.hasParam("CanRepeatModes");
 
-        // Pawprint
         final int pawprintLimit = sa.hasParam("Pawprint") ? AbilityUtils.calculateAmount(sa.getHostCard(), sa.getParam("Pawprint"), sa) : 0;
         if (pawprintLimit > 0) {
             // try to pay for the more expensive subs first
@@ -107,6 +106,7 @@ private List<AbilitySub> chooseOptionsAi(SpellAbility sa, List<AbilitySub> choic
 
         // First pass using standard canPlayAi() for good choices
         for (AbilitySub sub : choices) {
+            handleDependentModes(sa, chosen, sub);
             sub.setActivatingPlayer(ai);
             // TODO refactor to obtain the AiAbilityDecision instead, then we can check all to sort by value
             if (AiPlayDecision.WillPlay == aic.canPlaySa(sub)) {
@@ -117,42 +117,45 @@ private List<AbilitySub> chooseOptionsAi(SpellAbility sa, List<AbilitySub> choic
                     }
                     pawprintAmount += curPawprintAmount;
                 }
-                chosenList.add(sub);
-                if (chosenList.size() == num) {
+                chosen.add(sub);
+                if (chosen.size() == num) {
                     // maximum choices reached
-                    return chosenList;
+                    break;
                 }
             }
         }
-        if (isTrigger && chosenList.size() < min) {
+        if (isTrigger && chosen.size() < min) {
             // Second pass using doTrigger(false) to fulfill minimum choice
-            choices.removeAll(chosenList);
+            choices.removeAll(chosen);
             for (AbilitySub sub : choices) {
+                handleDependentModes(sa, chosen, sub);
                 if (aic.doTrigger(sub, false)) {
-                    chosenList.add(sub);
-                    if (chosenList.size() == min) {
-                        return chosenList;
+                    chosen.add(sub);
+                    if (chosen.size() == min) {
+                        break;
                     }
                 }
             }
             // Third pass using doTrigger(true) to force fill minimum choices
-            if (chosenList.size() < min) {
-                choices.removeAll(chosenList);
+            if (chosen.size() < min) {
+                choices.removeAll(chosen);
                 for (AbilitySub sub : choices) {
+                    handleDependentModes(sa, chosen, sub);
                     if (aic.doTrigger(sub, true)) {
-                        chosenList.add(sub);
-                        if (chosenList.size() == min) {
+                        chosen.add(sub);
+                        if (chosen.size() == min) {
                             break;
                         }
                     }
                 }
             }
         }
-        if (chosenList.size() < min) {
+        if (chosen.size() < min) {
             // not enough choices
-            chosenList.clear();
+            chosen.clear();
         }
-        return chosenList;
+        sa.setSubAbility(null);
+        return chosen;
     }
 
     private List<AbilitySub> chooseTriskaidekaphobia(List<AbilitySub> choices, final Player ai) {
@@ -242,32 +245,43 @@ else if (aiLife < 13 || ((aiLife - 13) % 2) == 1) {
         return chosenList;
     }
 
-    // Choice selection for charms that require multiple choices (eg. Cryptic Command, DTK commands)
-    private List<AbilitySub> chooseMultipleOptionsAi(List<AbilitySub> choices, final Player ai, int min) {
+    // Choice selection for charms that require multiple choices (e.g. Cryptic Command)
+    private List<AbilitySub> chooseMultipleOptionsAi(SpellAbility sa, List<AbilitySub> choices, final Player ai, int min) {
         AbilitySub goodChoice = null;
-        List<AbilitySub> chosenList = Lists.newArrayList();
+        List<AbilitySub> chosen = Lists.newArrayList();
         AiController aic = ((PlayerControllerAi) ai.getController()).getAi();
         for (AbilitySub sub : choices) {
+            handleDependentModes(sa, chosen, sub);
             sub.setActivatingPlayer(ai);
             // Assign generic good choice to fill up choices if necessary 
             if ("Good".equals(sub.getParam("AILogic")) && aic.doTrigger(sub, false)) {
                 goodChoice = sub;
             } else if (AiPlayDecision.WillPlay == aic.canPlaySa(sub)) {
-                chosenList.add(sub);
-                if (chosenList.size() == min) {
+                chosen.add(sub);
+                if (chosen.size() == min) {
                     break; // enough choices
                 }
             }
         }
         // Add generic good choice if one more choice is needed
-        if (chosenList.size() == min - 1 && goodChoice != null) {
-            chosenList.add(0, goodChoice);  // hack to make Dromoka's Command fight targets work
+        if (chosen.size() == min - 1 && goodChoice != null) {
+            chosen.add(0, goodChoice);  // hack to make Dromoka's Command fight targets work
         }
-        if (chosenList.size() != min) {
-            chosenList.clear();
+        if (chosen.size() != min) {
+            chosen.clear();
         }
-        return chosenList;
-    } 
+        sa.setSubAbility(null);
+        return chosen;
+    }
+
+    private void handleDependentModes(SpellAbility sa, List<AbilitySub> chosen, AbilitySub sub) {
+        if (sub.hasParam("TargetUnique") && !chosen.isEmpty()) {
+            // support "Each mode must target a different..."
+            sa.setSubAbility(null);
+            CharmEffect.chainAbilities(sa, chosen);
+            sa.appendSubAbility(sub);
+        }
+    }
 
     @Override
     public Player chooseSinglePlayer(Player ai, SpellAbility sa, Iterable<Player> opponents, Map<String, Object> params) {

forge-ai/src/main/java/forge/ai/ability/CountersPutAi.java

@@ -335,17 +335,14 @@ protected AiAbilityDecision checkApiLogic(Player ai, final SpellAbility sa) {
             amount = 1; // TODO: improve this to possibly account for some variability depending on the roll outcome (e.g. 4 for 1d8, perhaps)
         }
 
-        if (ph.is(PhaseType.COMBAT_DECLARE_BLOCKERS)) {
-            Combat combat = game.getCombat();
-            if (sourceName.equals("Psychic Frog")) {
-                return doCombatAdaptLogic(source, amount, combat);
-            }
-            if (sa.hasParam("Adapt")) {
-                if (!source.canReceiveCounters(CounterEnumType.P1P1) || source.getCounters(CounterEnumType.P1P1) > 0) {
-                    return new AiAbilityDecision(0, AiPlayDecision.CantPlayAi);
-                }
-                return doCombatAdaptLogic(source, amount, combat);
-            }
+        if (sa.hasParam("Adapt") &&
+                (!source.canReceiveCounters(CounterEnumType.P1P1) || source.getCounters(CounterEnumType.P1P1) > 0)) {
+            return new AiAbilityDecision(0, AiPlayDecision.CantPlayAi);
+        }
+
+        if (ph.is(PhaseType.COMBAT_DECLARE_BLOCKERS) &&
+                (sa.hasParam("Adapt") || sourceName.equals("Psychic Frog"))) {
+            return doCombatAdaptLogic(source, amount, game.getCombat());
         }
 
         if ("Fight".equals(logic) || "PowerDmg".equals(logic)) {

forge-ai/src/main/java/forge/ai/ability/CountersPutAllAi.java

@@ -139,8 +139,7 @@ protected AiAbilityDecision doTriggerNoCost(final Player aiPlayer, final SpellAb
 
             for (final Player p : players) {
                 if (sa.canTarget(p)) {
-                    boolean preferred = false;
-                    preferred = (sa.isCurse() && p.isOpponentOf(aiPlayer)) || (!sa.isCurse() && p == aiPlayer);
+                    boolean preferred = (sa.isCurse() && p.isOpponentOf(aiPlayer)) || (!sa.isCurse() && p == aiPlayer);
                     sa.resetTargets();
                     sa.getTargets().add(p);
                     if (preferred) {
@@ -149,9 +148,8 @@ protected AiAbilityDecision doTriggerNoCost(final Player aiPlayer, final SpellAb
 
                     if (mandatory) {
                         return new AiAbilityDecision(100, AiPlayDecision.WillPlay);
-                    } else {
-                        return new AiAbilityDecision(0, AiPlayDecision.CantPlayAi);
                     }
+                    return new AiAbilityDecision(0, AiPlayDecision.CantPlayAi);
                 }
             }
         }

forge-ai/src/main/java/forge/ai/ability/DrawAi.java

@@ -289,23 +289,19 @@ private boolean targetAI(final Player ai, final SpellAbility sa, final boolean m
         // TODO: if xPaid and one of the below reasons would fail, instead of
         // bailing reduce toPay amount to acceptable level
         if (sa.usesTargeting()) {
-            // ability is targeted
             sa.resetTargets();
 
             // if it wouldn't draw anything and its not mandatory, skip it
             if (numCards == 0 && !mandatory && !drawback) {
                 return false;
             }
 
-            // filter player that can be targeted
             PlayerCollection players = game.getPlayers().filter(PlayerPredicates.isTargetableBy(sa));
 
-            // no targets skip it
             if (players.isEmpty()) {
                 return false;
             }
 
-            // filter opponents
             PlayerCollection opps = players.filter(PlayerPredicates.isOpponentOf(ai));
 
             for (Player oppA : opps) {
@@ -530,7 +526,7 @@ private boolean targetAI(final Player ai, final SpellAbility sa, final boolean m
             }
         }
         return true;
-    } // drawTargetAI()
+    }
 
     @Override
     protected AiAbilityDecision doTriggerNoCost(Player ai, SpellAbility sa, boolean mandatory) {

…n/java/forge/ai/ability/FlipACoinAi.java …in/java/forge/ai/ability/FlipCoinAi.javaforge-ai/src/main/java/forge/ai/ability/FlipACoinAi.java renamed to forge-ai/src/main/java/forge/ai/ability/FlipCoinAi.java forge-ai/src/main/java/forge/ai/ability/FlipACoinAi.java renamed to forge-ai/src/main/java/forge/ai/ability/FlipCoinAi.java

@@ -9,7 +9,7 @@
 import forge.game.player.Player;
 import forge.game.spellability.SpellAbility;
 
-public class FlipACoinAi extends SpellAbilityAi {
+public class FlipCoinAi extends SpellAbilityAi {
 
     /* (non-Javadoc)
      * @see forge.card.abilityfactory.SpellAiLogic#checkApiLogic(forge.game.player.Player, java.util.Map, forge.card.spellability.SpellAbility)

forge-ai/src/main/java/forge/ai/ability/TokenAi.java

@@ -256,20 +256,21 @@ protected AiAbilityDecision doTriggerNoCost(Player ai, SpellAbility sa, boolean
                 if (tgtRoleAura(ai, sa, actualToken, mandatory)) {
                     // Targeting handled in tgtRoleAura
                     return new AiAbilityDecision(100, AiPlayDecision.WillPlay);
-                } else {
-                    return new AiAbilityDecision(0, AiPlayDecision.TargetingFailed);
                 }
+                return new AiAbilityDecision(0, AiPlayDecision.TargetingFailed);
             }
 
-            if (tgt.canOnlyTgtOpponent()) {
+            if (sa.canTarget(ai)) {
+                sa.getTargets().add(ai);
+            } else if (mandatory || tgt.canOnlyTgtOpponent()) {
                 PlayerCollection targetableOpps = ai.getOpponents().filter(PlayerPredicates.isTargetableBy(sa));
-                if (mandatory && targetableOpps.isEmpty()) {
-                    return new AiAbilityDecision(0, AiPlayDecision.CantPlayAi);
+                if (targetableOpps.isEmpty()) {
+                    return new AiAbilityDecision(0, AiPlayDecision.TargetingFailed);
                 }
                 Player opp = targetableOpps.min(PlayerPredicates.compareByLife());
                 sa.getTargets().add(opp);
             } else {
-                sa.getTargets().add(ai);
+                return new AiAbilityDecision(0, AiPlayDecision.TargetingFailed);
             }
         }