Risan\OAuth1\Signature\BaseStringBuilder

Complexity

Total Complexity

14

Size/Duplication

Total Lines	130
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	build()	0	15	1
A	normalizeParameters()	0	20	3
A	buildUriComponent()	0	9	1
B	buildQueryString()	0	17	5
A	__construct()	0	3	1
A	buildMethodComponent()	0	3	1
A	buildParametersComponent()	0	5	1
A	getUriParser()	0	3	1

<?php

namespace Risan\OAuth1\Signature;

use Risan\OAuth1\Request\UriParserInterface;

class BaseStringBuilder implements BaseStringBuilderInterface
{
    /**
     * The UriParserInterface instance.
     *
     * @var \Risan\OAuth1\Request\UriParserInterface
     */
    protected $uriParser;

    public function __construct(UriParserInterface $uriParser)
    {
        $this->uriParser = $uriParser;
    }

    /**
     * {@inheritdoc}
     */
    public function getUriParser()
    {
        return $this->uriParser;
    }

    /**
     * {@inheritdoc}
     */
    public function build($httpMethod, $uri, array $parameters = [])
    {
        $uri = $this->uriParser->toPsrUri($uri);

        $components = [];

        $components[] = rawurlencode($this->buildMethodComponent($httpMethod));

        $components[] = rawurlencode($this->buildUriComponent($uri));

        parse_str($uri->getQuery(), $queryParameters);

        $components[] = rawurlencode($this->buildParametersComponent(array_merge($queryParameters, $parameters)));

        return implode('&', $components);
    }

    /**
     * {@inheritdoc}
     */
    public function buildMethodComponent($httpMethod)
    {
        return strtoupper($httpMethod);
    }

    /**
     * {@inheritdoc}
     */
    public function buildUriComponent($uri)
    {
        $uri = $this->uriParser->toPsrUri($uri);

        return $this->uriParser->buildFromParts([
            'scheme' => $uri->getScheme(),
            'host' => $uri->getHost(),
            'port' => $uri->getPort(),
            'path' => $uri->getPath(),
        ]);
    }

    /**
     * {@inheritdoc}
     */
    public function buildParametersComponent(array $parameters)
    {
        $parameters = $this->normalizeParameters($parameters);

        return $this->buildQueryString($parameters);
    }

    /**
     * Normalize the given request parameters.
     *
     * @param array $parameters
     *
     * @return array
     */
    public function normalizeParameters(array $parameters)
    {
        $normalized = [];

        // [1] Encode both the keys and values.
        // Decode it frist, in case the given data is already encoded.
        foreach ($parameters as $key => $value) {
            $key = rawurlencode(rawurldecode($key));

            if (is_array($value)) {
                $normalized[$key] = $this->normalizeParameters($value);
            } else {
                $normalized[$key] = rawurlencode(rawurldecode($value));
            }
        }

        // [2] Sort by the encoded key.
        ksort($normalized);

        return $normalized;
    }

    /**
     * Build query string from the given parameters.
     *
     * @param array  $parameters
     * @param array  $initialQueryParameters
     * @param string $previousKey
     *
     * @return string
     */
    public function buildQueryString(array $parameters, array $initialQueryParameters = [], $previousKey = null)
    {
        $queryParameters = $initialQueryParameters;

        foreach ($parameters as $key => $value) {
            if (null !== $previousKey) {
                $key = "{$previousKey}[{$key}]";
            }

            if (is_array($value)) {
                $queryParameters = $this->buildQueryString($value, $queryParameters, $key);
            } else {
                $queryParameters[] = "{$key}={$value}";
            }
        }

        return null !== $previousKey ? $queryParameters : implode('&', $queryParameters);

It seems like $queryParameters can also be of type string; however, parameter $pieces of implode() does only seem to accept array, maybe add an additional type check?

    }
}