fix infer normalizer for nullable types

Author: DavidBadura

src/Metadata/AttributeMetadataFactory.php

@@ -29,6 +29,7 @@
 use ReflectionProperty;
 use Symfony\Component\TypeInfo\Type;
 use Symfony\Component\TypeInfo\Type\CollectionType;
+use Symfony\Component\TypeInfo\Type\NullableType;
 use Symfony\Component\TypeInfo\Type\ObjectType;
 use Symfony\Component\TypeInfo\TypeResolver\TypeResolver;
 
@@ -396,13 +397,21 @@ private function findNormalizer(ReflectionProperty $reflectionProperty, Type $ty
             return $attributeReflectionList[0]->newInstance();
         }
 
+        if ($type instanceof NullableType) {
+            $type = $type->getWrappedType();
+        }
+
         if ($type instanceof ObjectType) {
             return $this->findNormalizerOnClass(new ReflectionClass($type->getClassName()));
         }
 
         if ($type instanceof CollectionType && $type->getCollectionValueType() instanceof ObjectType) {
             $valueType = $type->getCollectionValueType();
 
+            if ($valueType instanceof NullableType) {
+                $valueType = $type->getWrappedType();
+            }
+
             if (!$valueType instanceof ObjectType) {
                 return null;
             }

tests/Unit/Fixture/InferNormalizerWithNullableDto.php

@@ -15,6 +15,7 @@ public function __construct(
         public DateTimeImmutable|null $dateTimeImmutable,
         public DateTime|null $dateTime = null,
         public DateTimeZone|null $dateTimeZone = null,
+        public ProfileId|null $profileId = null,
     ) {
     }
 }

tests/Unit/Metadata/AttributeMetadataFactoryTest.php

@@ -27,6 +27,7 @@
 use Patchlevel\Hydrator\Tests\Unit\Fixture\MissingSubjectIdDto;
 use Patchlevel\Hydrator\Tests\Unit\Fixture\ParentDto;
 use Patchlevel\Hydrator\Tests\Unit\Fixture\ParentWithPersonalDataDto;
+use Patchlevel\Hydrator\Tests\Unit\Fixture\ProfileId;
 use Patchlevel\Hydrator\Tests\Unit\Fixture\Status;
 use PHPUnit\Framework\TestCase;
 
@@ -199,6 +200,27 @@ public function __construct(
         self::assertSame(Status::class, $normalizer->getEnum());
     }
 
+    public function testEventWithInferNormalizer(): void
+    {
+        $object = new class {
+            public function __construct(
+                public ProfileId|null $profileId = null,
+            ) {
+            }
+        };
+
+        $metadataFactory = new AttributeMetadataFactory();
+        $metadata = $metadataFactory->metadata($object::class);
+
+        $properties = $metadata->properties();
+
+        self::assertCount(1, $properties);
+
+        $propertyMetadata = $metadata->propertyForField('profileId');
+
+        self::assertEquals(new IdNormalizer(ProfileId::class), $propertyMetadata->normalizer());
+    }
+
     public function testExtends(): void
     {
         $metadataFactory = new AttributeMetadataFactory();

tests/Unit/MetadataHydratorTest.php

@@ -106,6 +106,28 @@ public function testExtractCircularReference(): void
         $this->hydrator->extract($dto1);
     }
 
+    public function testExtractWithInferNormalizer2(): void
+    {
+        $result = $this->hydrator->extract(
+            new InferNormalizerWithNullableDto(
+                null,
+                null,
+                profileId: ProfileId::fromString('1'),
+            ),
+        );
+
+        self::assertEquals(
+            [
+                'status' => null,
+                'dateTimeImmutable' => null,
+                'dateTime' => null,
+                'dateTimeZone' => null,
+                'profileId' => '1',
+            ],
+            $result,
+        );
+    }
+
     public function testExtractWithInferNormalizerFailed(): void
     {
         $this->expectException(NormalizationMissing::class);