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

Vehicle falls forever on skylight #56514

Merged
merged 1 commit into from
Apr 6, 2022

Conversation

RoyBerube
Copy link
Contributor

Summary

Bugfixes "Vehicle falls forever on skylight"

Purpose of change

Fixes #56477

Vehicles would fall infinitely if on a skylight, never breaking it or actually falling.

Also items or characters would fall through a skylight without breaking it.

Describe the solution

Remove the NO_FLOOR flag and the trap. This allows items to be supported by skylights.

Fix spelling error.

Justification: simplified skylight behaviour is better than its current buggy state.

Describe alternatives you've considered

Fix the code. Since this is such a rarely encountered terrain type, I am not enthusiastic about spending more time here.

Testing

Dragged luggage cart onto a hotel skylight. Luggage cart and player stay supported without falling. Furniture is not allowed on the tile, which is ok.

Additional context

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Apr 1, 2022
@kevingranade kevingranade merged commit 5188e38 into CleverRaven:master Apr 6, 2022
@RoyBerube RoyBerube deleted the skylight-crash branch April 6, 2022 01:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Luggage cart falling forever creating lots of lag and smashing noises
2 participants