From d110c283ab9ac7a7e85ec1e4ca3b08577489666f Mon Sep 17 00:00:00 2001 From: Sol Date: Mon, 15 Apr 2013 03:18:56 +0000 Subject: [PATCH] - Blockers with Trample should not open the AssignDamage dialog when there is only one creature it's blocking --- src/main/java/forge/game/player/PlayerControllerHuman.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/main/java/forge/game/player/PlayerControllerHuman.java b/src/main/java/forge/game/player/PlayerControllerHuman.java index a21e53edaca..9362cb43a8a 100644 --- a/src/main/java/forge/game/player/PlayerControllerHuman.java +++ b/src/main/java/forge/game/player/PlayerControllerHuman.java @@ -188,12 +188,14 @@ public class PlayerControllerHuman extends PlayerController { */ @Override public Map assignCombatDamage(Card attacker, List blockers, int damageDealt, GameEntity defender) { + // Attacker is a poor name here, since the creature assigning damage + // could just as easily be the blocker. Map map; if (defender != null && assignDamageAsIfNotBlocked(attacker)) { map = new HashMap(); map.put(null, damageDealt); } else { - if (attacker.hasKeyword("Trample") || (blockers.size() > 1)) { + if ((attacker.hasKeyword("Trample") && defender != null) || (blockers.size() > 1)) { map = CMatchUI.SINGLETON_INSTANCE.getDamageToAssign(attacker, blockers, damageDealt, defender); } else { map = new HashMap();