API Break;

[iZeaoGamer]

Hi. This issue should fix: the rename from getDamager() to getModifier(). This was a API break that pocketmine have done over the past few weeks or months, and I just thought I'd update it to that API, and make sure to update all the plugin's problems and issues I see.
This has not been tested yet however, you may want to test this commit before merging.
If more errors occur, please let me know, and I'll do more testing myself to double check that this issue get's resolved.

UPDATES

The updates which consists:
Updating the propertie's code (onDeath) event, to double check that getModifier is now involved and added, instead of getDamager(), the pre-resistance bug which has occurred for a while, and should now be fixed.
This issue should also fix: #5
If there's anything else I need to change, then please let me know.
Thank you.

[JaxkDev]

Hi, most of my plugins have gone private to the server I now work on, but I do still accept PR’s on public work.
I will need to test this first, I’ll get back to you when I have.

[iZeaoGamer]

Okay, thanks.

[iZeaoGamer]

I just noticed getDamager() is different to getModifier(). I've tested similar plugins that has this part of the code and they seem to work just fine. I'm not entirely sure what happened, but this PR should fix everything relating to the issue posts you were receiving.
Let's just see how this plays out, and let me know if this fixes it or not. Thanks.
And sorry for any confusions there.

[JaxkDev]

I’ll test this version also, but in reality I need to rewrite the whole plugin at some point.

[iZeaoGamer]

Alright.

[JaxkDev]

Tests were run on latest pmmp server, and the death of someone with a valid bounty failed no message or logged message by plugin, I can only assume $cause was not of player instance.

[JaxkDev]

So I had some time to myself and read the breaks,
I found that all I have to do is change getPlayer to getEntity.