PhpGitHooks\Module\PhpMd\Contract\Command\PhpMdToolHandler

Complexity

Total Complexity

5

Size/Duplication

Total Lines	62
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	5	5	1
A	execute()	22	22	3
A	handle()	4	4	1

<?php

namespace PhpGitHooks\Module\PhpMd\Contract\Command;

use Bruli\EventBusBundle\CommandBus\CommandHandlerInterface;
use Bruli\EventBusBundle\CommandBus\CommandInterface;
use PhpGitHooks\Module\Git\Contract\Response\BadJobLogoResponse;
use PhpGitHooks\Module\Git\Service\PreCommitOutputWriter;
use PhpGitHooks\Module\PhpMd\Contract\Exception\PhpMdViolationsException;
use PhpGitHooks\Module\PhpMd\Model\PhpMdToolProcessorInterface;
use Symfony\Component\Console\Output\OutputInterface;

class PhpMdToolHandler implements CommandHandlerInterface

This class seems to be duplicated in your project.

{
    const CHECKING_MESSAGE = 'Checking code mess with PHPMD';
    /**
     * @var OutputInterface
     */
    private $output;
    /**
     * @var PhpMdToolProcessorInterface
     */
    private $phpMdToolProcessor;

    /**
     * PhpMdTool constructor.
     *
     * @param OutputInterface             $output
     * @param PhpMdToolProcessorInterface $phpMdToolProcessor
     */
    public function __construct(OutputInterface $output, PhpMdToolProcessorInterface $phpMdToolProcessor)
    {
        $this->output = $output;
        $this->phpMdToolProcessor = $phpMdToolProcessor;
    }

    /**
     * @param array  $files
     * @param string $options
     * @param string $errorMessage
     *
     * @throws PhpMdViolationsException
     */
    private function execute(array  $files, $options, $errorMessage)
    {
        $outputMessage = new PreCommitOutputWriter(self::CHECKING_MESSAGE);
        $this->output->write($outputMessage->getMessage());

        $errors = [];
        foreach ($files as $file) {
            $errors[] = $this->phpMdToolProcessor->process($file, $options);
        }

        $errors = array_filter($errors);

        if (!empty($errors)) {
            $outputText = $outputMessage->setError(implode('', $errors));
            $this->output->writeln($outputMessage->getFailMessage());
            $this->output->writeln($outputText);
            $this->output->writeln(BadJobLogoResponse::paint($errorMessage));
            throw new PhpMdViolationsException();
        }

        $this->output->writeln($outputMessage->getSuccessfulMessage());
    }

    /**
     * @param CommandInterface|PhpMdTool $command
     */
    public function handle(CommandInterface $command)
    {
        $this->execute($command->getFiles(), $command->getOptions(), $command->getErrorMessage());

It seems like you code against a concrete implementation and not the interface Bruli\EventBusBundle\CommandBus\CommandInterface as the method getFiles() does only exist in the following implementations of said interface: PhpGitHooks\Module\Composer\Contract\Command\ComposerTool, PhpGitHooks\Module\JsonLint\Contract\Command\JsonLintTool, PhpGitHooks\Module\PhpCsFixer\Contract\Command\PhpCsFixerTool, PhpGitHooks\Module\PhpCs\Contract\Command\PhpCsTool, PhpGitHooks\Module\PhpLint\Contract\Command\PhpLintTool, PhpGitHooks\Module\PhpMd\Contract\Command\PhpMdTool.

It seems like you code against a concrete implementation and not the interface Bruli\EventBusBundle\CommandBus\CommandInterface as the method getOptions() does only exist in the following implementations of said interface: PhpGitHooks\Module\PhpCsFixer\Contract\Command\PhpCsFixerTool, PhpGitHooks\Module\PhpMd\Contract\Command\PhpMdTool, PhpGitHooks\Module\PhpUnit\Contract\Command\PhpUnitTool.

It seems like you code against a concrete implementation and not the interface Bruli\EventBusBundle\CommandBus\CommandInterface as the method getErrorMessage() does only exist in the following implementations of said interface: PhpGitHooks\Module\Composer\Contract\Command\ComposerTool, PhpGitHooks\Module\JsonLint\Contract\Command\JsonLintTool, PhpGitHooks\Module\PhpCsFixer\Contract\Command\PhpCsFixerTool, PhpGitHooks\Module\PhpCs\Contract\Command\PhpCsTool, PhpGitHooks\Module\PhpLint\Contract\Command\PhpLintTool, PhpGitHooks\Module\PhpMd\Contract\Command\PhpMdTool, PhpGitHooks\Module\PhpUnit\Contract\Command\PhpUnitTool, PhpGitHooks\Module\PhpUnit\Contract\Command\StrictCoverage.

    }
}