Skip writing readonly properties if the value did not change

Author: derrabus

lib/Doctrine/ORM/Mapping/ClassMetadataInfo.php

@@ -701,7 +701,7 @@ class ClassMetadataInfo implements ClassMetadata
     /**
      * The ReflectionProperty instances of the mapped class.
      *
-     * @var ReflectionProperty[]|null[]
+     * @var array<string, ReflectionProperty|null>
      */
     public $reflFields = [];
 

lib/Doctrine/ORM/UnitOfWork.php

@@ -40,6 +40,7 @@
 use Doctrine\Persistence\PropertyChangedListener;
 use Exception;
 use InvalidArgumentException;
+use LogicException;
 use RuntimeException;
 use Throwable;
 use UnexpectedValueException;
@@ -53,6 +54,7 @@
 use function array_pop;
 use function array_sum;
 use function array_values;
+use function assert;
 use function count;
 use function current;
 use function get_class;
@@ -66,6 +68,8 @@
 use function spl_object_id;
 use function sprintf;
 
+use const PHP_VERSION_ID;
+
 /**
  * The UnitOfWork is responsible for tracking changes to objects during an
  * "object-level" transaction and for writing out changes to the database
@@ -2723,9 +2727,25 @@ public function createEntity($className, array $data, &$hints = [])
         }
 
         foreach ($data as $field => $value) {
-            if (isset($class->fieldMappings[$field])) {
-                $class->reflFields[$field]->setValue($entity, $value);
+            if (! isset($class->fieldMappings[$field])) {
+                continue;
             }
+
+            assert($class->reflFields[$field] !== null);
+
+            if (
+                PHP_VERSION_ID >= 80100
+                && $class->reflFields[$field]->isReadOnly()
+                && $class->reflFields[$field]->isInitialized($entity)
+            ) {
+                if ($class->reflFields[$field]->getValue($entity) === $value) {
+                    continue;
+                }
+
+                throw new LogicException(sprintf('Attempting to change readonly field %s.', $field));
+            }
+
+            $class->reflFields[$field]->setValue($entity, $value);
         }
 
         // Loading the entity right here, if its in the eager loading map get rid of it there.

phpstan-baseline.neon

@@ -1001,18 +1001,17 @@ parameters:
 			path: lib/Doctrine/ORM/Query/Parser.php
 
 		-
-			message:
-				"""
-					#^PHPDoc tag @return has invalid value \\(AST\\\\BetweenExpression\\|
-					        AST\\\\CollectionMemberExpression\\|
-					        AST\\\\ComparisonExpression\\|
-					        AST\\\\EmptyCollectionComparisonExpression\\|
-					        AST\\\\ExistsExpression\\|
-					        AST\\\\InExpression\\|
-					        AST\\\\InstanceOfExpression\\|
-					        AST\\\\LikeExpression\\|
-					        AST\\\\NullComparisonExpression\\)\\: Unexpected token "\\\\n     \\* ", expected type at offset 344$#
-				"""
+			message: """
+				#^PHPDoc tag @return has invalid value \\(AST\\\\BetweenExpression\\|
+				        AST\\\\CollectionMemberExpression\\|
+				        AST\\\\ComparisonExpression\\|
+				        AST\\\\EmptyCollectionComparisonExpression\\|
+				        AST\\\\ExistsExpression\\|
+				        AST\\\\InExpression\\|
+				        AST\\\\InstanceOfExpression\\|
+				        AST\\\\LikeExpression\\|
+				        AST\\\\NullComparisonExpression\\)\\: Unexpected token "\\\\n     \\* ", expected type at offset 344$#
+			"""
 			count: 1
 			path: lib/Doctrine/ORM/Query/Parser.php
 
@@ -1876,6 +1875,16 @@ parameters:
 			count: 1
 			path: lib/Doctrine/ORM/UnitOfWork.php
 
+		-
+			message: "#^Call to an undefined method ReflectionProperty\\:\\:isInitialized\\(\\)\\.$#"
+			count: 1
+			path: lib/Doctrine/ORM/UnitOfWork.php
+
+		-
+			message: "#^Call to an undefined method ReflectionProperty\\:\\:isReadOnly\\(\\)\\.$#"
+			count: 1
+			path: lib/Doctrine/ORM/UnitOfWork.php
+
 		-
 			message: "#^Parameter \\#1 \\$collection of method Doctrine\\\\ORM\\\\Persisters\\\\Collection\\\\CollectionPersister\\:\\:delete\\(\\) expects Doctrine\\\\ORM\\\\PersistentCollection, Doctrine\\\\Common\\\\Collections\\\\Collection\\<\\(int\\|string\\), object\\> given\\.$#"
 			count: 1

psalm-baseline.xml

@@ -3563,7 +3563,7 @@
       <code>$assoc['targetEntity']</code>
       <code>$assoc['type']</code>
     </PossiblyNullArrayAccess>
-    <PossiblyNullReference occurrences="33">
+    <PossiblyNullReference occurrences="32">
       <code>buildCachedCollectionPersister</code>
       <code>buildCachedEntityPersister</code>
       <code>getCacheFactory</code>