ChunkManager->setChunk() no longer accepts NULL

This commit is contained in:
Dylan K. Taylor 2020-12-05 18:54:30 +00:00
parent b6df5b974d
commit 59cb11dc83
3 changed files with 3 additions and 11 deletions

View File

@ -42,7 +42,7 @@ interface ChunkManager{
public function getChunk(int $chunkX, int $chunkZ) : ?Chunk;
public function setChunk(int $chunkX, int $chunkZ, ?Chunk $chunk) : void;
public function setChunk(int $chunkX, int $chunkZ, Chunk $chunk) : void;
/**
* Returns the height of the world

View File

@ -63,11 +63,7 @@ class SimpleChunkManager implements ChunkManager{
return $this->chunks[World::chunkHash($chunkX, $chunkZ)] ?? null;
}
public function setChunk(int $chunkX, int $chunkZ, ?Chunk $chunk) : void{
if($chunk === null){
unset($this->chunks[World::chunkHash($chunkX, $chunkZ)]);
return;
}
public function setChunk(int $chunkX, int $chunkZ, Chunk $chunk) : void{
$this->chunks[World::chunkHash($chunkX, $chunkZ)] = $chunk;
}

View File

@ -2025,11 +2025,7 @@ class World implements ChunkManager{
/**
* @param bool $deleteEntitiesAndTiles Whether to delete entities and tiles on the old chunk, or transfer them to the new one
*/
public function setChunk(int $chunkX, int $chunkZ, ?Chunk $chunk, bool $deleteEntitiesAndTiles = true) : void{
if($chunk === null){
return;
}
public function setChunk(int $chunkX, int $chunkZ, Chunk $chunk, bool $deleteEntitiesAndTiles = true) : void{
$chunkHash = World::chunkHash($chunkX, $chunkZ);
$oldChunk = $this->loadChunk($chunkX, $chunkZ);
if($oldChunk !== null and $oldChunk !== $chunk){