JincorTech\AuthClient\UserTokenVerificationResult

Complexity

Total Complexity

11

Size/Duplication

Total Lines	109
Duplicated Lines	10.09 %

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	setSub()	0	7	2
A	getSub()	0	3	1
A	setDeviceId()	0	7	2
A	setExp()	0	3	1
A	getExp()	0	3	1
A	getScope()	0	3	1
A	getDeviceId()	0	3	1
A	__construct()	10	10	1
A	setScope()	0	3	1

<?php

namespace JincorTech\AuthClient;

use InvalidArgumentException;
use JincorTech\AuthClient\Abstracts\VerificationResult;

/**
 * Class UserTokenVerificationResult.
 */
class UserTokenVerificationResult extends VerificationResult
{
    /**
     * @var string
     */
    private $scope;

    /**
     * @var string
     */
    private $deviceId;

    /**
     * @var string
     */
    private $sub;

    /**
     * @var int
     */
    private $exp;

    /**
     * UserTokenVerificationResult constructor.
     *
     * @param array $params
     */
    public function __construct(array $params)

This method seems to be duplicated in your project.

    {
        parent::__construct($params);

        $this->validateParams($params, ['deviceId', 'sub', 'exp']);

        $this->setDeviceId($params['deviceId']);
        $this->setSub($params['sub']);
        $this->setExp($params['exp']);

It seems like $params['exp'] can also be of type string; however, parameter $exp of JincorTech\AuthClient\UserTokenVerificationResult::setExp() does only seem to accept integer, maybe add an additional type check?

        $this->setScope($params['scope'] ?? '');
    }

    /**
     * @return string
     */
    public function getScope(): string
    {
        return $this->scope;
    }

    /**
     * @return string
     */
    public function getDeviceId(): string
    {
        return $this->deviceId;
    }

    /**
     * @return string
     */
    public function getSub(): string
    {
        return $this->sub;
    }

    /**
     * @return int
     */
    public function getExp(): int
    {
        return $this->exp;
    }

    /**
     * @param string $scope
     */
    private function setScope(string $scope)
    {
        $this->scope = $scope;
    }

    /**
     * @param string $deviceId
     */
    private function setDeviceId(string $deviceId)
    {
        if (empty($deviceId)) {
            throw new InvalidArgumentException('DeviceId value can not be empty');
        }

        $this->deviceId = $deviceId;
    }

    /**
     * @param string $sub
     */
    private function setSub(string $sub)
    {
        if (empty($sub)) {
            throw new InvalidArgumentException('Sub value can not be empty');
        }

        $this->sub = $sub;
    }

    /**
     * @param int $exp
     */
    private function setExp(int $exp)
    {
        $this->exp = $exp;
    }
}