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 synthetics being in a weird limbo state when they have their head reattached. #5916

Merged
merged 2 commits into from
Mar 18, 2024

Conversation

TheGamerdk
Copy link
Contributor

@TheGamerdk TheGamerdk commented Mar 10, 2024

About the pull request

Did some digging.
Turns out, when the Queen gibs someone the flag 'undefibbable' gets set.
Then, next time the Life() proc is called we get removed from the 'processable_humans_list' on SSHuman. (Makes sense, no reason to process someone unrevivable, which non-synth gibbed people are)
This means when the synth has their head reattached and they get revived they no longer process Life() ticks. This means their blindness + critical state will never get cleared.

I fixed this by not making synths 'undefibbable'. You already can't revive someone without a head, so you won't be able to revive a headless corpse.

In addition I checked the usages of 'undefibbable' and didn't find anything that would break from this not being set.

Explain why it's good for the game

Testing Photographs and Procedure

Screenshots & Videos

Tested the following:
Not able to revive headless corpse
Not able to revive before head is fully attached
Revived synth can speak normally + see
Revived synth can walk and do normal actions when limbs are reattached.

Changelog

🆑
fix: Synths can now have their heads reattached without being sent to limbo-crit-hell.
/:cl:

@github-actions github-actions bot added the Fix Fix one bug, make ten more label Mar 10, 2024
Copy link
Member

@harryob harryob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

redundant else after a return clause

code/modules/mob/living/carbon/human/death.dm Outdated Show resolved Hide resolved
@TheGamerdk TheGamerdk requested a review from harryob March 15, 2024 11:18
@harryob harryob added this pull request to the merge queue Mar 18, 2024
Merged via the queue into cmss13-devs:master with commit 3dfe21e Mar 18, 2024
26 checks passed
cm13-github added a commit that referenced this pull request Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants