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

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;