Sinergi\Users\User\UserController

Complexity

Total Complexity

15

Size/Duplication

Total Lines	129
Duplicated Lines	12.4 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	2
A	createUser()	0	19	2
B	updateUser()	0	27	3
A	updatePassword()	0	13	2
A	deleteUser()	8	8	1
A	banUser()	8	8	1
A	getUserStatusLabel()	0	15	4

<?php

namespace Sinergi\Users\User;

use Interop\Container\ContainerInterface;
use Sinergi\Users\Container;
use Sinergi\Users\User\Exception\InvalidUserException;

class UserController
{
    private $container;

    public function __construct(ContainerInterface $container)
    {
        if ($container instanceof Container) {
            $this->container = $container;
        } else {
            $this->container = new Container($container);
        }
    }

    public function createUser(UserEntityInterface $user): UserEntityInterface
    {
        /** @var UserValidatorInterface $userValidator */
        $userValidator = $this->container->get(UserValidatorInterface::class);
        $errors = $userValidator($user);

        if (!empty($errors)) {
            throw new InvalidUserException($errors);
        }

        /** @var UserRepositoryInterface $userRepository */
        $userRepository = $this->container->get(UserRepositoryInterface::class);
        $userRepository->save($user);

        $this->getEmailConfirmationController()

The method getEmailConfirmationController() does not seem to exist on object<Sinergi\Users\User\UserController>.

            ->sendConfirmationEmail($user);

        return $user;
    }

    /**
     * @param UserEntity $userEntity
     * @param array      $parameters
     */
    public function updateUser(UserEntity $userEntity, array $parameters)
    {
        $options = [UserValidator::OPTION_SKIP_PASSWORD_VALIDATION];

        //todo : refactored this :

        $parameters = array_merge($userEntity->toArray(), $parameters);

        if ($userEntity->getEmail() === $parameters['email']) {
            $options[] = UserValidator::OPTION_SKIP_EMAIL_UNIQUENESS;
        }

        $this->getUserValidator()->validateParameters($parameters, $options);

The method getUserValidator() does not seem to exist on object<Sinergi\Users\User\UserController>.

        if ($userEntity->getEmail() !== $parameters['email']) {
            $userEntity->setPendingEmail($parameters['email']);

            (new EmailConfirmationController($this->getContainer()))->emailUpdated($userEntity);

The method getContainer() does not seem to exist on object<Sinergi\Users\User\UserController>.

        }

        $userEntity
            ->setName($parameters['name'])
            ->setPhone($parameters['phone']);

        $this->getEntityManager()->persist($userEntity);

The method getEntityManager() does not seem to exist on object<Sinergi\Users\User\UserController>.

        $this->getEntityManager()->flush($userEntity);

The method getEntityManager() does not seem to exist on object<Sinergi\Users\User\UserController>.

    }

    /**
     * @param userEntity $userEntity
     * @param array      $parameters
     * @throws AuthenticationException
     */
    public function updatePassword(userEntity $userEntity, array $parameters)
    {
        if (!$userEntity->testPassword($parameters['current-password'])) {
            throw new AuthenticationException($this->getDictionary()->get('user.password.error.wrong_password'));

The method getDictionary() does not seem to exist on object<Sinergi\Users\User\UserController>.

        }

        $this->getUserValidator()->validateParameters($parameters,

The method getUserValidator() does not seem to exist on object<Sinergi\Users\User\UserController>.

            [UserValidator::OPTION_JUST_PASSWORD]);

        $userEntity->setPassword($parameters['password']);
        $this->getEntityManager()->persist($userEntity);

The method getEntityManager() does not seem to exist on object<Sinergi\Users\User\UserController>.

        $this->getEntityManager()->flush($userEntity);

The method getEntityManager() does not seem to exist on object<Sinergi\Users\User\UserController>.

    }

    /**
     * @param userEntity $userEntity
     */
    public function deleteUser(userEntity $userEntity)

This method seems to be duplicated in your project.

    {
        $userEntity->setStatus(UserEntity::STATUS_DELETED);
        $userEntity->setDeletedEmail($userEntity->getEmail());
        $userEntity->setEmail(null);
        $this->getContainer()->getEntityManager()->persist($userEntity);

The method getContainer() does not seem to exist on object<Sinergi\Users\User\UserController>.

        $this->getContainer()->getEntityManager()->flush($userEntity);

The method getContainer() does not seem to exist on object<Sinergi\Users\User\UserController>.

    }

    /**
     * @param userEntity $userEntity
     */
    public function banUser(userEntity $userEntity)

This method seems to be duplicated in your project.

    {
        $userEntity->setStatus(UserEntity::STATUS_BANNED);
        $userEntity->setDeletedEmail($userEntity->getEmail());
        $userEntity->setEmail(null);
        $this->getContainer()->getEntityManager()->persist($userEntity);

The method getContainer() does not seem to exist on object<Sinergi\Users\User\UserController>.

        $this->getContainer()->getEntityManager()->flush($userEntity);

The method getContainer() does not seem to exist on object<Sinergi\Users\User\UserController>.

    }

    /**
     * @param $status
     *
     * @return mixed
     */
    public function getUserStatusLabel($status)
    {
        switch ($status) {
            case UserEntity::STATUS_ACTIVE:
                return $this->getDictionary()->get('user.status.active');

The method getDictionary() does not seem to exist on object<Sinergi\Users\User\UserController>.

            case UserEntity::STATUS_DELETED:
                return $this->getDictionary()->get('user.status.deleted');

The method getDictionary() does not seem to exist on object<Sinergi\Users\User\UserController>.

                break;

break is not strictly necessary here and could be removed.

            case UserEntity::STATUS_BANNED:
                return $this->getDictionary()->get('user.status.banned');

The method getDictionary() does not seem to exist on object<Sinergi\Users\User\UserController>.

                break;

break is not strictly necessary here and could be removed.

            default:
                return $this->getDictionary()->get('user.status.inactive');

The method getDictionary() does not seem to exist on object<Sinergi\Users\User\UserController>.

        }
    }
}