symfony · nicolas-grekas · Nov 21, 2025 · Nov 20, 2025
diff --git a/src/Symfony/Component/ObjectMapper/ObjectMapper.php b/src/Symfony/Component/ObjectMapper/ObjectMapper.php
@@ -150,10 +150,6 @@ private function doMap(object $source, object|string|null $target, \WeakMap $obj
                 }
 
                 $targetPropertyName = $mapping->target ?? $propertyName;
-                if (!$targetRefl->hasProperty($targetPropertyName)) {
-                    continue;
-                }
-
                 $value = $this->getSourceValue($source, $mappedTarget, $value, $objectMap, $mapping);
                 $this->storeValue($targetPropertyName, $mapToProperties, $ctorArguments, $value);
             }
@@ -186,7 +182,19 @@ private function doMap(object $source, object|string|null $target, \WeakMap $obj
         }
 
         foreach ($mapToProperties as $property => $value) {
-            $this->propertyAccessor ? $this->propertyAccessor->setValue($mappedTarget, $property, $value) : ($mappedTarget->{$property} = $value);
+            if ($this->propertyAccessor) {
+                if ($this->propertyAccessor->isWritable($mappedTarget, $property)) {
+                    $this->propertyAccessor->setValue($mappedTarget, $property, $value);
+                }
+
+                continue;
+            }
+
+            if (!$targetRefl->hasProperty($property)) {
+                continue;
+            }
+
+            $mappedTarget->{$property} = $value;
         }
 
         return $mappedTarget;

diff --git a/src/Symfony/Component/ObjectMapper/Tests/Fixtures/EmbeddedMapping/Address.php b/src/Symfony/Component/ObjectMapper/Tests/Fixtures/EmbeddedMapping/Address.php
@@ -0,0 +1,9 @@
+<?php
+
+namespace Symfony\Component\ObjectMapper\Tests\Fixtures\EmbeddedMapping;
+
+class Address
+{
+    public string $zipcode;
+    public string $city;
+}
diff --git a/src/Symfony/Component/ObjectMapper/Tests/Fixtures/EmbeddedMapping/User.php b/src/Symfony/Component/ObjectMapper/Tests/Fixtures/EmbeddedMapping/User.php
@@ -0,0 +1,14 @@
+<?php
+
+namespace Symfony\Component\ObjectMapper\Tests\Fixtures\EmbeddedMapping;
+
+class User
+{
+    public string $name;
+    public Address $address;
+
+    public function __construct()
+    {
+        $this->address = new Address();
+    }
+}
diff --git a/src/Symfony/Component/ObjectMapper/Tests/Fixtures/EmbeddedMapping/UserDto.php b/src/Symfony/Component/ObjectMapper/Tests/Fixtures/EmbeddedMapping/UserDto.php
@@ -0,0 +1,17 @@
+<?php
+
+namespace Symfony\Component\ObjectMapper\Tests\Fixtures\EmbeddedMapping;
+
+use Symfony\Component\ObjectMapper\Attribute\Map;
+
+class UserDto
+{
+    public function __construct(
+        #[Map(target: 'address.zipcode')]
+        public string $userAddressZipcode,
+        #[Map(target: 'address.city')]
+        public string $userAddressCity,
+        public string $name,
+    ) {
+    }
+}
diff --git a/src/Symfony/Component/ObjectMapper/Tests/ObjectMapperTest.php b/src/Symfony/Component/ObjectMapper/Tests/ObjectMapperTest.php
@@ -37,6 +37,9 @@
 use Symfony\Component\ObjectMapper\Tests\Fixtures\DefaultLazy\UserSource;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\DefaultLazy\UserTarget;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\DefaultValueStdClass\TargetDto;
+use Symfony\Component\ObjectMapper\Tests\Fixtures\EmbeddedMapping\Address;
+use Symfony\Component\ObjectMapper\Tests\Fixtures\EmbeddedMapping\User as UserEmbeddedMapping;
+use Symfony\Component\ObjectMapper\Tests\Fixtures\EmbeddedMapping\UserDto;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\Flatten\TargetUser;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\Flatten\User;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\Flatten\UserProfile;
@@ -611,4 +614,22 @@ public function testSkipLazyGhostWithClassTransform()
         $this->assertSame($result->relation, $service->get());
         $this->assertSame($result->relation->name, 'loaded');
     }
+
+    public function testMapEmbeddedProperties()
+    {
+        $dto = new UserDto(
+            userAddressZipcode: '12345',
+            userAddressCity: 'Test City',
+            name: 'John Doe'
+        );
+
+        $mapper = new ObjectMapper(propertyAccessor: PropertyAccess::createPropertyAccessor());
+        $user = $mapper->map($dto, UserEmbeddedMapping::class);
+
+        $this->assertInstanceOf(UserEmbeddedMapping::class, $user);
+        $this->assertSame('John Doe', $user->name);
+        $this->assertInstanceOf(Address::class, $user->address);
+        $this->assertSame('12345', $user->address->zipcode);
+        $this->assertSame('Test City', $user->address->city);
+    }
 }