SumoCoders\FrameworkMultiUserBundle\Security\ObjectUserEmailProvider

Complexity

Total Complexity

7

Size/Duplication

Total Lines	56
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	4	4	1
A	loadUserByUsername()	15	15	3
A	refreshUser()	10	10	2
A	supportsClass()	4	4	1

<?php

namespace SumoCoders\FrameworkMultiUserBundle\Security;

use SumoCoders\FrameworkMultiUserBundle\User\Interfaces\User;
use SumoCoders\FrameworkMultiUserBundle\User\BaseUserRepositoryCollection;
use Symfony\Component\Security\Core\User\UserProviderInterface;
use Symfony\Component\Security\Core\User\UserInterface;
use Symfony\Component\Security\Core\Exception\UnsupportedUserException;
use Symfony\Component\Security\Core\Exception\UsernameNotFoundException;

class ObjectUserEmailProvider implements UserProviderInterface

This class seems to be duplicated in your project.

{
    /** @var BaseUserRepositoryCollection */
    private $userRepositoryCollection;

    public function __construct(BaseUserRepositoryCollection $userRepositoryCollection)
    {
        $this->userRepositoryCollection = $userRepositoryCollection;
    }

    /**
     * @param string $emailAddress
     *
     * @return User
     *
     * @throws UsernameNotFoundException
     */
    public function loadUserByUsername($emailAddress): User
    {
        foreach ($this->userRepositoryCollection->all() as $repository) {
            $user = $repository->findByEmailAddress($emailAddress);

            if ($user instanceof User) {
                return $user;
            }
        }

        // Since we are using the email as username we keep this exception since it is a part of symfony
        throw new UsernameNotFoundException(
            sprintf('Email "%s" does not exist.', $emailAddress)
        );
    }

    /**
     * @param UserInterface $user
     *
     * @return User
     *
     * @throws UnsupportedUserException
     */
    public function refreshUser(UserInterface $user): User
    {
        if (!$this->supportsClass(get_class($user))) {
            throw new UnsupportedUserException(
                sprintf('Instances of "%s" are not supported.', get_class($user))
            );
        }

        return $this->loadUserByUsername($user->getEmail());

It seems like you code against a concrete implementation and not the interface Symfony\Component\Security\Core\User\UserInterface as the method getEmail() does only exist in the following implementations of said interface: SumoCoders\FrameworkMultiUserBundle\Entity\BaseUser.

    }

    public function supportsClass($class): bool
    {
        return $this->userRepositoryCollection->supportsClass($class);
    }
}