Fix #4252 Cascade first before refreshing the entity itself so toMany…

… associations are not reset to empty collections
doctrine · Oct 30, 2017 · 9d9f684 · 9d9f684
1 parent a8465a3
commit 9d9f684
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/Doctrine/ORM/UnitOfWork.php b/lib/Doctrine/ORM/UnitOfWork.php
@@ -2095,12 +2095,12 @@ private function doRefresh($entity, array &$visited)
             throw ORMInvalidArgumentException::entityNotManaged($entity);
         }
 
+        $this->cascadeRefresh($entity, $visited);
+
         $this->getEntityPersister($class->name)->refresh(
             array_combine($class->getIdentifierFieldNames(), $this->entityIdentifiers[$oid]),
             $entity
         );
-
-        $this->cascadeRefresh($entity, $visited);
     }
 
     /**