Surfnet\StepupSelfService\SelfServiceBundle\Security\Authentication\EventSubscriber\CustomLogoutListener::onLogout()

Complexity

Conditions	1
Paths	1

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    #[AsEventListener(event: LogoutEvent::class, priority: 127)]
    public function onLogout(LogoutEvent $event): void

Missing doc comment for function onLogout()

    {
        $identity = $this->security->getUser()->getIdentity();

The method getIdentity() does not exist on Symfony\Component\Security\Core\User\UserInterface. It seems like you code against a sub-type of Symfony\Component\Security\Core\User\UserInterface such as Surfnet\StepupSelfService\SelfServiceBundle\Security\Authentication\AuthenticatedIdentity.

        $logoutRedirectUrl = $this->logoutRedirectUrl[$identity->preferredLocale];

        $response = $event->getResponse();

The assignment to $response is dead and can be removed.

        $response = new RedirectResponse($logoutRedirectUrl);

        $event->setResponse($response);
    }