Removed parentheses and added unit tests to verify the behavior

phalcon · Nov 12, 2014 · 2c2bb31 · 2c2bb31
1 parent af452dc
commit 2c2bb31
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/phalcon/config.zep b/phalcon/config.zep
@@ -59,7 +59,7 @@ class Config implements \ArrayAccess, \Countable
 		for key, value in arrayConfig {
 
 			let key = strval(key);
-			if (typeof value === "array") {
+			if typeof value === "array" {
 				let this->{key} = new self(value);
 			} else {
 				let this->{key} = value;

diff --git a/unit-tests/ConfigTest.php b/unit-tests/ConfigTest.php
@@ -263,4 +263,16 @@ public function testYamlConfigCallback()
         ));
         $this->assertTrue($this->_compareConfig($this->_config, $config));
     }
+
+    public function testNumericConfig()
+    {
+		$config = new \Phalcon\Config(["abc"]);
+		$this->assertEquals($config->{0}, "abc");
+    }
+
+    public function testChildArrayToConfigObject()
+    {
+		$config = new \Phalcon\Config(['childsettings' => ['A', 'B', 'C']]);
+		$this->assertInstanceOf('Phalcon\Config', $config->childsettings);
+    }
 }