Added a catch for if a player is caged and kicked. Resolves #1789 #2041
Conversation
| @@ -1,6 +1,8 @@ | |||
| package me.totalfreedom.totalfreedommod.command; | |||
|
|
|||
| import me.totalfreedom.totalfreedommod.banning.Ban; | |||
| import me.totalfreedom.totalfreedommod.caging.CageData; | |||
| import me.totalfreedom.totalfreedommod.player.FPlayer; | |||
There was a problem hiding this comment.
I think you meant to put these in the other file..
There was a problem hiding this comment.
These will need to be removed, methinks.
There was a problem hiding this comment.
Yes, thats where I changed implementation mid-way through. Let me fix that.
| @@ -96,6 +97,18 @@ public void onPlayerQuit(PlayerQuitEvent event) | |||
| } | |||
|
|
|||
| @EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true) | |||
| public void onPlayerKick(PlayerKickEvent event) | |||
There was a problem hiding this comment.
Why ignoreCancelled? If the event is cancelled, why do you need to remove the cage?
There was a problem hiding this comment.
Wild's implementation is correct. If the kick event is cancelled, the player isn't kicked, so the cage data shouldn't be removed. ignoreCancelled causes the event handler to skip processing if the event is cancelled.
There was a problem hiding this comment.
Oh, never mind then. I was thinking the other way around.
| @@ -1,6 +1,8 @@ | |||
| package me.totalfreedom.totalfreedommod.command; | |||
|
|
|||
| import me.totalfreedom.totalfreedommod.banning.Ban; | |||
| import me.totalfreedom.totalfreedommod.caging.CageData; | |||
| import me.totalfreedom.totalfreedommod.player.FPlayer; | |||
There was a problem hiding this comment.
These will need to be removed, methinks.
|
@JeromSar Unused imports removed. |
|
Thanks for contributing! Your PR will be merged soon. |
Resolves #1789