Ticket #1375: string_pkey_in_admin.patch

django/contrib/admin/models.py

@@ -4 +4 @@
-from django.utils.translation import ugettext_lazy as _
-from django.utils.encoding import smart_unicode
-from django.utils.safestring import mark_safe
+from urllib import quote, unquote
-
-ADDITION = 1
-CHANGE = 2
-DELETION = 3
-
+def pk_url_quote(s):
+    """
+    Ensure that primary key values do not confuse the admin URLs by escaping
+    any special URL characters. Similar to urllib.quote, except that the
+    quoting is slightly different so that it doesn't get automatically
+    unquoted by the Web browser.
+    """
+    if type(s) != str and type(s) != unicode:
+        return s
+    res = list(s)
+    for i in range(len(res)):
+        c = res[i]
+        if c in ':/.#?@=&;':
+            res[i] = '.%02X' % ord(c)
+    return ''.join(res)
+
+def pk_url_unquote(s):
+    """
+    Undo the effects of pk_url_quote(). Based heavily on urllib.unquote().
+    """
+    mychr = chr
+    myatoi = int
+    list = s.split('.')
+    res = [list[0]]
+    myappend = res.append
+    del list[0]
+    for item in list:
+        if item[1:2]:
+            try:
+                myappend(mychr(myatoi(item[:2], 16)) + item[2:])
+            except ValueError:
+                myappend('.' + item)
+        else:
+            myappend('.' + item)
+    return "".join(res)
+
-class LogEntryManager(models.Manager):
-    def log_action(self, user_id, content_type_id, object_id, object_repr, action_flag, change_message=''):
-        e = self.model(None, None, user_id, content_type_id, smart_unicode(object_id), object_repr[:200], action_flag, change_message)
@@ -50 +87 @@
-        Returns the admin URL to edit the object represented by this log entry.
-        This is relative to the Django admin index page.
-        """
-self.object_id
+quote(pk_url_quote(self.object_id))

django/contrib/admin/templatetags/admin_list.py

@@ -11 +11 @@
-from django.utils.encoding import smart_unicode, smart_str, force_unicode
-from django.template import Library
-import datetime
+from urllib import quote
-
-register = Library()
-
@@ -197 +198 @@
-            # Problem cases are long ints (23L) and non-ASCII strings.
-            result_id = repr(force_unicode(getattr(result, pk)))[1:]
-            yield mark_safe(u'<%s%s><a href="%s"%s>%s</a></%s>' % \
-url
+quote(url)
-        else:
-            yield mark_safe(u'<td%s>%s</td>' % (row_class, conditional_escape(result_repr)))
-

django/contrib/admin/views/main.py

@@ -22 +22 @@
-except NameError:
-    from sets import Set as set   # Python 2.3 fallback
-
-
+, pk_url_quote, pk_url_unquote
-if not LogEntry._meta.installed:
-    raise ImproperlyConfigured, "You'll need to put 'django.contrib.admin' in your INSTALLED_APPS setting before you can use the admin application."
-
@@ -50 +50 @@
-class IncorrectLookupParameters(Exception):
-    pass
-
-def quote(s):
-    """
-    Ensure that primary key values do not confuse the admin URLs by escaping
-    any '/', '_' and ':' characters. Similar to urllib.quote, except that the
-    quoting is slightly different so that it doesn't get automatically
-    unquoted by the Web browser.
-    """
-    if type(s) != type(''):
-        return s
-    res = list(s)
-    for i in range(len(res)):
-        c = res[i]
-        if c in ':/_':
-            res[i] = '_%02X' % ord(c)
-    return ''.join(res)
-
-def unquote(s):
-    """
-    Undo the effects of quote(). Based heavily on urllib.unquote().
-    """
-    mychr = chr
-    myatoi = int
-    list = s.split('_')
-    res = [list[0]]
-    myappend = res.append
-    del list[0]
-    for item in list:
-        if item[1:2]:
-            try:
-                myappend(mychr(myatoi(item[:2], 16)) + item[2:])
-            except ValueError:
-                myappend('_' + item)
-        else:
-            myappend('_' + item)
-    return "".join(res)
-
-def get_javascript_imports(opts, auto_populated_fields, field_sets):
-# Put in any necessary JavaScript imports.
-    js = ['js/core.js', 'js/admin/RelatedObjectLookups.js']
@@ -309 +273 @@
-
-def change_stage(request, app_label, model_name, object_id):
-    model = models.get_model(app_label, model_name)
-
+pk_url_
-    if model is None:
-        raise Http404("App %r, model %r, not found" % (app_label, model_name))
-    opts = model._meta
@@ -501 +465 @@
-
-def delete_stage(request, app_label, model_name, object_id):
-    model = models.get_model(app_label, model_name)
-
+pk_url_
-    if model is None:
-        raise Http404("App %r, model %r, not found" % (app_label, model_name))
-    opts = model._meta
@@ -511 +475 @@
-
-    # Populate deleted_objects, a data structure of all related objects that
-    # will also be deleted.
-force_unicode(object_id
+pk_url_quote(force_unicode(object_id)
-    perms_needed = set()
-    _get_deleted_objects(deleted_objects, perms_needed, request.user, obj, opts, 1)
-
@@ -538 +502 @@
-
-def history(request, app_label, model_name, object_id):
-    model = models.get_model(app_label, model_name)
-
+pk_url_
-    if model is None:
-        raise Http404("App %r, model %r, not found" % (app_label, model_name))
-    action_list = LogEntry.objects.filter(object_id=object_id,
@@ -764 +728 @@
-        return qs
-
-    def url_for_result(self, result):
-
+pk_url_
-
-def change_list(request, app_label, model_name):
-    model = models.get_model(app_label, model_name)