Skip to content

Comments

fix: use dynohook for take damage old for compatibility#1159

Merged
roflmuffin merged 6 commits intomainfrom
fix/use-dynohook-for-take-damage
Dec 23, 2025
Merged

fix: use dynohook for take damage old for compatibility#1159
roflmuffin merged 6 commits intomainfrom
fix/use-dynohook-for-take-damage

Conversation

@roflmuffin
Copy link
Owner

@roflmuffin roflmuffin commented Dec 23, 2025

Hook Result Behaviour Comparison

This PR changes the dyno hook handling to be somewhat more consistent with other hook pattern handling, in that a hook result of Stop will prevent other hooks from running immediately. It will also prevent any Post hooks from running if the result is Handled or Stop. This was required for cases such as "TakeDamageOld" which should not run the post damage hook if the damage has been prevented by a plugin in CS#.

It also adds a way for dyno hook hooks to be added on the native side.

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.

1 participant