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

[MIRROR] Fixes a small issue with disposal outlets. #640

Merged
merged 1 commit into from
Nov 15, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25018
Original PR: tgstation/tgstation#79706

About The Pull Request

Disposal outlets eject the contents of the holder with a 2 seconds delay. Delete the outlet before the delay is over and the contents will stay stuck inside the holder.

Also merging disposal holders doesn't properly register a signal for mobs transfered to the new holder.

Why It's Good For The Game

Bugfixing. Disposal code is a mess. Perhaps related to #79629 but I couldn't exactly reproduce it.

Changelog

🆑 Ghommie
fix: Fixed a small issue with disposal outlets leaving contents about to be ejected stuck inside the pipe beneath it if deleted.
/:cl:

…018)

* Fixes a small issue with disposal outlets. (#79706)

## About The Pull Request
Disposal outlets eject the contents of the holder with a 2 seconds
delay. Delete the outlet before the delay is over and the contents will
stay stuck inside the holder.

Also merging disposal holders doesn't properly register a signal for
mobs transfered to the new holder.

## Why It's Good For The Game
Bugfixing. Disposal code is a mess. Perhaps related to #79629 but I
couldn't exactly reproduce it.

## Changelog

:cl:
fix: Fixed a small issue with disposal outlets leaving contents about to
be ejected stuck inside the pipe beneath it if deleted.
/:cl:

* Fixes a small issue with disposal outlets.

---------

Co-authored-by: Ghom <[email protected]>
@Iajret Iajret merged commit d26011c into master Nov 15, 2023
24 checks passed
@Iajret Iajret deleted the upstream-mirror-25018 branch November 15, 2023 09:42
AnywayFarus added a commit that referenced this pull request Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants