Alchemy\RestBundle\EventListener\DecodeJsonBodyRequestListener

Complexity

Total Complexity

9

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes	3
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	2
A	onKernelRequest()	0	11	3
A	decodeBody()	0	17	3
A	getSubscribedEvents()	0	4	1

<?php

namespace Alchemy\RestBundle\EventListener;

use Alchemy\Rest\Request\ContentTypeMatcher;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpKernel\Event\GetResponseEvent;
use Symfony\Component\HttpKernel\KernelEvents;

class DecodeJsonBodyRequestListener implements EventSubscriberInterface
{
    /**
     * @var ContentTypeMatcher
     */
    private $contentTypeMatcher;

    /**
     * @var string[]
     */
    private $contentTypes;

    /**
     * @param ContentTypeMatcher $contentTypeMatcher
     * @param null|string[] $contentTypes
     */
    public function __construct(ContentTypeMatcher $contentTypeMatcher, array $contentTypes = null)
    {
        $this->contentTypeMatcher = $contentTypeMatcher;
        $this->contentTypes = $contentTypes ?: array('application/json');
    }

    public function onKernelRequest(GetResponseEvent $event)
    {
        $request = $event->getRequest();
        $restAttributes = $request->attributes->get('_rest', []);

        if (! isset($restAttributes['decode_request']) || ! $restAttributes['decode_request']) {
            return;
        }

        $this->decodeBody($request);
    }

    /**
     * @param Request $request
     */
    public function decodeBody(Request $request)
    {
        $acceptHeader = $request->headers->get('Content-Type', '*/*');

        if (!$this->contentTypeMatcher->matches($acceptHeader, $this->contentTypes)) {

It seems like $acceptHeader defined by $request->headers->get('Content-Type', '*/*') on line 50 can also be of type array; however, Alchemy\Rest\Request\ContentTypeMatcher::matches() does only seem to accept string, maybe add an additional type check?

            return;
        }

        $jsonBody = $request->getContent(false);
        $decodedBody = json_decode($jsonBody, true);

        if (! is_array($decodedBody)) {
            $decodedBody = array();
        }

        $request->request->replace($decodedBody);
    }

    public static function getSubscribedEvents()
    {
        return array(KernelEvents::REQUEST => array('onKernelRequest', -2048));
    }
}