On Jun 17, 8:17 pm, Rajesh D <rajesh.dha...@gmail.com> wrote:
> Hi Josh,
>
> On Jun 17, 10:25 am, Joshua Russo <joshua.rupp...@gmail.com> wrote:
>
> > Is there a generally accepted workaround for the problem of sorting
> > relational models by more than one field? I have a model that
> > represents recurring payments between a person and an object and it
> > would be really nice to order by the object, year, and creation date
> > in the admin app change list.
>
> You can also override the queryset method in your admin class to
> return a queryset that's sorted in the order that you need to display
> in the admin change list view:
>
> http://code.djangoproject.com/browser/django/trunk/django/contrib/adm...
>
> -Raj D
Ok, so that didn't work. It behaves exactly the same as if you set the
ordering instance variable for the admin or the sort order in the
model. The order is hijacked by the get_ordering() method of the
ChangeList object, and still only uses the first field.
Here is what I came up with for a solution. I made some "simple" mods
to the ChangeList object, the Result_Headers tag, and the change list
template.
#contrib\admin\views\main.py
class ChangeList(object):
def __init__(...):
...
# MODIFIED: Added self.ordering
self.order_field, self.order_type, self.ordering =
self.get_ordering()
...
def get_ordering(self):
...
# COMMENTED: The IF statement
#if ordering[0].startswith('-'):
# order_field, order_type = ordering[0][1:], 'desc'
#else:
# order_field, order_type = ordering[0], 'asc'
# ADDED: init order_field and order_type
order_field = None
order_type = None
...
# MODIFIED: Added the third return value
return order_field, order_type, ordering
def get_query_set(self):
...
# Set ordering.
if self.order_field:
qs = qs.order_by('%s%s' % ((self.order_type == 'desc' and
'-' or ''), self.order_field))
# ADDED: if the self.order_field is not set check the new
self.ordering value
elif self.ordering:
qs = qs.order_by(*self.ordering)
...
# contrib\admin\templatetags\admin_list.py
def result_headers(cl):
...
# MODIFIED: Added a check that the cl.order_field had been set
if cl.order_field != None and (field_name == cl.order_field or
admin_order_field == cl.order_field):
th_classes.append('sorted %sending' % cl.order_type.lower())
new_order_type = {'asc': 'desc', 'desc': 'asc'}
[cl.order_type.lower()]
...
# contrib\admin\templates\admin\change_list.html
{# Added an <li> item to object-tools block #}
<ul class="object-tools">
<li><a href=".">{% trans 'Reset sorting' %}{% if cl.has_filters %}
{% trans ' and filters' %}{% endif %}</a></li>
<li><a href="add/{% if is_popup %}?_popup=1{% endif %}"
class="addlink">{% blocktrans with cl.opts.verbose_name as name %}Add
{{ name }}{% endblocktrans %}</a></li>
</ul>
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django users" group.
To post to this group, send email to django-users@googlegroups.com
To unsubscribe from this group, send email to
django-users+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/django-users?hl=en
-~----------~----~----~----~------~----~------~--~---
