Arbitrary multiple card choice now works correctly.
This commit is contained in:
@@ -52,7 +52,7 @@ public abstract class ChooseCardsFromHandEffect extends AbstractEffect {
|
||||
cardsSelected(game, selectableCards);
|
||||
else {
|
||||
game.getUserFeedback().sendAwaitingDecision(_playerId,
|
||||
new CardsSelectionDecision(1, "Choose card from hand", selectableCards, minimum, _maximum) {
|
||||
new CardsSelectionDecision(1, "Choose card(s) from hand", selectableCards, minimum, _maximum) {
|
||||
@Override
|
||||
public void decisionMade(String result) throws DecisionResultInvalidException {
|
||||
Set<PhysicalCard> selectedCards = getSelectedCardsByResponse(result);
|
||||
|
||||
@@ -2,7 +2,7 @@ package com.gempukku.lotro.cards.set1.gandalf;
|
||||
|
||||
import com.gempukku.lotro.cards.AbstractAlly;
|
||||
import com.gempukku.lotro.cards.PlayConditions;
|
||||
import com.gempukku.lotro.cards.effects.ChooseArbitraryCardsEffect;
|
||||
import com.gempukku.lotro.cards.effects.ChooseCardsFromHandEffect;
|
||||
import com.gempukku.lotro.cards.effects.ExertCharactersEffect;
|
||||
import com.gempukku.lotro.common.Culture;
|
||||
import com.gempukku.lotro.common.Keyword;
|
||||
@@ -50,7 +50,7 @@ public class Card1_080 extends AbstractAlly {
|
||||
final ActivateCardAction action = new ActivateCardAction(self, Keyword.FELLOWSHIP);
|
||||
action.appendCost(new ExertCharactersEffect(self, self));
|
||||
action.appendEffect(
|
||||
new ChooseArbitraryCardsEffect(playerId, "Choose card(s) to discard", game.getGameState().getHand(playerId), 0, 3) {
|
||||
new ChooseCardsFromHandEffect(playerId, 0, 3, Filters.any()) {
|
||||
@Override
|
||||
protected void cardsSelected(LotroGame game, Collection<PhysicalCard> selectedCards) {
|
||||
action.appendEffect(new DiscardCardsFromHandEffect(self, selectedCards));
|
||||
|
||||
@@ -10,6 +10,8 @@ even though after page reload (F5) they were showing up.
|
||||
- Added game manual.
|
||||
- The "Card information" window will be bigger by default. In cases of most screen sizes it should be enough to see the
|
||||
whole card without scrolling window.
|
||||
- Arbitrary multiple card choice now works correctly.
|
||||
- Saruman no longer can be assigned to skirmish.
|
||||
|
||||
<b>26 Sept. 2011</b>
|
||||
- Changed the cost-to-effect actions to follow the "do as much as possible" route, rathern than can/can't do.
|
||||
|
||||
Reference in New Issue
Block a user