Merge pull request #5 from moufmouf/master

Making sure a default value for a property or parameter can contain a non empty array.

Author: gossi

src/visitor/DefaultVisitor.php

@@ -224,18 +224,11 @@ public function visitProperty(PhpProperty $property) {
 	}
 
 	protected function getPhpExport($value) {
-		if (is_bool($value)) {
-			return $value ? 'true' : 'false';
-		}
-
+		// Simply to be sure a null value is displayed in lowercase.
 		if (null === $value) {
 			return 'null';
 		}
 
-		if (is_array($value)) {
-			return 'array()';
-		}
-
 		return var_export($value, true);
 	}
 

tests/Visitor/DefaultVisitorTest.php

@@ -3,6 +3,7 @@
 
 use gossi\codegen\model\PhpFunction;
 use gossi\codegen\model\PhpParameter;
+use gossi\codegen\model\PhpProperty;
 use gossi\codegen\visitor\DefaultVisitor;
 use gossi\codegen\model\PhpMethod;
 use gossi\codegen\utils\Writer;
@@ -52,6 +53,17 @@ public function testVisitMethodWithCallable() {
 		$this->assertEquals($this->getContent('callable_parameter.php'), $visitor->getContent());
 	}
 
+	public function testDefaultNotEmptyArray() {
+		$property = new PhpProperty('fooArray');
+		$visitor = new DefaultVisitor();
+
+		$property->setDefaultValue(array("some value"));
+		$visitor->visitProperty($property);
+
+		$this->assertContains("some value", $visitor->getContent());
+	}
+
+
 	/**
 	 *
 	 * @param string $filename