Store PersistentCollection changes in entity change set as array

The property $entityChangeSets of UnitOfWork stores entity changes
as an array of two items for each property changed. Before this
change, when an entity with a property holding a Collection was
changed, $entityChangeSets stored only the old Collection.

UPGRADE.md

@@ -141,6 +141,12 @@ Use one of the dedicated event classes instead:
 * `Doctrine\ORM\Event\PostRemoveEventArgs`
 * `Doctrine\ORM\Event\PostLoadEventArgs`
 
+## BC BREAK: Changed entity changeset when a `Collection` is modified
+
+When working with an entity with a `Collection` property, when that property
+is changed, the changeset stored in `UnitOfWork` includes the old and the new
+value of this property as an array of two items.
+
 ## BC BREAK: Removed `AttributeDriver::$entityAnnotationClasses` and `AttributeDriver::getReader()`
 
 * If you need to change the behavior of `AttributeDriver::isTransient()`,

lib/Doctrine/ORM/Event/PreUpdateEventArgs.php

@@ -5,7 +5,6 @@
 namespace Doctrine\ORM\Event;
 
 use Doctrine\ORM\EntityManagerInterface;
-use Doctrine\ORM\PersistentCollection;
 use Doctrine\Persistence\Event\LifecycleEventArgs;
 use InvalidArgumentException;
 
@@ -19,12 +18,12 @@
  */
 class PreUpdateEventArgs extends LifecycleEventArgs
 {
-    /** @var array<string, array{mixed, mixed}|PersistentCollection> */
+    /** @var array<string, array{mixed, mixed}> */
     private array $entityChangeSet;
 
     /**
      * @param mixed[][] $changeSet
-     * @psalm-param array<string, array{mixed, mixed}|PersistentCollection> $changeSet
+     * @psalm-param array<string, array{mixed, mixed}> $changeSet
      */
     public function __construct(object $entity, EntityManagerInterface $em, array &$changeSet)
     {
@@ -37,7 +36,7 @@ public function __construct(object $entity, EntityManagerInterface $em, array &$
      * Retrieves entity changeset.
      *
      * @return mixed[][]
-     * @psalm-return array<string, array{mixed, mixed}|PersistentCollection>
+     * @psalm-return array<string, array{mixed, mixed}>
      */
     public function getEntityChangeSet(): array
     {

lib/Doctrine/ORM/UnitOfWork.php

@@ -514,7 +514,7 @@ private function executeExtraUpdates(): void
      * Gets the changeset for an entity.
      *
      * @return mixed[][]
-     * @psalm-return array<string, array{mixed, mixed}|PersistentCollection>
+     * @psalm-return array<string, array{mixed, mixed}>
      */
     public function & getEntityChangeSet(object $entity): array
     {
@@ -715,7 +715,7 @@ public function computeChangeSet(ClassMetadata $class, object $entity): void
                     }
 
                     $this->collectionDeletions[$coid] = $orgValue;
-                    $changeSet[$propName]             = $orgValue; // Signal changeset, to-many assocs will be ignored.
+                    $changeSet[$propName]             = [$orgValue, $actualValue];
 
                     continue;
                 }

psalm-baseline.xml

@@ -1214,11 +1214,7 @@
       <code>$collectionToDelete</code>
       <code>$collectionToUpdate</code>
     </InvalidArgument>
-    <InvalidPropertyAssignmentValue>
-      <code><![CDATA[$this->entityChangeSets]]></code>
-      <code><![CDATA[$this->entityChangeSets]]></code>
-    </InvalidPropertyAssignmentValue>
-    <NoValue>
+    <NoValue occurrences="2">
       <code>$entityState</code>
       <code>$entityState</code>
     </NoValue>