Flipbox\OAuth2\Client\Provider\AbstractGuardian

Complexity

Total Complexity

9

Size/Duplication

Total Lines	86
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getScopeSeparator()	0	4	1
A	getAuthorizationParameters()	0	9	1
A	getDefaultHeaders()	0	6	1
A	getAccessTokenBody()	0	4	1
A	getAccessTokenOptions()	0	13	1
A	checkResponse()	0	8	3
A	createResourceOwner()	0	4	1

<?php

namespace Flipbox\OAuth2\Client\Provider;

use Flipbox\OAuth2\Client\Provider\Exception\GuardianIdentityProviderException;
use League\OAuth2\Client\Provider\AbstractProvider;
use League\OAuth2\Client\Token\AccessToken;
use League\OAuth2\Client\Tool\BearerAuthorizationTrait;
use Psr\Http\Message\ResponseInterface;

abstract class AbstractGuardian extends AbstractProvider
{
    use BearerAuthorizationTrait;

    /**
     * @inheritdoc
     */
    protected function getScopeSeparator()
    {
        return ' ';
    }

    /**
     * @inheritdoc
     */
    protected function getAuthorizationParameters(array $options)
    {
        return array_merge(
            parent::getAuthorizationParameters($options),
            [
                'client_secret' => $this->clientSecret
            ]
        );
    }

    /**
     * @inheritdoc
     */
    protected function getDefaultHeaders()
    {
        return [
            'accept' => 'application/json'
        ];
    }

    /**
     * Returns the request body for requesting an access token.
     *
     * @param  array $params
     * @return string
     */
    protected function getAccessTokenBody(array $params)
    {
        return json_encode(array_filter($params));
    }

    /**
     * Builds request options used for requesting an access token.
     *
     * @param  array $params
     * @return array
     */
    protected function getAccessTokenOptions(array $params)
    {
        $option = array_merge(
            parent::getAccessTokenOptions($params),
            [
                'headers' => [
                    'content-type' => 'application/json'
                ]
            ]
        );

        return $option;
    }

    /**
     * @inheritdoc
     */
    protected function checkResponse(ResponseInterface $response, $data)
    {
        if ($response->getStatusCode() >= 400) {
            throw GuardianIdentityProviderException::clientException($response, $data);

It seems like $data defined by parameter $data on line 80 can also be of type string; however, Flipbox\OAuth2\Client\Provider\Exception\GuardianIdentityProviderException::clientException() does only seem to accept array, maybe add an additional type check?

        } elseif (isset($data['error'])) {
            throw GuardianIdentityProviderException::oauthException($response, $data);

It seems like $data defined by parameter $data on line 80 can also be of type string; however, Flipbox\OAuth2\Client\Provider\Exception\GuardianIdentityProviderException::oauthException() does only seem to accept array, maybe add an additional type check?

        }
    }

    /**
     * @inheritdoc
     */
    protected function createResourceOwner(array $response, AccessToken $token)
    {
        return new GuardianResourceOwner($response);
    }
}