From 5a19c9ed7e31c658cec4c52392a73422c5a6e24d Mon Sep 17 00:00:00 2001 From: Agetian Date: Wed, 8 Oct 2025 10:25:45 +0300 Subject: [PATCH] Revert "- Fix AI not ordering combatants for damage in non-legacy mode." This reverts commit ff784183dc1ca8c0e75d8231efb388360909a45a. --- forge-game/src/main/java/forge/game/combat/Combat.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/forge-game/src/main/java/forge/game/combat/Combat.java b/forge-game/src/main/java/forge/game/combat/Combat.java index 4797c94c038..4f81d53ef7e 100644 --- a/forge-game/src/main/java/forge/game/combat/Combat.java +++ b/forge-game/src/main/java/forge/game/combat/Combat.java @@ -492,7 +492,7 @@ public class Combat { /** If there are multiple blockers, the Attacker declares the Assignment Order */ public void orderBlockersForDamageAssignment(Card attacker, CardCollection blockers) { // this method performs controller's role - if (blockers.size() <= 1 || (!this.legacyOrderCombatants && !playerWhoAttacks.isAI())) { + if (blockers.size() <= 1 || !this.legacyOrderCombatants) { blockersOrderedForDamageAssignment.get().put(attacker, new CardCollection(blockers)); return; } @@ -529,7 +529,7 @@ public class Combat { final CardCollection oldBlockers = blockersOrderedForDamageAssignment.get().get(attacker); if (oldBlockers == null || oldBlockers.isEmpty()) { blockersOrderedForDamageAssignment.get().put(attacker, new CardCollection(blocker)); - } else if (this.legacyOrderCombatants || playerWhoAttacks.isAI()) { + } else if (this.legacyOrderCombatants) { CardCollection orderedBlockers = playerWhoAttacks.getController().orderBlocker(attacker, blocker, oldBlockers); blockersOrderedForDamageAssignment.get().put(attacker, orderedBlockers); } else { @@ -550,7 +550,7 @@ public class Combat { // They need a reverse map here: Blocker => List Player blockerCtrl = blocker.getController(); - CardCollection orderedAttacker = attackers.size() <= 1 || (!this.legacyOrderCombatants && !blockerCtrl.getController().isAI()) ? attackers : blockerCtrl.getController().orderAttackers(blocker, attackers); + CardCollection orderedAttacker = attackers.size() <= 1 || !this.legacyOrderCombatants ? attackers : blockerCtrl.getController().orderAttackers(blocker, attackers); // Damage Ordering needs to take cards like Melee into account, is that happening? attackersOrderedForDamageAssignment.get().put(blocker, orderedAttacker);