[BUG] Phalcon\Config cannot merge the same section when using Phalcon\Config\Adapter\Ini

[breezewish]

$config1 = new \Phalcon\Config\Adapter\Ini("a.ini");
$config2 = new \Phalcon\Config\Adapter\Ini("b.ini");
$config1->merge($config2)

Case 1: Merge different sections (OK)

a.ini:

[a]
a_1 = 1

b.ini:

[b]
b_1 = 1

Merge result:

[a]
a_1 = 1
[b]
b_1 = 1

Case 2: Merge same section (Fail)

a.ini:

[a]
a_1 = 1

b.ini:

[a]
a_2 = 1

Merge result:

[a]
a_2 = 1

Case 3: Merge \Phalcon\Config (OK)

The following code could get an expected result:

$config1 = new \Phalcon\Config((array)new Phalcon\Config\Adapter\Ini('a.ini'));
$config2 = new \Phalcon\Config((array)new Phalcon\Config\Adapter\Ini('b.ini'));
$config1->merge($config2);

May be a bug? (Phalcon 1.2.1, Windows)

[dreamsxin]

Case 2 is same this:

$a = array('a' => array('a_1' => 1));
$b = array('a' => array('a_2' => 2));

$a = array_merge($a, $b);
var_dump($a);

output:

array(1) {
  ["a"]=>
  array(1) {
    ["a_1"]=>
    int(1)
  }
}

A two-dimensional array of merger, may need to write their own Arr class
@sjinks What do you think needs?

[ghost]

The issue is that Phalcon\Config::toArray() is too recursive for merge(). I will fix this.

[phalcon]

This was fixed in 1.3.0