Yoanm\DefaultPhpRepository\Command\Helper\CommandTemplateHelper

Complexity

Total Complexity

10

Size/Duplication

Total Lines	88
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	19	1
C	dumpTemplate()	0	26	8
A	doOverwrite()	0	9	1

<?php
namespace Yoanm\DefaultPhpRepository\Command\Helper;

use Symfony\Component\Console\Helper\QuestionHelper;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Question\ConfirmationQuestion;
use Symfony\Component\Filesystem\Filesystem;
use Yoanm\DefaultPhpRepository\Exception\TargetFileExistsException;
use Yoanm\DefaultPhpRepository\Helper\TemplateHelper;

/**
 * Class CommandTemplateHelper
 */
class CommandTemplateHelper extends TemplateHelper
{
    /** @var InputInterface */
    private $input;
    /** @var OutputInterface */
    private $output;
    /** @var Filesystem */
    private $fs;

Consider using a different property name as you override a private property of the parent class.

    /** @var bool */
    private $skipExisting;
    /** @var bool */
    private $forceOverride;

    /**
     * @param QuestionHelper  $questionHelper
     * @param InputInterface  $input
     * @param OutputInterface $output
     * @param array           $variableList
     * @param bool            $skipExisting
     * @param bool            $forceOverride
     * @param array           $extraTemplatePath
     */
    public function __construct(
        QuestionHelper $questionHelper,
        InputInterface $input,
        OutputInterface $output,
        array $variableList,
        $skipExisting = true,
        $forceOverride = false,
        array $extraTemplatePath = []
    ) {
        parent::__construct($variableList, $extraTemplatePath);

        $this->questionHelper = $questionHelper;

The property questionHelper does not exist. Did you maybe forget to declare it?

        $this->fs = new Filesystem();
        $this->input = $input;
        $this->output = $output;

        $this->skipExisting = $skipExisting;
        $this->forceOverride = $forceOverride;
    }

    /**
     * @param string $templateFilePath
     * @param string $outputFilePath
     * @param bool   $overwrite overwrite even if present
     *
     * @throws TargetFileExistsException
     */
    public function dumpTemplate($templateFilePath, $outputFilePath, $overwrite = false)
    {
        $this->output->write("            <info>$outputFilePath</info> : ");
        $fileExist = $this->fs->exists($outputFilePath);
        $process = true;
        if ($fileExist) {
            if (false === $this->forceOverride && true === $this->skipExisting) {
                $this->output->writeln('<comment>Skipped !</comment>');
                $process = false;
            } else {
                $process = false;
                if (true === $this->forceOverride) {
                    $process = true;
                    $this->output->writeln('<comment>Overriden !</comment>');
                } elseif ($this->doOverwrite()) {
                    $process = true;
                }
            }
        }
        if (true === $process) {
            parent::dumpTemplate($templateFilePath, $outputFilePath);
            if (false === $fileExist) {
                $this->output->writeln('<info>Done</info>');
            }
        }
    }

    /**@return bool
     */
    protected function doOverwrite()
    {
        $question = new ConfirmationQuestion(
            '<question>Overwrite (y/n) ? [n]</question>',
            false
        );

        return $this->questionHelper->ask($this->input, $this->output, $question);
    }
}