#3275: [patch] select_related() additions (depth=N, fields=[])
------------------------------------------------+---------------------------
Reporter: David Cramer <[EMAIL PROTECTED]> | Owner: nobody
Status: new | Component:
Database wrapper
Version: SVN | Resolution:
Keywords: select_related() | Stage:
Accepted
Has_patch: 1 | Needs_docs: 1
Needs_tests: 1 | Needs_better_patch: 1
------------------------------------------------+---------------------------
Comment (by Gábor Farkas <[EMAIL PROTECTED]>):
unfortunately in this case i think improving the patch won't help.
there was a patch in ticket 4879, that had tests and everything, and it
was not committed.
as Malcom described here: http://groups.google.com/group/django-
developers/browse_thread/thread/eb3e315abdba5ce9/d0401b4d31717e70 ,
it will be fixed when the queryset-refactor branch is merged.
btw. my opinion is that this bug should get a little higher priority,
because without this patch select_related(depth) simply returns wrong
data,
but probably different priorities for different people :-)
p.s: please note, that the "patch needs improvement" flag is probably
meant for the other patches in this ticket. the fix_depth_bug.diff patch
is a one-line-patch,
which adds exactly one character. it's hard to imagine how such a patch
could be improved :)
--
Ticket URL: <http://code.djangoproject.com/ticket/3275#comment:20>
Django Code <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 [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
