Opened 18 years ago
Closed 18 years ago
#5045 closed (duplicate)
Primary keys containing URL characters breaks admin links
Description ¶
The quote
function, used by the admin application to escape url characters, incorrectly checks for strings:
if type(s) != type(''): return s
This obviously fails now that Django can return a unicode string for field values, resulting in broken links for objects with url characters in their primary key.
Checking against basestring would seem to be one way to go:
if not isinstance(s, basestring): return s
...but perhaps the admin application should be changed to use django.utils.http.urlquote
since this performs a similar function?
Change History (3)
by , 18 years ago
Attachment: | admin-quote_unicode.patch added |
---|
comment:2 by , 18 years ago
Resolution: | → duplicate |
---|---|
Status: | new → closed |
Closing as a duplicate of #5041, since that incorporates this change and has been accepted.
Simple patch to correctly catch strings & unicode strings