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

No more runtimes when a limb takes damage with no owner #23341

Merged
merged 2 commits into from
Nov 29, 2023

Conversation

Qwertytoforty
Copy link
Contributor

What Does This PR Do

Checks for an owner before checking droplimb from max damage on limbs since limbs don't droplimb with no owner and it causes null check

Why It's Good For The Game

bugs bad

Testing

Compiled, now has checks protected via the owner check

Changelog

🆑
fix: Limbs no longer runtime when taking damage with no owner
/:cl:

@ParadiseSS13-Bot ParadiseSS13-Bot added the -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally label Nov 23, 2023
Co-authored-by: Contrabang <91113370+Contrabang@users.noreply.github.com>
@farie82 farie82 added the Fix This PR will fix an issue in the game label Nov 29, 2023
@ParadiseSS13-Bot ParadiseSS13-Bot added -Status: Awaiting merge This PR is ready for merge and removed -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally labels Nov 29, 2023
@lewcc lewcc merged commit d63e2c9 into ParadiseSS13:master Nov 29, 2023
Oyu07 pushed a commit to Oyu07/Paradise that referenced this pull request Jan 2, 2024
…#23341)

* Update organ_external.dm

* Update code/modules/surgery/organs/organ_external.dm

Co-authored-by: Contrabang <91113370+Contrabang@users.noreply.github.com>

---------

Co-authored-by: Contrabang <91113370+Contrabang@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-Status: Awaiting merge This PR is ready for merge Fix This PR will fix an issue in the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants