App\Command\SetPasswordCommand

Complexity

Total Complexity

7

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	configure()	0	6	1
A	execute()	0	53	5

<?php

namespace App\Command;

use App\Entity\User;
use Doctrine\ORM\EntityManagerInterface;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Style\SymfonyStyle;
use Symfony\Component\Security\Core\Encoder\UserPasswordEncoderInterface;

class SetPasswordCommand extends Command
{
    protected static $defaultName = 'app:set-password';

    protected $entityManager;
    protected $encoder;

    public function __construct(EntityManagerInterface $entityManager, UserPasswordEncoderInterface $passwordEncoder)
    {
        $this->entityManager = $entityManager;
        $this->encoder = $passwordEncoder;


        parent::__construct();
    }


    protected function configure()
    {
        $this
            ->setDescription('Sets the password of a user')
            ->setHelp('This password allows you to set the password of a user, without knowing the old password.')
            ->addArgument('user', InputArgument::REQUIRED, 'The name of the user')
        ;
    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $io = new SymfonyStyle($input, $output);
        $user_name = $input->getArgument('user');

        /**
         * @var User $user
         */
        $users = $this->entityManager->getRepository(User::class)->findBy(['name' => $user_name]);
        $user = $users[0];


        if($user == null)
        {
            $io->error(sprintf('No user with the given username %s found in the database!', $user_name));

It seems like $user_name can also be of type string[]; however, parameter $args of sprintf() does only seem to accept string, maybe add an additional type check?

            return;
        }

        $io->note('User found!');

        $proceed = $io->confirm(
            sprintf('You are going to change the password of %s with ID %d. Proceed?',
                $user->getFullName(true), $user->getID()));

        if(!$proceed)
        {
            return;
        }

        $success = false;
        $new_password = "";

        while(!$success) {
            $pw1 = $io->askHidden("Please enter new password:");
            $pw2 = $io->askHidden('Please confirm:');
            if($pw1 !== $pw2) {
                $io->error('The entered password did not match! Please try again.');
            } else {
                //Exit loop
                $success = true;
                $new_password = $pw1;
            }
        }

        //Encode password
        $hash = $this->encoder->encodePassword($user, $new_password);
        $user->setPassword($hash);

        //And save it to databae
        $this->entityManager->persist($user);
        $this->entityManager->flush();

        $io->success('Password was set successful! You can now log in using the new password.');
    }
}