LoginCidadao\CoreBundle\Security\OIDC\UserProvider

Complexity

Total Complexity

8

Size/Duplication

Total Lines	113
Duplicated Lines	0 %

Test Coverage

Coverage

12.12%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	supportsClass()	0	7	2
A	createUser()	0	20	1
A	__construct()	0	5	1
A	refreshUser()	0	8	2
A	loadUserByUsername()	0	12	2

<?php
/*
 * This file is part of the login-cidadao project or it's bundles.
 *
 * (c) Guilherme Donato <guilhermednt on github>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace LoginCidadao\CoreBundle\Security\OIDC;

use Doctrine\ORM\EntityManager;
use Symfony\Component\Security\Core\Exception\UsernameNotFoundException;
use Symfony\Component\Security\Core\Exception\UnsupportedUserException;
use Symfony\Component\Security\Core\Encoder\EncoderFactoryInterface;
use Symfony\Component\Security\Core\User\UserProviderInterface;
use Symfony\Component\Security\Core\User\UserInterface;
use OAuth2\ServerBundle\User\OAuth2UserProvider;

class UserProvider extends OAuth2UserProvider implements UserProviderInterface
{
    private $em;
    private $encoderFactory;

    public function __construct(EntityManager $entityManager,

The first parameter of a multi-line function declaration must be on the line after the opening bracket

                                EncoderFactoryInterface $encoderFactory)

Multi-line function declaration not indented correctly; expected 8 spaces but found 32

The closing parenthesis of a multi-line function declaration must be on a new line

    {

The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line

        $this->em             = $entityManager;
        $this->encoderFactory = $encoderFactory;
    }

    /**
     * Loads the user for the given username.
     *
     * This method must throw UsernameNotFoundException if the user is not
     * found.
     *
     * @param string $username The username
     *
     * @return UserInterface
     *
     * @see UsernameNotFoundException
     *
     * @throws UsernameNotFoundException if the user is not found
     *
     */
    public function loadUserByUsername($username)
    {
        $user = $this->em->getRepository('LoginCidadaoCoreBundle:Person')
            ->find($username);

        if (!$user) {
            throw new UsernameNotFoundException(sprintf(
                'Username "%s" not found.', $username
            ));
        }

        return $user;
    }

    /**
     * Refreshes the user for the account interface.
     *
     * It is up to the implementation to decide if the user data should be
     * totally reloaded (e.g. from the database), or if the UserInterface
     * object can just be merged into some internal array of users / identity
     * map.
     * @param UserInterface $user
     *
     * @return UserInterface
     *
     * @throws UnsupportedUserException if the account is not supported
     */
    public function refreshUser(UserInterface $user)
    {
        if (!$user instanceof OAuth2User) {

The type LoginCidadao\CoreBundle\Security\OIDC\OAuth2User was not found. Maybe you did not declare it correctly or list all dependencies?

            throw new UnsupportedUserException(sprintf('Instances of "%s" are not supported.',
                get_class($user)));
        }

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

    /**
     * Whether this provider supports the given user class
     *
     * @param string $class
     *
     * @return Boolean
     */
    public function supportsClass($class)
    {
        if ($class == 'Person') {
            return true;
        }

        return false;
    }

    /**
     * Creates a new user
     *
     * @param string $username
     *
     * @param string $password
     *
     * @param array $roles
     *
     * @param array $scopes
     *
     * @return UserInterface
     */
    public function createUser($username, $password, array $roles = array(),

The first parameter of a multi-line function declaration must be on the line after the opening bracket

Multi-line function declarations must define one parameter per line

                                array $scopes = array())

Multi-line function declaration not indented correctly; expected 8 spaces but found 32

The closing parenthesis of a multi-line function declaration must be on a new line

    {

The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line

        $user = parent::createUser();

The call to OAuth2\ServerBundle\User\OAuth2UserProvider::createUser() has too few arguments starting with username.

        // Generate password
        $salt     = $this->generateSalt();
        $password = $this->encoderFactory->getEncoder($user)
            ->encodePassword($password, $salt);

        $user->setSalt($salt);
        $user->setPassword($password);
        $user->setUsername($username);
        $user->setRoles($roles);
        $user->setScopes($scopes);

        $this->em->persist($user);
        $this->em->flush();

        return $user;
    }
}