Merge branch 'develop' of github.com:saxix/django-concurrency into develop

* 'develop' of github.com:saxix/django-concurrency:
  fixes admin action bug occurring when setting `select all` checkbox (`select_across`)

Author: saxix

src/concurrency/admin.py

@@ -85,7 +85,7 @@ def response_action(self, request, queryset):  # noqa
 
             # Get the list of selected PKs. If nothing's selected, we can't
             # perform an action on it, so bail.
-            if int(request.POST.get('select_across', 0)):
+            if action_form.cleaned_data['select_across']:
                 selected = ALL
             else:
                 selected = request.POST.getlist(helpers.ACTION_CHECKBOX_NAME)

tests/test_admin_actions.py

@@ -33,6 +33,26 @@ def test_dummy_action(self):
         self.assertIn('**concurrent_update**', res)
         self.assertNotIn('**action_update**', res)
 
+    def test_dummy_action_select_across(self):
+        id = next(unique_id)
+        SimpleConcurrentModel.objects.get_or_create(pk=id)
+        res = self.app.get('/admin/', user='sax')
+
+        res = res.click('^SimpleConcurrentModels')
+        assert 'SimpleConcurrentModel #%s' % id in res  # sanity check
+
+        self._create_conflict(id)
+
+        form = res.forms['changelist-form']
+        form['action'].value = 'dummy_action'
+        form['select_across'] = 'True'
+        sel = form.get('_selected_action', index=0)  # needed
+        sel.checked = True  # needed
+        res = form.submit()
+        res = res.follow()
+
+        self.assertIn('Selecting all records, you will avoid the concurrency check', res)
+
     @pytest.mark.skipif(django.VERSION[:2] >= (1, 7), reason="Skip django>=1.9")
     def test_delete_allowed_if_no_updates(self):
         id = next(unique_id)