Comments on Profile Post by Praya

You shouldn't need to use that method add all, as it would affect anyone else using the bonuses. Instead, use addMaxHealth or addMaxMana, providing the negative equivalent of the bonus you've already given.

yeah I know that, but I think you should try to making an addon that using addMaxHealth() and addMaxMana() without clearBonusMaxHealth()

that very buggy and not constant, especially when player change class or server restart/reload/crash

better to make an event like PlayerMaxHealthChange and PlayerMaxManaChange rather than use that method

Clearing is handled by SkillAPI and done before they disconnect or switch classes. You just need to worry about applying your own bonuses and removing them if they end early.

or make getBonusMaxHealth and getBonusMaxMana then separate the set method with the update method

for a plugin that often to change max health, addMaxHealth() is very buggy because will calculate the previous bonus.

I already tried by using a hashmap to save last bonus, but when the server restarts the last bonus off course will be removed. that causing old max health affect to new max health.