[PR #148] [MERGED] Do not load entity on retrieving identifier from a proxy #7780

Closed
opened 2026-01-22 15:56:47 +01:00 by admin · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/doctrine/orm/pull/148
Author: @asm89
Created: 10/15/2011
Status: Merged
Merged: 10/15/2011
Merged by: @guilhermeblanco

Base: masterHead: ProxyIdentifer


📝 Commits (4)

  • bc4e14a Prototype for a proxy extension that avoids loads when calling for a getter that is named after an identifier.
  • f47e1fe Merge branch 'master' of git://github.com/doctrine/doctrine2 into ProxyIdentifer
  • 8d1b852 Added tests for not loading the entity + fixed a test
  • d46352d Fixed tests + added dedicated tests for proxy loading and identifiers

📊 Changes

6 files changed (+64 additions, -6 deletions)

View changed files

📝 lib/Doctrine/ORM/Proxy/ProxyFactory.php (+20 -0)
📝 tests/Doctrine/Tests/ORM/Functional/LifecycleCallbackTest.php (+6 -2)
📝 tests/Doctrine/Tests/ORM/Functional/ReferenceProxyTest.php (+24 -0)
📝 tests/Doctrine/Tests/ORM/Functional/Ticket/DDC1238Test.php (+4 -0)
📝 tests/Doctrine/Tests/ORM/Functional/Ticket/DDC381Test.php (+7 -2)
📝 tests/Doctrine/Tests/ORM/Proxy/ProxyClassGeneratorTest.php (+3 -2)

📄 Description

Fork of the branch created by beberlei. Merged master + added and fixed tests.


🔄 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/148 **Author:** [@asm89](https://github.com/asm89) **Created:** 10/15/2011 **Status:** ✅ Merged **Merged:** 10/15/2011 **Merged by:** [@guilhermeblanco](https://github.com/guilhermeblanco) **Base:** `master` ← **Head:** `ProxyIdentifer` --- ### 📝 Commits (4) - [`bc4e14a`](https://github.com/doctrine/orm/commit/bc4e14a99f26338e83c1e643293074c064bc5bdd) Prototype for a proxy extension that avoids loads when calling for a getter that is named after an identifier. - [`f47e1fe`](https://github.com/doctrine/orm/commit/f47e1feac69a03bbac09933e22b0880b4895bf0f) Merge branch 'master' of git://github.com/doctrine/doctrine2 into ProxyIdentifer - [`8d1b852`](https://github.com/doctrine/orm/commit/8d1b852aa22569fc62c0b9a3ea9b13713b08de84) Added tests for not loading the entity + fixed a test - [`d46352d`](https://github.com/doctrine/orm/commit/d46352da0106278ce3b271bca907a381254c42d8) Fixed tests + added dedicated tests for proxy loading and identifiers ### 📊 Changes **6 files changed** (+64 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `lib/Doctrine/ORM/Proxy/ProxyFactory.php` (+20 -0) 📝 `tests/Doctrine/Tests/ORM/Functional/LifecycleCallbackTest.php` (+6 -2) 📝 `tests/Doctrine/Tests/ORM/Functional/ReferenceProxyTest.php` (+24 -0) 📝 `tests/Doctrine/Tests/ORM/Functional/Ticket/DDC1238Test.php` (+4 -0) 📝 `tests/Doctrine/Tests/ORM/Functional/Ticket/DDC381Test.php` (+7 -2) 📝 `tests/Doctrine/Tests/ORM/Proxy/ProxyClassGeneratorTest.php` (+3 -2) </details> ### 📄 Description Fork of the branch created by beberlei. Merged master + added and fixed tests. --- <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 15:56:47 +01:00
admin closed this issue 2026-01-22 15:56:48 +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#7780