From f09e09f505b156eff94847a4d7dff13b5deaf545 Mon Sep 17 00:00:00 2001
From: marcins78 <marcins78@eb7970ca-0f6f-de4b-2938-835b230b9d1f>
Date: Mon, 25 Mar 2013 11:12:49 +0000
Subject: [PATCH] 20_389

---
 .../src/main/web/js/gemp-019/set20.js         |  2 +-
 .../lotro/cards/set20/shire/Card20_389.java   | 35 ++++++++++++-------
 2 files changed, 23 insertions(+), 14 deletions(-)

diff --git a/gemp-lotr/gemp-lotr-async/src/main/web/js/gemp-019/set20.js b/gemp-lotr/gemp-lotr-async/src/main/web/js/gemp-019/set20.js
index f0edeef02..822294607 100644
--- a/gemp-lotr/gemp-lotr-async/src/main/web/js/gemp-019/set20.js
+++ b/gemp-lotr/gemp-lotr-async/src/main/web/js/gemp-019/set20.js
@@ -379,7 +379,7 @@ var set20 = {
     '20_386': 'http://lotrtcg.org/coreset/shire/feastingandsmoking.jpg',
     '20_387': 'http://lotrtcg.org/coreset/shire/frodobotsb.jpg',
     '20_388': 'http://lotrtcg.org/coreset/shire/frodoer.jpg',
-    '20_389': 'http://lotrtcg.org/coreset/shire/frodospipe.jpg',
+    '20_389': 'http://lotrtcg.org/coreset/shire/frodospipe(r2).jpg',
     '20_390': 'http://lotrtcg.org/coreset/shire/hamfastgamgeetg.jpg',
     '20_391': 'http://lotrtcg.org/coreset/shire/herbloreoftheshire.jpg',
     '20_392': 'http://lotrtcg.org/coreset/shire/hobbitpartyguest.jpg',
diff --git a/gemp-lotr/gemp-lotr-cards/src/main/java/com/gempukku/lotro/cards/set20/shire/Card20_389.java b/gemp-lotr/gemp-lotr-cards/src/main/java/com/gempukku/lotro/cards/set20/shire/Card20_389.java
index 764495771..e1a4e2d51 100644
--- a/gemp-lotr/gemp-lotr-cards/src/main/java/com/gempukku/lotro/cards/set20/shire/Card20_389.java
+++ b/gemp-lotr/gemp-lotr-cards/src/main/java/com/gempukku/lotro/cards/set20/shire/Card20_389.java
@@ -2,15 +2,15 @@ package com.gempukku.lotro.cards.set20.shire;
 
 import com.gempukku.lotro.cards.AbstractAttachableFPPossession;
 import com.gempukku.lotro.cards.PlayConditions;
-import com.gempukku.lotro.cards.TriggerConditions;
-import com.gempukku.lotro.cards.effects.RemoveBurdenEffect;
+import com.gempukku.lotro.cards.effects.ForEachYouSpotEffect;
 import com.gempukku.lotro.cards.effects.choose.ChooseAndDiscardCardsFromPlayEffect;
 import com.gempukku.lotro.common.*;
 import com.gempukku.lotro.filters.Filters;
 import com.gempukku.lotro.game.PhysicalCard;
 import com.gempukku.lotro.game.state.LotroGame;
-import com.gempukku.lotro.logic.actions.OptionalTriggerAction;
-import com.gempukku.lotro.logic.timing.EffectResult;
+import com.gempukku.lotro.logic.actions.ActivateCardAction;
+import com.gempukku.lotro.logic.effects.ChooseAndHealCharactersEffect;
+import com.gempukku.lotro.logic.timing.Action;
 
 import java.util.Collections;
 import java.util.List;
@@ -18,9 +18,11 @@ import java.util.List;
 /**
  * 1
  * •Frodo's Pipe
- * Shire	Possession • Pipe
- * Bearer must be Frodo.
- * At the beginning of the regroup phase, you may discard a pipeweed possession to remove a burden.
+ * Possession • Pipe
+ * Bearer must be a Hobbit.
+ * Fellowship: If bearer is Frodo, you may discard a pipeweed possession and spot X pipes to heal a companion bearing
+ * a pipe X times (limit once per phase).
+ * http://lotrtcg.org/coreset/shire/frodospipe(r2).jpg
  */
 public class Card20_389 extends AbstractAttachableFPPossession {
     public Card20_389() {
@@ -29,18 +31,25 @@ public class Card20_389 extends AbstractAttachableFPPossession {
 
     @Override
     protected Filterable getValidTargetFilter(String playerId, LotroGame game, PhysicalCard self) {
-        return Filters.frodo;
+        return Race.HOBBIT;
     }
 
     @Override
-    public List<OptionalTriggerAction> getOptionalAfterTriggers(String playerId, LotroGame game, EffectResult effectResult, PhysicalCard self) {
-        if (TriggerConditions.startOfPhase(game, effectResult, Phase.REGROUP)
+    protected List<? extends Action> getExtraInPlayPhaseActions(final String playerId, LotroGame game, PhysicalCard self) {
+        if (PlayConditions.canUseFPCardDuringPhase(game, Phase.FELLOWSHIP, self)
+                && PlayConditions.isActive(game, Filters.frodo, Filters.hasAttached(self))
                 && PlayConditions.canDiscardFromPlay(self, game, CardType.POSSESSION, Keyword.PIPEWEED)) {
-            OptionalTriggerAction action = new OptionalTriggerAction(self);
+            final ActivateCardAction action =new ActivateCardAction(self);
             action.appendCost(
                     new ChooseAndDiscardCardsFromPlayEffect(action, playerId, 1, 1, CardType.POSSESSION, Keyword.PIPEWEED));
-            action.appendEffect(
-                    new RemoveBurdenEffect(playerId, self));
+            action.appendCost(
+                    new ForEachYouSpotEffect(playerId, PossessionClass.PIPE) {
+                        @Override
+                        protected void spottedCards(int spotCount) {
+                            action.appendEffect(
+                                    new ChooseAndHealCharactersEffect(action, playerId, 1, 1, spotCount, CardType.COMPANION, Filters.hasAttached(PossessionClass.PIPE)));
+                        }
+                    });
             return Collections.singletonList(action);
         }
         return null;