DDC-3623: [GH-1337] Paginator OrderBy fix take 2 #4448

Closed
opened 2026-01-22 14:41:50 +01:00 by admin · 5 comments
Owner

Originally created by @doctrinebot on GitHub (Mar 17, 2015).

Originally assigned to: @Ocramius on GitHub.

Jira issue originally created by user @doctrinebot:

This issue is created automatically through a Github pull request on behalf of zeroedin-bill:

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

Message:

This PR fixes issues created in #1220, reported in #1267. The original PR was reverted in #1283.

The issue was that in queries ordered by joined association columns, the joined column aliases did not exist in the outer query created in LimitSubqueryOutputWalker.

This PR adds functionality to LimitSubqueryOutputWalker which finds any such columns referenced in the OrderBy clause, and adds them to the SelectStatement prior to SQL generation.

Originally created by @doctrinebot on GitHub (Mar 17, 2015). Originally assigned to: @Ocramius on GitHub. Jira issue originally created by user @doctrinebot: This issue is created automatically through a Github pull request on behalf of zeroedin-bill: Url: https://github.com/doctrine/doctrine2/pull/1337 Message: This PR fixes issues created in #1220, reported in #1267. The original PR was reverted in #1283. The issue was that in queries ordered by joined association columns, the joined column aliases did not exist in the outer query created in LimitSubqueryOutputWalker. This PR adds functionality to LimitSubqueryOutputWalker which finds any such columns referenced in the OrderBy clause, and adds them to the SelectStatement prior to SQL generation.
admin added the Bug label 2026-01-22 14:41:50 +01:00
admin closed this issue 2026-01-22 14:41:51 +01:00
Author
Owner

@doctrinebot commented on GitHub (Mar 17, 2015):

@doctrinebot commented on GitHub (Mar 17, 2015): - is required for [DDC-3606: [GH-1325] fixed PostgreSQL and Oracle pagination issues](http://www.doctrine-project.org/jira/browse/DDC-3606) - is required for [DDC-3629: [GH-1342] Paginator functional tests](http://www.doctrine-project.org/jira/browse/DDC-3629)
Author
Owner

@doctrinebot commented on GitHub (Mar 17, 2015):

Comment created by @doctrinebot:

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

@doctrinebot commented on GitHub (Mar 17, 2015): Comment created by @doctrinebot: A related Github Pull-Request [GH-1337] was labeled: https://github.com/doctrine/doctrine2/pull/1337
Author
Owner

@doctrinebot commented on GitHub (Mar 17, 2015):

Comment created by @doctrinebot:

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

@doctrinebot commented on GitHub (Mar 17, 2015): Comment created by @doctrinebot: A related Github Pull-Request [GH-1337] was labeled: https://github.com/doctrine/doctrine2/pull/1337
Author
Owner

@doctrinebot commented on GitHub (Mar 24, 2015):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Mar 24, 2015): Issue was closed with resolution "Fixed"
Author
Owner

@doctrinebot commented on GitHub (Mar 24, 2015):

Comment created by @doctrinebot:

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

@doctrinebot commented on GitHub (Mar 24, 2015): Comment created by @doctrinebot: A related Github Pull-Request [GH-1337] was assigned: https://github.com/doctrine/doctrine2/pull/1337
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#4448