[DDC-2322] [GH-592] Added ability to specify INDEX BY when creating a QueryBuilder from a Repository Created: 27/Feb/13 Updated: 12/Mar/13 Resolved: 12/Mar/13
|Project:||Doctrine 2 - ORM|
|Affects Version/s:||2.2, 2.3, Git Master|
|Reporter:||Benjamin Eberlei||Assignee:||Benjamin Eberlei|
This issue is created automatically through a Github pull request on behalf of cmenning:
This is a handy shortcut when using indexed results.
|Comment by Benjamin Eberlei [ 12/Mar/13 ]|
A related Github Pull-Request [GH-592] was closed
|Comment by Craig Menning [ 12/Mar/13 ]|
I disagree with the reasoning behind closing this request. In DoctrineBundle,
has been deprecated, which leaves you to use
Adding ->from() at that point is redundant since it's already been specified inside the createQueryBuilder() call. It seems wasteful to not allow a passthru of the $indexBy argument during the initial call when it's already supported by the from() called inside createQueryBuilder().
|Comment by Marco Pivetta [ 12/Mar/13 ]|
The repository is already (probably) doing too much. A simpler approach with:
Is no big deal in my opinion.