LunixREST\RequestFactory\HeaderParser\DefaultHeaderParser

Complexity

Total Complexity

15

Size/Duplication

Total Lines	86
Duplicated Lines	25.58 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	parse()	0	8	1
B	findAcceptableMIMETypes()	0	18	5
A	findAPIKey()	11	11	4
A	getContentType()	11	11	4

<?php
namespace LunixREST\RequestFactory\HeaderParser;

use LunixREST\RequestFactory\HeaderParser\Exceptions\UnableToParseHeadersException;

/**
 * Parse out all the common headers from a request.
 * Class DefaultHeaderParser
 * @package LunixREST\RequestFactory\HeaderParser
 */
class DefaultHeaderParser implements HeaderParser
{

    /**
     * @var string
     */
    private $apiKeyHeaderKey;

    /**
     * DefaultHeaderParser constructor.
     * @param string $apiKeyHeaderKey
     */
    public function __construct($apiKeyHeaderKey = 'x-api-key')
    {
        $this->apiKeyHeaderKey = $apiKeyHeaderKey;
    }

    /**
     * @param array $headers
     * @return ParsedHeaders
     * @throws UnableToParseHeadersException
     */
    public function parse(array $headers): ParsedHeaders
    {
        $contentType = $this->getContentType($headers);

Are you sure the assignment to $contentType is correct as $this->getContentType($headers) (which targets LunixREST\RequestFactory\HeaderParser\DefaultHeaderParser::getContentType()) seems to always return null.

        $acceptableMIMETypes = $this->findAcceptableMIMETypes($headers);
        $apiKey = $this->findAPIKey($headers);

Are you sure the assignment to $apiKey is correct as $this->findAPIKey($headers) (which targets LunixREST\RequestFactory\HeaderParser\DefaultHeaderParser::findAPIKey()) seems to always return null.

        return new ParsedHeaders($contentType, $acceptableMIMETypes, $apiKey);
    }

    /**
     * @param array $headers
     * @return array
     */
    protected function findAcceptableMIMETypes(array $headers): array
    {
        //TODO: follow RFC2616 order
        $acceptedMIMETypes = [];

        foreach ($headers as $key => $values) {
            if (strtolower($key) == 'http-accept') {
                foreach ($values as $acceptedType) {
                    $typeParts = explode(';', $acceptedType);
                    if (count($typeParts) > 0) {
                        $acceptedMIMETypes[] = trim($typeParts[0]);
                    }
                }
                break;
            }
        }
        return $acceptedMIMETypes;
    }

    /**
     * @param array $headers
     * @return null
     */
    protected function findAPIKey(array $headers)

This method seems to be duplicated in your project.

    {
        foreach ($headers as $key => $values) {
            if (strtolower($key) == strtolower($this->apiKeyHeaderKey)) {
                foreach($values as $value) {
                    return $value;
                }
            }
        }
        return null;
    }

    /**
     * @param array $headers
     * @return null
     */
    protected function getContentType(array $headers)

This method seems to be duplicated in your project.

    {
        foreach ($headers as $key => $values) {
            if (strtolower($key) == 'content-type') {
                foreach ($values as $value) {
                    return $value;
                }
            }
        }
        return null;
    }
}