AbterPhp\Admin\Oauth2\Repository\Client

Complexity

Total Complexity

6

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getClientEntity()	0	20	3
A	query()	0	20	2
A	__construct()	0	4	1

<?php

declare(strict_types=1);

namespace AbterPhp\Admin\Oauth2\Repository;

use AbterPhp\Admin\Oauth2\Entity\Client as Entity;
use AbterPhp\Framework\Crypto\Crypto;
use League\OAuth2\Server\Entities\ClientEntityInterface;

The type League\OAuth2\Server\Entities\ClientEntityInterface was not found. Maybe you did not declare it correctly or list all dependencies?

use League\OAuth2\Server\Repositories\ClientRepositoryInterface;

The type League\OAuth2\Server\Repositories\ClientRepositoryInterface was not found. Maybe you did not declare it correctly or list all dependencies?

use Opulence\Databases\ConnectionPools\ConnectionPool;
use Opulence\QueryBuilders\MySql\QueryBuilder;

class Client implements ClientRepositoryInterface
{
    /** @var Crypto */
    protected $crypto;

    /** @var ConnectionPool */
    protected $connectionPool;

    /**
     * Client constructor.
     *
     * @param Crypto         $crypto
     * @param ConnectionPool $connectionPool
     */
    public function __construct(Crypto $crypto, ConnectionPool $connectionPool)
    {
        $this->crypto         = $crypto;
        $this->connectionPool = $connectionPool;
    }

    /**
     * @SuppressWarnings(PHPMD.UnusedFormalParameter)
     *
     * @param string $clientIdentifier
     * @param null   $grantType

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

     * @param null   $clientSecret

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

     * @param bool   $mustValidateSecret
     *
     * @return ClientEntityInterface
     */
    public function getClientEntity(
        $clientIdentifier,
        $grantType = null,
        $clientSecret = null,
        $mustValidateSecret = true
    ) {
        $clientData = $this->query($clientIdentifier);

        if (empty($clientData['secret'])) {
            return null;
        }

        $clientSecret = $this->crypto->prepareSecret($clientSecret);

$clientSecret of type null is incompatible with the type string expected by parameter $rawText of AbterPhp\Framework\Crypto\Crypto::prepareSecret().

        if (!$this->crypto->verifySecret($clientSecret, $clientData['secret'])) {
            return null;
        }

        $client = new Entity($clientIdentifier, $clientIdentifier, '');

        return $client;
    }

    /**
     * @param string $clientId
     *
     * @return array|bool
     */
    protected function query(string $clientId)
    {
        // TODO: Implement getClientEntity() method.
        $query = (new QueryBuilder())
            ->select('ac.secret')
            ->from('api_clients', 'ac')
            ->where('ac.deleted = 0')
            ->andWhere('ac.id = :clientId');

        $sql    = $query->getSql();
        $params = ['clientId' => $clientId];

        $connection = $this->connectionPool->getReadConnection();
        $statement  = $connection->prepare($sql);
        $statement->bindValues($params);
        if (!$statement->execute()) {
            return false;
        }

        return $statement->fetch(\PDO::FETCH_ASSOC);
    }
}