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

Fixes seeds able to mutate through infusions being able to infused beyond 99% damage #12446

Merged

Conversation

M-Earthfire
Copy link
Contributor

[hydroponics][bug]

About the PR

Moves the check for seed damage out of proc/HYPInfusionP into proc/HYPInfusionS. All plants that override proc/HYPInfusionP were able to be damaged over 100% by the used chem (most notable encountered when making seethers with phlogiston, which often generated seeds with 106% or up to 190% damage when the first infusion hit like 90%). Instead of changing all children of proc/HYPInfusionP, the parent was changed to accomodate to how the proc was used to prevent further incidents of that kind.

Why's this needed?

Seeds shouldn't be damaged by over 99% and survive that. This should fix issue #9768.

Changelog

(u)Lord_Earthfire
(*)Fixes seeds being able to be damaged by over 100%. Prepare to have your phlog infusions to fail more often!

… when they have an infusion-triggered mutation
@keywordlabeler keywordlabeler bot added the C-Bug A bug that impacts usage of a feature label Dec 18, 2022
@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Dec 18, 2022
@Tarmunora Tarmunora merged commit f61529a into goonstation:master Dec 18, 2022
github-actions bot pushed a commit that referenced this pull request Dec 18, 2022
@M-Earthfire M-Earthfire deleted the Infusion_seed_destruction_fix branch December 18, 2022 17:09
@flrsh
Copy link
Contributor

flrsh commented Dec 18, 2022

For future reference, fixes like this warrant a minor changelog at most.

@M-Earthfire
Copy link
Contributor Author

M-Earthfire commented Dec 18, 2022

To be honest, o thought i put it as a mino change. I was kinda surprised to find it as major change. That was a mistake on my part, sorry for that.

I better check my other bugfix-prs if i made them minor changes correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Bug A bug that impacts usage of a feature size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants