Author: adrian
Date: 2008-09-08 00:45:17 -0500 (Mon, 08 Sep 2008)
New Revision: 8985

Modified:
   django/trunk/django/contrib/admin/options.py
   django/trunk/django/contrib/admin/sites.py
   django/trunk/django/contrib/admin/widgets.py
Log:
Fixed #7929 -- Removed some snippets of admin code that weren't doing anything, 
and tightened up a few more lines. Plus, an indentation fix. Thanks, julien

Modified: django/trunk/django/contrib/admin/options.py
===================================================================
--- django/trunk/django/contrib/admin/options.py        2008-09-08 05:19:28 UTC 
(rev 8984)
+++ django/trunk/django/contrib/admin/options.py        2008-09-08 05:45:17 UTC 
(rev 8985)
@@ -471,18 +471,10 @@
         "The 'add' admin view for this model."
         model = self.model
         opts = model._meta
-        app_label = opts.app_label
 
         if not self.has_add_permission(request):
             raise PermissionDenied
 
-        if self.has_change_permission(request, None):
-            # redirect to list view
-            post_url = '../'
-        else:
-            # Object list will give 'Permission Denied', so go back to admin 
home
-            post_url = '../../../'
-
         ModelForm = self.get_form(request)
         formsets = []
         if request.method == 'POST':
@@ -541,7 +533,7 @@
             'inline_admin_formsets': inline_admin_formsets,
             'errors': helpers.AdminErrorList(form, formsets),
             'root_path': self.admin_site.root_path,
-            'app_label': app_label,
+            'app_label': opts.app_label,
         }
         context.update(extra_context or {})
         return self.render_change_form(request, context, add=True)
@@ -551,7 +543,6 @@
         "The 'change' admin view for this model."
         model = self.model
         opts = model._meta
-        app_label = opts.app_label
 
         try:
             obj = model._default_manager.get(pk=object_id)
@@ -620,7 +611,7 @@
             'inline_admin_formsets': inline_admin_formsets,
             'errors': helpers.AdminErrorList(form, formsets),
             'root_path': self.admin_site.root_path,
-            'app_label': app_label,
+            'app_label': opts.app_label,
         }
         context.update(extra_context or {})
         return self.render_change_form(request, context, change=True, obj=obj)
@@ -738,8 +729,8 @@
         }
         context.update(extra_context or {})
         return render_to_response(self.object_history_template or [
-            "admin/%s/%s/object_history.html" % (opts.app_label, 
opts.object_name.lower()),
-            "admin/%s/object_history.html" % opts.app_label,
+            "admin/%s/%s/object_history.html" % (app_label, 
opts.object_name.lower()),
+            "admin/%s/object_history.html" % app_label,
             "admin/object_history.html"
         ], context, context_instance=template.RequestContext(request))
 

Modified: django/trunk/django/contrib/admin/sites.py
===================================================================
--- django/trunk/django/contrib/admin/sites.py  2008-09-08 05:19:28 UTC (rev 
8984)
+++ django/trunk/django/contrib/admin/sites.py  2008-09-08 05:45:17 UTC (rev 
8985)
@@ -105,7 +105,6 @@
         The default implementation checks that LogEntry, ContentType and the
         auth context processor are installed.
         """
-        from django.conf import settings
         from django.contrib.admin.models import LogEntry
         from django.contrib.contenttypes.models import ContentType
 

Modified: django/trunk/django/contrib/admin/widgets.py
===================================================================
--- django/trunk/django/contrib/admin/widgets.py        2008-09-08 05:19:28 UTC 
(rev 8984)
+++ django/trunk/django/contrib/admin/widgets.py        2008-09-08 05:45:17 UTC 
(rev 8985)
@@ -114,7 +114,7 @@
         else:
             url = ''
         if not attrs.has_key('class'):
-          attrs['class'] = 'vForeignKeyRawIdAdminField' # The JavaScript looks 
for this hook.
+            attrs['class'] = 'vForeignKeyRawIdAdminField' # The JavaScript 
looks for this hook.
         output = [super(ForeignKeyRawIdWidget, self).render(name, value, 
attrs)]
         # TODO: "id_" is hard-coded here. This should instead use the correct
         # API to determine the ID dynamically.


--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to