DDC-182: Changing entity in preUpdate hook tramples existing changes #225

New Issue

admin · 2026-01-22T12:31:30+01:00

admin commented

2026-01-22 12:31:30 +01:00

Originally created by @doctrinebot on GitHub (Nov 28, 2009).

Jira issue originally created by user amirabiri:

If the object is changed during the pre-update callback, this causes the previous changes to be discarded.

Steps to reproduce:

Create entity with pre-update lifecycle callback
Change property A,
Call flush.
Change property B in pre-update callback

Result is that changes to property A are discarded.

The reason for that is that at the end of Doctrine\ORM\UnitOfWork::_computeEntityChanges() these two statements appear:

$this->_entityChangeSets[$oid] = $changeSet;
$this->_originalEntityData[$oid] = $actualData;

However at the end of Doctrine\ORM\UnitOfWork::computeSingleEntityChangeSet(), which is called after the pre-update callbacks are called, the same thing happens:

$this->_entityChangeSets[$oid] = $changeSet;
$this->_originalEntityData[$oid] = $actualData;

Because the original data is recorded as equal to the state of the object after the first call to _computeEntityChanges(), the call to computeSingleEntityChangeSet() does not detect the original set of changes, it only detects the changes made in the pre-update hooks. But at the same time, both methods completely overwrite the changeset buffer for the entity rather than merge it. The result is that the first changeset is completely discarded.

Originally created by @doctrinebot on GitHub (Nov 28, 2009). Jira issue originally created by user amirabiri: If the object is changed during the pre-update callback, this causes the previous changes to be discarded. Steps to reproduce: - Create entity with pre-update lifecycle callback - Change property A, - Call flush. - Change property B in pre-update callback Result is that changes to property A are discarded. The reason for that is that at the end of Doctrine\ORM\UnitOfWork::_computeEntityChanges() these two statements appear: ``` $this->_entityChangeSets[$oid] = $changeSet; $this->_originalEntityData[$oid] = $actualData; ``` However at the end of Doctrine\ORM\UnitOfWork::computeSingleEntityChangeSet(), which is called after the pre-update callbacks are called, the same thing happens: ``` $this->_entityChangeSets[$oid] = $changeSet; $this->_originalEntityData[$oid] = $actualData; ``` Because the original data is recorded as equal to the state of the object after the first call to _computeEntityChanges(), the call to computeSingleEntityChangeSet() does not detect the original set of changes, it only detects the changes made in the pre-update hooks. But at the same time, both methods completely overwrite the changeset buffer for the entity rather than merge it. The result is that the first changeset is completely discarded.

admin added the Bug label 2026-01-22 12:31:30 +01:00

admin referenced this issue

2026-01-22 15:57:22 +01:00

[PR #225] [MERGED] [DDC-551] Fix testcase on pgsql #7882

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/archived-orm#225