Details
-
Type:
Improvement
-
Status:
Closed
-
Priority:
Minor
-
Resolution: Won't Fix
-
Affects Version/s: None
-
Fix Version/s: None
-
Component/s: DQL
-
Security Level: All
-
Labels:None
Description
Currently, QueryBuilder allows assembling broken DQL like
SELECT u FROM User u JOIN u.posts p JOIN u.logs p
or
SELECT u FROM User u JOIN u.posts u
It may be interesting if we either replace DQL parts by associating them to their alias.
Another solution, since Alexander noted that it is a dumb string builder, is to throw exceptions early to disallow broken DQL generation (such as in the second case).
It may also be useful to allow giving insights about the currently existing joined aliases.
public function hasJoinedAlias($alias);
Activity
Marco Pivetta
made changes -
| Field | Original Value | New Value |
|---|---|---|
| Attachment | DDC2046Test.php [ 11328 ] |
Marco Pivetta
made changes -
| Status | Open [ 1 ] | Closed [ 6 ] |
| Resolution | Won't Fix [ 2 ] |
This list may be incomplete, as errors occurred whilst retrieving source from linked applications:
- Request to http://www.doctrine-project.org/fisheye/ failed: Error in remote call to 'FishEye 0 (http://www.doctrine-project.org/fisheye/)' (http://www.doctrine-project.org/fisheye) [AbstractRestCommand{path='/rest-service-fe/search-v1/crossRepositoryQuery', params={query=DDC-2046, expand=changesets[-21:-1].revisions[0:29],reviews}, methodType=GET}] : Received status code 503 (Service Temporarily Unavailable)
Attaching failing test case