PhpGitHooks\Module\JsonLint\Contract\Command\JsonLintToolHandler

Complexity

Total Complexity

5

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	execute()	0	8	2
A	jsonFilesExists()	0	4	1
A	handle()	0	4	1

<?php

namespace PhpGitHooks\Module\JsonLint\Contract\Command;

use Bruli\EventBusBundle\CommandBus\CommandHandlerInterface;
use Bruli\EventBusBundle\CommandBus\CommandInterface;
use Bruli\EventBusBundle\QueryBus\QueryBus;
use PhpGitHooks\Module\Files\Contract\Query\JsonFilesExtractor;
use PhpGitHooks\Module\JsonLint\Service\JsonLintToolExecutor;

class JsonLintToolHandler implements CommandHandlerInterface
{
    /**
     * @var JsonLintToolExecutor
     */
    private $jsonLintToolExecutor;
    /**
     * @var QueryBus
     */
    private $queryBus;

    /**
     * JsonLintTool constructor.
     *
     * @param JsonLintToolExecutor $jsonLintToolExecutor
     * @param QueryBus             $queryBus
     */
    public function __construct(
        JsonLintToolExecutor $jsonLintToolExecutor,
        QueryBus $queryBus
    ) {
        $this->jsonLintToolExecutor = $jsonLintToolExecutor;
        $this->queryBus = $queryBus;
    }

    /**
     * @param array  $files
     * @param string $errorMessage
     */
    private function execute(array $files, $errorMessage)
    {
        $jsonFilesResponse = $this->queryBus->handle(new JsonFilesExtractor($files));

        if (true === $this->jsonFilesExists($jsonFilesResponse->getFiles())) {
            $this->jsonLintToolExecutor->execute($jsonFilesResponse->getFiles(), $errorMessage);
        }
    }

    /**
     * @param array $files
     *
     * @return bool
     */
    private function jsonFilesExists(array $files)
    {
        return 0 < count($files);
    }

    /**
     * @param CommandInterface|JsonLintTool $command
     */
    public function handle(CommandInterface $command)
    {
        $this->execute($command->getFiles(), $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 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.

    }
}