id,summary,reporter,owner,description,type,status,component,version,severity,resolution,keywords,cc,stage,has_patch,needs_docs,needs_tests,needs_better_patch,easy,ui_ux 5045,Primary keys containing URL characters breaks admin links,robbie@…,Adrian Holovaty,"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? ",,closed,contrib.admin,dev,,duplicate,unicode,,Unreviewed,1,0,0,0,0,0