FMUP\Config

Complexity

Total Complexity

7

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	get()	0	7	3
A	set()	0	5	1
A	has()	0	4	1
A	mergeConfig()	0	9	2

<?php
namespace FMUP;

/**
 * Class Config
 * @package FMUP
 */
class Config implements Config\ConfigInterface
{
    /**
     * @var array
     */
    private $params = array();

    /**
     * Retrieve defined param
     * @param string $paramName
     * @return array|null
     */
    public function get($paramName = null)
    {
        if (true == is_null($paramName)) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            return $this->params;
        }
        return $this->has($paramName) ? $this->params[$paramName] : null;
    }

    /**
     * Define a param
     * @param string $paramName
     * @param mixed $value default null
     * @return $this
     */
    public function set($paramName, $value = null)
    {
        $this->params[$paramName] = $value;
        return $this;
    }

    /**
     * Merge an array with current config
     * @param array $paramArray
     * @param bool $before Merge before (default : false)
     * @return $this
     */
    public function mergeConfig(array $paramArray = array(), $before = false)
    {
        if ($before) {
            $this->params = $this->params + $paramArray;
        } else {
            $this->params = $paramArray + $this->params;
        }
        return $this;
    }

    /**
     * Is a param defined
     * @param string $paramName
     * @return bool
     */
    public function has($paramName)
    {
        return isset($this->params[$paramName]);
    }
}