PhpGitHooks\Infrastructure\CodeSniffer\CodeSnifferHandler

Complexity

Total Complexity

9

Size/Duplication

Total Lines	83
Duplicated Lines	8.43 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes	7
Bugs	0	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	run()	7	28	5
A	setNeddle()	0	4	1
A	setFiles()	0	4	1
A	setStandard()	0	4	1

<?php

namespace PhpGitHooks\Infrastructure\CodeSniffer;

use IgnoreFiles\IgnoreFiles;
use PhpGitHooks\Application\Message\MessageConfigData;
use PhpGitHooks\Command\BadJobLogo;
use PhpGitHooks\Command\OutputHandlerInterface;
use PhpGitHooks\Infrastructure\Common\ToolHandler;
use Symfony\Component\Process\Process;
use Symfony\Component\Process\ProcessBuilder;

/**
 * Class CodeSnifferHandler.
 */
class CodeSnifferHandler extends ToolHandler
{
    /** @var array */
    private $files;
    /** @var string */
    private $neddle;
    /** @var string */
    private $standard = 'PSR2';
    /**
     * @var IgnoreFiles
     */
    private $ignoreFiles;

    /**
     * CodeSnifferHandler constructor.
     *
     * @param OutputHandlerInterface $outputHandler
     * @param IgnoreFiles            $ignoreFiles
     */
    public function __construct(OutputHandlerInterface $outputHandler, IgnoreFiles $ignoreFiles)
    {
        parent::__construct($outputHandler);
        $this->ignoreFiles = $ignoreFiles;
    }

    /**
     * @param array $messages
     *
     * @throws InvalidCodingStandardException
     */
    public function run(array $messages)
    {
        $this->outputHandler->setTitle('Checking code style with PHPCS');
        $this->output->write($this->outputHandler->getTitle());

        foreach ($this->files as $file) {
            if (!preg_match($this->neddle, $file)) {
                continue;
            }

            if (false === $this->ignoreFiles->isIgnored($file)) {
                $processBuilder = new ProcessBuilder(array('php', 'bin/phpcs', '--standard='.$this->standard, $file));
                /** @var Process $phpCs */
                $phpCs = $processBuilder->getProcess();
                $phpCs->run();

                if (false === $phpCs->isSuccessful()) {

This code seems to be duplicated across your project.

                    $this->outputHandler->setError($phpCs->getOutput());
                    $this->output->writeln($this->outputHandler->getError());
                    $this->output->writeln(BadJobLogo::paint($messages[MessageConfigData::KEY_ERROR_MESSAGE]));

                    throw new InvalidCodingStandardException();
                }
            }
        }

        $this->output->writeln($this->outputHandler->getSuccessfulStepMessage());
    }

    /**
     * @param string $neddle
     */
    public function setNeddle($neddle)
    {
        $this->neddle = $neddle;
    }

    /**
     * @param array $files
     */
    public function setFiles($files)
    {
        $this->files = $files;
    }

    /**
     * @param array $standard
     */
    public function setStandard($standard)
    {
        $this->standard = $standard;

It seems like $standard of type array is incompatible with the declared type string of property $standard.

    }
}