Small de-spaghettification of login handling

no need for these to be split up, it just makes the player login code flow look confusing.
This commit is contained in:
Dylan K. Taylor 2018-02-28 18:40:32 +00:00
parent 533d139385
commit 973d5dc251
2 changed files with 2 additions and 6 deletions

View File

@ -2080,9 +2080,10 @@ class Player extends Human implements CommandSender, ChunkLoader, IPlayer{
$this->armorInventory->sendContents($this);
$this->inventory->sendCreativeContents();
$this->inventory->sendHeldItem($this);
$this->dataPacket($this->server->getCraftingManager()->getCraftingDataPacket());
$this->server->addOnlinePlayer($this);
$this->server->onPlayerCompleteLoginSequence($this);
$this->server->sendFullPlayerListData($this);
}
/**

View File

@ -2253,11 +2253,6 @@ class Server{
$this->loggedInPlayers[$player->getRawUniqueId()] = $player;
}
public function onPlayerCompleteLoginSequence(Player $player){
$this->sendFullPlayerListData($player);
$player->dataPacket($this->craftingManager->getCraftingDataPacket());
}
public function onPlayerLogout(Player $player){
unset($this->loggedInPlayers[$player->getRawUniqueId()]);
}