The result of the given event invocation compares against the ActionResult.SUCCESS object (for 1.21.5 atleast) which is wrong I believe as such should check if the given instance objects isAccepted() since you can create a different SUCCESS object by adjusting the different held stack. This means that if done so on the client to just keep common code, you will never match against the SUCCESS object.