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

Fix play_more_possible for Mole Man and Professor X #3

Open
michaeljb opened this issue Nov 8, 2014 · 2 comments
Open

Fix play_more_possible for Mole Man and Professor X #3

michaeljb opened this issue Nov 8, 2014 · 2 comments

Comments

@michaeljb
Copy link
Owner

Mole Man can play up to 6 Subterranea (or whichever group he's leading if you play Advanced Solo Mode) Villains; the current implementation calculates turns based on him playing up to 6 of any Villain or Henchman card.

Note: no known interaction where it definitely affects the calculation of the turn range; if one is found, add a feature test for it and implement correctly.

michaeljb added a commit that referenced this issue Nov 8, 2014
michaeljb added a commit that referenced this issue Nov 21, 2014
This method will need further refactoring should a new Adversary group
come out featuring 2 or fewer cards with "X-Treme Attack". This will
also need to be changed if some edge case can be found involving
henchmen (like Mole Man's problem in GH-3).

[GH-15]
@michaeljb
Copy link
Owner Author

Similarly, Professor X can play up to 3 Adversaries with X-Treme Attack; the current implementation calculates turns based on him playing up to 3 of any Villain or Henchman card, given Uncanny Avengers, Uncanny X-Men, or X-Men First Class are in use.

@michaeljb michaeljb changed the title Fix Mole Man's play_more_possible Fix play_more_possible for Mole Man and Professor X Nov 21, 2014
@michaeljb
Copy link
Owner Author

Any input on setup where these details actually affect when the Deck empties would be appreciated.

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

No branches or pull requests

1 participant