symfony · soyuka · Nov 25, 2025
diff --git a/src/Symfony/Component/ObjectMapper/CHANGELOG.md b/src/Symfony/Component/ObjectMapper/CHANGELOG.md
@@ -1,6 +1,11 @@
 CHANGELOG
 =========
 
+8.0
+---
+
+* Merge nested properties when targeting the same class
+
 7.4
 ---
 

diff --git a/src/Symfony/Component/ObjectMapper/ObjectMapper.php b/src/Symfony/Component/ObjectMapper/ObjectMapper.php
@@ -154,14 +154,30 @@ private function doMap(object $source, object|string|null $target, \WeakMap $obj
                 $this->storeValue($targetPropertyName, $mapToProperties, $ctorArguments, $value);
             }
 
-            if (!$mappings && $targetRefl->hasProperty($propertyName)) {
-                $sourceProperty = $refl->getProperty($propertyName);
-                if ($refl->isInstance($source) && !$sourceProperty->isInitialized($source)) {
-                    continue;
-                }
+            if ($mappings) {
+                continue;
+            }
+
+            $sourceProperty = $refl->getProperty($propertyName);
+            if ($refl->isInstance($source) && !$sourceProperty->isInitialized($source)) {
+                continue;
+            }
 
-                $value = $this->getSourceValue($source, $mappedTarget, $this->getRawValue($source, $propertyName), $objectMap);
+            $rawValue = $this->getRawValue($source, $propertyName);
+            if ($targetRefl->hasProperty($propertyName)) {
+                $value = $this->getSourceValue($source, $mappedTarget, $rawValue, $objectMap);
                 $this->storeValue($propertyName, $mapToProperties, $ctorArguments, $value);
+                continue;
+            }
+
+            if (
+                \is_object($rawValue)
+                && ($innerMetadata = $this->metadataFactory->create($rawValue))
+                && ($mapTo = $this->getMapTarget($innerMetadata, $rawValue, $source, $mappedTarget))
+                && \is_string($mapTo->target)
+                && $mapTo->target === $targetRefl->getName()
+            ) {
+                ($this->objectMapper ?? $this)->map($rawValue, $mappedTarget);
             }
         }
 

diff --git a/src/Symfony/Component/ObjectMapper/Tests/Fixtures/NestedMapping/NestedBankDataDto.php b/src/Symfony/Component/ObjectMapper/Tests/Fixtures/NestedMapping/NestedBankDataDto.php
@@ -0,0 +1,22 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\ObjectMapper\Tests\Fixtures\NestedMapping;
+
+use Symfony\Component\ObjectMapper\Attribute\Map;
+
+#[Map(target: NestedBankDataResource::class)]
+class NestedBankDataDto
+{
+    public string $iban;
+    public NestedBankDto $bank;
+}
+
diff --git a/src/Symfony/Component/ObjectMapper/Tests/Fixtures/NestedMapping/NestedBankDataResource.php b/src/Symfony/Component/ObjectMapper/Tests/Fixtures/NestedMapping/NestedBankDataResource.php
@@ -0,0 +1,21 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\ObjectMapper\Tests\Fixtures\NestedMapping;
+
+class NestedBankDataResource
+{
+    public string $iban;
+    public string $bic;
+    public string $bankCode;
+    public string $bankName;
+}
+
diff --git a/src/Symfony/Component/ObjectMapper/Tests/Fixtures/NestedMapping/NestedBankDto.php b/src/Symfony/Component/ObjectMapper/Tests/Fixtures/NestedMapping/NestedBankDto.php
@@ -0,0 +1,25 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\ObjectMapper\Tests\Fixtures\NestedMapping;
+
+use Symfony\Component\ObjectMapper\Attribute\Map;
+
+#[Map(target: NestedBankDataResource::class)]
+class NestedBankDto
+{
+    public string $bic;
+    #[Map(target: 'bankCode')]
+    public string $code;
+    #[Map(target: 'bankName')]
+    public string $name;
+}
+
diff --git a/src/Symfony/Component/ObjectMapper/Tests/ObjectMapperTest.php b/src/Symfony/Component/ObjectMapper/Tests/ObjectMapperTest.php
@@ -64,6 +64,9 @@
 use Symfony\Component\ObjectMapper\Tests\Fixtures\MultipleTargets\A as MultipleTargetsA;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\MultipleTargets\C as MultipleTargetsC;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\MyProxy;
+use Symfony\Component\ObjectMapper\Tests\Fixtures\NestedMapping\NestedBankDataDto;
+use Symfony\Component\ObjectMapper\Tests\Fixtures\NestedMapping\NestedBankDataResource;
+use Symfony\Component\ObjectMapper\Tests\Fixtures\NestedMapping\NestedBankDto;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\PartialInput\FinalInput;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\PartialInput\PartialInput;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\PromotedConstructor\Source as PromotedConstructorSource;
@@ -651,4 +654,25 @@ public function testBugReportLazyLoadingPromotedReadonlyProperty()
         $this->assertSame('foo', $out->var1);
         $this->assertSame('bar', $out->b->var2);
     }
+
+    public function testNestedBankDataMapping()
+    {
+        $bankDto = new NestedBankDto();
+        $bankDto->bic = 'BIC123';
+        $bankDto->code = 'BANK001';
+        $bankDto->name = 'Test Bank';
+
+        $bankDataDto = new NestedBankDataDto();
+        $bankDataDto->iban = 'IBAN12345';
+        $bankDataDto->bank = $bankDto;
+
+        $mapper = new ObjectMapper();
+        $bankDataResource = $mapper->map($bankDataDto, NestedBankDataResource::class);
+
+        $this->assertInstanceOf(NestedBankDataResource::class, $bankDataResource);
+        $this->assertSame('IBAN12345', $bankDataResource->iban);
+        $this->assertSame('BIC123', $bankDataResource->bic);
+        $this->assertSame('BANK001', $bankDataResource->bankCode);
+        $this->assertSame('Test Bank', $bankDataResource->bankName);
+    }
 }