[PR #617] [CLOSED] Wrong UnitOfWork::computeChangeSet() #8453

Closed
opened 2026-01-22 16:00:01 +01:00 by admin · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/doctrine/orm/pull/617
Author: @fchris82
Created: 3/16/2013
Status: Closed

Base: masterHead: master


📝 Commits (1)

  • b236d21 Check the compared values isn't Proxy.

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 lib/Doctrine/ORM/UnitOfWork.php (+1 -1)

📄 Description

Sometimes some fields are Proxy when compute "changeSet". If it is Proxy, some listeners - example Gedmo sortable listener - belive the value has changed and this leads to chaos.

I check the $actualValue, if it is Proxy, the value didn't change.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/doctrine/orm/pull/617 **Author:** [@fchris82](https://github.com/fchris82) **Created:** 3/16/2013 **Status:** ❌ Closed **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`b236d21`](https://github.com/doctrine/orm/commit/b236d2101fdc054b3542c8f175853b5404cc927e) Check the compared values isn't Proxy. ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `lib/Doctrine/ORM/UnitOfWork.php` (+1 -1) </details> ### 📄 Description Sometimes some fields are Proxy when compute "changeSet". If it is Proxy, some listeners - example Gedmo sortable listener - belive the value has changed and this leads to chaos. I check the $actualValue, if it is Proxy, the value didn't change. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
admin added the pull-request label 2026-01-22 16:00:01 +01:00
admin closed this issue 2026-01-22 16:00:01 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#8453