Ticket #5520: 00-admin-saveas.diff

django/contrib/admin/options.py

@@ -452 +452 @@
             request.user.message_set.create(message=msg)
             return HttpResponseRedirect("../")
 
-    def render_change_form(self, request, model, context, add=False, change=False, form_url='', obj=None):
+    def render_change_form(self, request, model, context, add=False, change=False, obj=None):
         opts = model._meta
         app_label = opts.app_label
         ordered_objects = opts.get_ordered_objects()
@@ -465 +465 @@
             'has_file_field': True, # FIXME - this should check if form or formsets have a FileField,
             'has_absolute_url': hasattr(model, 'get_absolute_url'),
             'ordered_objects': ordered_objects,
-            'form_url': mark_safe(form_url),
             'opts': opts,
             'content_type_id': ContentType.objects.get_for_model(model).id,
             'save_on_top': self.save_on_top,
+            'save_as': self.save_as,
+            'is_save_as': request.POST.has_key('_saveasnew'),
         }
         context.update(extra_context)
         return render_to_response([
@@ -476 +477 @@
             "admin/%s/change_form.html" % app_label,
             "admin/change_form.html"], context_instance=context)
 
-    def add_view(self, request, form_url=''):
+    def add_view(self, request):
         "The 'add' admin view for this model."
         model = self.model
         opts = model._meta
@@ -523 +524 @@
             'title': _('Add %s') % opts.verbose_name,
             'adminform': adminForm,
             'is_popup': request.REQUEST.has_key('_popup'),
-            'show_delete': False,
             'media': mark_safe(media),
             'inline_admin_formsets': inline_admin_formsets,
         })
@@ -549 +549 @@
         if obj is None:
             raise Http404('%s object with primary key %r does not exist.' % (opts.verbose_name, escape(object_id)))
 
-        if request.POST and request.POST.has_key("_saveasnew"):
-            return self.add_view(request, form_url='../../add/')
+        if request.POST and request.POST.has_key('_saveasnew'):
+            return self.add_view(request)
 
         ModelForm = self.form_change(request, obj)
         inline_formsets = []

django/contrib/admin/templates/admin/change_form.html

@@ -30 +30 @@
   </ul>
 {% endif %}{% endif %}
 {% endblock %}
-<form {% if has_file_field %}enctype="multipart/form-data" {% endif %}action="{{ form_url }}" method="post" id="{{ opts.module_name }}_form">{% block form_top %}{% endblock %}
+<form {% if has_file_field %}enctype="multipart/form-data" {% endif %}action="." method="post" id="{{ opts.module_name }}_form">{% block form_top %}{% endblock %}
 <div>
 {% if is_popup %}<input type="hidden" name="_popup" value="1" />{% endif %}
 {% if save_on_top %}{% submit_row %}{% endif %}

django/contrib/admin/templatetags/admin_modify.py

@@ -18 +18 @@
 
 def submit_row(context):
     opts = context['opts']
+    add = context['add']
     change = context['change']
     is_popup = context['is_popup']
-    # TODO: Fix this hack.
-    # save_as = opts.admin.save_as
-    save_as = False
+    save_as = context['save_as']
+    is_save_as = context['is_save_as'] 
     return {
         'onclick_attrib': (opts.get_ordered_objects() and change
                             and 'onclick="submitOrderForm();"' or ''),
-        'show_delete_link': (not is_popup and context['has_delete_permission']
-                              and (change or context['show_delete'])),
-        'show_save_as_new': not is_popup and change and save_as,
-        'show_save_and_add_another': context['has_add_permission'] and 
-                            not is_popup and (not save_as or context['add']),
-        'show_save_and_continue': not is_popup and context['has_change_permission'],
-        'show_save': True
+        'show_delete_link': change and context['has_delete_permission'] and not is_popup and not is_save_as,
+        'show_save_as_new': save_as and (is_save_as or change) and context['has_add_permission'] and not is_popup,
+        'show_save_and_add_another': (add or not save_as) and context['has_add_permission'] and not is_popup and not is_save_as,
+        'show_save_and_continue': context['has_change_permission'] and not is_popup and not is_save_as,
+        'show_save': not is_save_as, 
     }
 submit_row = register.inclusion_tag('admin/submit_line.html', takes_context=True)(submit_row)
 

django/contrib/auth/admin.py

@@ -57 +57 @@
             'auto_populated_fields': (),
             'opts': User._meta,
             'username_help_text': User._meta.get_field('username').help_text,
+            'save_as': False,
+            'is_save_as': False,
         }, context_instance=template.RequestContext(request))
 
 admin.site.register(Group, GroupAdmin)

django/contrib/auth/views.py

@@ -129 +129 @@
         'opts': User._meta,
         'original': user,
         'show_save': True,
+        'save_as': False,
+        'is_save_as': False,
     }, context_instance=RequestContext(request))