Player: Don't use iusername in cases where it's useless

these calls all do strtolower() anyway, so having iusername here just makes it confusing.
This commit is contained in:
Dylan K. Taylor 2018-08-18 13:42:00 +01:00
parent 25890e76e2
commit 39360f127a

View File

@ -393,7 +393,7 @@ class Player extends Human implements CommandSender, ChunkLoader, IPlayer{
}
public function isBanned() : bool{
return $this->server->getNameBans()->isBanned($this->iusername);
return $this->server->getNameBans()->isBanned($this->username);
}
public function setBanned(bool $value){
@ -406,14 +406,14 @@ class Player extends Human implements CommandSender, ChunkLoader, IPlayer{
}
public function isWhitelisted() : bool{
return $this->server->isWhitelisted($this->iusername);
return $this->server->isWhitelisted($this->username);
}
public function setWhitelisted(bool $value){
if($value){
$this->server->addWhitelist($this->iusername);
$this->server->addWhitelist($this->username);
}else{
$this->server->removeWhitelist($this->iusername);
$this->server->removeWhitelist($this->username);
}
}
@ -1931,7 +1931,7 @@ class Player extends Human implements CommandSender, ChunkLoader, IPlayer{
return true;
}
if(!$this->server->isWhitelisted($this->iusername) and $this->kick("Server is white-listed", false)){
if(!$this->server->isWhitelisted($this->username) and $this->kick("Server is white-listed", false)){
return true;
}