Atog\Api\Console\EndpointCommand

Complexity

Total Complexity

5

Size/Duplication

Total Lines	87
Duplicated Lines	26.44 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	17	1
A	execute()	23	23	3
B	template()	0	26	1

<?php
namespace Atog\Api\Console;

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

/**
 * Class EndpointCommand
 * @package Atog\Console
 */
class EndpointCommand extends Command
{
    /**
     * Command Config
     */
    protected function configure()
    {
        $this
            ->setName('create:endpoint')
            ->setDescription('Create a new Endpoint')
            ->addArgument(
                'namespace',
                InputArgument::REQUIRED,
                'Namespace in Dot-Notation e.g. Acme.Api'
            )
            ->addArgument(
                'name',
                InputArgument::REQUIRED,
                'Name of the Endpoint'
            )
        ;
    }

    /**
     * Execute command
     * @param \Symfony\Component\Console\Input\InputInterface   $input
     * @param \Symfony\Component\Console\Output\OutputInterface $output
     * @return void
     */
    protected function execute(InputInterface $input, OutputInterface $output)

This method seems to be duplicated in your project.

    {
        // args
        $namespace = str_replace('.', '\\', $input->getArgument('namespace'));
        $name = $input->getArgument('name');
        $path = $this->getApplication()->endpointPath();

It seems like you code against a specific sub-type and not the parent class Symfony\Component\Console\Application as the method endpointPath() does only exist in the following sub-classes of Symfony\Component\Console\Application: Atog\Api\Console\Application. Maybe you want to instanceof check for one of these explicitly?

        $file = $path . str_plural($name) . '.php';

        // only write file if path exists and file does not exist yet
        if (file_exists($path) && !file_exists($file)) {
            // write file
            $endpoint = fopen($file, "w+");
            fwrite($endpoint, $this->template($namespace, $name));
            fclose($endpoint);

            // output
            $output->writeln("<info>Created Endpoint: $name</info>");
            return;
        }

        // error
        $output->writeln("<error>Could not create file $file at $path</error>");
    }

    /**
     * File Template
     * @param string $namespace
     * @param string $name
     * @param string $endpointNS
     * @return string
     */
    protected function template($namespace, $name, $endpointNS = "Endpoints")
    {
        $className = str_plural($name);
        $endpointName = strtolower($name);
        $namespace = "{$namespace}\\{$endpointNS}";

        return <<< EOT
<?php
namespace $namespace;

use Atog\Api\Endpoint;

/**
 * Class $className
 * @package $namespace
 */
class $className extends Endpoint
{
    /**
     * @var string
     */
    protected \$endpoint = '$endpointName';
}

EOT;
    }
}