[PR #369] [MERGED] Show the advice only when relevant. #8091

Open
opened 2026-01-22 15:58:21 +01:00 by admin · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/doctrine/orm/pull/369
Author: @greg0ire
Created: 6/10/2012
Status: Merged
Merged: 6/18/2012
Merged by: @beberlei

Base: masterHead: master


📝 Commits (2)

  • 190e63a Show the advice only when necessary.
  • 111bb52 delimit ternary

📊 Changes

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

View changed files

📝 lib/Doctrine/ORM/ORMInvalidArgumentException.php (+5 -3)

📄 Description

This is indeed a good advice, but people who
already have implemented __toString() might
get confused about it.


🔄 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/369 **Author:** [@greg0ire](https://github.com/greg0ire) **Created:** 6/10/2012 **Status:** ✅ Merged **Merged:** 6/18/2012 **Merged by:** [@beberlei](https://github.com/beberlei) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (2) - [`190e63a`](https://github.com/doctrine/orm/commit/190e63a6bb995f0f17c4a13cd32014ceef08a369) Show the advice only when necessary. - [`111bb52`](https://github.com/doctrine/orm/commit/111bb52add924223ad51854c4b632037fc86305c) delimit ternary ### 📊 Changes **1 file changed** (+5 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `lib/Doctrine/ORM/ORMInvalidArgumentException.php` (+5 -3) </details> ### 📄 Description This is indeed a good advice, but people who already have implemented __toString() might get confused about it. --- <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:58:21 +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#8091