AbstractAuthorizeGrant::makeRedirectUri() - Code Metrics - Inspection of "Deny access if user is not authenticated" - thephpleague/oauth2-server - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#1408)

unknown

created 2024-05-17 21:25 UTC

AbstractAuthorizeGrant::makeRedirectUri() A

↳ Parent: AbstractAuthorizeGrant

Complexity

Conditions	2
Paths	2

Size

Total Lines	5
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	2

Importance

Changes

Metric	Value
cc	2
eloc	2
c	0
b	0
f	0
nc	2
nop	3
dl	0
loc	5
ccs	3
cts	3
cp	1
crap	2
rs	10

<?php

/**
 * Abstract authorization grant.
 *
 * @author      Julián Gutiérrez <[email protected]>
 * @copyright   Copyright (c) Alex Bilbie
 * @license     http://mit-license.org/
 *
 * @link        https://github.com/thephpleague/oauth2-server
 */

declare(strict_types=1);

namespace League\OAuth2\Server\Grant;

use League\OAuth2\Server\RequestTypes\AuthorizationRequest;
use League\OAuth2\Server\RequestTypes\AuthorizationRequestInterface;

use function http_build_query;

abstract class AbstractAuthorizeGrant extends AbstractGrant
{
    /**
     * @param array<array-key,mixed> $params

     */
    public function makeRedirectUri(string $uri, array $params = [], string $queryDelimiter = '?'): string
    {
        $uri .= str_contains($uri, $queryDelimiter) ? '&' : $queryDelimiter;

        return $uri . http_build_query($params);
    }

    protected function createAuthorizationRequest(): AuthorizationRequestInterface
    {
        return new AuthorizationRequest();
    }

    /**
     * Get the client redirect URI.
     */
    protected function getClientRedirectUri(AuthorizationRequestInterface $authorizationRequest): string
    {
        return is_array($authorizationRequest->getClient()->getRedirectUri())
            ? $authorizationRequest->getClient()->getRedirectUri()[0]
            : $authorizationRequest->getClient()->getRedirectUri();
    }
}


1		<?php
2
3		/**
4		* Abstract authorization grant.
5		*
6		* @author Julián Gutiérrez <[email protected]>
7		* @copyright Copyright (c) Alex Bilbie
8		* @license http://mit-license.org/
9		*
10		* @link https://github.com/thephpleague/oauth2-server
11		*/
12
13		declare(strict_types=1);
14
15		namespace League\OAuth2\Server\Grant;
16
17		use League\OAuth2\Server\RequestTypes\AuthorizationRequest;
18		use League\OAuth2\Server\RequestTypes\AuthorizationRequestInterface;
19
20		use function http_build_query;
21
22		abstract class AbstractAuthorizeGrant extends AbstractGrant
23		{
24		/**
25		* @param array<array-key,mixed> $params
		0 ignored issues – show Documentation Bug introduced 2024-05-13 21:07 UTC by Report Bug Copy Issue Report The doc comment `array<array-key,mixed>` at position `2` could not be parsed: Unknown type name 'array-key' at position 2 in array<array-key,mixed>. Loading history...
26		*/
27	8	public function makeRedirectUri(string $uri, array $params = [], string $queryDelimiter = '?'): string
28		{
29	8	$uri .= str_contains($uri, $queryDelimiter) ? '&' : $queryDelimiter;
30
31	8	return $uri . http_build_query($params);
32		}
33
34	9	protected function createAuthorizationRequest(): AuthorizationRequestInterface
35		{
36	9	return new AuthorizationRequest();
37		}
38
39		/**
40		* Get the client redirect URI.
41		*/
42		protected function getClientRedirectUri(AuthorizationRequestInterface $authorizationRequest): string
43		{
44		return is_array($authorizationRequest->getClient()->getRedirectUri())
45		? $authorizationRequest->getClient()->getRedirectUri()[0]
46		: $authorizationRequest->getClient()->getRedirectUri();
47		}
48		}
49

thephpleague / oauth2-server

Pull Request — master (#1408)

AbstractAuthorizeGrant::makeRedirectUri() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like