#30259: Support the use of admin_order_field on properties in the admin.
---------------------------------+----------------------------------------
     Reporter:  Tobias Wiese     |                    Owner:  Jani Tiainen
         Type:  New feature      |                   Status:  new
    Component:  contrib.admin    |                  Version:  master
     Severity:  Release blocker  |               Resolution:
     Keywords:                   |             Triage Stage:  Accepted
    Has patch:  0                |      Needs documentation:  0
  Needs tests:  0                |  Patch needs improvement:  0
Easy pickings:  0                |                    UI/UX:  0
---------------------------------+----------------------------------------

Comment (by Jani Tiainen):

 Replying to [comment:7 Tim Graham]:
 > A selenium test fails after the patch:
 `admin_views.tests.SeleniumTests.test_list_editable_raw_id_fields`:
 > {{{
 >   File
 "/home/tim/code/django/django/contrib/admin/templatetags/admin_list.py",
 line 133, in result_headers
 >     admin_order_field = getattr(attr.fget, 'admin_order_field')
 > AttributeError: 'function' object has no attribute 'admin_order_field'
 > }}}

 Apparently that `getattr` should be:
 {{{
 admin_order_field = getattr(attr.fget, 'admin_order_field', None)
 }}}

-- 
Ticket URL: <https://code.djangoproject.com/ticket/30259#comment:8>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/069.f7a3072cb737d3c1ccc058c213df8d7a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to