mirror of
https://github.com/django/django.git
synced 2025-07-06 18:59:13 +00:00
queryset-refactor: Removed handle_legacy_orderlist() call from admin. Refs #245.
git-svn-id: http://code.djangoproject.com/svn/django/branches/queryset-refactor@7093 bcc190cf-cafb-0310-a4f2-bffc1f526a37
This commit is contained in:
parent
c23436be59
commit
0a07dac880
@ -8,7 +8,7 @@ from django.core.exceptions import ImproperlyConfigured, ObjectDoesNotExist, Per
|
|||||||
from django.core.paginator import ObjectPaginator, InvalidPage
|
from django.core.paginator import ObjectPaginator, InvalidPage
|
||||||
from django.shortcuts import get_object_or_404, render_to_response
|
from django.shortcuts import get_object_or_404, render_to_response
|
||||||
from django.db import models
|
from django.db import models
|
||||||
from django.db.models.query import handle_legacy_orderlist, QuerySet
|
from django.db.models.query import QuerySet
|
||||||
from django.http import Http404, HttpResponse, HttpResponseRedirect
|
from django.http import Http404, HttpResponse, HttpResponseRedirect
|
||||||
from django.utils.html import escape
|
from django.utils.html import escape
|
||||||
from django.utils.text import capfirst, get_text_list
|
from django.utils.text import capfirst, get_text_list
|
||||||
@ -653,15 +653,12 @@ class ChangeList(object):
|
|||||||
|
|
||||||
def get_ordering(self):
|
def get_ordering(self):
|
||||||
lookup_opts, params = self.lookup_opts, self.params
|
lookup_opts, params = self.lookup_opts, self.params
|
||||||
# For ordering, first check the "ordering" parameter in the admin options,
|
# For ordering, first check the "ordering" parameter in the admin
|
||||||
# then check the object's default ordering. If neither of those exist,
|
# options, then check the object's default ordering. If neither of
|
||||||
# order descending by ID by default. Finally, look for manually-specified
|
# those exist, order descending by ID by default. Finally, look for
|
||||||
# ordering from the query string.
|
# manually-specified ordering from the query string.
|
||||||
ordering = lookup_opts.admin.ordering or lookup_opts.ordering or ['-' + lookup_opts.pk.name]
|
ordering = lookup_opts.admin.ordering or lookup_opts.ordering or ['-' + lookup_opts.pk.name]
|
||||||
|
|
||||||
# Normalize it to new-style ordering.
|
|
||||||
ordering = handle_legacy_orderlist(ordering)
|
|
||||||
|
|
||||||
if ordering[0].startswith('-'):
|
if ordering[0].startswith('-'):
|
||||||
order_field, order_type = ordering[0][1:], 'desc'
|
order_field, order_type = ordering[0][1:], 'desc'
|
||||||
else:
|
else:
|
||||||
|
Loading…
x
Reference in New Issue
Block a user