From 2c755f8e6bbbf6714dc7555a0fa6dd754061f1fb Mon Sep 17 00:00:00 2001 From: magenoxx Date: Wed, 10 Aug 2011 18:14:19 +0400 Subject: [PATCH] Fixed Govlin Arsonist not dealing damage. --- .../sets/riseoftheeldrazi/GoblinArsonist.java | 24 +++++++++---------- 1 file changed, 11 insertions(+), 13 deletions(-) diff --git a/Mage.Sets/src/mage/sets/riseoftheeldrazi/GoblinArsonist.java b/Mage.Sets/src/mage/sets/riseoftheeldrazi/GoblinArsonist.java index 52479a3a9c..2155b35efa 100644 --- a/Mage.Sets/src/mage/sets/riseoftheeldrazi/GoblinArsonist.java +++ b/Mage.Sets/src/mage/sets/riseoftheeldrazi/GoblinArsonist.java @@ -87,20 +87,18 @@ class GoblinArsonistEffect extends OneShotEffect { Player player = game.getPlayer(source.getControllerId()); player.choose(Outcome.Damage, target, game); - Permanent arsonist = game.getPermanent(source.getSourceId()); - if (arsonist != null) { - Permanent permanent = game.getPermanent(target.getFirstTarget()); - if (permanent != null) { - permanent.damage(1, arsonist.getId(), game, true, false); - return true; - } + Permanent permanent = game.getPermanent(target.getFirstTarget()); + if (permanent != null) { + permanent.damage(1, source.getSourceId(), game, true, false); + return true; + } + + Player targetPlayer = game.getPlayer(target.getFirstTarget()); + if (targetPlayer != null) { + targetPlayer.damage(1, source.getSourceId(), game, true, false); + return true; + } - Player targetPlayer = game.getPlayer(target.getFirstTarget()); - if (targetPlayer != null) { - targetPlayer.damage(1, arsonist.getId(), game, true, false); - return true; - } - } return false; }