App\EventSubscriber\User\UserDeleteAccountSubscriber

Complexity

Total Complexity

4

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getSubscribedEvents()	0	6	1
A	__construct()	0	4	1
A	deleteAccount()	0	3	1
A	removeSession()	0	4	1

<?php

namespace App\EventSubscriber\User;

use App\Event\User\UserDeleteAccountEvent;
use App\Event\User\UserEvent;
use App\FlashMessage\FlashMessageCategory;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\HttpFoundation\Session\Session;
use Symfony\Component\HttpFoundation\Session\SessionInterface;
use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;


class UserDeleteAccountSubscriber extends UserSubscriber implements EventSubscriberInterface
{

    /**
 * @var Session
 */
    private $session;
    /**
     * @var TokenStorageInterface
     */
    private $tokenStorage;

    public function __construct(SessionInterface $session, TokenStorageInterface $tokenStorage)
    {
        $this->session = $session;

$session is of type Symfony\Component\HttpFoundation\Session\SessionInterface, but the property $session was declared to be of type Symfony\Component\HttpFoundation\Session\Session. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->tokenStorage = $tokenStorage;
    }

    public function deleteAccount(UserEvent $event)
    {
        $this->deleteFromDatabase($event);
    }

    public function removeSession()
    {
        $this->tokenStorage->setToken(null);
        $this->session->invalidate();
    }


    /**
     * @return array The event names to listen to
     */
    public static function getSubscribedEvents()
    {
        return [
            UserDeleteAccountEvent::NAME => [
                ['deleteAccount', 50],
                ['removeSession', 40],
            ],
        ];
    }
}