fix: error when mapping from stdClass to constructor with nullable/op…

…tional arguments
jolicode · Sep 12, 2024 · 018ab22 · 018ab22
1 parent 8fa3035
commit 018ab22
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/Transformer/NullableTransformerFactory.php b/src/Transformer/NullableTransformerFactory.php
@@ -21,6 +21,10 @@ final class NullableTransformerFactory implements TransformerFactoryInterface, P
 
     public function getTransformer(TypesMatching $types, SourcePropertyMetadata $source, TargetPropertyMetadata $target, MapperMetadata $mapperMetadata): ?TransformerInterface
     {
+        if (null !== $target->parameterInConstructor) {
+            return null;
+        }
+
         $sourceType = $types->getSourceUniqueType();
 
         if (null === $sourceType) {

diff --git a/tests/AutoMapperTest.php b/tests/AutoMapperTest.php
@@ -525,6 +525,15 @@ public function testConstructorForcedException(): void
         $this->autoMapper->map($data, ConstructorWithDefaultValues::class);
     }
 
+    public function testConstructorWithDefaultFromStdClass(): void
+    {
+        $data = (object) ['baz' => 'baz'];
+        /** @var ConstructorWithDefaultValues $object */
+        $object = $this->autoMapper->map($data, ConstructorWithDefaultValues::class);
+
+        self::assertInstanceOf(ConstructorWithDefaultValues::class, $object);
+    }
+
     public function testConstructorWithDefault(): void
     {
         $user = new Fixtures\UserDTONoAge();