Hi, thanks for the PR!

SA changes entity health directly as well, so this implementation doesn't cover all cases.
The only way you could make this event work reliably is to dispatch it from the net code [since it tracks entity health for sync].

Almost after a month's break, lol. I decided to use a server-side event only. It is the most accurate and insta calls when the health value changes. Works for player, ped, vehicle and object.

As @Pirulax mentioned, I think it would be nice for onElementHealthChange to trigger whenever element health changes, so not just via setElementHealth, but also when changed in-game (from fall damage, explosion, etc). There should be a parameter to the event for whether it was changed via script.

Currently, regardless of the circumstances, this event will always be triggered. The parameter idea seems interesting.

Any update?

@znvjder:

Currently, regardless of the circumstances, this event will always be triggered.

As i see it, this is useful enough (that it triggers in all cases) to be in MTA as-is, so i consider this PR pretty mature.
We can merge it now, and add parameters later, either by you or someone else. It will prevent this PR from becoming very stale.

For this line of action, please resolve conflicts, and thanks for all the effort!