#17198: In admin results can be omitted due to pagination and inadequate 
ordering
clauses
-------------------------------+------------------------------------
     Reporter:  lukeplant      |                    Owner:  julien
         Type:  Bug            |                   Status:  new
    Component:  contrib.admin  |                  Version:
     Severity:  Normal         |               Resolution:
     Keywords:                 |             Triage Stage:  Accepted
    Has patch:  1              |      Needs documentation:  0
  Needs tests:  0              |  Patch needs improvement:  0
Easy pickings:  0              |                    UI/UX:  0
-------------------------------+------------------------------------
Changes (by julien):

 * has_patch:  0 => 1


Comment:

 The attach patch ensures that 'pk' is systematically used as an ordering
 field in the changelist. Tests are also added for the related ticket
 #16819.

 It breaks an existing test though (`admin_views.
 AdminViewBasicTest.testChangeListSortingPreserveQuerySetOrdering`), which
 was added in r16316 to fix #7309. Luke, do you remember the logic for that
 test? It seems like it expects the default order to be '-pk', so I'd just
 like to check if it's ok to make it 'pk' instead.

 Thanks!

-- 
Ticket URL: <https://code.djangoproject.com/ticket/17198#comment:6>
Django <https://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-updates@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.

Reply via email to