PhpGitHooks\Module\PhpMd\Infrastructure\Tool\PhpMdToolProcessor

Complexity

Total Complexity

6

Size/Duplication

Total Lines	62
Duplicated Lines	12.9 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	process()	0	6	1
A	execute()	0	9	1
A	setError()	8	8	3

<?php

namespace PhpGitHooks\Module\PhpMd\Infrastructure\Tool;

use PhpGitHooks\Infrastructure\Tool\ToolPathFinder;
use PhpGitHooks\Module\PhpMd\Model\PhpMdToolProcessorInterface;
use Symfony\Component\Process\Process;

class PhpMdToolProcessor implements PhpMdToolProcessorInterface
{
    /**
     * @var ToolPathFinder
     */
    private $toolPathFinder;

    /**
     * PhpMdToolProcessor constructor.
     *
     * @param ToolPathFinder $toolPathFinder
     */
    public function __construct(ToolPathFinder $toolPathFinder)
    {
        $this->toolPathFinder = $toolPathFinder;
    }

    /**
     * @param string $file
     * @param string $options
     *
     * @return string
     */
    public function process($file, $options)
    {
        $process = $this->execute($file, $options);

        return $this->setError($process);
    }

    /**
     * @param string $file
     * @param string $options
     *
     * @return Process
     */
    private function execute($file, $options)
    {
        $command = sprintf('php %s %s text ./PmdRules.xml %s', $this->toolPathFinder->find('phpmd'), $file, $options);

        $process = new Process($command);

$command is of type string, but the function expects a array.

        $process->run();

        return $process;
    }

    /**
     * @param Process $process
     *
     * @return null|string
     *
     * @throws \Symfony\Component\Process\Exception\LogicException
     */
    private function setError(Process $process)

This method seems to be duplicated in your project.

    {
        if (false === $process->isSuccessful()) {
            return $process->getErrorOutput() ?: $process->getOutput();
        }

        return null;
    }
}