diff --git a/src/Symfony/Component/Serializer/Normalizer/PropertyNormalizer.php b/src/Symfony/Component/Serializer/Normalizer/PropertyNormalizer.php
index 6a5d0acd8904d..fdd48b047a654 100644
--- a/src/Symfony/Component/Serializer/Normalizer/PropertyNormalizer.php
+++ b/src/Symfony/Component/Serializer/Normalizer/PropertyNormalizer.php
@@ -13,6 +13,7 @@
use Symfony\Component\PropertyAccess\Exception\UninitializedPropertyException;
use Symfony\Component\PropertyInfo\PropertyTypeExtractorInterface;
+use Symfony\Component\Serializer\Exception\LogicException;
use Symfony\Component\Serializer\Mapping\ClassDiscriminatorResolverInterface;
use Symfony\Component\Serializer\Mapping\Factory\ClassMetadataFactoryInterface;
use Symfony\Component\Serializer\NameConverter\NameConverterInterface;
@@ -202,7 +203,22 @@ protected function setAttributeValue(object $object, string $attribute, mixed $v
return;
}
- $reflectionProperty->setValue($object, $value);
+ if (!$reflectionProperty->isReadOnly()) {
+ $reflectionProperty->setValue($object, $value);
+
+ return;
+ }
+
+ if (!$reflectionProperty->isInitialized($object)) {
+ $declaringClass = $reflectionProperty->getDeclaringClass();
+ $declaringClass->getProperty($reflectionProperty->getName())->setValue($object, $value);
+
+ return;
+ }
+
+ if ($reflectionProperty->getValue($object) !== $value) {
+ throw new LogicException(\sprintf('Attempting to change readonly property "%s"::$%s.', $object::class, $reflectionProperty->getName()));
+ }
}
/**
diff --git a/src/Symfony/Component/Serializer/Tests/Fixtures/BookDummy.php b/src/Symfony/Component/Serializer/Tests/Fixtures/BookDummy.php
new file mode 100644
index 0000000000000..2b62667c0dbc3
--- /dev/null
+++ b/src/Symfony/Component/Serializer/Tests/Fixtures/BookDummy.php
@@ -0,0 +1,27 @@
+
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\Serializer\Tests\Fixtures;
+
+class BookDummy
+{
+ public function __construct(
+ public private(set) string $title,
+ public protected(set) string $author,
+ protected private(set) int $pubYear,
+ ) {
+ }
+
+ public function getPubYear(): int
+ {
+ return $this->pubYear;
+ }
+}
diff --git a/src/Symfony/Component/Serializer/Tests/Fixtures/ChildClassDummy.php b/src/Symfony/Component/Serializer/Tests/Fixtures/ChildClassDummy.php
new file mode 100644
index 0000000000000..f8d4f0743a4c4
--- /dev/null
+++ b/src/Symfony/Component/Serializer/Tests/Fixtures/ChildClassDummy.php
@@ -0,0 +1,17 @@
+
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\Serializer\Tests\Fixtures;
+
+readonly class ChildClassDummy extends ParentClassDummy
+{
+ public string $childProp;
+}
diff --git a/src/Symfony/Component/Serializer/Tests/Fixtures/ParentClassDummy.php b/src/Symfony/Component/Serializer/Tests/Fixtures/ParentClassDummy.php
new file mode 100644
index 0000000000000..730d71aecfec9
--- /dev/null
+++ b/src/Symfony/Component/Serializer/Tests/Fixtures/ParentClassDummy.php
@@ -0,0 +1,22 @@
+
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\Serializer\Tests\Fixtures;
+
+readonly class ParentClassDummy
+{
+ private string $parentProp;
+
+ public function getParentProp(): string
+ {
+ return $this->parentProp;
+ }
+}
diff --git a/src/Symfony/Component/Serializer/Tests/Fixtures/SpecialBookDummy.php b/src/Symfony/Component/Serializer/Tests/Fixtures/SpecialBookDummy.php
new file mode 100644
index 0000000000000..44679e2f205f0
--- /dev/null
+++ b/src/Symfony/Component/Serializer/Tests/Fixtures/SpecialBookDummy.php
@@ -0,0 +1,16 @@
+
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\Serializer\Tests\Fixtures;
+
+class SpecialBookDummy extends BookDummy
+{
+}
diff --git a/src/Symfony/Component/Serializer/Tests/Normalizer/PropertyNormalizerTest.php b/src/Symfony/Component/Serializer/Tests/Normalizer/PropertyNormalizerTest.php
index 0601a2d602084..386f0613d60ed 100644
--- a/src/Symfony/Component/Serializer/Tests/Normalizer/PropertyNormalizerTest.php
+++ b/src/Symfony/Component/Serializer/Tests/Normalizer/PropertyNormalizerTest.php
@@ -30,10 +30,12 @@
use Symfony\Component\Serializer\SerializerInterface;
use Symfony\Component\Serializer\Tests\Fixtures\Attributes\GroupDummy;
use Symfony\Component\Serializer\Tests\Fixtures\Attributes\GroupDummyChild;
+use Symfony\Component\Serializer\Tests\Fixtures\ChildClassDummy;
use Symfony\Component\Serializer\Tests\Fixtures\Dummy;
use Symfony\Component\Serializer\Tests\Fixtures\Php74Dummy;
use Symfony\Component\Serializer\Tests\Fixtures\PropertyCircularReferenceDummy;
use Symfony\Component\Serializer\Tests\Fixtures\PropertySiblingHolder;
+use Symfony\Component\Serializer\Tests\Fixtures\SpecialBookDummy;
use Symfony\Component\Serializer\Tests\Normalizer\Features\CacheableObjectAttributesTestTrait;
use Symfony\Component\Serializer\Tests\Normalizer\Features\CallbacksTestTrait;
use Symfony\Component\Serializer\Tests\Normalizer\Features\CircularReferenceTestTrait;
@@ -174,6 +176,39 @@ public function testDenormalize()
$this->assertEquals('bar', $obj->getBar());
}
+ /**
+ * @requires PHP 8.2
+ */
+ public function testDenormalizeWithReadOnlyClass()
+ {
+ /** @var ChildClassDummy $object */
+ $object = $this->normalizer->denormalize(
+ ['parentProp' => 'parentProp', 'childProp' => 'childProp'],
+ ChildClassDummy::class,
+ 'any'
+ );
+
+ $this->assertSame('parentProp', $object->getParentProp());
+ $this->assertSame('childProp', $object->childProp);
+ }
+
+ /**
+ * @requires PHP 8.4
+ */
+ public function testDenormalizeWithAsymmetricPropertyVisibility()
+ {
+ /** @var SpecialBookDummy $object */
+ $object = $this->normalizer->denormalize(
+ ['title' => 'life', 'author' => 'Santiago San Martin', 'pubYear' => 2000],
+ SpecialBookDummy::class,
+ 'any'
+ );
+
+ $this->assertSame('life', $object->title);
+ $this->assertSame('Santiago San Martin', $object->author);
+ $this->assertSame(2000, $object->getPubYear());
+ }
+
public function testNormalizeWithParentClass()
{
$group = new GroupDummyChild();
pFad - Phonifier reborn
Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.
Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.
Alternative Proxies:
Alternative Proxy
pFad Proxy
pFad v3 Proxy
pFad v4 Proxy