Lej\Command\CreateUuidIdCommand

Complexity

Total Complexity

2

Size/Duplication

Total Lines	36
Duplicated Lines	100 %

Coupling/Cohesion

Components	2
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	9	9	1
A	execute()	17	17	1

<?php

declare(strict_types=1);

namespace Lej\Command;

use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

class CreateUuidIdCommand extends AbstractCommand

This class seems to be duplicated in your project.

{
    /**
     * {@inheritdoc}
     */
    protected function configure()
    {
        $this
            ->setName('lej-ddd:create-uuid-id')
            ->setDescription('Creates a UUID-based identity.')
            ->addArgument('domain', InputArgument::REQUIRED, 'The name of the domain.')
            ->addArgument('context', InputArgument::REQUIRED, 'The name of the context.')
            ->addArgument('name', InputArgument::REQUIRED, 'The name of the identity.');
    }

    /**
     * {@inheritdoc}
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $twig = $this->twig();

        $path = $this->modelPath($input->getArgument('domain'), $input->getArgument('context'));
        $namespace = $this->modelNamespace($input->getArgument('domain'), $input->getArgument('context'));
        @mkdir($path, 0755, true);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        $idClassName = $input->getArgument('name');

        $id = $twig->render('uuid_id.php.twig', [
            'namespace' => $namespace,
            'idClassName' => $idClassName
        ]);

        file_put_contents($path . '/' . $idClassName . '.php', $id);
    }
}