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

warrior decap doesnt check for the hugged flagged #6087

Open
3 tasks done
Red-byte3D opened this issue Apr 5, 2024 · 2 comments
Open
3 tasks done

warrior decap doesnt check for the hugged flagged #6087

Red-byte3D opened this issue Apr 5, 2024 · 2 comments
Labels
Bug Admin events aren't bugs

Comments

@Red-byte3D
Copy link
Contributor

Testmerges

dnm

Round ID

dnm

Description of the bug

#4979 this doesnt work as you can still start the decap and get the decap off before the hugger is finished

What's the difference with what should have happened?

not this

How do we reproduce this bug?

1.hug man
2. grab man
3. behead man
...

Issue Bingo

@Red-byte3D Red-byte3D added the Bug Admin events aren't bugs label Apr 5, 2024
@vero5123
Copy link
Contributor

vero5123 commented May 1, 2024

it does though

@Drulikar
Copy link
Contributor

Drulikar commented May 2, 2024

So impregnation occurs after a random amount of time when a hugger attaches. What they're saying is that it doesn't recheck can_not_harm after the do_after. This is really niche though. See https://github.com/cmss13-devs/cmss13/blob/357e4fe2dd4fd58dedddd1f75040350fc792783c/code/modules/mob/living/carbon/xenomorph/castes/Warrior.dm#L192C5-L192C17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Admin events aren't bugs
Projects
None yet
Development

No branches or pull requests

3 participants