Fixed lazy-loading ghost objects generation with property hooks with default values.

cheack · cheack · commit 07d390662454 · 2025-05-14T12:41:00.000+02:00
diff --git a/src/Symfony/Component/VarExporter/ProxyHelper.php b/src/Symfony/Component/VarExporter/ProxyHelper.php
@@ -86,7 +86,7 @@ public static function generateLazyGhost(\ReflectionClass $class): string
                 .($p->isProtected() ? 'protected' : 'public')
                 .($p->isProtectedSet() ? ' protected(set)' : '')
                 ." {$type} \${$name}"
-                .($p->hasDefaultValue() ? ' = '.$p->getDefaultValue() : '')
+                .($p->hasDefaultValue() ? ' = '.VarExporter::export($p->getDefaultValue()) : '')
                 ." {\n";
 
             foreach ($p->getHooks() as $hook => $method) {
diff --git a/src/Symfony/Component/VarExporter/Tests/Fixtures/LazyProxy/HookedWithDefaultValue.php b/src/Symfony/Component/VarExporter/Tests/Fixtures/LazyProxy/HookedWithDefaultValue.php
@@ -4,8 +4,18 @@
 
 class HookedWithDefaultValue
 {
-    public int $backedWithDefault = 321 {
-        get => $this->backedWithDefault;
-        set => $this->backedWithDefault = $value;
+    public int $backedIntWithDefault = 321 {
+        get => $this->backedIntWithDefault;
+        set => $this->backedIntWithDefault = $value;
+    }
+
+    public string $backedStringWithDefault = '321' {
+        get => $this->backedStringWithDefault;
+        set => $this->backedStringWithDefault = $value;
+    }
+
+    public bool $backedBoolWithDefault = false {
+        get => $this->backedBoolWithDefault;
+        set => $this->backedBoolWithDefault = $value;
     }
 }
diff --git a/src/Symfony/Component/VarExporter/Tests/LegacyLazyGhostTraitTest.php b/src/Symfony/Component/VarExporter/Tests/LegacyLazyGhostTraitTest.php
@@ -332,16 +332,22 @@ public function testPropertyHooksWithDefaultValue()
             $initialized = true;
         });
 
-        $this->assertSame(321, $object->backedWithDefault);
+        $this->assertSame(321, $object->backedIntWithDefault);
+        $this->assertSame('321', $object->backedStringWithDefault);
+        $this->assertSame(false, $object->backedBoolWithDefault);
         $this->assertTrue($initialized);
 
         $initialized = false;
         $object = $this->createLazyGhost(HookedWithDefaultValue::class, function ($instance) use (&$initialized) {
             $initialized = true;
         });
-        $object->backedWithDefault = 654;
+        $object->backedIntWithDefault = 654;
+        $object->backedStringWithDefault = '654';
+        $object->backedBoolWithDefault = true;
         $this->assertTrue($initialized);
-        $this->assertSame(654, $object->backedWithDefault);
+        $this->assertSame(654, $object->backedIntWithDefault);
+        $this->assertSame('654', $object->backedStringWithDefault);
+        $this->assertSame(true, $object->backedBoolWithDefault);
     }
 
     /**
