enumag
diff --git a/‎src/Symfony/Component/PropertyInfo/Extractor/ReflectionExtractor.php
Lines changed: 6 additions & 2 deletions b/‎src/Symfony/Component/PropertyInfo/Extractor/ReflectionExtractor.php
Lines changed: 6 additions & 2 deletions
diff --git a/‎src/Symfony/Component/PropertyInfo/Tests/Extractor/ReflectionExtractorTest.php
Lines changed: 12 additions & 0 deletions b/‎src/Symfony/Component/PropertyInfo/Tests/Extractor/ReflectionExtractorTest.php
Lines changed: 12 additions & 0 deletions
diff --git a/‎src/Symfony/Component/PropertyInfo/Tests/Fixtures/Php81Dummy.php
Lines changed: 4 additions & 0 deletions b/‎src/Symfony/Component/PropertyInfo/Tests/Fixtures/Php81Dummy.php
Lines changed: 4 additions & 0 deletions
@@ -180,7 +180,7 @@ public function isReadable($class, $property, array $context = []): ?bool
      */
     public function isWritable($class, $property, array $context = []): ?bool
     {
-        if ($this->isAllowedProperty($class, $property)) {
+        if ($this->isAllowedProperty($class, $property, true)) {
             return true;
         }
 
@@ -389,11 +389,15 @@ private function isNullableProperty(string $class, string $property): bool
         return false;
     }
 
-    private function isAllowedProperty(string $class, string $property): bool
+    private function isAllowedProperty(string $class, string $property, bool $writeAccessRequired = false): bool
     {
         try {
             $reflectionProperty = new \ReflectionProperty($class, $property);
 
+            if (\PHP_VERSION_ID >= 80100 && $writeAccessRequired && $reflectionProperty->isReadOnly()) {
+                return false;
+            }
+
             if ($this->accessFlags & self::ALLOW_PUBLIC && $reflectionProperty->isPublic()) {
                 return true;
             }
 
@@ -22,6 +22,7 @@
 use Symfony\Component\PropertyInfo\Tests\Fixtures\Php74Dummy;
 use Symfony\Component\PropertyInfo\Tests\Fixtures\Php7Dummy;
 use Symfony\Component\PropertyInfo\Tests\Fixtures\Php7ParentDummy;
+use Symfony\Component\PropertyInfo\Tests\Fixtures\Php81Dummy;
 use Symfony\Component\PropertyInfo\Type;
 
 /**
@@ -234,6 +235,7 @@ public function php71TypesProvider()
 
     /**
      * @dataProvider php80TypesProvider
+     *
      * @requires PHP 8
      */
     public function testExtractPhp80Type($property, array $type = null)
@@ -257,6 +259,7 @@ public function php80TypesProvider()
 
     /**
      * @dataProvider php81TypesProvider
+     *
      * @requires PHP 8.1
      */
     public function testExtractPhp81Type($property, array $type = null)
@@ -272,8 +275,17 @@ public function php81TypesProvider()
         ];
     }
 
+    /**
+     * @requires PHP 8.1
+     */
+    public function testReadonlyPropertiesAreNotWriteable()
+    {
+        $this->assertFalse($this->extractor->isWritable(Php81Dummy::class, 'foo'));
+    }
+
     /**
+     *
      * @requires PHP 8.2
      */
     public function testExtractPhp82Type($property, array $type = null)
 
@@ -13,6 +13,10 @@
 
 class Php81Dummy
 {
+    public function __construct(public readonly string $foo)
+    {
+    }
+
     public function getNothing(): never
     {
         throw new \Exception('Oops');
Original file line number	Diff line number	Diff line change
`@@ -180,7 +180,7 @@ public function isReadable($class, $property, array $context = []): ?bool`
`180`	`180`	`*/`
`181`	`181`	`public function isWritable($class, $property, array $context = []): ?bool`
`182`	`182`	`{`
`183`		`- if ($this->isAllowedProperty($class, $property)) {`
	`183`	`+ if ($this->isAllowedProperty($class, $property, true)) {`
`184`	`184`	`return true;`
`185`	`185`	`}`
`186`	`186`
`@@ -389,11 +389,15 @@ private function isNullableProperty(string $class, string $property): bool`
`389`	`389`	`return false;`
`390`	`390`	`}`
`391`	`391`
`392`		`- private function isAllowedProperty(string $class, string $property): bool`
	`392`	`+ private function isAllowedProperty(string $class, string $property, bool $writeAccessRequired = false): bool`
`393`	`393`	`{`
`394`	`394`	`try {`
`395`	`395`	`$reflectionProperty = new \ReflectionProperty($class, $property);`
`396`	`396`
	`397`	`+ if (\PHP_VERSION_ID >= 80100 && $writeAccessRequired && $reflectionProperty->isReadOnly()) {`
	`398`	`+ return false;`
	`399`	`+ }`
	`400`	`+`
`397`	`401`	`if ($this->accessFlags & self::ALLOW_PUBLIC && $reflectionProperty->isPublic()) {`
`398`	`402`	`return true;`
`399`	`403`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,10 @@`
`13`	`13`
`14`	`14`	`class Php81Dummy`
`15`	`15`	`{`
	`16`	`+ public function __construct(public readonly string $foo)`
	`17`	`+ {`
	`18`	`+ }`
	`19`	`+`
`16`	`20`	`public function getNothing(): never`
`17`	`21`	`{`
`18`	`22`	`throw new \Exception('Oops');`