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

[s] Disposals now checks for density before expelling #22274

Merged
merged 2 commits into from
Sep 15, 2023

Conversation

Adrer
Copy link
Contributor

@Adrer Adrer commented Sep 7, 2023

What Does This PR Do

When an unconnected disposals pipe tries to expel someone, it first checks for dense objects on the tile in front of it: If there is a dense objects on the tile in front of it, it will instead expel you on the tile with the pipe.
Fixes: #13249

Why It's Good For The Game

Currently you are able to bypass any door/window/non-turf dense object by placing an unconnected disposals trunk/pipe in front of it and getting into a chute, this PR aims to fix that.

Testing

Be a skrell with and RPD and tools
Point a disposals trunk into an airlock and add a chute to enter the trunk.
Enter the trunk.
Get thrown out of the trunk on the side of the airlock I started on, and not behind it.

Changelog

🆑
fix: Disposals now checks for density before expelling
/:cl:

@ParadiseSS13-Bot ParadiseSS13-Bot added the -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally label Sep 7, 2023
@matttheficus
Copy link
Contributor

Loony Toons'ing under a wall is kinda wacky. This is a good change.

@SteelSlayer SteelSlayer added the Fix This PR will fix an issue in the game label Sep 10, 2023
@ParadiseSS13-Bot ParadiseSS13-Bot added -Status: Awaiting review This PR is awaiting review from the review team and removed -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally labels Sep 10, 2023
Co-authored-by: SteelSlayer <42044220+SteelSlayer@users.noreply.github.com>
Copy link
Member

@hal9000PR hal9000PR left a comment

Choose a reason for hiding this comment

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

This kills a part of my soul to merge

@hal9000PR hal9000PR merged commit 0b36b64 into ParadiseSS13:master Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-Status: Awaiting review This PR is awaiting review from the review team Fix This PR will fix an issue in the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use disposals to go through shutters
5 participants