PhpGitHooks\Module\PhpCs\Contract\Command\PhpCsTool

Complexity

Total Complexity

6

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	getErrorMessage()	0	4	1
A	isEnableFaces()	0	4	1
A	getFiles()	0	4	1
A	getStandard()	0	4	1
A	getIgnore()	0	4	1

<?php

namespace PhpGitHooks\Module\PhpCs\Contract\Command;

use Bruli\EventBusBundle\CommandBus\CommandInterface;

class PhpCsTool implements CommandInterface
{
    /**
     * @var array
     */
    private $files;
    /**
     * @var string
     */
    private $standard;
    /**
     * @var string
     */
    private $errorMessage;
    /**
     * @var string
     */
    private $enableFaces;
    /**
     * @var string
     */
    private $ignore;

    /**
     * PhpCsToolCommand constructor.
     *
     * @param array $files
     * @param string $standard
     * @param string $errorMessage
     * @param bool $enableFaces
     * @param string $ignore
     */
    public function __construct(array $files, $standard, $errorMessage, $enableFaces, $ignore)
    {
        $this->files = $files;
        $this->standard = $standard;
        $this->errorMessage = $errorMessage;
        $this->enableFaces = $enableFaces;

The property $enableFaces was declared of type string, but $enableFaces is of type boolean. Maybe add a type cast?

        $this->ignore = $ignore;
    }

    /**
     * @return string
     */
    public function getErrorMessage()
    {
        return $this->errorMessage;
    }

    /**
     * @return string
     */
    public function isEnableFaces()
    {
        return $this->enableFaces;
    }

    /**
     * @return array
     */
    public function getFiles()
    {
        return $this->files;
    }

    /**
     * @return string
     */
    public function getStandard()
    {
        return $this->standard;
    }

    /**
     * @return string
     */
    public function getIgnore()
    {
        return $this->ignore;
    }
}