ambidextrous/gemp-lotr
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixed discard triggers causing reconciliation to draw back to the incorrect number of cards

Browse Source
This commit is contained in:
Christian 'ketura' McCarty
2024-11-20 22:24:23 -06:00
parent f122369419
commit 51f6482714
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
gemp-lotr/gemp-lotr-logic/src/main/java/com/gempukku/lotro/logic/actions/PlayerReconcilesAction.java
View File

@@ -110,16 +110,20 @@ public class PlayerReconcilesAction implements Action {
new TriggeringResultEffect(new ReconcileResult(_playerId), "Player reconciled"));
}
}));
} else if (cardsInHand.size() > 0) {
} else if (!cardsInHand.isEmpty()) {
_effectQueue.add(new PlayoutDecisionEffect(_playerId,
new CardsSelectionDecision(1, "Reconcile - choose card to discard or press DONE", cardsInHand, 0, 1) {
@Override
public void decisionMade(String result) throws DecisionResultInvalidException {
Set<PhysicalCard> selectedCards = getSelectedCardsByResponse(result);
if (selectedCards.size() > 0) {
if (!selectedCards.isEmpty()) {
_effectQueue.add(new DiscardCardsFromHandEffect(null, _playerId, selectedCards, false));
}
int cardsInHandAfterDiscard = cardsInHand.size() - selectedCards.size();
//We refresh the hand size here because effects could have reacted to the discard, causing
// the previous hand size calculation to be stale.
//See https://github.com/PlayersCouncil/gemp-lotr/issues/89
var currentCards = new HashSet<PhysicalCard>(gameState.getHand(_playerId));
int cardsInHandAfterDiscard = currentCards.size();
if (cardsInHandAfterDiscard < handSize) {
_effectQueue.add(new DrawCardsEffect(PlayerReconcilesAction.this, _playerId, handSize - cardsInHandAfterDiscard));
}