Fix AI never casting paired commanders

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

@@ -279,6 +279,11 @@ public class AiController {
                 continue;
             }
 
+            // can't fetch partner isn't problematic
+            if (tr.getKeyword() != null && tr.getKeyword().getOriginal().startsWith("Partner")) {
+                continue;
+            }
+
             if (!ZoneType.Battlefield.toString().equals(tr.getParam("Destination"))) {
                 continue;
             }

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

@@ -593,15 +593,6 @@ public class PlayerControllerAi extends PlayerController {
         return !ComputerUtil.wantMulligan(player, cardsToReturn);
     }
 
-    @Override
-    public CardCollectionView getCardsToMulligan(Player firstPlayer)  {
-        if (!ComputerUtil.wantMulligan(player, 0)) {
-            return null;
-        }
-
-        return player.getCardsIn(ZoneType.Hand);
-    }
-
     @Override
     public CardCollectionView londonMulliganReturnCards(final Player mulliganingPlayer, int cardsToReturn) {
         // TODO This is better than it was before, but still suboptimal (but fast).

forge-game/src/main/java/forge/game/player/PlayerController.java

@@ -191,7 +191,6 @@ public abstract class PlayerController {
 
     public abstract Object vote(SpellAbility sa, String prompt, List<Object> options, ListMultimap<Object, Player> votes, Player forPlayer);
 
-    public abstract CardCollectionView getCardsToMulligan(Player firstPlayer);
     public abstract boolean mulliganKeepHand(Player player, int cardsToReturn);
     public abstract CardCollectionView londonMulliganReturnCards(Player mulliganingPlayer, int cardsToReturn);
 

forge-gui-desktop/src/test/java/forge/gamesimulationtests/util/PlayerControllerForTests.java

@@ -319,11 +319,6 @@ public class PlayerControllerForTests extends PlayerController {
         return true;
     }
 
-    @Override
-    public CardCollectionView getCardsToMulligan(Player firstPlayer) {
-        return null;
-    }
-
     @Override
     public void declareAttackers(Player attacker, Combat combat) {
         //Doing nothing is safe in most cases, but not all (creatures that must attack etc).  TODO: introduce checks?

forge-gui/src/main/java/forge/player/PlayerControllerHuman.java

@@ -1460,14 +1460,6 @@ public class PlayerControllerHuman extends PlayerController implements IGameCont
         return inp.getSelectedCards();
     }
 
-    @Override
-    public CardCollectionView getCardsToMulligan(final Player firstPlayer) {
-        // Partial Paris is gone, so it being commander doesn't really matter anymore...
-        final InputConfirmMulligan inp = new InputConfirmMulligan(this, player, firstPlayer);
-        inp.showAndWait();
-        return inp.isKeepHand() ? null : player.getCardsIn(ZoneType.Hand);
-    }
-
     @Override
     public void declareAttackers(final Player attackingPlayer, final Combat combat) {
         if (mayAutoPass()) {