Added recommended work-around for handling player movement in such a way that it doesn't trigger the movement speed check. Thanks for the help Rigby!

Plugins are advised to no longer teleport players in the PlayerMove event and to use setTo instead, as we now trigger a teleport when setTo is used.

1 files changed, 12 insertions, 2 deletions

diff --git a/src/main/java/net/minecraft/server/NetServerHandler.java b/src/main/java/net/minecraft/server/NetServerHandler.java
index 052a2b80..c5b2b9c4 100644
--- a/src/main/java/net/minecraft/server/NetServerHandler.java
+++ b/src/main/java/net/minecraft/server/NetServerHandler.java
@@ -139,8 +139,18 @@ public class NetServerHandler extends NetHandler implements ICommandListener {
 
                 server.getPluginManager().callEvent(event);
 
-                from = event.getFrom();
-                to = event.isCancelled() ? from : event.getTo();
+                // If the event is cancelled we move the player back to their old location.
+                if (event.isCancelled()) {
+                    to = from;
+                }
+
+                /* If a Plugin has changed the To destination then we teleport the Player
+                   there to avoid any 'Moved wrongly' or 'Moved too quickly' errors.
+                   We only do this if the Event was not cancelled. */
+                if (!to.equals(event.getTo()) && !event.isCancelled()) {
+                    ((CraftPlayer) this.player.getBukkitEntity()).teleport(event.getTo());
+                    return;
+                }
 
                 this.player.locX = to.getX();
                 this.player.locY = to.getY();