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

Blood Filter CBM update to remove Badly Poisoned status #1440

Conversation

KheirFerrum
Copy link
Collaborator

Summary

Blood Filter now removes Badly Poisoned status.

Purpose of change

Didn't seem like there was a good reason to have Badly Poisoned not be affected at all by Blood Filter CBM. It is still a toxin in your bloodstream after all.

Describe alternatives you've considered

One of my considerations was that Badly Poisoned could be downgraded to just Poisoned before being removed on the 2nd "flush", this would throw in resource and turn management considerations into avoiding Badly Poisoned to begin with. However I lack the technical knowhow to make this work, so this is what you get.

Testing

Walked into Poison Gas, activated Blood Filter CBM, saw poison removed.

Additional context

Blood Analysis CBM considers both Badly Poisoned and Poisoned to be separate conditions but simultaneously calls them poisoned. I'd hope that this could be changed later on.

@KheirFerrum KheirFerrum changed the title Update bionics.cpp Blood Filter CBM update to remove Badly Poisoned status Apr 5, 2022
@Coolthulhu Coolthulhu self-assigned this Apr 6, 2022
@Coolthulhu Coolthulhu merged commit c67bb9d into cataclysmbnteam:upload Apr 6, 2022
@KheirFerrum KheirFerrum deleted the Blood-Filter-Removes-Badly-Poisoned branch April 20, 2022 21:25
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.

2 participants