#5863: list_display does not allow functions of referenced objects
-------------------------------------+-------------------------------------
Reporter: Beat | Owner: nobody
Bolli <me+django@…> | Status: closed
Type: | Component: contrib.admin
Uncategorized | Severity: Normal
Milestone: | Keywords: list_display
Version: SVN | Has patch: 1
Resolution: wontfix | Needs tests: 0
Triage Stage: Design |
decision needed |
Needs documentation: 1 |
Patch needs improvement: 1 |
-------------------------------------+-------------------------------------
Changes (by brillgen):
* cc: brillgen (added)
* type: => Uncategorized
* severity: => Normal
Comment:
@kmtracey:
We would have to use over a 100 extra boilerplate functions at last
estimate since callables is being stated as the only supported means of
getting direct fields from foreign keys...some of them are simply a text
field that needs to be displayed as such..I have 2 questions for the
community:
1. Isn't there a performance impact of using a callable (select_related)
benefit loss which could happen in case of simple FK fields required?
If there is no impact, its irrelevant because the boiler plate code is not
thaat much trouble
However, if there is (and I believe there would be),
2. Would the django admin accept a patch as long as it met the
requirements of test cases, documentation etc etc.
I ask because there would be nothing more frustrating than developing a
fully featured patch against trunk just to hear that it'll never get
accepted due to contrary design ideas :(
--
Ticket URL: <http://code.djangoproject.com/ticket/5863#comment:31>
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-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.
