From dcabbda57c2b3d1af21e652ba9d896da13eca455 Mon Sep 17 00:00:00 2001
From: Dinnerbone <dinnerbone@dinnerbone.com>
Date: Sun, 6 Feb 2011 14:03:28 +0000
Subject: [PATCH] Fixed invisibility after cross-world teleportation

---
 .../net/minecraft/server/InventoryPlayer.java |  2 +-
 .../minecraft/server/NetServerHandler.java    |  2 +-
 .../craftbukkit/entity/CraftPlayer.java       | 35 ++++++++++++-------
 3 files changed, 24 insertions(+), 15 deletions(-)

diff --git a/src/main/java/net/minecraft/server/InventoryPlayer.java b/src/main/java/net/minecraft/server/InventoryPlayer.java
index 6867dcd5d2..c918c46e80 100644
--- a/src/main/java/net/minecraft/server/InventoryPlayer.java
+++ b/src/main/java/net/minecraft/server/InventoryPlayer.java
@@ -5,7 +5,7 @@ public class InventoryPlayer implements IInventory {
     public ItemStack[] a = new ItemStack[36];
     public ItemStack[] b = new ItemStack[4];
     public int c = 0;
-    private EntityHuman e;
+    public EntityHuman e; // Craftbukkit - public
     private ItemStack f;
     public boolean d = false;
 
diff --git a/src/main/java/net/minecraft/server/NetServerHandler.java b/src/main/java/net/minecraft/server/NetServerHandler.java
index 33dfd64242..0ae1d3e65c 100644
--- a/src/main/java/net/minecraft/server/NetServerHandler.java
+++ b/src/main/java/net/minecraft/server/NetServerHandler.java
@@ -33,7 +33,7 @@ public class NetServerHandler extends NetHandler implements ICommandListener {
     public NetworkManager b;
     public boolean c = false;
     private MinecraftServer d;
-    private EntityPlayer e;
+    public EntityPlayer e; // Craftbukkit - public
     private int f = 0;
     private double g;
     private double h;
diff --git a/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java b/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
index f59b6b70ab..f1c62d5039 100644
--- a/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
+++ b/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
@@ -116,28 +116,37 @@ public class CraftPlayer extends CraftHumanEntity implements Player {
 
     @Override
     public void teleportTo(Location location) {
-        boolean worldChange = location.getWorld() != getWorld();
+        WorldServer oldWorld = ((CraftWorld)getWorld()).getHandle();
+        WorldServer newWorld = ((CraftWorld)location.getWorld()).getHandle();
         ServerConfigurationManager manager = server.getHandle();
 
-        if (worldChange) {
+        if (oldWorld != newWorld) {
             manager.c.k.a(entity);
             manager.c.k.b(entity);
             manager.d.b(entity);
-            entity.world.e(entity);
+            manager.b.remove(entity);
+            oldWorld.e(entity);
 
-            entity.world = ((CraftWorld)location.getWorld()).getHandle();
+            EntityPlayer newEntity = new EntityPlayer(manager.c, newWorld, entity.name, new ItemInWorldManager(newWorld));
 
-            entity.c = new ItemInWorldManager(((CraftWorld)location.getWorld()).getHandle());
-            entity.c.a = entity;
+            newEntity.id = entity.id;
+            newEntity.a = entity.a;
+            newEntity.health = entity.health;
+            newEntity.fireTicks = entity.fireTicks;
+            newWorld.A.d((int) location.getBlockX() >> 4, (int) location.getBlockZ() >> 4);
 
-            ((WorldServer)entity.world).A.d((int) entity.locX >> 4, (int) entity.locZ >> 4);
-        }
+            newEntity.a.a(location.getX(), location.getY(), location.getZ(), location.getYaw(), location.getPitch());
+            manager.d.a(newEntity);
+            newWorld.a(newEntity);
+            manager.b.add(newEntity);
+            newEntity.l();
 
-        entity.a.a(location.getX(), location.getY(), location.getZ(), location.getYaw(), location.getPitch());
-
-        if (worldChange) {
-            manager.d.a(entity);
-            entity.world.a(entity);
+            entity.a.e = newEntity;
+            newEntity.inventory = entity.inventory;
+            newEntity.inventory.e = newEntity;
+            entity = newEntity;
+        } else {
+            entity.a.a(location.getX(), location.getY(), location.getZ(), location.getYaw(), location.getPitch());
         }
     }
 }