Skip to content

Add Reed Richards#10931

Open
tehdiplomat wants to merge 3 commits into
masterfrom
reed_richards
Open

Add Reed Richards#10931
tehdiplomat wants to merge 3 commits into
masterfrom
reed_richards

Conversation

@tehdiplomat

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread forge-game/src/main/java/forge/game/player/Player.java Outdated
tool4ever
tool4ever previously approved these changes Jun 9, 2026

@tool4ever tool4ever left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I couldn't find a way to avoid extra count

@tool4ever tool4ever requested a review from Hanmac June 9, 2026 18:02
if (game.getPhaseHandler().is(PhaseType.DRAW)) {
numDrawnThisDrawStep++;
if (numDrawnThisDrawStep == 1) {
numExtraDrawnThisTurn--;

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is wrong. What if you draw during a different players draw step?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also if you skip your "normal" draw to Pursuit of Knowledge, but then extra draw during your draw step. @tool4ever

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's why I'm checking for it in ReplaceDraw again
not sure if that can be consolidated differently

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants