bytic / hello

src/Models/Clients/Client.php

<?php

namespace ByTIC\Hello\Models\Clients;

use ByTIC\Hello\Utility\RandomHelper;
use League\OAuth2\Server\Entities\ClientEntityInterface;
use League\OAuth2\Server\Entities\Traits\ClientTrait;
use League\OAuth2\Server\Entities\Traits\EntityTrait;

/**
 * Class Token
 * @package ByTIC\Hello\Models\Clients
 */
class Client extends \Nip\Records\Record implements ClientEntityInterface
{
    use ClientTrait;
    use EntityTrait;
    use Traits\ClientHasGrantsTrait;

The trait ByTIC\Hello\Models\Clients\Traits\ClientHasGrantsTrait requires the property $grant_types which is not provided by ByTIC\Hello\Models\Clients\Client.

    use Traits\ClientHasSecretTrait;
    use Traits\ClientHasRedirectTrait;

    public function __construct()
    {
        $this->setIdentifier(RandomHelper::generateIdentifier());
        $this->setSecret(RandomHelper::generateToken());
    }

    /**
     * @inheritDoc
     */
    public function writeData($data = false)
    {
        parent::writeData($data);

        if (isset($data['grant_types']) && !empty($data['grant_types'])) {
            $this->initGrantsFromDb($data['grant_types']);
        }

        if (isset($data['identifier']) && !empty($data['identifier'])) {
            $this->initIdentifierFromDb($data['identifier']);
        }
    }

    /**
     * @inheritDoc
     */
    public function setIdentifier($identifier)
    {
        $this->setDataValue('identifier', $identifier);

The function ByTIC\DataObjects\AbstractDto::setDataValue() has been deprecated: use

        $this->identifier = $identifier;
    }

    /**
     * @param null $data

Are you sure the doc-type for parameter $data is correct as it would always require null to be passed?

     */
    public function initIdentifierFromDb($data = null)
    {
        $data = $data === null ? $this->_data['identifier'] : $data;

The property _data does not exist on ByTIC\Hello\Models\Clients\Client. Since you implemented __get, consider adding a @property annotation.

The condition $data === null is always true.

        if (!empty($data)) {
            $this->setIdentifier($data);
        }
    }

    /**
     * @param mixed $name
     */
    public function setName($name)
    {
        $this->name = $name;
        parent::setDataValue('name', $name);

The function ByTIC\DataObjects\Legacy\Behaviors\OldAccessingPatternsTrait::setDataValue() has been deprecated: use

    }
}