Merge pull request #166 from MidnightDesign/omit-default-null

Omit default property value if it is null

Author: michalbundyra

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);
+    }
 }