Custom player class

Ok so i have a custom player class for my Core plugin and i am trying to get the players UUID but it throws an error from the Raklib saying toString on null. Is there any other way i can get the players UUID ?

Here is the code:

PHP:

<?php
/**
 * Created by PhpStorm.
 * User: Savion
 * Date: 3/24/2017
 * Time: 6:15 PM
 */

namespace Auth;

use pocketmine\event\Listener;
use pocketmine\network\SourceInterface;
use pocketmine\Player;
use pocketmine\utils\TextFormat;
use pocketmine\utils\UUID;


class CustomPlayer extends Player{

    private $loginTrys = [];

    private $lastUuid;

    public function __construct(SourceInterface $interface, $clientID, $ip, $port)
    {
        parent::__construct($interface, $clientID, $ip, $port);
    }


    public function getPlugin(){
        return Main::getInstance();
    }

    public function isRegistered(){
        if($this->getPlugin()->database->exists(strtolower($this->getName()))){
            return true;
        }else{
            return false;
        }
    }

    public function isLoggedIn(){
        if(isset($this->getPlugin()->loggedIn[$this->getName()])){
            return true;
        }else{
            return false;
        }
    }

    public function sameUuid(){
        //have to fix ;P
        return false;
    }

    public function login($type){
        if($type === Main::UUID){
            $this->getPlugin()->loggedIn[$this->getName()] = $this->getName();
            $this->sendMessage(Main::PREFIX_GOOD.Main::LOGGED_IN_BY_UUID);
            $this->setNameTag($this->getName());
            date_default_timezone_set('UTC');
            if(isset($this->loginTrys[$this->getName()])){
                unset($this->loginTrys[$this->getName()]);
            }
        }

        if($type === Main::PASSWORD){
            $this->getPlugin()->loggedIn[$this->getName()] = $this->getName();
            $this->sendMessage(Main::PREFIX_GOOD.Main::LOGGED_IN_BY_PASSWORD);
            $this->setNameTag($this->getName());
            if(isset($this->loginTrys[$this->getName()])){
                unset($this->loginTrys[$this->getName()]);
            }
        }
        $this->sendMessage(Main::PREFIX_GOOD."Today is ".date("D, F d, Y, H:i T")."!");
        $this->sendTitle(TextFormat::GOLD."Welcome to AzeronPE!", TextFormat::AQUA."Follow us on Twitter: @AzeronPE", 20*3, 20*3);
    }

    public function logout(){
        unset($this->getPlugin()->loggedIn[$this->getName()]);
    }
}

 

In your CustomPlayer class, execute this:

PHP:

$this->getUniqueId();

CustomPlayer extends Player class, getUniqueId() is inherited from parent class, Player.

 

I am wanting to add my own stuff to the Player class without modifying the actually software

 

I highly recommend not using a custom player class unless you know what you are doing. PlayerCreationEvent can only be made use of in one plugin.

Back to the question:

PHP:

Player::getUniqueId()->toString();


 

You may not need to but he is. This thread isn't about discussing wether he should overwrite the player class but was merely a question about retrieving the players UUID.

I understand that it's not recommended for plugin developers to make use of the ability to extend the player classes but the fact remains that the ability is there.

Incorrect. It can be used in as many plugins as you like, you just have to make sure every new player class you use extends an existing one (preferably one that hasn't been extended yet so you don't loose part of the functionality) instead of PocketMines.

 

1. Seriously, the main question was Is there any other way i can get the players UUID? And yes, it is unneeded to override the player class to achieve that.

2. Of course. Anything is possible in code. You'll need to check which plugins get enabled first and sequence PlayerCreationEvent as per it. But chances are, it'll still be a mess.
Quote from PEMapModder
"If more than one plugin does this, it will conflict. Only the plugin with the highest priority wins.

Unless it's a private plugin, avoid using it. PlayerCreationEvent was originally for things that are really impossible to do without using it. For example, to override the sendTranslation function."
So what SOF3 said was correct either ways. It's not needed, and if the sole purpose is obtaining UUID, then it's just useless, unneeded, not recommended.