FSi\Bundle\AdminSecurityBundle\EventListener\SetEmailAsUsernameListener

Complexity

Total Complexity

3

Size/Duplication

Total Lines	17
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getSubscribedEvents()	0	4	1
A	setEmailAsUsername()	0	8	2

<?php

/**
 * (c) FSi sp. z o.o. <info@fsi.pl>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace FSi\Bundle\AdminSecurityBundle\EventListener;

use FSi\Bundle\AdminSecurityBundle\Event\AdminSecurityEvents;
use FSi\Bundle\AdminSecurityBundle\Event\UserEvent;
use FSi\Bundle\AdminSecurityBundle\Security\User\UserInterface;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;

class SetEmailAsUsernameListener implements EventSubscriberInterface
{
    public static function getSubscribedEvents(): array
    {
        return [
            AdminSecurityEvents::USER_CREATED => 'setEmailAsUsername'
        ];
    }

    public function setEmailAsUsername(UserEvent $event): void
    {
        $user = $event->getUser();
        if (!$user instanceof UserInterface) {
            return;
        }

        $user->setUsername($user->getEmail());

It seems like $user->getEmail() can also be of type null; however, parameter $username of FSi\Bundle\AdminSecurityBundle\Security\User\UserInterface::setUsername() does only seem to accept string, maybe add an additional type check?

    }
}