Puppy\Config\ArrayConfig

Complexity

Total Complexity

12

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes	7
Bugs	0	Features	2

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	2
A	restrict()	0	4	1
A	offsetSet()	0	5	1
A	buildConfig()	0	4	1
A	formatKeys()	0	10	3
A	replace()	0	11	4

<?php
namespace Puppy\Config;

/**
 * Class SimpleConfig
 * @package Puppy\Config
 * @author Raphaël Lefebvre <raphael@raphaellefebvre.be>
 */
class ArrayConfig extends \ArrayObject
{

    /**
     * @param array|\ArrayObject $data
     */
    public function __construct($data)
    {
        if(!($data instanceof self)){
            $data = $this->buildConfig($data);
        }
        parent::__construct($data);
    }

    /**
     * restricts the config to a specific visibility.
     *
     * @param string $restriction
     * @param string $separator
     * @return RestrictedConfig
     */
    public function restrict($restriction, $separator = '.')
    {
        return new RestrictedConfig($restriction, $separator, $this);
    }

    /**
     * @param mixed $key
     * @param mixed $value
     */
    public function offsetSet($key, $value)
    {
        parent::offsetSet($key, $value);
        $this->exchangeArray($this->buildConfig(parent::getArrayCopy()));

It seems like you call parent on a different method (getArrayCopy() instead of offsetSet()). Are you sure this is correct? If so, you might want to change this to $this->getArrayCopy().

    }

    /**
     * @param array|\ArrayObject $data
     * @return array
     */
    private function buildConfig($data)
    {
        return $this->replace($data, $this->formatKeys($data));
    }

    /**
     * @param array|\ArrayObject $data
     * @return array
     */
    private function formatKeys($data)
    {
        $result = [];
        foreach ($data as $key => $value) {
            if (is_string($value)) {
                $result['%' . $key . '%'] = $value;
            }
        }
        return $result;
    }

    /**
     * @param $data
     * @param array $replacements
     * @return mixed
     */
    private function replace($data, array $replacements)
    {
        foreach ($data as $key => $var) {
            if (is_string($var)) {
                $data[$key] = strtr($var, $replacements);
            }elseif(is_array($var)){
                $data[$key] = $this->replace($var, $replacements);
            }
        }
        return $data;
    }
}