Majora\Framework\Loader\Bridge\Security\UserProviderLoaderTrait

Complexity

Total Complexity

6

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	loadUserByUsername()	0	16	3
A	refreshUser()	0	11	2
A	supportsClass()	0	4	1

<?php

namespace Majora\Framework\Loader\Bridge\Security;

use Symfony\Component\Security\Core\Exception\UnsupportedUserException;
use Symfony\Component\Security\Core\Exception\UsernameNotFoundException;
use Symfony\Component\Security\Core\User\UserInterface;

/**
 * Trait which provides a bridge between majora loader and symfony security
 * user providers
 *
 * @property string entityClass
 * @property \ReflectionClass entityReflection
 */
trait UserProviderLoaderTrait
{
    /**
     * @see UserProviderInterface::loadUserByUsername()
     */
    public function loadUserByUsername($username)
    {
        $queryParameters = array('email' => $username);
        if ($this->entityReflection->implementsInterface('Majora\Framework\Model\EnablableInterface')) {
            $queryParameters['enabled'] = true;
        }

        if (!$person = $this->retrieveOne($queryParameters)) {

It seems like retrieveOne() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            throw new UsernameNotFoundException(sprintf(
                'User "%s" is not an active person.',
                $username
            ));
        }

        return $person;
    }

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

        return $this->loadUserByUsername($user->getUsername());
    }

    /**
     * @see UserProviderInterface::supportsClass()
     */
    public function supportsClass($class)
    {
        return $class == $this->entityClass;
    }
}