Obblm\Core\Command\RevokeCoachCommand

Complexity

Total Complexity

14

Size/Duplication

Total Lines	78
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	1	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
C	execute()	0	55	12
A	configure()	0	4	1
A	__construct()	0	4	1

<?php

namespace Obblm\Core\Command;

use Doctrine\ORM\EntityManagerInterface;
use Obblm\Core\Entity\Coach;
use Obblm\Core\Event\RegisterCoachEvent;
use Obblm\Core\Security\Roles;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Style\SymfonyStyle;

class RevokeCoachCommand extends Command
{
    /** @var string */
    protected static $defaultName = 'obblm:coach:revoke';
    /** @var EntityManagerInterface */
    private $em;
    /** @var SymfonyStyle */
    private $io;
    private $passwordEncoder;

The private property $passwordEncoder is not used, and could be removed.

    private $dispatcher;

The private property $dispatcher is not used, and could be removed.

    public function __construct(EntityManagerInterface $em)
    {
        $this->em = $em;
        parent::__construct();
    }

    protected function configure():void
    {
        $this->setDescription('Remoke the administrator role to an existing user.')
            ->setHelp('This command will revoke the administrator role of an existing user.');
    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $this->io = new SymfonyStyle($input, $output);

        $this->io->title('Revoke an OBBLM Administrator');
        $this->io->caution('Be carefull, this new user will not have anymore the highest right on the application');
        $continue = $this->io->confirm('Are you sure you want to continue ?', true);
        if($continue) {
            $coachRepository = $this->em->getRepository(Coach::class);

            $choice = $this->io->choice('Search him by login or email ', ['login', 'email']);

            if($choice !== 'login' && $choice !== 'email') {
                throw new \RuntimeException("Something went wrong.");
                return 0;

return 0 is not reachable.

            }
            elseif($choice === 'login') {
                $coach = $this->io->ask('User login', null, function ($username) use ($coachRepository) {
                    if (empty($username)) {
                        throw new \RuntimeException('The login cannot be empty.');
                    }
                    $coach = $coachRepository->findOneByUsername($username);
                    if (!$coach) {
                        throw new \RuntimeException("This login doesn't exist.");
                    }
                    return $coach;
                });
            }
            elseif ($choice === 'email') {
                $coach = $this->io->ask('User email', null, function ($email) use ($coachRepository) {
                    if (empty($email)) {
                        throw new \RuntimeException('The email cannot be empty.');
                    }
                    $coach = $coachRepository->findOneByEmail($email);
                    if (!$coach) {
                        throw new \RuntimeException("This email doesn't exist.");
                    }
                    return $coach;
                });
            }
            /** @var Coach $coach */
            $continue = $this->io->confirm("Are you sure you want to revoke {$coach->getUsername()} ?", true);

The variable $coach does not seem to be defined for all execution paths leading up to this point.

            if($continue) {
                if(in_array(Roles::ADMIN, $coach->getRoles())) {
                    $coach
                        ->setRoles([Roles::COACH]);
                    $this->em->persist($coach);
                    $this->em->flush();
                }
                $this->io->success("The coach {$coach->getUsername()} has been revoked !");
                return 1;
            }
        }
        $this->io->text('Aborted.');
        return 0;
    }
}