DDC-2685: [GH-793] Improved error messages in Paginator: LimitSubqueryOutputWalker #3366

Closed
opened 2026-01-22 14:18:39 +01:00 by admin · 3 comments
Owner

Originally created by @doctrinebot on GitHub (Sep 17, 2013).

Originally assigned to: @guilhermeblanco on GitHub.

Jira issue originally created by user @doctrinebot:

This issue is created automatically through a Github pull request on behalf of flip111:

Url: https://github.com/doctrine/doctrine2/pull/793

Message:

The already existing follow-up check throwing "Not all identifier properties can be found in the ResultSetMapping: %s" is a much more technical description.

If $sqlIdentifier is empty (count 0) it can be concluded that there are no mapped results. An alternative error message could thus be "The Paginator does not support queries without mapped results." (line 153).

Possibly '!=' on line 154 could be replaced by '>' because i think this is the only situation that can occur. But since i don't know this for sure, i left it like i found it.

Originally created by @doctrinebot on GitHub (Sep 17, 2013). Originally assigned to: @guilhermeblanco on GitHub. Jira issue originally created by user @doctrinebot: This issue is created automatically through a Github pull request on behalf of flip111: Url: https://github.com/doctrine/doctrine2/pull/793 Message: The already existing follow-up check throwing "Not all identifier properties can be found in the ResultSetMapping: %s" is a much more technical description. If $sqlIdentifier is empty (count 0) it can be concluded that there are no mapped results. An alternative error message could thus be "The Paginator does not support queries without mapped results." (line 153). Possibly '!=' on line 154 could be replaced by '>' because i think this is the only situation that can occur. But since i don't know this for sure, i left it like i found it.
admin added the Bug label 2026-01-22 14:18:39 +01:00
admin closed this issue 2026-01-22 14:18:39 +01:00
Author
Owner

@doctrinebot commented on GitHub (Sep 30, 2013):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-793] was closed:
https://github.com/doctrine/doctrine2/pull/793

@doctrinebot commented on GitHub (Sep 30, 2013): Comment created by @doctrinebot: A related Github Pull-Request [GH-793] was closed: https://github.com/doctrine/doctrine2/pull/793
Author
Owner

@doctrinebot commented on GitHub (Oct 18, 2013):

Comment created by @ocramius:

Merged: d8335eee7b

@doctrinebot commented on GitHub (Oct 18, 2013): Comment created by @ocramius: Merged: https://github.com/doctrine/doctrine2/commit/d8335eee7b1e8e8c3bdd11c30d81fe96d80f4510
Author
Owner

@doctrinebot commented on GitHub (Oct 18, 2013):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Oct 18, 2013): Issue was closed with resolution "Fixed"
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#3366