search

flebel / django-tagging

Automatically exported from code.google.com/p/django-tagging
Other
0 stars 0 forks source link

queryset-refactor breaks advanced queries #118

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
The changes being made in queryset-refactor make 'order_by' an argument to 
extra() for cases 
when they're used together to create custom ordering; as such, uses of extra() 
in django-tagging 
which do custom ordering will need to chnage to use the 'order_by' argument to 
that method.

Original issue reported on code.google.com by ubernost...@gmail.com on 25 Apr 2008 at 4:13

GoogleCodeExporter commented 9 years ago 
Could you explain exactly where django-tagging is using order_by in conjunction 
with
an extra()? I can find no such instance in the code. There is some 'ORDER BY' 
sql
code, but that is all executed via the cursor.

(maybe these were fixed long ago, but looking back at the change logs I cant 
find it)

Much of the cursor code could now be replaces in part or totally by
extra(select=,where=,order_by=) calls, but there are still some django bugs 
which
need to first be fixed before I would risk making those changes.

Original comment by doug.nap...@gmail.com on 12 Aug 2008 at 4:44