Del\Console\UserCommand

Complexity

Total Complexity

4

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	configure()	0	16	1
A	execute()	0	15	2

<?php

namespace Del\Console;

use DateTime;
use Del\Common\ContainerService;
use Del\Criteria\UserCriteria;
use Del\Service\UserService;
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;

class UserCommand extends Command
{
    /** @var UserService $userService */
    private $userService;

    /**
     * UserCommand constructor.
     * @throws \Doctrine\ORM\ORMException
     */
    public function __construct(UserService $userService)
    {
        parent::__construct();
        $this->userService = $userService;
    }

    /**
     *
     */
    protected function configure()
    {
        $this
            ->setName('reset-pass')
            ->setDescription('Resets a user\'s password')
            ->addArgument(
                'email',
                InputArgument::REQUIRED,
                'The email of the user'
            )->addArgument(
                'newPassword',
                InputArgument::REQUIRED,
                'The email of the user'
            )
        ;
    }

    /**
     * @param InputInterface $input
     * @param OutputInterface $output
     * @return int|void|null
     * @throws \Doctrine\ORM\OptimisticLockException
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $email = $input->getArgument('email');
        $pass = $input->getArgument('newPassword');
        $criteria = new UserCriteria();
        $criteria->setEmail($email);

It seems like $email defined by $input->getArgument('email') on line 58 can also be of type array<integer,string> or null; however, Del\Criteria\UserCriteria::setEmail() does only seem to accept string, maybe add an additional type check?

        $user = $this->userService->findOneByCriteria($criteria);
        if ($user === null) {
            $output->writeln('No User Found.');
        } else {
            $this->userService->changePassword($user, $pass);

It seems like $pass defined by $input->getArgument('newPassword') on line 59 can also be of type array<integer,string> or null; however, Del\Service\UserService::changePassword() does only seem to accept string, maybe add an additional type check?

            $this->userService->saveUser($user);
            $output->writeln('Password changed for '.$email);
        }
    }
}