Ticket #12962: with_test_change-12812.diff

django/contrib/admin/options.py

@@ -698 +698 @@
         changelist; it returns an HttpResponse if the action was handled, and
         None otherwise.
         """
-        if 'index' not in request.POST:
-            # If "Go" was not pushed then we can assume the POST was for
-            # an inline edit save and we do not need to validate the form.
-            return None
 
         # There can be multiple action forms on the page (at the top
         # and bottom of the change list, for example). Get the action
@@ -978 +974 @@
             return HttpResponseRedirect(request.path + '?' + ERROR_FLAG + '=1')
 
         # If the request was POSTed, this might be a bulk action or a bulk edit.
-        # Try to look up an action first, but if this isn't an action the POST
+        # Try to look up an action or confirmation first, but if this isn't an action the POST
         # will fall through to the bulk edit check, below.
-        if actions and request.method == 'POST':
+        if actions and request.method == 'POST' and (helpers.ACTION_CHECKBOX_NAME in request.POST or 'index' in request.POST):
             response = self.response_action(request, queryset=cl.get_query_set())
             if response:
                 return response

tests/regressiontests/admin_views/tests.py

@@ -1340 +1340 @@
         delete_confirmation_data = {
             ACTION_CHECKBOX_NAME: [1, 2],
             'action' : 'delete_selected',
-            'index': 0,
             'post': 'yes',
         }
         confirmation = self.client.post('/test_admin/admin/admin_views/subscriber/', action_data)