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

… associations are not reset to empty collections
doctrine · Oct 27, 2017 · 9d21362 · 9d21362
1 parent c7cde75
commit 9d21362
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/Doctrine/ORM/UnitOfWork.php b/lib/Doctrine/ORM/UnitOfWork.php
@@ -2062,12 +2062,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);
     }
 
     /**