Fixed - reveal and choose card(s) effects now populate the list of cards into game log
This commit is contained in:
@@ -4,6 +4,7 @@ import com.gempukku.lotro.common.Filterable;
|
||||
import com.gempukku.lotro.filters.Filters;
|
||||
import com.gempukku.lotro.game.PhysicalCard;
|
||||
import com.gempukku.lotro.game.state.LotroGame;
|
||||
import com.gempukku.lotro.logic.GameUtils;
|
||||
import com.gempukku.lotro.logic.PlayOrder;
|
||||
import com.gempukku.lotro.logic.decisions.ArbitraryCardsSelectionDecision;
|
||||
import com.gempukku.lotro.logic.decisions.DecisionResultInvalidException;
|
||||
@@ -47,6 +48,7 @@ public abstract class RevealAndChooseCardsFromOpponentHandEffect extends Abstrac
|
||||
public void playEffect(LotroGame game) {
|
||||
if (game.getModifiersQuerying().canLookOrRevealCardsInHand(game.getGameState(), _opponentId, _playerId)) {
|
||||
List<PhysicalCard> opponentHand = new LinkedList<PhysicalCard>(game.getGameState().getHand(_opponentId));
|
||||
game.getGameState().sendMessage(GameUtils.getCardLink(_source) + " revealed " + _opponentId + " cards in hand - " + getAppendedNames(opponentHand));
|
||||
|
||||
final PlayOrder playOrder = game.getGameState().getPlayerOrder().getCounterClockwisePlayOrder(_opponentId, false);
|
||||
// Skip hand owner (opponent)
|
||||
|
||||
@@ -1,8 +1,12 @@
|
||||
package com.gempukku.lotro.logic.timing;
|
||||
|
||||
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.SubAction;
|
||||
|
||||
import java.util.Collection;
|
||||
|
||||
public abstract class AbstractSubActionEffect implements Effect {
|
||||
private SubAction _subAction;
|
||||
|
||||
@@ -11,6 +15,10 @@ public abstract class AbstractSubActionEffect implements Effect {
|
||||
game.getActionsEnvironment().addActionToStack(_subAction);
|
||||
}
|
||||
|
||||
protected final String getAppendedNames(Collection<? extends PhysicalCard> cards) {
|
||||
return GameUtils.getAppendedNames(cards);
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean wasCarriedOut() {
|
||||
return _subAction != null && _subAction.wasCarriedOut();
|
||||
|
||||
Reference in New Issue
Block a user