Skip to content

Fix grappling hook ladder death bug - #106

Merged
Pyritie merged 2 commits into
TerraFirmaGreg-Team:devfrom
Zippity:ladder-fix
Aug 20, 2025
Merged

Pyritie merged 2 commits into
TerraFirmaGreg-Team:devfrom
Zippity:ladder-fix

Conversation

@Zippity

@Zippity Zippity commented Aug 20, 2025

Copy link
Copy Markdown
Contributor

What is the new behavior?

Fixes #1647 on our side while my PR for Grappling Hook Reforged is reviewed

Implementation Details

I copied this fix from a PR from the original Grappling Hook mod. It adds an additional ladder check as well as resets fall distance on grapple release.

Outcome

Grappling hooks no longer kill you if you jump from one onto a ladder.

Additional Information

This apparently fixes other issues with how grappling hooks and gliders interact according to #123, so that's good?

Potential Compatibility Issues

If my PR gets accepted by the grapple mod author then this fix will become redundant and will need to be removed.

Redeix
Redeix previously approved these changes Aug 20, 2025

@Redeix Redeix left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

you sure love mixins huh?

@Redeix
Redeix requested a review from gustovafing August 20, 2025 05:08
@Zippity

Zippity commented Aug 20, 2025

Copy link
Copy Markdown
Contributor Author

I really do lol

@Redeix

Redeix commented Aug 20, 2025

Copy link
Copy Markdown
Member

I really do lol

Well thats good because I dont like making them lmao. Btw when you make a mixin I recommend adding the mods license to the top of the class like I did with the others. 99% of the time it wont be a problem but we dont want to risk curseforge taking the project down.

@Pyritie
Pyritie merged commit 331bdc6 into TerraFirmaGreg-Team:dev Aug 20, 2025
@Zippity
Zippity deleted the ladder-fix branch August 21, 2025 03:06
Zippity added a commit to Zippity/Core-Modern-ZPT that referenced this pull request Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🐛]: Grappling hooking onto a ladder doesn't allow climbing

3 participants