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

Categories for Weapons Bonuses #119

Merged
merged 1 commit into from Jul 21, 2019

Conversation

@crnormand
Copy link
Contributor

commented Jul 21, 2019

The last of the "categories" change sets. This adds category filtering to weapons bonuses. The only real difference is that the "owner" of a WeaponStat could be something other than Equipment, and if so, we won't know what categories it belongs to (so we send null). And we had to protect the WeaponBonus matchesCategories() method in case the categories were null.

@richardwilkes richardwilkes merged commit 5883eb1 into richardwilkes:master Jul 21, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.