#59 Check for null before fetching target properties

mark-gerarts · May 5, 2020 · 0e9aa94 · 0e9aa94
1 parent 420c43d
commit 0e9aa94
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 0 deletions.
diff --git a/src/PropertyAccessor/PropertyAccessor.php b/src/PropertyAccessor/PropertyAccessor.php
@@ -122,6 +122,10 @@ private function isPublic($object, string $propertyName): bool
      * @return iterable|\ReflectionProperty[]
      */
     private function getReflectionProperties($object): iterable {
+        if ($object === null) {
+            return;
+        }
+
         $reflectionClass = new \ReflectionObject($object);
         $properties = $reflectionClass->getProperties();
         foreach ($properties as $property) {

diff --git a/test/AutoMapperTest.php b/test/AutoMapperTest.php
@@ -719,4 +719,27 @@ public function testAnExceptionIsThrownForNoIterableSourceInMultpleMappings()
 
         $mapper->mapMultiple($sourceCollection, Destination::class);
     }
+
+    /**
+     * @group test
+     */
+    public function testItMapsANullObjectReturnedFromConstructorToNull()
+    {
+        $this->config->registerMapping(DataType::ARRAY, Address::class)
+            ->beConstructedUsing(function () { return null; });
+        $this->config->registerMapping(DataType::ARRAY, Person::class)
+            ->forMember('adres', Operation::mapTo(Address::class, true));
+        $mapper = new AutoMapper($this->config);
+
+        $source = [
+            'adres' => [
+                'street' => 'Main Street',
+                'number' => '314'
+            ]
+        ];
+
+        $result = $mapper->map($source, Person::class);
+
+        $this->assertNull($result->adres);
+    }
 }