jeboehm / mailserver-admin

src/Command/DomainAddCommand.php

<?php

declare(strict_types=1);
/**
 * This file is part of the mailserver-admin package.
 * (c) Jeffrey Boehm <https://github.com/jeboehm/mailserver-admin>
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace App\Command;

use App\Entity\Domain;
use Doctrine\Persistence\ManagerRegistry;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Validator\ConstraintViolation;
use Symfony\Component\Validator\Validator\ValidatorInterface;

class DomainAddCommand extends Command
{
    private ManagerRegistry $manager;

    private ValidatorInterface $validator;

    public function __construct(ManagerRegistry $manager, ValidatorInterface $validator)
    {
        parent::__construct();

        $this->manager = $manager;
        $this->validator = $validator;
    }

    protected function configure(): void
    {
        $this
            ->setName('domain:add')
            ->setDescription('Add domains.')
            ->addArgument('domain', InputArgument::REQUIRED, 'Domain-part (after @)');
    }

    protected function execute(InputInterface $input, OutputInterface $output): int
    {
        $domain = new Domain();
        $domain->setName(\mb_strtolower($input->getArgument('domain')));

It seems like $input->getArgument('domain') can also be of type null and string[]; however, parameter $string of mb_strtolower() does only seem to accept string, maybe add an additional type check?

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

        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->getManager()->persist($domain);
        $this->manager->getManager()->flush();

        return 0;
    }
}