AuthorizationCodeFlow::handleAuthorizationRequest() - Code Metrics - Inspection of "Merge remote-tracking branch 'origin/master'" - php-guard/oauth-server - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 64cd9f...411f65 )

by Alexandre

created 2018-03-07 22:38 UTC

handleAuthorizationRequest() A

↳ Parent: AuthorizationCodeFlow

Complexity

Conditions	1
Paths	1

Size

Total Lines	10
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	7
nc	1
nop	2
dl	0
loc	10
rs	9.4285
c	0
b	0
f	0

<?php
/**
 * Created by PhpStorm.
 * User: Alexandre
 * Date: 18/02/2018
 * Time: 18:08
 */

namespace OAuth2\Flows;


use OAuth2\Endpoints\AuthorizationEndpoint;
use OAuth2\Storages\AuthorizationCodeStorageInterface;

class AuthorizationCodeFlow implements FlowInterface
{
    /**
     * @var AuthorizationCodeStorageInterface
     */
    private $authorizationCodeStorage;

    public function __construct(AuthorizationCodeStorageInterface $authorizationCodeStorage)
    {
        $this->authorizationCodeStorage = $authorizationCodeStorage;
    }

    function getResponseTypes(): array

    {
        return ['code'];
    }

    function handleAuthorizationRequest(AuthorizationEndpoint $authorizationEndpoint, array $requestData): array

    {
        $authorizationCode = $this->authorizationCodeStorage->generate(
            implode(' ', $authorizationEndpoint->getScopes()),
            $authorizationEndpoint->getClient()->getIdentifier(),
            $authorizationEndpoint->getResourceOwner()->getIdentifier(),
            $requestData['scope'] ?? null,
            $requestData['redirect_uri'] ?? null
        );
        return ['code' => $authorizationCode->getCode()];
    }

    function getDefaultResponseMode(): string

    {
        return 'query';
    }

    function getUnsupportedResponseModes(): array

    {
        return [];
    }
}

1			<?php
2			/**
3			* Created by PhpStorm.
4			* User: Alexandre
5			* Date: 18/02/2018
6			* Time: 18:08
7			*/
8
9			namespace OAuth2\Flows;
10
11
12			use OAuth2\Endpoints\AuthorizationEndpoint;
13			use OAuth2\Storages\AuthorizationCodeStorageInterface;
14
15			class AuthorizationCodeFlow implements FlowInterface
16			{
17			/**
18			* @var AuthorizationCodeStorageInterface
19			*/
20			private $authorizationCodeStorage;
21
22			public function __construct(AuthorizationCodeStorageInterface $authorizationCodeStorage)
23			{
24			$this->authorizationCodeStorage = $authorizationCodeStorage;
25			}
26
27			function getResponseTypes(): array
			0 ignored issues – show Best Practice introduced 2018-03-04 18:03 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
28			{
29			return ['code'];
30			}
31
32			function handleAuthorizationRequest(AuthorizationEndpoint $authorizationEndpoint, array $requestData): array
			0 ignored issues – show Best Practice introduced 2018-03-07 22:41 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
33			{
34			$authorizationCode = $this->authorizationCodeStorage->generate(
35			implode(' ', $authorizationEndpoint->getScopes()),
36			$authorizationEndpoint->getClient()->getIdentifier(),
37			$authorizationEndpoint->getResourceOwner()->getIdentifier(),
38			$requestData['scope'] ?? null,
39			$requestData['redirect_uri'] ?? null
40			);
41			return ['code' => $authorizationCode->getCode()];
42			}
43
44			function getDefaultResponseMode(): string
			0 ignored issues – show Best Practice introduced 2018-03-04 18:03 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
45			{
46			return 'query';
47			}
48
49			function getUnsupportedResponseModes(): array
			0 ignored issues – show Best Practice introduced 2018-03-04 18:03 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
50			{
51			return [];
52			}
53			}

php-guard / oauth-server

Push — master ( 64cd9f...411f65 )

handleAuthorizationRequest() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like