diff options
author | LogicParrot <LogicParrot@users.noreply.github.com> | 2016-02-02 16:37:21 +0100 |
---|---|---|
committer | LogicParrot <LogicParrot@users.noreply.github.com> | 2016-02-02 16:55:00 +0100 |
commit | 07b7fd4ad3fef714399009efd5d4a903a561a6f2 (patch) | |
tree | 27253c8e468f068dd03d7bf8b434e4e78ca78144 /src/Entities | |
parent | Merge pull request #2936 from mathias-github/master (diff) | |
download | cuberite-07b7fd4ad3fef714399009efd5d4a903a561a6f2.tar cuberite-07b7fd4ad3fef714399009efd5d4a903a561a6f2.tar.gz cuberite-07b7fd4ad3fef714399009efd5d4a903a561a6f2.tar.bz2 cuberite-07b7fd4ad3fef714399009efd5d4a903a561a6f2.tar.lz cuberite-07b7fd4ad3fef714399009efd5d4a903a561a6f2.tar.xz cuberite-07b7fd4ad3fef714399009efd5d4a903a561a6f2.tar.zst cuberite-07b7fd4ad3fef714399009efd5d4a903a561a6f2.zip |
Diffstat (limited to '')
-rw-r--r-- | src/Entities/Entity.cpp | 13 | ||||
-rw-r--r-- | src/Entities/Player.cpp | 18 |
2 files changed, 25 insertions, 6 deletions
diff --git a/src/Entities/Entity.cpp b/src/Entities/Entity.cpp index d4097f734..ff03bae3c 100644 --- a/src/Entities/Entity.cpp +++ b/src/Entities/Entity.cpp @@ -1499,8 +1499,17 @@ bool cEntity::DoMoveToWorld(cWorld * a_World, bool a_ShouldSendRespawn, Vector3d return false; } - // Remove all links to the old world - SetWorldTravellingFrom(GetWorld()); // cChunk::Tick() handles entity removal + // Remove entity from chunk + if (!GetWorld()->DoWithChunk(GetChunkX(), GetChunkZ(), [this](cChunk & a_Chunk) -> bool + { + a_Chunk.SafeRemoveEntity(this); + return true; + })) + { + LOGD("Entity Teleportation failed! Didn't find the source chunk!\n"); + return false; + } + GetWorld()->BroadcastDestroyEntity(*this); SetPosition(a_NewPosition); diff --git a/src/Entities/Player.cpp b/src/Entities/Player.cpp index 767ee2061..7ba6b2bf6 100644 --- a/src/Entities/Player.cpp +++ b/src/Entities/Player.cpp @@ -1695,6 +1695,20 @@ bool cPlayer::DoMoveToWorld(cWorld * a_World, bool a_ShouldSendRespawn, Vector3d return false; } + // Remove player from chunk + if (!GetWorld()->DoWithChunk(GetChunkX(), GetChunkZ(), [this](cChunk & a_Chunk) -> bool + { + a_Chunk.SafeRemoveEntity(this); + return true; + })) + { + LOGD("Entity Teleportation failed! Didn't find the source chunk!\n"); + return false; + } + + // Remove player from world + GetWorld()->RemovePlayer(this, false); + // Send the respawn packet: if (a_ShouldSendRespawn && (m_ClientHandle != nullptr)) { @@ -1704,10 +1718,6 @@ bool cPlayer::DoMoveToWorld(cWorld * a_World, bool a_ShouldSendRespawn, Vector3d // Broadcast for other people that the player is gone. GetWorld()->BroadcastDestroyEntity(*this); - // Remove player from the old world - SetWorldTravellingFrom(GetWorld()); // cChunk handles entity removal - GetWorld()->RemovePlayer(this, false); - SetPosition(a_NewPosition); // Queue adding player to the new world, including all the necessary adjustments to the object |