#11058: list_display_links doesn't allow callables not defined in the model
-------------------------------------------+--------------------------------
Reporter: dvine | Owner:
Status: new | Milestone:
Component: django.contrib.admin | Version: SVN
Resolution: | Keywords: list_display_links
Stage: Accepted | Has_patch: 1
Needs_docs: 0 | Needs_tests: 0
Needs_better_patch: 1 |
-------------------------------------------+--------------------------------
Comment (by acdha):
I have a github branch against Django 1.2.3 which includes working tests
and a doc update to clarify that anything which you put in `list_display`
can be used in `list_display_links`:
https://github.com/acdha/django/tree/ticket-11058-list_display_links
Here's the
[https://github.com/acdha/django/compare/1.2.3...ticket-11058-list_display_links
comparison] and
[https://github.com/acdha/django/compare/1.2.3...ticket-11058-list_display_links.patch
patch] against Django 1.2.3.
One note regarding tests: the old doctest hard-coded the error message
you'd get for a field which is not defined on the model, whether or not it
was otherwise valid. I believe `ImproperlyConfigured: …
list_display_links[0]'refers to 'non_existent_field' which is not defined
in 'list_display'` is now more appropriate and have updated the doctest to
match.
--
Ticket URL: <http://code.djangoproject.com/ticket/11058#comment:23>
Django <http://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-upda...@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.
