Ticket #4326: django_new_admin_2_bug_fixed.patch

django/contrib/admin/sites.py

@@ -2 +2 @@
 from django.contrib.admin import ModelAdmin
 from django.contrib.auth import authenticate, login
 from django.db.models import Model
+from django.db.models.base import ModelBase
 from django.shortcuts import render_to_response
 from django.utils.text import capfirst
 from django.utils.translation import gettext_lazy
+from django.utils.datastructures import SortedDict
 import base64
 import cPickle as pickle
 import datetime
@@ -69 +71 @@
         """
         admin_class = admin_class or ModelAdmin
         # TODO: Handle options
-        if issubclass(model_or_iterable, Model):
+        if isinstance(model_or_iterable, ModelBase) and issubclass(model_or_iterable, Model):
             model_or_iterable = [model_or_iterable]
         for model in model_or_iterable:
             if model in self._registry:
@@ -234 +236 @@
         Displays the main admin index page, which lists all of the installed
         apps that have been registered in this site.
         """
-        app_list = []
+        apps = SortedDict()
         user = request.user
         for model, model_admin in self._registry.items():
             app_label = model._meta.app_label
@@ -254 +256 @@
                         'admin_url': '%s/%s/' % (app_label, model.__name__.lower()),
                         'perms': perms,
                     }
-                    app_list.append({
-                        'name': app_label.title(),
-                        'has_module_perms': has_module_perms,
-                        'models': [model_dict],
-                    })
+                    if app_label in apps:
+                        apps[app_label]['models'].append(model_dict)
+                    else:
+                        apps[app_label] = {
+                            'name': app_label.title(),
+                            # I found that this isn't used in template.
+                            #'has_module_perms': has_module_perms,
+                            'models': [model_dict],
+                        }
         return render_to_response('admin/index.html', {
             'title': _('Site administration'),
-            'app_list': app_list,
+            'app_list': apps.values(),
         }, context_instance=template.RequestContext(request))
 
 # This global object represents the default admin site, for the common case.