DDC-1426: Github-PR-115 by mpdeimos: findByXXX does not accept orderBy/limit/offset #1789

Closed
opened 2026-01-22 13:26:00 +01:00 by admin · 2 comments
Owner

Originally created by @doctrinebot on GitHub (Oct 17, 2011).

Originally assigned to: @asm89 on GitHub.

Jira issue originally created by user @guilhermeblanco:

Our modification to pass findByXXX functions orderBy/limit/offset params for entity repositories like one is used from normal findBy function.

We're using an if cascade in favor of the call_user_function_array since the latter is considered as slow.

PS: If i need to target this pull request to your master, I'll resubmit - it's just that we're building against a stable codebase.

Reference link: https://github.com/doctrine/doctrine2/pull/115

Originally created by @doctrinebot on GitHub (Oct 17, 2011). Originally assigned to: @asm89 on GitHub. Jira issue originally created by user @guilhermeblanco: Our modification to pass findByXXX functions orderBy/limit/offset params for entity repositories like one is used from normal findBy function. We're using an if cascade in favor of the call_user_function_array since the latter is considered as slow. PS: If i need to target this pull request to your master, I'll resubmit - it's just that we're building against a stable codebase. Reference link: https://github.com/doctrine/doctrine2/pull/115
admin added the Improvement label 2026-01-22 13:26:00 +01:00
admin closed this issue 2026-01-22 13:26:01 +01:00
Author
Owner

@doctrinebot commented on GitHub (Mar 11, 2012):

Comment created by @asm89:

Added in master. See here: https://github.com/doctrine/doctrine2/pull/115.

@doctrinebot commented on GitHub (Mar 11, 2012): Comment created by @asm89: Added in master. See here: https://github.com/doctrine/doctrine2/pull/115.
Author
Owner

@doctrinebot commented on GitHub (Mar 11, 2012):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Mar 11, 2012): 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#1789