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