Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Thirst for Discovery didn't force me to discard #13291

Closed
alexander-novo opened this issue Feb 3, 2025 · 2 comments
Closed

Thirst for Discovery didn't force me to discard #13291

alexander-novo opened this issue Feb 3, 2025 · 2 comments

Comments

@alexander-novo
Copy link
Contributor

Image

Played and resolved a [[Thirst for Discovery]], which normally asks if I'd like to discard a basic land or two cards, but instead didn't ask me that and I didn't discard any cards.

Haven't checked the code yet, but I suspect it might be related to the fact that I have no basic lands in my hand?

Copy link

github-actions bot commented Feb 3, 2025

Thirst for Discovery - (Gatherer) (Scryfall) (EDHREC)

{2}{U}
Instant
Draw three cards. Then discard two cards unless you discard a basic land card.

@alexander-novo
Copy link
Contributor Author

Indeed, this line short-circuits the logic:

if (!this.cost.canPay(source, source, player.getId(), game)) {

If you can't pay the cost, then just nothing happens

@JayDi85 JayDi85 closed this as completed Feb 3, 2025
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

No branches or pull requests

2 participants