- Reworked "Orkish Sneak".

gemp-lotr/gemp-lotr-cards/src/main/java/com/gempukku/lotro/cards/set18/orc/Card18_091.java

@@ -6,6 +6,7 @@ import com.gempukku.lotro.cards.actions.SubCostToEffectAction;
 import com.gempukku.lotro.cards.effects.AddBurdenEffect;
 import com.gempukku.lotro.cards.effects.ChoiceEffect;
 import com.gempukku.lotro.cards.effects.OptionalEffect;
+import com.gempukku.lotro.cards.effects.choose.ChooseAndAssignCharacterToMinionEffect;
 import com.gempukku.lotro.cards.effects.choose.ChooseAndExertCharactersEffect;
 import com.gempukku.lotro.common.Culture;
 import com.gempukku.lotro.common.Phase;
@@ -15,8 +16,6 @@ import com.gempukku.lotro.game.PhysicalCard;
 import com.gempukku.lotro.game.state.LotroGame;
 import com.gempukku.lotro.logic.GameUtils;
 import com.gempukku.lotro.logic.actions.ActivateCardAction;
-import com.gempukku.lotro.logic.effects.AssignmentEffect;
-import com.gempukku.lotro.logic.effects.ChooseActiveCardEffect;
 import com.gempukku.lotro.logic.effects.DiscardCardsFromPlayEffect;
 import com.gempukku.lotro.logic.effects.StackActionEffect;
 import com.gempukku.lotro.logic.timing.Action;
@@ -48,38 +47,28 @@ public class Card18_091 extends AbstractMinion {
         if (PlayConditions.canUseShadowCardDuringPhase(game, Phase.ASSIGNMENT, self, 0)
                 && PlayConditions.canSpot(game, Filters.not(self), Culture.ORC, Race.ORC)) {
             final ActivateCardAction action = new ActivateCardAction(self);
+            action.appendEffect(
+                    new ChooseAndAssignCharacterToMinionEffect(action, playerId, self, Filters.ringBearer, Filters.not(Culture.SHIRE)));
+
+            String fpPlayer = game.getGameState().getCurrentPlayerId();
+            List<Effect> possibleCosts = new LinkedList<Effect>();
+            possibleCosts.add(
+                    new AddBurdenEffect(self, 1));
+            possibleCosts.add(
+                    new ChooseAndExertCharactersEffect(action, fpPlayer, 1, 1, 2, Filters.ringBearer));
+            SubCostToEffectAction subAction = new SubCostToEffectAction(action);
             action.appendCost(
-                    new ChooseActiveCardEffect(self, playerId, "Choose another ORC Orc", Filters.not(self), Culture.ORC, Race.ORC) {
+                    new ChoiceEffect(subAction, fpPlayer, possibleCosts));
-                        @Override
+            action.appendEffect(
-                        protected void cardSelected(LotroGame game, final PhysicalCard minion) {
+                    new DiscardCardsFromPlayEffect(self, self));
-                            action.appendEffect(
+            action.appendEffect(
-                                    new ChooseActiveCardEffect(self, playerId, "Choose a non-SHIRE Ring-bearer", Filters.not(Culture.SHIRE), Filters.ringBearer) {
+                    new OptionalEffect(action, game.getGameState().getCurrentPlayerId(),
-                                        @Override
+                            new StackActionEffect(subAction) {
-                                        protected void cardSelected(LotroGame game, PhysicalCard ringBearer) {
+                                @Override
-                                            action.appendEffect(
+                                public String getText(LotroGame game) {
-                                                    new AssignmentEffect(playerId, ringBearer, minion, false));
+                                    return "Add burden or exert the Ring-bearer twice to discard " + GameUtils.getCardLink(self);
-                                        }
+                                }
-                                    });
+                            }));
-                            List<Effect> possibleCosts = new LinkedList<Effect>();
-                            possibleCosts.add(
-                                    new AddBurdenEffect(self, 1));
-                            possibleCosts.add(
-                                    new ChooseAndExertCharactersEffect(action, playerId, 1, 1, 2, Filters.ringBearer));
-                            SubCostToEffectAction subAction = new SubCostToEffectAction(action);
-                            action.appendCost(
-                                    new ChoiceEffect(subAction, game.getGameState().getCurrentPlayerId(), possibleCosts));
-                            action.appendEffect(
-                                    new DiscardCardsFromPlayEffect(self, minion));
-                            action.appendEffect(
-                                    new OptionalEffect(action, game.getGameState().getCurrentPlayerId(),
-                                            new StackActionEffect(subAction) {
-                                                @Override
-                                                public String getText(LotroGame game) {
-                                                    return "Add burden or exert the Ring-bearer twice to discard " + GameUtils.getCardLink(minion);
-                                                }
-                                            }));
-                        }
-                    });
             return Collections.singletonList(action);
         }
         return null;

gemp-lotr/gemp-lotr-web/src/main/webapp/includes/changeLog.html

@@ -1,4 +1,7 @@
 <pre style="font-size:80%">
+<b>4 Jun. 2012</b>
+- Reworked "Orkish Sneak".
+
 <b>2 Jun. 2012</b>
 - The drop-downs for deck and format in Game Hall no longer reset periodically.
 - The formats for each week in League tab are now clickable to show the details for the format.