[PR #1082] [MERGED] added more informative error messages when invalid parameter count #9131

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

📋 Pull Request Information

Original PR: https://github.com/doctrine/orm/pull/1082
Author: @rodnaph
Created: 7/15/2014
Status: Merged
Merged: 7/15/2014
Merged by: @guilhermeblanco

Base: masterHead: invalid-parameter-count


📝 Commits (1)

  • ad10a18 added more informative error messages when invalid parameter count

📊 Changes

3 files changed (+36 additions, -6 deletions)

View changed files

📝 lib/Doctrine/ORM/Query.php (+6 -2)
📝 lib/Doctrine/ORM/Query/QueryException.php (+16 -2)
📝 tests/Doctrine/Tests/ORM/Functional/QueryTest.php (+14 -2)

📄 Description

When an invalid number of parameters is bound current it's not indicated if there are too few or too many (which could help a lot in trying to debug the problem). This change alters the exception message to be more informative as to whether too few or too many parameters have been bound in the query.


🔄 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/1082 **Author:** [@rodnaph](https://github.com/rodnaph) **Created:** 7/15/2014 **Status:** ✅ Merged **Merged:** 7/15/2014 **Merged by:** [@guilhermeblanco](https://github.com/guilhermeblanco) **Base:** `master` ← **Head:** `invalid-parameter-count` --- ### 📝 Commits (1) - [`ad10a18`](https://github.com/doctrine/orm/commit/ad10a18071f9f6b01a2ae56590695ddd54876786) added more informative error messages when invalid parameter count ### 📊 Changes **3 files changed** (+36 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `lib/Doctrine/ORM/Query.php` (+6 -2) 📝 `lib/Doctrine/ORM/Query/QueryException.php` (+16 -2) 📝 `tests/Doctrine/Tests/ORM/Functional/QueryTest.php` (+14 -2) </details> ### 📄 Description When an invalid number of parameters is bound current it's not indicated if there are too few or too many (which could help a lot in trying to debug the problem). This change alters the exception message to be more informative as to whether too few or too many parameters have been bound in the query. --- <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:03:20 +01:00
admin closed this issue 2026-01-22 16:03:20 +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#9131