SumoCoders\FrameworkMultiUserBundle\Console\DeleteUserCommand

Complexity

Total Complexity

4

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	configure()	0	12	1
A	execute()	0	17	2

<?php

namespace SumoCoders\FrameworkMultiUserBundle\Console;

use SumoCoders\FrameworkMultiUserBundle\Command\DeleteUserHandler;
use SumoCoders\FrameworkMultiUserBundle\DataTransferObject\Form\BaseUser;
use SumoCoders\FrameworkMultiUserBundle\User\UserRepository;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

final class DeleteUserCommand extends Command
{
    /**
     * @var UserRepository
     */
    private $userRepository;

    /**
     * @var DeleteUserHandler
     */
    private $handler;

    /**
     * DeleteUserCommand constructor.
     *
     * @param UserRepository $userRepository
     * @param DeleteUserHandler $handler
     */
    public function __construct(UserRepository $userRepository, DeleteUserHandler $handler)
    {
        parent::__construct();
        $this->userRepository = $userRepository;
        $this->handler = $handler;
    }

    protected function configure()
    {
        $this
            ->setName('sumocoders:multiuser:delete')
            ->setDescription('Delete a user entity')
            ->addArgument(
                'username',
                InputArgument::REQUIRED,
                'The username of the user'
            )
        ;
    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $username = $input->getArgument('username');
        $user = $this->userRepository->findByUsername($username);

        if (!$user) {
            $output->writeln('<error>'.$username.' doesn\'t exists');

            return;
        }

        $dataTransferObject = BaseUser::fromUser($user);

$user of type object<SumoCoders\FrameworkMultiUserBundle\User\UserInterface> is not a sub-type of object<SumoCoders\FrameworkMultiUserBundle\User\User>. It seems like you assume a concrete implementation of the interface SumoCoders\FrameworkMultiUserBundle\User\UserInterface to be always present.

        $this->handler->handle($dataTransferObject);

        $output->writeln($username . ' has been deleted');
    }
}