Magium\Util\Configuration\ClassConfigurationReader

Complexity

Total Complexity

10

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__set()	0	4	1
A	__call()	0	4	1
A	__construct()	0	6	1
A	setConfigurationDir()	0	4	1
B	configure()	0	28	6

<?php

namespace Magium\Util\Configuration;

class ClassConfigurationReader
{

    protected $configurationDir;
    protected $object;

    public function __construct(
        $configurationDir = null
    )
    {
        $this->configurationDir = $configurationDir;
    }

    /**
     * @param null $configurationDir
     */
    public function setConfigurationDir($configurationDir)
    {
        $this->configurationDir = $configurationDir;
    }


    public function __set($name, $value)
    {
        $this->object->$name = $value;
    }

    public function __call($method, $args)
    {
        call_user_func_array([$this->object, $method], $args);
    }


    public function configure(ConfigurableObjectInterface $config)
    {
        $this->object = $config;
        $configurationDir = $this->configurationDir;
        if ($configurationDir === null) {
            $path = realpath(__DIR__ . '/../');
            $count = 0;
            while ($count++ < 10) {
                $filename = "{$path}/configuration";
                if (is_dir($filename)) {
                    $configurationDir = realpath($filename);
                    break;
                }
                $path .= '/../';
                $path = realpath($path); // More for debugging clarity.
            }
        }

        if (!$configurationDir) return;

The expression $configurationDir of type string|null is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

        $configurationFile = get_class($config) . '.php';
        $configurationFile = $configurationDir . DIRECTORY_SEPARATOR . str_replace('\\', DIRECTORY_SEPARATOR, $configurationFile);

        if (file_exists($configurationFile)) {
            include $configurationFile;
        }

    }

}