Merge branch 'release/2.2'

* release/2.2: (21 commits)
  updates
  updates
  update github actions
  bump version
  add support for ModelAdmin callable - #181
  renamed `get_export_form` as `get_aa_export_form`
  remvoves admin-extra-url depencency
  removes pytest-pythonpath plugin
  add pre-commit
  let use black for formatting
  lint
  add suppport Django 4.2
  improved bulk_update
  lint
  lint
  updates tests
  lint
  lint
  lint
  updates
  ...

.bumpversion.cfg

@@ -1,5 +1,5 @@
 [bumpversion]
-current_version = 2.1.0
+current_version = 2.2.0
 parse = (?P<major>\d+)\.(?P<minor>\d+)\.(?P<patch>\d+)
 serialize = {major}.{minor}.{patch}
 commit = False

.github/workflows/test.yml

@@ -16,10 +16,10 @@ jobs:
 
             -   name: Install dependencies
                 run: |
-                    python -m pip install --upgrade pip tox
+                    python -m pip install --upgrade pip pre-commit black isort flake8
             -   name: Lint with flake8
                 run: |
-                    tox -e lint
+                    pre-commit run --all
 
     test:
         services:
@@ -31,7 +31,7 @@ jobs:
             fail-fast: false
             matrix:
                 python-version: [ "3.9", "3.10", "3.11" ]
-                django-version: [ "3.2", "4.0", "4.1" ]
+                django-version: [ "3.2", "4.2" ]
         env:
             PY_VER: ${{ matrix.python-version}}
             DJ_VER: ${{ matrix.django-version}}

.gitignore

@@ -19,5 +19,4 @@ sonar-project.properties
 Pipfile
 Pipfile.lock
 poetry.lock
-pyproject.toml
 .venv/

.pre-commit-config.yaml

@@ -0,0 +1,21 @@
+repos:
+-   repo: local
+    hooks:
+    # Configuration for black exists in pyproject.toml,
+    # but we let pre-commit take care of the file filtering.
+    - id: black
+      name: black
+      entry: black
+      language: python
+      types: [python]
+      require_serial: true
+    - id: isort
+      name: isort
+      entry: isort
+      language: python
+      types: [python]
+    - id: flake8
+      name: flake8
+      entry: flake8
+      language: python
+      types: [python]

CHANGES

