RetailCRM-Mirror/doctrine2
1
0
Fork 0
mirror of synced 2025-02-09 00:39:25 +03:00
Code Issues Projects Releases Wiki Activity

[DDC-2775] Bugfix

Browse Source
This commit is contained in:
Matthieu Napoli 2013-11-04 12:42:23 +01:00 committed by Benjamin Eberlei
parent 65bcdbf4c7
commit 3829b9c28b
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
lib/Doctrine/ORM/UnitOfWork.php
View File

@ -2233,6 +2233,10 @@ class UnitOfWork implements PropertyChangedListener
function ($assoc) { return $assoc['isCascadeRemove']; } function ($assoc) { return $assoc['isCascadeRemove']; }
); );
$entitiesToCascade = array();
// We need to load all related entities beforehand so that lazy collection loading doesn't
// reload entities after they have been removed (bug DDC-2775)
foreach ($associationMappings as $assoc) { foreach ($associationMappings as $assoc) {
if ($entity instanceof Proxy && !$entity->__isInitialized__) { if ($entity instanceof Proxy && !$entity->__isInitialized__) {
$entity->__load(); $entity->__load();
@ -2245,18 +2249,22 @@ class UnitOfWork implements PropertyChangedListener
case (is_array($relatedEntities)): case (is_array($relatedEntities)):
// If its a PersistentCollection initialization is intended! No unwrap! // If its a PersistentCollection initialization is intended! No unwrap!
foreach ($relatedEntities as $relatedEntity) { foreach ($relatedEntities as $relatedEntity) {
$this->doRemove($relatedEntity, $visited); $entitiesToCascade[] = $relatedEntity;
} }
break; break;
case ($relatedEntities !== null): case ($relatedEntities !== null):
$this->doRemove($relatedEntities, $visited); $entitiesToCascade[] = $relatedEntities;
break; break;
default: default:
// Do nothing // Do nothing
} }
} }
foreach ($entitiesToCascade as $relatedEntity) {
$this->doRemove($relatedEntity, $visited);
}
} }
/** /**