diff --git a/src/pocketmine/Player.php b/src/pocketmine/Player.php index b4886852e..43aec00cb 100644 --- a/src/pocketmine/Player.php +++ b/src/pocketmine/Player.php @@ -1722,6 +1722,20 @@ class Player extends Human implements CommandSender, InventoryHolder, ChunkLoade return true; } + public function doFoodTick(int $tickDiff = 1){ + if($this->isSurvival()){ + parent::doFoodTick($tickDiff); + } + } + + public function exhaust(float $amount, int $cause = PlayerExhaustEvent::CAUSE_CUSTOM) : float{ + if($this->isSurvival()){ + return parent::exhaust($amount, $cause); + } + + return 0.0; + } + public function checkNetwork(){ if(!$this->isOnline()){ return; @@ -4108,7 +4122,9 @@ class Player extends Human implements CommandSender, InventoryHolder, ChunkLoade } public function onChunkChanged(Chunk $chunk){ - unset($this->usedChunks[Level::chunkHash($chunk->getX(), $chunk->getZ())]); + if(isset($this->usedChunks[$hash = Level::chunkHash($chunk->getX(), $chunk->getZ())])){ + $this->usedChunks[$hash] = false; + } } public function onChunkLoaded(Chunk $chunk){ diff --git a/src/pocketmine/entity/Human.php b/src/pocketmine/entity/Human.php index 5d8db656c..9797ead87 100644 --- a/src/pocketmine/entity/Human.php +++ b/src/pocketmine/entity/Human.php @@ -27,6 +27,7 @@ use pocketmine\event\player\PlayerExhaustEvent; use pocketmine\inventory\InventoryHolder; use pocketmine\inventory\PlayerInventory; use pocketmine\item\Item as ItemItem; +use pocketmine\level\Level; use pocketmine\nbt\NBT; use pocketmine\nbt\tag\ByteTag; use pocketmine\nbt\tag\CompoundTag; @@ -61,13 +62,21 @@ class Human extends Creature implements ProjectileSource, InventoryHolder{ public $eyeHeight = 1.62; protected $skinId; - protected $skin; + protected $skin = null; protected $foodTickTimer = 0; protected $totalXp = 0; protected $xpSeed; + public function __construct(Level $level, CompoundTag $nbt){ + if($this->skin === null and (!isset($nbt->Skin) or !isset($nbt->Skin->Data) or !Player::isValidSkin($nbt->Skin->Data->getValue()))){ + throw new \InvalidStateException((new \ReflectionClass($this))->getShortName() . " must have a valid skin set"); + } + + parent::__construct($level, $nbt); + } + public function getSkinData(){ return $this->skin; } @@ -95,6 +104,10 @@ class Human extends Creature implements ProjectileSource, InventoryHolder{ * @param string $skinId */ public function setSkin($str, $skinId){ + if(!Player::isValidSkin($str)){ + throw new \InvalidStateException("Specified skin is not valid, must be 8KiB or 16KiB"); + } + $this->skin = $str; $this->skinId = $skinId; } @@ -361,12 +374,18 @@ class Human extends Creature implements ProjectileSource, InventoryHolder{ public function entityBaseTick($tickDiff = 1){ $hasUpdate = parent::entityBaseTick($tickDiff); + $this->doFoodTick($tickDiff); + + return $hasUpdate; + } + + public function doFoodTick(int $tickDiff = 1){ if($this->isAlive()){ $food = $this->getFood(); $health = $this->getHealth(); $difficulty = $this->server->getDifficulty(); - $this->foodTickTimer++; + $this->foodTickTimer += $tickDiff; if($this->foodTickTimer >= 80){ $this->foodTickTimer = 0; } @@ -399,8 +418,6 @@ class Human extends Creature implements ProjectileSource, InventoryHolder{ } } } - - return $hasUpdate; } public function getName(){ @@ -483,7 +500,7 @@ class Human extends Creature implements ProjectileSource, InventoryHolder{ if($player !== $this and !isset($this->hasSpawned[$player->getLoaderId()])){ $this->hasSpawned[$player->getLoaderId()] = $player; - if(strlen($this->skin) < 64 * 32 * 4){ + if(!Player::isValidSkin($this->skin)){ throw new \InvalidStateException((new \ReflectionClass($this))->getShortName() . " must have a valid skin set"); } diff --git a/src/pocketmine/level/format/Chunk.php b/src/pocketmine/level/format/Chunk.php index e6070eb87..7ff262d56 100644 --- a/src/pocketmine/level/format/Chunk.php +++ b/src/pocketmine/level/format/Chunk.php @@ -709,9 +709,16 @@ class Chunk{ continue; //Fixes entities allocated in wrong chunks. } - if(($entity = Entity::createEntity($nbt["id"], $level, $nbt)) instanceof Entity){ - $entity->spawnToAll(); - }else{ + try{ + $entity = Entity::createEntity($nbt["id"], $level, $nbt); + if($entity instanceof Entity){ + $entity->spawnToAll(); + }else{ + $changed = true; + continue; + } + }catch(\Throwable $t){ + $level->getServer()->getLogger()->logException($t); $changed = true; continue; }