ArrayConfig::offsetSet() - Code Metrics - Raphhh/puppy-config - Measure and Improve Code Quality continuously with Scrutinizer

ArrayConfig::offsetSet() A
last analyzed 2016-03-23 08:27 UTC

↳ Parent: ArrayConfig

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	1

Importance

Changes	2
Bugs	0	Features	1

Metric	Value
c	2
b	0
f	1
dl	0
loc	5
ccs	4
cts	4
cp	1
rs	9.4285
cc	1
eloc	3
nc	1
nop	2
crap	1

<?php
namespace Puppy\Config;

/**
 * Class SimpleConfig
 * @package Puppy\Config
 * @author Raphaël Lefebvre <[email protected]>
 */
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()));
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
    }

    /**
     * @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;
    }
}
 

1		<?php
2		namespace Puppy\Config;
3
4		/**
5		* Class SimpleConfig
6		* @package Puppy\Config
7		* @author Raphaël Lefebvre <[email protected]>
8		*/
9		class ArrayConfig extends \ArrayObject
10		{
11
12		/**
13		* @param array\|\ArrayObject $data
14		*/
15	23	public function __construct($data)
16		{
17	23	if(!($data instanceof self)){
18	23	$data = $this->buildConfig($data);
19	23	}
20	23	parent::__construct($data);
21	23	}
22
23		/**
24		* restricts the config to a specific visibility.
25		*
26		* @param string $restriction
27		* @param string $separator
28		* @return RestrictedConfig
29		*/
30	2	public function restrict($restriction, $separator = '.')
31		{
32	2	return new RestrictedConfig($restriction, $separator, $this);
33		}
34
35		/**
36		* @param mixed $key
37		* @param mixed $value
38		*/
39	6	public function offsetSet($key, $value)
40		{
41	6	parent::offsetSet($key, $value);
42	6	$this->exchangeArray($this->buildConfig(parent::getArrayCopy()));
		0 ignored issues – show Comprehensibility Bug introduced 2016-03-21 17:00 UTC by Report Bug Copy Issue Report 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()`. This check looks for a call to a parent method whose name is different than the method from which it is called. Consider the following code: class Daddy { protected function getFirstName() { return "Eidur"; } protected function getSurName() { return "Gudjohnsen"; } } class Son { public function getFirstName() { return parent::getSurname(); } } The `getFirstName()` method in the `Son` calls the wrong method in the parent class. Loading history...
43	6	}
44
45		/**
46		* @param array\|\ArrayObject $data
47		* @return array
48		*/
49	23	private function buildConfig($data)
50		{
51	23	return $this->replace($data, $this->formatKeys($data));
52		}
53
54		/**
55		* @param array\|\ArrayObject $data
56		* @return array
57		*/
58	23	private function formatKeys($data)
59		{
60	23	$result = [];
61	23	foreach ($data as $key => $value) {
62	21	if (is_string($value)) {
63	21	$result['%' . $key . '%'] = $value;
64	21	}
65	23	}
66	23	return $result;
67		}
68
69		/**
70		* @param $data
71		* @param array $replacements
72		* @return mixed
73		*/
74	23	private function replace($data, array $replacements)
75		{
76	23	foreach ($data as $key => $var) {
77	21	if (is_string($var)) {
78	21	$data[$key] = strtr($var, $replacements);
79	21	}elseif(is_array($var)){
80	2	$data[$key] = $this->replace($var, $replacements);
81	2	}
82	23	}
83	23	return $data;
84		}
85		}
86

Raphhh / puppy-config

ArrayConfig::offsetSet() A last analyzed 2016-03-23 08:27 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

ArrayConfig::offsetSet() A
last analyzed 2016-03-23 08:27 UTC