Webhook\Bundle\EventListener\TerminateEventListener

Complexity

Total Complexity

3

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getSubscribedEvents()	0	6	1
A	handle()	0	4	1

<?php

declare(strict_types=1);

namespace Webhook\Bundle\EventListener;

use Doctrine\ORM\EntityManager;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\HttpKernel\Event\PostResponseEvent;
use Symfony\Component\HttpKernel\KernelEvents;

final class TerminateEventListener implements EventSubscriberInterface
{
    /** @var EntityManager */
    private $entityManager;

    /**
     * TerminateEventListener constructor.
     *
     * @param EntityManager $entityManager
     */
    public function __construct(EntityManager $entityManager)

You have injected the EntityManager via parameter $entityManager. This is generally not recommended as it might get closed and become unusable. Instead, it is recommended to inject the ManagerRegistry and retrieve the EntityManager via getManager() each time you need it.

    {
        $this->entityManager = $entityManager;
    }

    /**
     * @return array
     */
    public static function getSubscribedEvents(): array
    {
        return [
            KernelEvents::TERMINATE => 'handle',
        ];
    }

    /**
     * @param PostResponseEvent $event
     *
     * @throws \Doctrine\Common\Persistence\Mapping\MappingException
     */
    public function handle(PostResponseEvent $event): void

The parameter $event is not used and could be removed.

    {
        $this->entityManager->clear();
    }
}