Ticket #4507: newforms-admin-form-save-hooks.diff

django/contrib/admin/options.py

@@ -292 +292 @@
         """
         return self.queryset(request)
 
+    def form_save(self, request, form):
+        """
+        Returns object saved from curent form.
+        """
+        return form.save(commit=True)
+
+    def form_save_add(self, request, form):
+        """
+        Returns object saved from current form in add stage.
+        """
+        return self.form_save(form, request)
+
+    def form_save_change(self, request, form):
+        """
+        Returns object saved from current form in change stage.
+        """
+        return self.form_save(form, request)
+
     def save_add(self, request, model, form, post_url_continue):
         """
         Saves the object in the "add" stage and returns an HttpResponseRedirect.
@@ -301 +319 @@
         from django.contrib.admin.models import LogEntry, ADDITION
         from django.contrib.contenttypes.models import ContentType
         opts = model._meta
-        new_object = form.save(commit=True)
+        new_object = self.form_save_add(request, form)
         pk_value = new_object._get_pk_val()
         LogEntry.objects.log_action(request.user.id, ContentType.objects.get_for_model(model).id, pk_value, str(new_object), ADDITION)
         msg = _('The %(name)s "%(obj)s" was added successfully.') % {'name': opts.verbose_name, 'obj': new_object}
@@ -340 +358 @@
         from django.contrib.admin.models import LogEntry, CHANGE
         from django.contrib.contenttypes.models import ContentType
         opts = model._meta
-        new_object = form.save(commit=True)
+        new_object = self.form_save_change(request, form)
         pk_value = new_object._get_pk_val()
 
         # Construct the change message. TODO: Temporarily commented-out,