@@ -1,3 +1,14 @@
+Release 2.2
+===========
+* new `MassUpdateForm.sort_fields`. Make optional MassUpdateForm fields sorting
+* make possible globally customize MassUpdateForm
+* removes async feature from Bulk update
+* fixes support for 'MemoryFileUploadHandler' in bulk_update
+* add `dry_run` option to bulk_update
+* bulk_update now returns a page with pre/post action field values
+* renamed `get_export_form` as `get_aa_export_form` ( @see https://github.com/saxix/django-adminactions/issues/217)
+* add support for ModelAdmin callable (@see https://github.com/saxix/django-adminactions/issues/181)
+
 Release 2.1
 ===========
 * new action "Bulk Update"

docs/source/_ext/djangodocs.py

@@ -4,49 +4,50 @@
 import json
 import os
 import re
-from docutils import nodes, transforms
-from sphinx import __version__ as sphinx_ver, addnodes, roles
+
+from sphinx import __version__ as sphinx_ver
+from sphinx import addnodes
 from sphinx.builders.html import StandaloneHTMLBuilder
 from sphinx.util.compat import Directive
 from sphinx.util.console import bold
 from sphinx.writers.html import SmartyPantsHTMLTranslator
 
 # RE for option descriptions without a '--' prefix
-simple_option_desc_re = re.compile(
-    r'([-_a-zA-Z0-9]+)(\s*.*?)(?=,\s+(?:/|-|--)|$)')
+simple_option_desc_re = re.compile(r"([-_a-zA-Z0-9]+)(\s*.*?)(?=,\s+(?:/|-|--)|$)")
+
 
 def setup(app):
     app.add_crossref_type(
-        directivename = "setting",
-        rolename      = "setting",
-        indextemplate = "pair: %s; setting",
+        directivename="setting",
+        rolename="setting",
+        indextemplate="pair: %s; setting",
     )
     app.add_crossref_type(
-        directivename = "templatetag",
-        rolename      = "ttag",
-        indextemplate = "pair: %s; template tag"
+        directivename="templatetag",
+        rolename="ttag",
+        indextemplate="pair: %s; template tag",
     )
     app.add_crossref_type(
-        directivename = "templatefilter",
-        rolename      = "tfilter",
-        indextemplate = "pair: %s; template filter"
+        directivename="templatefilter",
+        rolename="tfilter",
+        indextemplate="pair: %s; template filter",
     )
     app.add_crossref_type(
-        directivename = "fieldlookup",
-        rolename      = "lookup",
-        indextemplate = "pair: %s; field lookup type",
+        directivename="fieldlookup",
+        rolename="lookup",
+        indextemplate="pair: %s; field lookup type",
     )
     app.add_description_unit(
-        directivename = "django-admin",
-        rolename      = "djadmin",
-        indextemplate = "pair: %s; django-admin command",
-        parse_node    = parse_django_admin_node,
+        directivename="django-admin",
+        rolename="djadmin",
+        indextemplate="pair: %s; django-admin command",
+        parse_node=parse_django_admin_node,
     )
     app.add_description_unit(
-        directivename = "django-admin-option",
-        rolename      = "djadminopt",
-        indextemplate = "pair: %s; django-admin command-line option",
-        parse_node    = parse_django_adminopt_node,
+        directivename="django-admin-option",
+        rolename="djadminopt",
+        indextemplate="pair: %s; django-admin command-line option",
+        parse_node=parse_django_adminopt_node,
     )
     # app.add_config_value('django_next_version', '0.0', True)
     # app.add_directive('versionadded', VersionDirective)
@@ -67,17 +68,17 @@ def run(self):
         node = addnodes.versionmodified()
         ret.append(node)
         if self.arguments[0] == env.config.django_next_version:
-            node['version'] = "Development version"
+            node["version"] = "Development version"
         else:
-            node['version'] = self.arguments[0]
-        node['type'] = self.name
+            node["version"] = self.arguments[0]
+        node["type"] = self.name
         if len(self.arguments) == 2:
-            inodes, messages = self.state.inline_text(self.arguments[1], self.lineno+1)
+            inodes, messages = self.state.inline_text(self.arguments[1], self.lineno + 1)
             node.extend(inodes)
             if self.content:
                 self.state.nested_parse(self.content, self.content_offset, node)
             ret = ret + messages
-        env.note_versionchange(node['type'], node['version'], node, self.lineno)
+        env.note_versionchange(node["type"], node["version"], node, self.lineno)
         return ret
 
 
@@ -88,19 +89,19 @@ class DjangoHTMLTranslator(SmartyPantsHTMLTranslator):
 
     # Don't use border=1, which docutils does by default.
     def visit_table(self, node):
-        self._table_row_index = 0 # Needed by Sphinx
-        self.body.append(self.starttag(node, 'table', CLASS='docutils'))
+        self._table_row_index = 0  # Needed by Sphinx
+        self.body.append(self.starttag(node, "table", CLASS="docutils"))
 
     # <big>? Really?
     def visit_desc_parameterlist(self, node):
-        self.body.append('(')
+        self.body.append("(")
         self.first_param = 1
         self.param_separator = node.child_text_separator
 
     def depart_desc_parameterlist(self, node):
-        self.body.append(')')
+        self.body.append(")")
 
-    if sphinx_ver < '1.0.8':
+    if sphinx_ver < "1.0.8":
         #
         # Don't apply smartypants to literal blocks
         #
@@ -122,18 +123,16 @@ def depart_literal_block(self, node):
     # that work.
     #
     version_text = {
-        'deprecated':       'Deprecated in Django %s',
-        'versionchanged':   'Changed in Django %s',
-        'versionadded':     'New in Django %s',
+        "deprecated": "Deprecated in Django %s",
+        "versionchanged": "Changed in Django %s",
+        "versionadded": "New in Django %s",
     }
 
     def visit_versionmodified(self, node):
-        self.body.append(
-            self.starttag(node, 'div', CLASS=node['type'])
-        )
+        self.body.append(self.starttag(node, "div", CLASS=node["type"]))
         title = "%s%s" % (
-            self.version_text[node['type']] % node['version'],
-            len(node) and ":" or "."
+            self.version_text[node["type"]] % node["version"],
+            len(node) and ":" or ".",
         )
         self.body.append('<span class="title">%s</span> ' % title)
 
@@ -142,28 +141,31 @@ def depart_versionmodified(self, node):
 
     # Give each section a unique ID -- nice for custom CSS hooks
     def visit_section(self, node):
-        old_ids = node.get('ids', [])
-        node['ids'] = ['s-' + i for i in old_ids]
-        node['ids'].extend(old_ids)
+        old_ids = node.get("ids", [])
+        node["ids"] = ["s-" + i for i in old_ids]
+        node["ids"].extend(old_ids)
         SmartyPantsHTMLTranslator.visit_section(self, node)
-        node['ids'] = old_ids
+        node["ids"] = old_ids
+
 
 def parse_django_admin_node(env, sig, signode):
-    command = sig.split(' ')[0]
+    command = sig.split(" ")[0]
     env._django_curr_admin_command = command
     title = "django-admin.py %s" % sig
     signode += addnodes.desc_name(title, title)
     return sig
 
+
 def parse_django_adminopt_node(env, sig, signode):
     """A copy of sphinx.directives.CmdoptionDesc.parse_signature()"""
     from sphinx.domains.std import option_desc_re
+
     count = 0
-    firstname = ''
+    firstname = ""
     for m in option_desc_re.finditer(sig):
         optname, args = m.groups()
         if count:
-            signode += addnodes.desc_addname(', ', ', ')
+            signode += addnodes.desc_addname(", ", ", ")
         signode += addnodes.desc_name(optname, optname)
         signode += addnodes.desc_addname(args, args)
         if not count:
@@ -173,7 +175,7 @@ def parse_django_adminopt_node(env, sig, signode):
         for m in simple_option_desc_re.finditer(sig):
             optname, args = m.groups()
             if count:
-                signode += addnodes.desc_addname(', ', ', ')
+                signode += addnodes.desc_addname(", ", ", ")
             signode += addnodes.desc_name(optname, optname)
             signode += addnodes.desc_addname(args, args)
             if not count:
@@ -189,20 +191,26 @@ class DjangoStandaloneHTMLBuilder(StandaloneHTMLBuilder):
     Subclass to add some extra things we need.
     """
 
-    name = 'djangohtml'
+    name = "djangohtml"
 
     def finish(self):
         super().finish()
         self.info(bold("writing templatebuiltins.js..."))
         xrefs = self.env.domaindata["std"]["objects"]
         templatebuiltins = {
-            "ttags": [n for ((t, n), (l, a)) in xrefs.items()
-                        if t == "templatetag" and l == "ref/templates/builtins"],
-            "tfilters": [n for ((t, n), (l, a)) in xrefs.items()
-                        if t == "templatefilter" and l == "ref/templates/builtins"],
+            "ttags": [
+                n
+                for ((t, n), (l, a)) in xrefs.items()
+                if t == "templatetag" and l == "ref/templates/builtins"
+            ],
+            "tfilters": [
+                n
+                for ((t, n), (l, a)) in xrefs.items()
+                if t == "templatefilter" and l == "ref/templates/builtins"
+            ],
         }
         outfilename = os.path.join(self.outdir, "templatebuiltins.js")
-        with open(outfilename, 'wb') as fp:
-            fp.write('var django_template_builtins = ')
+        with open(outfilename, "wb") as fp:
+            fp.write("var django_template_builtins = ")
             json.dump(templatebuiltins, fp)
-            fp.write(';\n')
+            fp.write(";\n")