Kunstmaan\AdminBundle\EventListener\ExceptionSubscriber

Complexity

Total Complexity

7

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getSubscribedEvents()	0	6	1
A	__construct()	0	5	1
A	onKernelException()	0	36	5

<?php

namespace Kunstmaan\AdminBundle\EventListener;

use Doctrine\ORM\EntityManagerInterface;
use Kunstmaan\AdminBundle\Entity\Exception;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent;
use Symfony\Component\HttpKernel\Exception\HttpExceptionInterface;
use Symfony\Component\HttpKernel\KernelEvents;

class ExceptionSubscriber implements EventSubscriberInterface
{
    /**
     * @var EntityManagerInterface
     */
    private $em;

    /**
     * @var array
     */
    private $excludes;

    /**
     * @return array

Consider making the return type a bit more specific; maybe use array<*,string>.

     */
    public static function getSubscribedEvents()
    {
        return [
            KernelEvents::EXCEPTION => 'onKernelException',
        ];
    }

    /**
     * ExceptionSubscriber constructor.
     *
     * @param EntityManagerInterface $em
     * @param array                  $excludes
     */
    public function __construct(EntityManagerInterface $em, array $excludes = [])
    {
        $this->em = $em;
        $this->excludes = $excludes;
    }

    /**
     * @param GetResponseForExceptionEvent $event
     */
    public function onKernelException(GetResponseForExceptionEvent $event)
    {
        $request = $event->getRequest();
        $exception = $event->getException();

        if ($exception instanceof HttpExceptionInterface) {
            $uri = $request->getUri();

            if (count($this->excludes) > 0) {
                $excludes = array_filter($this->excludes, function ($pattern) use ($uri) {
                    return preg_match($pattern, $uri);
                });

                if (count($excludes) > 0) {
                    return;
                }
            }

            $hash = md5(
                $exception->getStatusCode().$uri.$request->headers->get('referer')
            );

            if ($model = $this->em->getRepository(Exception::class)->findOneBy(['hash' => $hash])) {
                $model->increaseEvents();
                $model->setResolved(false);
            } else {
                $model = new Exception();
                $model->setCode($exception->getStatusCode());
                $model->setUrl($uri);
                $model->setUrlReferer($request->headers->get('referer'));

It seems like $request->headers->get('referer') targeting Symfony\Component\HttpFoundation\HeaderBag::get() can also be of type array<integer,string> or null; however, Kunstmaan\AdminBundle\Entity\Exception::setUrlReferer() does only seem to accept string, maybe add an additional type check?

                $model->setHash($hash);
            }
            $this->em->persist($model);
            $this->em->flush();
        }
    }
}