From 7b220c450080724862209496862440aa83731991 Mon Sep 17 00:00:00 2001 From: Christian 'ketura' McCarty Date: Thu, 21 Nov 2024 22:46:23 -0600 Subject: [PATCH] Fixed the reconcile fix drawing 1 too few cards when the player discards a card during reconciliation --- .../gempukku/lotro/logic/actions/PlayerReconcilesAction.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gemp-lotr/gemp-lotr-logic/src/main/java/com/gempukku/lotro/logic/actions/PlayerReconcilesAction.java b/gemp-lotr/gemp-lotr-logic/src/main/java/com/gempukku/lotro/logic/actions/PlayerReconcilesAction.java index 8b5d12e4b..65c45ea81 100644 --- a/gemp-lotr/gemp-lotr-logic/src/main/java/com/gempukku/lotro/logic/actions/PlayerReconcilesAction.java +++ b/gemp-lotr/gemp-lotr-logic/src/main/java/com/gempukku/lotro/logic/actions/PlayerReconcilesAction.java @@ -122,8 +122,8 @@ public class PlayerReconcilesAction implements Action { //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(gameState.getHand(_playerId)); - int cardsInHandAfterDiscard = currentCards.size(); + //As the discard selection is only queued, we need to subtract it from the current hand size. + int cardsInHandAfterDiscard = gameState.getHand(_playerId).size() - selectedCards.size(); if (cardsInHandAfterDiscard < handSize) { _effectQueue.add(new DrawCardsEffect(PlayerReconcilesAction.this, _playerId, handSize - cardsInHandAfterDiscard)); }