DDC-444: Deferred explicit policy doesn't update data #554

Closed
opened 2026-01-22 12:42:20 +01:00 by admin · 6 comments
Owner

Originally created by @doctrinebot on GitHub (Mar 19, 2010).

Jira issue originally created by user bjori:

When using the deferred explicit policy the entities no longer get updated and undefined indexes notices get thrown.

This seems to be caused by a typo in UnitOfWork::scheduleForDirtyCheck():
The _scheduledForDirtyCheck entries are expected to be an array of entities (see UOW::computeChangeSets() foreach($entitiesToProcess..)).

Attached patch fixes both issues :]

Originally created by @doctrinebot on GitHub (Mar 19, 2010). Jira issue originally created by user bjori: When using the deferred explicit policy the entities no longer get updated and undefined indexes notices get thrown. This seems to be caused by a typo in UnitOfWork::scheduleForDirtyCheck(): The _scheduledForDirtyCheck entries are expected to be an array of entities (see UOW::computeChangeSets() foreach($entitiesToProcess..)). Attached patch fixes both issues :]
admin added the Bug label 2026-01-22 12:42:20 +01:00
admin closed this issue 2026-01-22 12:42:21 +01:00
Author
Owner

@doctrinebot commented on GitHub (Mar 19, 2010):

Comment created by bjori:

Added a testcase

@doctrinebot commented on GitHub (Mar 19, 2010): Comment created by bjori: Added a testcase
Author
Owner

@doctrinebot commented on GitHub (Mar 19, 2010):

Comment created by @beberlei:

I think we need a specific testmodel for the non-default change tracking policies also.

@doctrinebot commented on GitHub (Mar 19, 2010): Comment created by @beberlei: I think we need a specific testmodel for the non-default change tracking policies also.
Author
Owner

@doctrinebot commented on GitHub (Mar 19, 2010):

Comment created by romanb:

Thanks. I will apply the patches and commit them as soon as I can.

@doctrinebot commented on GitHub (Mar 19, 2010): Comment created by romanb: Thanks. I will apply the patches and commit them as soon as I can.
Author
Owner

@doctrinebot commented on GitHub (Mar 20, 2010):

Comment created by romanb:

Should be fixed now. Thanks for the patches.

@doctrinebot commented on GitHub (Mar 20, 2010): Comment created by romanb: Should be fixed now. Thanks for the patches.
Author
Owner

@doctrinebot commented on GitHub (Mar 20, 2010):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Mar 20, 2010): Issue was closed with resolution "Fixed"
Author
Owner

@doctrinebot commented on GitHub (Dec 13, 2015):

Imported 2 attachments from Jira into https://gist.github.com/f48236ab9f321b1c3f4a

@doctrinebot commented on GitHub (Dec 13, 2015): Imported 2 attachments from Jira into https://gist.github.com/f48236ab9f321b1c3f4a - [10502_UOW.patch](https://gist.github.com/f48236ab9f321b1c3f4a#file-10502_UOW-patch) - [10503_ddc444.patch](https://gist.github.com/f48236ab9f321b1c3f4a#file-10503_ddc444-patch)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#554