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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drugs call parent on affect_ingest(), removes duplicate definition of affect_blood() #8145

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

RikuTheKiller
Copy link
Contributor

About The Pull Request

I started wondering why hyperzine doesn't work on ingest, well, here's why.

/drug/affect_ingest() was not calling parent, meaning the affect_blood() called in /reagent/affect_ingest() was not getting called.

There was also a duplicate definition of affect_blood() for whatever reason, so I removed that.

Why It's Good For The Game

Fixes a long-standing bug (3 years) and removes an useless piece of code.

Testing

Compiles, no other testing done.
This can't cause any new bugs.
...don't quote me on that.

Changelog

馃啈
fix: Drugs work properly on ingest.
/:cl:

@RikuTheKiller RikuTheKiller marked this pull request as draft April 30, 2023 11:19
@Firefox13
Copy link
Contributor

If you didn't test it it doesn't work

@RikuTheKiller
Copy link
Contributor Author

If you didn't test it it doesn't work

Wrong, also, draft cause I decided on possibly porting the WHOLE tg metab system

@RikuTheKiller
Copy link
Contributor Author

My brain is going to be so joever after this...

@RikuTheKiller
Copy link
Contributor Author

Also this might be a bad PR to implement, it nerfs drugs hard

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.

None yet

2 participants