Symplify\MultiCodingStandard\Command\FixCommand

Complexity

Total Complexity

7

Size/Duplication

Total Lines	83
Duplicated Lines	16.87 %

Coupling/Cohesion

Components	2
Dependencies	6

Importance

Changes	2
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	1
A	configure()	0	6	1
A	execute()	0	17	3
A	fixDirectory()	14	14	2

<?php

/*
 * This file is part of Symplify
 * Copyright (c) 2016 Tomas Votruba (http://tomasvotruba.cz).
 */

namespace Symplify\MultiCodingStandard\Command;

use Exception;
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\Console\Style\StyleInterface;
use Symfony\Component\Finder\Finder;
use Symfony\CS\Fixer;
use Symplify\MultiCodingStandard\CodeSniffer\CodeBeautifier;
use Symplify\MultiCodingStandard\CodeSniffer\CodeBeautifierFactory;
use Symplify\MultiCodingStandard\Console\ExitCode;
use Symplify\MultiCodingStandard\Contract\CodeSniffer\CodeBeautifierFactoryInterface;

final class FixCommand extends Command
{
    /**
     * @var CodeBeautifierFactoryInterface
     */
    private $codeBeautifierFactory;

    /**
     * @var StyleInterface
     */
    private $style;

    /**
     * @var Fixer
     */
    private $fixer;

    /**
     * @var CodeBeautifier
     */
    private $codeBeautifier;

    public function __construct(
        CodeBeautifierFactoryInterface $codeBeautifierFactory,
        Fixer $fixer,
        StyleInterface $style
    ) {
        parent::__construct();

        $this->codeBeautifierFactory = $codeBeautifierFactory;
        $this->fixer = $fixer;
        $this->style = $style;
    }

    /**
     * {@inheritdoc}
     */
    protected function configure()
    {
        $this->setName('fix');
        $this->addArgument('path', InputArgument::REQUIRED | InputArgument::IS_ARRAY, 'The path(s)', null);
        $this->setDescription('Fix coding standard in one or more directories.');
    }

    /**
     * {@inheritdoc}
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        // note: needs to be lazy created, due to constant-options in PHP_CodeSniffer
        $this->codeBeautifier = $this->codeBeautifierFactory->create();

        try {
            foreach ($input->getArgument('path') as $path) {
                $this->fixDirectory($path);
            }

            return ExitCode::SUCCESS;
        } catch (Exception $exception) {
            $this->style->error($exception->getMessage());

            return ExitCode::ERROR;
        }
    }

    /**
     * @param string $path
     */
    private function fixDirectory($path)

This method seems to be duplicated in your project.

    {
        // code sniffer
        foreach (Finder::create()->in($path)->files() as $filePath => $fileInfo) {
            $file = $this->codeBeautifier->processFile($filePath);
            var_dump($file->getErrorCount());

var_dump($file->getErrorCount()); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

        }

        // php-cs-fixer

        $this->style->success(
            sprintf('Directory "%s" was checked!', $path)
        );
    }
}