bytic / hello

src/Models/AccessTokens/Token.php

<?php

namespace ByTIC\Hello\Models\AccessTokens;

use ByTIC\Hello\Utility\ModelsHelper;
use DateTimeImmutable;
use League\OAuth2\Server\Entities\AccessTokenEntityInterface;
use League\OAuth2\Server\Entities\ClientEntityInterface;
use League\OAuth2\Server\Entities\Traits\AccessTokenTrait;
use League\OAuth2\Server\Entities\Traits\EntityTrait;
use League\OAuth2\Server\Entities\Traits\TokenEntityTrait;

/**
 * Class Token
 * @package ByTIC\Auth\Models\AccessTokens
 *
 * @property int $user_id
 * @property int $client_id
 * @property string $revoked
 * @property string $expires_at
 */
class Token extends \Nip\Records\Record implements AccessTokenEntityInterface
{
    use EntityTrait {
        setIdentifier as setIdentifierTrait;
    }
    use TokenEntityTrait {
        TokenEntityTrait::getClient as getClientTrait;
        TokenEntityTrait::setUserIdentifier as setUserIdentifierTrait;
    }
    use AccessTokenTrait;

    /**
     * @param ClientEntityInterface $clientEntity
     */
    public function populateFromClient(ClientEntityInterface $clientEntity)
    {
        $this->setClient($clientEntity);
        $this->client_id = $clientEntity->getIdentifier();

The property $client_id was declared of type integer, but $clientEntity->getIdentifier() is of type string. Maybe add a type cast?

    }

    /**
     * @inheritDoc
     */
    public function getClient(): ClientEntityInterface
    {
        $client = $this->getClientTrait();
        if ($client instanceof ClientEntityInterface) {

$client is always a sub-type of League\OAuth2\Server\Entities\ClientEntityInterface.

            return $client;
        }
        /** @var ClientEntityInterface $relationClient */
        $relationClient = $this->getRelation('Client')->getResults();
        $this->setClient($relationClient);
        return $relationClient;
    }

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

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

        $this->setIdentifierTrait($identifier);
    }

    /**
     * @inheritDoc
     */
    public function setExpiresAt($value)
    {
        /** @noinspection PhpUnhandledExceptionInspection */
        $date = new DateTimeImmutable($value);
        $this->setExpiryDateTime($date);
        $this->setDataValue('expires_at', $value);

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

    }

    /**
     * @param $identifier
     */
    public function setUserId($identifier)
    {
        $this->setUserIdentifier($identifier);
    }


    /**
     * @param int|string|null $identifier
     */
    public function setUserIdentifier($identifier)
    {
        $this->setUserIdentifierTrait($identifier);
        $this->setDataValue('user_id', $this->getUserIdentifier());

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

    }

    /**
     * @param $scopes
     */
    public function setScopes($scopes)
    {
        if (is_string($scopes)) {
            $this->addScopesFromString($scopes);
            return;
        }
        $this->scopes = $scopes;
    }

    /**
     * @param $scopes
     */
    public function addScopesFromString($scopes)
    {
        $scopes = array_filter(explode(',', $scopes));
        foreach ($scopes as $key => $scope) {
            $scopes[$key] = ModelsHelper::scopes()->getScopeEntityByIdentifier($scope);
        }
        $this->addScopes($scopes);
    }

    /**
     * @param $scopes
     */
    public function addScopes($scopes)
    {
        foreach ($scopes as $scope) {
            $this->addScope($scope);
        }
    }

    /**
     * @inheritDoc
     */
    public function insert()
    {
        $this->castExpireDateTime();
        return parent::insert();
    }

    /**
     * @inheritDoc
     */
    public function update()
    {
        $this->castExpireDateTime();
        return parent::insert();
    }

    protected function castExpireDateTime()
    {
        $date = $this->getExpiryDateTime();
        $this->expires_at = ($date) ? $date->format('Y-m-d') : '';

$date is of type DateTimeImmutable, thus it always evaluated to true.

    }
}