PhpGitHooks\Application\Config\PreCommitConfig

Complexity

Total Complexity

8

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

84.21%

Importance

Changes	6
Bugs	0	Features	2

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	isEnabled()	0	10	3
A	getData()	0	10	2
A	getMessages()	0	6	1
A	extraOptions()	0	6	1

<?php

namespace PhpGitHooks\Application\Config;

use PhpGitHooks\Application\Message\MessageConfigData;
use PhpGitHooks\Infrastructure\Disk\Config\ConfigFileReaderInterface;
use Symfony\Component\Config\Definition\Exception\Exception;

/**
 * Class PreCommitConfig.
 */
class PreCommitConfig implements HookConfigInterface, HookConfigExtraToolInterface
{
    const HOOK_NAME = 'pre-commit';
    /** @var  ConfigFileReaderInterface */
    private $configFileReader;

    /**
     * PreCommitConfig constructor.
     *
     * @param ConfigFileReaderInterface $configFileReader
     */
    public function __construct(ConfigFileReaderInterface $configFileReader)
    {
        $this->configFileReader = $configFileReader;
    }

    /**
     * @param $service
     *
     * @return bool
     */
    public function isEnabled($service)
    {
        $data = $this->getData();

        if (false === isset($data[$service]) || false === is_bool($data[$service])) {
            return false;
        }

        return $data[$service];
    }

    /**
     * @return array
     */
    private function getData()
    {
        $data = $this->configFileReader->getFileContents();

        if (!$data) {

The expression $data of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            throw new Exception('php-git-hooks.yml file not found');
        }

        return $data[self::HOOK_NAME]['execute'];
    }

    /**
     * @return array
     */
    public function getMessages()
    {
        $data = $this->configFileReader->getFileContents();

        return $data[self::HOOK_NAME][MessageConfigData::TOOL];
    }

    /**
     * @param array $tool
     *
     * @return array
     */
    public function extraOptions($tool)
    {
        $data = $this->getData();

        return $data[$tool];
    }
}