Merge branch 'feature/166' into develop

Close #166

Author: michalbundyra

CHANGELOG.md

@@ -10,7 +10,7 @@ All notable changes to this project will be documented in this file, in reverse
 
 ### Changed
 
-- Nothing.
+- [#166](https://github.com/zendframework/zend-code/pull/166) changes omitting default property value if it is null.
 
 ### Deprecated
 

src/Generator/PropertyGenerator.php

@@ -46,7 +46,11 @@ public static function fromReflection(PropertyReflection $reflectionProperty)
 
         $allDefaultProperties = $reflectionProperty->getDeclaringClass()->getDefaultProperties();
 
-        $property->setDefaultValue($allDefaultProperties[$reflectionProperty->getName()]);
+        $defaultValue = $allDefaultProperties[$reflectionProperty->getName()];
+        $property->setDefaultValue($defaultValue);
+        if ($defaultValue === null) {
+            $property->omitDefaultValue = true;
+        }
 
         if ($reflectionProperty->getDocComment() != '') {
             $property->setDocBlock(DocBlockGenerator::fromReflection($reflectionProperty->getDocBlock()));

test/Generator/PropertyGeneratorTest.php

@@ -308,4 +308,15 @@ public function testOmitType()
 
         self::assertEquals('    public $foo;', $property->generate());
     }
+
+    public function testFromReflectionOmitsDefaultValueIfItIsNull() : void
+    {
+        $reflectionClass = new ClassReflection(TestAsset\TestClassWithManyProperties::class);
+        $propertyReflection = $reflectionClass->getProperty('fooStaticProperty');
+
+        $generator = PropertyGenerator::fromReflection($propertyReflection);
+        $code = $generator->generate();
+
+        $this->assertEquals('    public static $fooStaticProperty;', $code);
+    }
 }