#14137: Django ORM adding unwanted parens when using extra()
------------------------------------+---------------------------------------
Reporter: mjs7231 | Owner: nobody
Status: closed | Milestone:
Component: Uncategorized | Version: 1.1
Resolution: invalid | Keywords:
Stage: Unreviewed | Has_patch: 0
Needs_docs: 0 | Needs_tests: 0
Needs_better_patch: 0 |
------------------------------------+---------------------------------------
Changes (by lrekucki):
* status: new => closed
* resolution: => invalid
Comment:
The "select" keyword in `extra()` should be used to select extra columns
which can't be expressed in ORM (like complex expressions, using stored
procedures, stuff like that) not to inject arbitrary SQL into a query. A
better solution would be to support field names in distinct(), but that's
a diffrent thing. After all, there is no reason why the ORM wouldn't put
extra select columns on the end, which would also break your code.
--
Ticket URL: <http://code.djangoproject.com/ticket/14137#comment:2>
Django <http://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to django-upda...@googlegroups.com.
To unsubscribe from this group, send email to
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en.
