From 6bb80c00a6d2c921c963d274dc4c1c1d5ff0a909 Mon Sep 17 00:00:00 2001 From: Sol Date: Thu, 25 Apr 2013 00:51:14 +0000 Subject: [PATCH] - Provide targeting hints when attempting to target a Planeswalker with direct damage --- .../java/forge/control/input/InputSelectTargets.java | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/main/java/forge/control/input/InputSelectTargets.java b/src/main/java/forge/control/input/InputSelectTargets.java index 362b057b925..3ac2ebae19e 100644 --- a/src/main/java/forge/control/input/InputSelectTargets.java +++ b/src/main/java/forge/control/input/InputSelectTargets.java @@ -107,12 +107,17 @@ public final class InputSelectTargets extends InputSyncronizedBase { } // leave this in temporarily, there some seriously wrong things going on here + // Can be targeted doesn't check if the target is a valid type, only if a card is generally "targetable" if (!card.canBeTargetedBy(sa)) { - showMessage("Cannot target this card (Shroud? Protection? Restrictions?)."); + showMessage("Cannot target this card (Shroud? Protection? Restrictions)."); return; } if (!choices.contains(card)) { - showMessage("This card is not a valid choice for some other reason besides (Shroud? Protection? Restrictions?)."); + if (card.isPlaneswalker() && sa.getApi() == ApiType.DealDamage) { + showMessage("To deal an opposing Planeswalker direct damage, target its controller and then redirect the damage on resolution."); + } else { + showMessage("The selected card is not a valid choice to be targeted."); + } return; }