App\Command\UserAddCommand::execute()

Complexity

Conditions	7
Paths	15

Size

Total Lines	54
Code Lines	30

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    protected function execute(InputInterface $input, OutputInterface $output): int
    {
        $user = new User();
        $domain = $this->getDomain($input->getArgument('domain'));

        if (!$domain) {
            $output->writeln(sprintf('<error>Domain %s was not found.</error>', $input->getArgument('domain')));

            return 1;
        }

        $user->setDomain($domain);
        $user->setName(\mb_strtolower($input->getArgument('name')));
        $user->setAdmin((bool)$input->getOption('admin'));
        $user->setSendOnly((bool)$input->getOption('sendonly'));

        if ($input->hasOption('quota')) {
            $user->setQuota((int)$input->getOption('quota'));
        }

        if (!$input->getOption('password')) {
            /** @var QuestionHelper $helper */
            $helper = $this->getHelper('question');
            $question = new Question('Password? (hidden)');
            $question->setHidden(true);

            $password = $helper->ask($input, $output, $question);

            if (!$password) {
                $output->writeln('<error>Please set a valid password.</error>');

                return 1;
            }

            $user->setPlainPassword($password);

It seems like $password can also be of type true; however, parameter $plainPassword of App\Entity\User::setPlainPassword() does only seem to accept null|string, maybe add an additional type check?

        } else {
            $user->setPlainPassword($input->getOption('password'));
        }

        $validationResult = $this->validator->validate($user);

        if ($validationResult->count() > 0) {
            foreach ($validationResult as $item) {
                /** @var $item ConstraintViolation */
                $output->writeln(sprintf('<error>%s: %s</error>', $item->getPropertyPath(), $item->getMessage()));
            }

            return 1;
        }

        $this->manager->persist($user);
        $this->manager->flush();

        return 0;
    }