Clients::getClientEntity() - Code Metrics - Inspection of "update auth2-server to 8.0" - bytic/hello - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 3991b6...e9cdac )

by Gabriel

created 2020-08-20 15:20 UTC

Clients::getClientEntity() A

↳ Parent: Clients

Complexity

Conditions	5
Paths	3

Size

Total Lines	17
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	30

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
eloc	6
c	1
b	0
f	1
dl	0
loc	17
rs	9.6111
ccs	0
cts	7
cp	0
cc	5
nc	3
nop	4
crap	30

<?php

namespace ByTIC\Hello\Models\Clients;

use League\OAuth2\Server\Repositories\ClientRepositoryInterface;
use Nip\Records\Collections\Collection;
use Nip\Utility\Traits\SingletonTrait;

/**
 * Class Tokens
 * @package ByTIC\Hello\Models\Clients
 *
 * @method Client getNew()
 * @method Client findOneByIdentifier(string $clientIdentifier)
 * @method Client findOne(string $clientIdentifier)
 *
 * @method Client[]|Collection findByRedirect(string $uri)
 */
class Clients extends \Nip\Records\RecordManager implements ClientRepositoryInterface
{
    use SingletonTrait;

    /**
     * {@inheritdoc}
     */
    public function getClientEntity(
        $clientIdentifier,
        $grantType = null,
        $clientSecret = null,
        $mustValidateSecret = true
    ) {
        // First, we will verify that the client exists
        $client = $this->findOneByIdentifier($clientIdentifier);

        if (!$client || !$client->handlesGrant($grantType)) {

            return;
        }

        if ($mustValidateSecret && !$client->validateSecret($clientSecret)) {
            return;
        }
        return $client;
    }

    /** @noinspection PhpMissingParentCallCommonInspection
     * @inheritDoc
     */
    protected function generateTable()
    {
        return 'oauth_clients';
    }

    /**
     * @inheritDoc
     * @TODO add some validation logic
     */
    public function validateClient($clientIdentifier, $clientSecret, $grantType)
    {
        return true;
    }
}


1			<?php
2
3			namespace ByTIC\Hello\Models\Clients;
4
5			use League\OAuth2\Server\Repositories\ClientRepositoryInterface;
6			use Nip\Records\Collections\Collection;
7			use Nip\Utility\Traits\SingletonTrait;
8
9			/**
10			* Class Tokens
11			* @package ByTIC\Hello\Models\Clients
12			*
13			* @method Client getNew()
14			* @method Client findOneByIdentifier(string $clientIdentifier)
15			* @method Client findOne(string $clientIdentifier)
16			*
17			* @method Client[]\|Collection findByRedirect(string $uri)
18			*/
19			class Clients extends \Nip\Records\RecordManager implements ClientRepositoryInterface
20			{
21			use SingletonTrait;
22
23			/**
24			* {@inheritdoc}
25			*/
26			public function getClientEntity(
27			$clientIdentifier,
28			$grantType = null,
29			$clientSecret = null,
30			$mustValidateSecret = true
31			) {
32			// First, we will verify that the client exists
33			$client = $this->findOneByIdentifier($clientIdentifier);
34
35			if (!$client \|\| !$client->handlesGrant($grantType)) {
			0 ignored issues – show introduced 2020-06-22 13:49 UTC by Report Bug Copy Issue Report `$client` is of type `ByTIC\Hello\Models\Clients\Client`, thus it always evaluated to `true`. Loading history...
36			return;
37			}
38
39			if ($mustValidateSecret && !$client->validateSecret($clientSecret)) {
40			return;
41			}
42			return $client;
43			}
44
45			/** @noinspection PhpMissingParentCallCommonInspection
46			* @inheritDoc
47			*/
48			protected function generateTable()
49			{
50			return 'oauth_clients';
51			}
52
53			/**
54			* @inheritDoc
55			* @TODO add some validation logic
56			*/
57			public function validateClient($clientIdentifier, $clientSecret, $grantType)
58			{
59			return true;
60			}
61			}
62

bytic / hello

Push — master ( 3991b6...e9cdac )

Clients::getClientEntity() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like