ProcessLoginTask: check connected status instead of closed status

connected is what we're looking for here, as opposed to an entity being marked as garbage.
This commit is contained in:
Dylan K. Taylor 2018-08-14 14:39:01 +01:00
parent 0273e2484e
commit ce58294305

View File

@ -219,7 +219,7 @@ class ProcessLoginTask extends AsyncTask{
public function onCompletion(Server $server) : void{
/** @var Player $player */
$player = $this->fetchLocal();
if($player->isClosed()){
if(!$player->isConnected()){
$server->getLogger()->error("Player " . $player->getName() . " was disconnected before their login could be verified");
}elseif($player->setAuthenticationStatus($this->authenticated, $this->error)){
if(!$this->useEncryption){