Actually close the inventory on a *cross*-world teleport.

1 files changed, 2 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 81099584..e4f1b160 100644
--- a/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
+++ b/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
@@ -313,12 +313,12 @@ public class CraftPlayer extends CraftHumanEntity implements Player {
 
         // Check if the fromWorld and toWorld are the same.
         if (fromWorld == toWorld) {
+            entity.netServerHandler.teleport(to);
+        } else {
             // Close any foreign inventory
             if (getHandle().activeContainer != getHandle().defaultContainer){
                 getHandle().closeInventory();
             }
-            entity.netServerHandler.teleport(to);
-        } else {
             server.getHandle().moveToWorld(entity, toWorld.dimension, true, to);
         }
         return true;