From 71b0510f88d1edf31dd8d823e4035f0b508c735f Mon Sep 17 00:00:00 2001
From: feildmaster <admin@feildmaster.com>
Date: Tue, 20 Nov 2012 18:15:00 -0600
Subject: [PATCH] Fix teleporting entities with vehicles/passengers teleporting
 you illegally. Fixes BUKKIT-2821

Also some formatting...
---
 .../org/bukkit/craftbukkit/entity/CraftPlayer.java  | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java b/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
index 46db36cc8e..efa4e05aac 100644
--- a/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
+++ b/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
@@ -346,7 +346,14 @@ public class CraftPlayer extends CraftHumanEntity implements Player {
     @Override
     public boolean teleport(Location location, PlayerTeleportEvent.TeleportCause cause) {
         EntityPlayer entity = getHandle();
-        if (entity.netServerHandler == null || entity.netServerHandler.disconnected) return false;
+
+        if (entity.netServerHandler == null || entity.netServerHandler.disconnected) {
+            return false;
+        }
+
+        if (entity.vehicle != null || entity.passenger != null) {
+            return false;
+        }
 
         // From = Players current Location
         Location from = this.getLocation();
@@ -355,10 +362,12 @@ public class CraftPlayer extends CraftHumanEntity implements Player {
         // Create & Call the Teleport Event.
         PlayerTeleportEvent event = new PlayerTeleportEvent((Player) this, from, to, cause);
         server.getPluginManager().callEvent(event);
+
         // Return False to inform the Plugin that the Teleport was unsuccessful/cancelled.
-        if (event.isCancelled() == true) {
+        if (event.isCancelled()) {
             return false;
         }
+
         // Update the From Location
         from = event.getFrom();
         // Grab the new To Location dependent on whether the event was cancelled.