-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
fix #31963: Stopped climbing movent while in containers #32033
base: master
Are you sure you want to change the base?
Conversation
thank you very much, also i suggest you to read this |
thanks, I was unaware that could be done automatically |
After interacting with this problem it appears that there is a problem with the methods registered to events, and will require further attention. It appears that event registers to stop the doAfter for Buckling and Storing |
issues from #31963 should be correctly addressed: awaiting further review
|
|
||
if (_containers.IsEntityInContainer(uid)) | ||
return; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be done up the callstack in canclimb or the likes not after climbing is happening.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The ClimbingComponent field CanClimb is effectively unused as it is never altered from a true state. The code alteration could be made by subscribing to container remove as well, but if CanClimb is to be used I would recommend scoping this up from a bug fix to a ClimbSystem Rework.
I have moved the inventory check into OnDoAfter where it will stop the call of climb. I have also observed that it is technicality unnecessary but will result in visible rollback if not checked before climbing is initiated.
glorious, test failed; I shall look into it. |
About the PR
This resolves a bug that allows players to shoot while inside crates/lockers
Why / Balance
resolves #31963
Technical details
Block final transform for climbing while entity (player) is in a container.
Final climb may succeed still while in container, but the player remains in place.
Media
Requirements
Breaking changes
Changelog
🆑