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

Addition of Versatile still triggers "Contains forbidden cards" alert for investigators that have limits of cards of other class #637

Open
VladShubnikov opened this issue Jan 13, 2024 · 3 comments

Comments

@VladShubnikov
Copy link

Issue: If Versatile is added to a deck of investigator who has limits of cards level 0 for other classes and you add one more card of level 0 the alert "Contains forbidden cards (cards not permitted by Investigator)" is displayed.

STR:

  1. Create Carolyn Fern's deck, she has deckbuilding rule: up to 15 other Seeker and/or Mystic cards level 0-1.
  2. Add Versatile, this should increase limit for cards to 16
  3. Add 16 level 0 cards of Seeker and/or Mystic that do not heal horror

Actual result: error "Contains forbidden cards (cards not permitted by Investigator)" is displayed
Expected result: no errors are displayed unless 17 level 0 cards of Seeker and/or Mystic that do not heal horror are added

@VladShubnikov
Copy link
Author

Hi, After some testing found that addition of customizable cards triggers this bug.

For example, adding of Empirical Hypothesis (https://arkhamdb.com/card/09041) with any upgrade counts as 2 cards of seeker class.

@zzorba
Copy link
Contributor

zzorba commented Jan 16, 2024

If you check a box on customizable, it is no longer a level 0 card and therefore can't be included with versatile.

@zzorba
Copy link
Contributor

zzorba commented Jan 16, 2024

Which means fix is probably to sort cards by descending XP before checking which rule they apply to, on assumption that higher XP cards have more constraints.

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