[ObjectMapper] initialize lazy objects

soyuka · soyuka · commit 1c820707e53e · 2025-07-17T20:18:17.000+02:00
diff --git a/src/Symfony/Component/ObjectMapper/ObjectMapper.php b/src/Symfony/Component/ObjectMapper/ObjectMapper.php
@@ -20,6 +20,7 @@
 use Symfony\Component\ObjectMapper\Metadata\ReflectionObjectMapperMetadataFactory;
 use Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException as PropertyAccessorNoSuchPropertyException;
 use Symfony\Component\PropertyAccess\PropertyAccessorInterface;
+use Symfony\Component\VarExporter\LazyObjectInterface;
 
 /**
  * Object to object mapper.
@@ -300,6 +301,12 @@ private function getCallable(string|callable $fn, ?ContainerInterface $locator =
      */
     private function getSourceReflectionClass(object $source, \ReflectionClass $targetRefl): \ReflectionClass
     {
+        if ($source instanceof LazyObjectInterface) {
+            $source->initializeLazyObject();
+        } elseif (\PHP_VERSION_ID >= 80400) {
+            (new \ReflectionClass($source))->initializeLazyObject($source);
+        }
+
         $metadata = $this->metadataFactory->create($source);
         try {
             $refl = new \ReflectionClass($source);
diff --git a/src/Symfony/Component/ObjectMapper/Tests/Fixtures/LazyFoo.php b/src/Symfony/Component/ObjectMapper/Tests/Fixtures/LazyFoo.php
@@ -0,0 +1,35 @@
+<?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;
+
+use Symfony\Component\VarExporter\Internal\LazyDecoratorTrait;
+use Symfony\Component\VarExporter\LazyObjectInterface;
+
+class LazyFoo extends \stdClass implements LazyObjectInterface
+{
+    use LazyDecoratorTrait;
+
+    public string $name = 'foo';
+    private ?bool $initialized;
+
+    public function initializeLazyObject(): static
+    {
+        $this->initialized = true;
+
+        return $this;
+    }
+
+    public function getInitialized(): ?bool
+    {
+        return $this->initialized;
+    }
+}
diff --git a/src/Symfony/Component/ObjectMapper/Tests/Fixtures/MyProxy.php b/src/Symfony/Component/ObjectMapper/Tests/Fixtures/MyProxy.php
@@ -0,0 +1,17 @@
+<?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;
+
+class MyProxy
+{
+    public string $name;
+}
diff --git a/src/Symfony/Component/ObjectMapper/Tests/ObjectMapperTest.php b/src/Symfony/Component/ObjectMapper/Tests/ObjectMapperTest.php
@@ -40,6 +40,7 @@
 use Symfony\Component\ObjectMapper\Tests\Fixtures\InstanceCallback\B as InstanceCallbackB;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\InstanceCallbackWithArguments\A as InstanceCallbackWithArgumentsA;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\InstanceCallbackWithArguments\B as InstanceCallbackWithArgumentsB;
+use Symfony\Component\ObjectMapper\Tests\Fixtures\LazyFoo;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\MapStruct\AToBMapper;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\MapStruct\MapStructMapperMetadataFactory;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\MapStruct\Source;
@@ -51,6 +52,7 @@
 use Symfony\Component\ObjectMapper\Tests\Fixtures\MultipleTargetProperty\C as MultipleTargetPropertyC;
 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\Recursion\AB;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\Recursion\Dto;
 use Symfony\Component\ObjectMapper\Tests\Fixtures\ServiceLocator\A as ServiceLocatorA;
@@ -345,4 +347,37 @@ public function testDefaultValueStdClassWithPropertyInfo()
         $this->assertSame('abc', $b->id);
         $this->assertNull($b->optional);
     }
+
+    public function testMapInitializesLazyObject()
+    {
+        $lazy = new LazyFoo();
+        $mapper = new ObjectMapper();
+        $mapper->map($lazy, \stdClass::class);
+        $this->assertTrue($lazy->getInitialized());
+    }
+
+    public function testMapInitializesNativePhp84LazyObject()
+    {
+        if (\PHP_VERSION_ID < 80400) {
+            $this->markTestSkipped('Native lazy objects require PHP 8.4 or higher.');
+        }
+
+        $initialized = false;
+        $initializer = function () use (&$initialized) {
+            $initialized = true;
+
+            $p = new MyProxy();
+            $p->name = 'test';
+
+            return $p;
+        };
+
+        $r = new \ReflectionClass(MyProxy::class);
+        $lazyObj = $r->newLazyProxy($initializer);
+        $this->assertFalse($initialized);
+        $mapper = new ObjectMapper();
+        $d = $mapper->map($lazyObj, MyProxy::class);
+        $this->assertSame('test', $d->name);
+        $this->assertTrue($initialized);
+    }
 }
diff --git a/src/Symfony/Component/ObjectMapper/composer.json b/src/Symfony/Component/ObjectMapper/composer.json
@@ -20,7 +20,8 @@
         "psr/container": "^2.0"
     },
     "require-dev": {
-        "symfony/property-access": "^7.2"
+        "symfony/property-access": "^7.2",
+        "symfony/var-exporter": "^7.2"
     },
     "autoload": {
         "psr-4": {
