#16257: contrib:admin dynamic list_display_links support
-------------------------------------+-------------------------------------
               Reporter:             |          Owner:  nobody
  graveyboat                         |         Status:  new
                   Type:  New        |      Component:  contrib.admin
  feature                            |       Severity:  Normal
              Milestone:             |       Keywords:  list_display
                Version:  1.3        |  override dynamic admin
             Resolution:             |      Has patch:  1
           Triage Stage:             |    Needs tests:  0
  Unreviewed                         |  Easy pickings:  0
    Needs documentation:  0          |
Patch needs improvement:  1          |
                  UI/UX:  0          |
-------------------------------------+-------------------------------------
Changes (by aaugustin):

 * needs_better_patch:  0 => 1


Comment:

 This looks useful.

 In the patch, it would be more readable to build `list_display_links =
 list(self.get_list_display_links(request, list_display))` outside the
 invocation of `ChangeList(...)`, just after building `list_display`.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/16257#comment:2>
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 [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Reply via email to