PermissibleBase: removed nonsensical code

it's not possible for this to be null, unless a child class doesn't call the constructor, and anything could break in that case anyway.
This commit is contained in:
Dylan K. Taylor 2018-07-08 13:04:51 +01:00
parent 7ee98ff139
commit 23829952c3

View File

@ -30,7 +30,7 @@ use pocketmine\timings\Timings;
class PermissibleBase implements Permissible{
/** @var ServerOperator */
private $opable = null;
private $opable;
/** @var Permissible */
private $parent = null;
@ -59,24 +59,14 @@ class PermissibleBase implements Permissible{
* @return bool
*/
public function isOp() : bool{
if($this->opable === null){
return false;
}else{
return $this->opable->isOp();
}
return $this->opable->isOp();
}
/**
* @param bool $value
*
* @throws \Exception
*/
public function setOp(bool $value){
if($this->opable === null){
throw new \LogicException("Cannot change op value as no ServerOperator is set");
}else{
$this->opable->setOp($value);
}
$this->opable->setOp($value);
}
/**