#12841 closed (duplicate)
In admin options class, blank short_description causes "string index out of range" exception
Reported by: | Greg Brown | Owned by: | nobody |
---|---|---|---|
Component: | contrib.admin | Version: | 1.2-beta |
Severity: | Keywords: | ||
Cc: | Triage Stage: | Accepted | |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Example admin.py:
from models import MyModel from django.contrib import admin class MyModelAdmin(admin.ModelAdmin): list_display = ('name', 'my_custom_method', ) def my_custom_method(self, instance): return 'custom text' my_custom_method.short_description = '' admin.site.register(MyModel, MyModelAdmin)
Descripion
This admin classes changelist seems to throw a TemplateSyntaxError when trying to prettify the short description (which I'd have thought shouldn't really be happening; the point of short description in this context is to customise the column header, which by default is created by prettifying the function name)
The full traceback is
Original Traceback (most recent call last): File "/Library/Frameworks/Python.framework/Versions/2.5/lib/python2.5/site-packages/django/template/debug.py", line 72, in render_node result = node.render(context) File "/Library/Frameworks/Python.framework/Versions/2.5/lib/python2.5/site-packages/django/template/__init__.py", line 945, in render dict = func(*args) File "/Library/Frameworks/Python.framework/Versions/2.5/lib/python2.5/site-packages/django/contrib/admin/templatetags/admin_list.py", line 194, in result_list 'result_headers': list(result_headers(cl)), File "/Library/Frameworks/Python.framework/Versions/2.5/lib/python2.5/site-packages/django/contrib/admin/templatetags/admin_list.py", line 91, in result_headers header = pretty_name(header) File "/Library/Frameworks/Python.framework/Versions/2.5/lib/python2.5/site-packages/django/forms/forms.py", line 22, in pretty_name name = name[0].upper() + name[1:] IndexError: string index out of range
Attachments (1)
Change History (5)
comment:1 by , 15 years ago
Component: | Uncategorized → django.contrib.admin |
---|---|
milestone: | → 1.2 |
Triage Stage: | Unreviewed → Accepted |
by , 15 years ago
Attachment: | 12841.patch added |
---|
comment:2 by , 15 years ago
I've attached just a simple patch that uses string's method capitalize() instead.
Note:
See TracTickets
for help on using tickets.
Simple patch