Correct vehicle movement issues. Fixes BUKKIT-2993, BUKKIT-4056

When a player comes into range of an entity in a vehicle they will often be
sent the spawn packet for the rider before receiving one for the vehicle.
This causes the attachment packet to fail client side because it attemts
to attach the rider to a vehicle that doesn't exist on the client. To
correct this we account for both possible ordering and send the
attachment packet appropriately.

In what appears to be an attempt to fix this in vanilla an attachment
packet is sent every 60 ticks even if the rider hasn't changed vehicles.
As this packet is a toggle this causes the rider to appear to toggle
between riding in the vehicle and appearing somewhere else in the world.
As this fix appears broken anyway and we have already corrected the problem
we simply remove the 60 tick logic here entirely.
Various things
Add MobEffect from mc-dev for diff visibility
Use wall time instead of ticks for more things