ByTIC\Hello\Models\AccessTokens\Token

Complexity

Total Complexity

13

Size/Duplication

Total Lines	104
Duplicated Lines	0 %

Test Coverage

Coverage

75%

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	populateFromClient()	0	4	1
A	setExpiresAt()	0	5	1
A	getClient()	0	10	2
A	setUserId()	0	3	1
A	setUserIdentifier()	0	4	1
A	castExpireDateTime()	0	4	2
A	setIdentifier()	0	4	1
A	insert()	0	4	1
A	update()	0	4	1
A	addScopes()	0	4	2

<?php

namespace ByTIC\Hello\Models\AccessTokens;

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 {
        getClient as getClientTrait;
        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()
    {
        $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($value)
    {
        $this->setDataValue('identifier', $value);
        $this->setIdentifierTrait($value);
    }

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

    /**
     * @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());
    }

    /**
     * @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.

    }
}