CloverToHtml\ConfigDAO

Complexity

Total Complexity

6

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	findConfig()	0	22	4
A	getConfigPath()	0	4	1
A	loadConfig()	0	4	1

<?php

/**
 * This file is part of the Clover to Html package.
 *
 * (c) Stéphane Demonchaux <demonchaux.stephane@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace CloverToHtml;

class ConfigDAO
{
    /**
     * @param string $templatePath
     * @param string $key
     * @throws \InvalidArgumentException
     * @return array
     */
    public function findConfig($templatePath, $key = null)
    {
        if (is_file($this->getConfigPath($templatePath)) === false) {
            throw new \InvalidArgumentException(
                sprintf('Template "%s" with key "%s" not found', $templatePath, $key)
            );
        }

        if ($key === null) {
            return $config;

The variable $config seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

        }

        $config = $this->loadConfig($templatePath);

        if (isset($config[$key])) {
            return $config[$key];
        }

        throw new \InvalidArgumentException(
            sprintf('Template "%s" with key "%s" not found', $templatePath, $key)
        );
    }

    /**
     * @param string $templatePath
     * @return string
     */
    private function getConfigPath($templatePath)
    {
        return $templatePath.'/config.json';
    }

    /**
     * @param string $templatePath
     * @return array
     */
    private function loadConfig($templatePath)
    {
        return json_decode(file_get_contents($this->getConfigPath($templatePath)), true);
    }
}