ISSUE:
Entities lack any sort of flag for if they are currently swimming. This causes all of the swimming checks for players to be constantly redone all over the code base.
EXPECTED RESULT:
There should be a flag for the Entity class that says if they are swimming or not. Then swimming can be calculated once somewhere else, perhaps when an Entity enters a new map tile.
ADDITIONAL INFO:
This is a necessary refactor to fix #142 and #143. Currently the animation manager handles the constant drain of MP in single player when the animation is active. It also doesn't actually cast the spell until the animation is finished, because of this, #143, cannot be efficiently fixed.
ISSUE:
Entities lack any sort of flag for if they are currently swimming. This causes all of the swimming checks for players to be constantly redone all over the code base.
EXPECTED RESULT:
There should be a flag for the Entity class that says if they are swimming or not. Then swimming can be calculated once somewhere else, perhaps when an Entity enters a new map tile.
ADDITIONAL INFO:
This is a necessary refactor to fix #142 and #143. Currently the animation manager handles the constant drain of MP in single player when the animation is active. It also doesn't actually cast the spell until the animation is finished, because of this, #143, cannot be efficiently fixed.