Alchemy\RestBundle\EventListener\ExceptionListener

Complexity

Total Complexity

7

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	3
A	onKernelException()	0	15	3
A	getSubscribedEvents()	0	4	1

<?php

namespace Alchemy\RestBundle\EventListener;

use Alchemy\Rest\Request\ContentTypeMatcher;
use Alchemy\Rest\Response\ExceptionTransformer;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\HttpFoundation\JsonResponse;
use Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent;
use Symfony\Component\HttpKernel\Exception\HttpExceptionInterface;
use Symfony\Component\HttpKernel\KernelEvents;

class ExceptionListener implements EventSubscriberInterface
{
    /**
     * @var array
     */
    private $handledContentTypes;

    /**
     * @var ContentTypeMatcher
     */
    private $contentTypeMatcher;

    /**
     * @var ExceptionTransformer
     */
    private $exceptionTransformer;

    /**
     * @param ContentTypeMatcher $matcher
     * @param ExceptionTransformer $exceptionTransformer
     * @param array $handledContentTypes
     */
    public function __construct(
        ContentTypeMatcher $matcher,
        ExceptionTransformer $exceptionTransformer = null,
        array $handledContentTypes = null
    ) {
        $this->contentTypeMatcher = $matcher;
        $this->exceptionTransformer = $exceptionTransformer ?: new ExceptionTransformer\DefaultExceptionTransformer();
        $this->handledContentTypes = $handledContentTypes ?: array('application/json');
    }

    /**
     * @param GetResponseForExceptionEvent $event
     */
    public function onKernelException(GetResponseForExceptionEvent $event)
    {
        $request = $event->getRequest();
        $acceptHeader = $request->headers->get('Accept', '*/*');

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

It seems like $acceptHeader defined by $request->headers->get('Accept', '*/*') on line 51 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;
        }

        $exception = $event->getException();
        $data = $this->exceptionTransformer->transformException($exception);
        $status = $exception instanceof HttpExceptionInterface ? $exception->getStatusCode() : 500;

        $event->setResponse(new JsonResponse($data, $status));
    }

    public static function getSubscribedEvents()
    {
        return array(KernelEvents::EXCEPTION => 'onKernelException');
    }
}