Opened 17 years ago
Closed 17 years ago
#5045 closed (duplicate)
Primary keys containing URL characters breaks admin links
Reported by: | Owned by: | Adrian Holovaty | |
---|---|---|---|
Component: | contrib.admin | Version: | dev |
Severity: | Keywords: | unicode | |
Cc: | Triage Stage: | Unreviewed | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
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?
Attachments (1)
Change History (3)
by , 17 years ago
Attachment: | admin-quote_unicode.patch added |
---|
comment:2 by , 17 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