I discovered this one as I was working on #498.

The problem is we catch absolutely all parameter mismatches, including "legitimate" ones which happen long after we've made the call to MAIN.

All handlers are written as followed:

[HandlerAction(PacketType.MOVEITEM)]
public void OnMoveItem(IWorldServerClient serverClient, MoveItemPacket packet)
{
    _inventorySystem.MoveItem(serverClient.Player, packet.SourceSlot, packet.DestinationSlot);
}

Most of the time, in handlers, we use the serverClient.Player property. I think it might be better to pass directly a `IPla