.github/workflows/release.yml

@@ -26,7 +26,7 @@ jobs:
           echo "dir=$(pip cache dir)" >> $GITHUB_OUTPUT
 
       - name: Cache
-        uses: actions/cache@v5
+        uses: actions/cache@v4
         with:
           path: ${{ steps.pip-cache.outputs.dir }}
           key: release-${{ hashFiles('**/setup.py') }}

.pre-commit-config.yaml

@@ -1,7 +1,7 @@
 ---
 repos:
   - repo: https://github.com/psf/black-pre-commit-mirror
-    rev: 26.3.0
+    rev: 25.11.0
     hooks:
       - id: black
         language_version: python3.10
@@ -14,7 +14,7 @@ repos:
       - id: flake8
         args: ["--max-line-length", "110"]
   - repo: https://github.com/PyCQA/isort
-    rev: 8.0.1
+    rev: 7.0.0
     hooks:
       - id: isort
   - repo: https://github.com/asottile/pyupgrade
@@ -23,7 +23,7 @@ repos:
     -   id: pyupgrade
         args: [--py310-plus]
   - repo: https://github.com/adamchainz/django-upgrade
-    rev: 1.30.0
+    rev: 1.29.1
     hooks:
     - id: django-upgrade
       args: [--target-version, "4.2"]

CHANGELOG.md

@@ -2,16 +2,6 @@
 
 ## Next Release
 
-#### Fixes
-
-- `KeyError` when calling `changes_str` on a log entry that tracks many-to-many field changes ([#798](https://github.com/jazzband/django-auditlog/pull/798))
-
-## 3.4.1 (2025-12-13)
-
-#### Fixes
-
-- Fix AttributeError when AUDITLOG_LOGENTRY_MODEL is not set ([#789](https://github.com/jazzband/django-auditlog/pull/789))
-
 ## 3.4.0 (2025-12-04)
 
 #### Improvements

auditlog/__init__.py

@@ -10,9 +10,10 @@ __version__ = version("django-auditlog")
 
 
 def get_logentry_model():
-    model_string = getattr(settings, "AUDITLOG_LOGENTRY_MODEL", "auditlog.LogEntry")
     try:
-        return django_apps.get_model(model_string, require_ready=False)
+        return django_apps.get_model(
+            settings.AUDITLOG_LOGENTRY_MODEL, require_ready=False
+        )
     except ValueError:
         raise ImproperlyConfigured(
             "AUDITLOG_LOGENTRY_MODEL must be of the form 'app_label.model_name'"
@@ -20,5 +21,5 @@ def get_logentry_model():
     except LookupError:
         raise ImproperlyConfigured(
             "AUDITLOG_LOGENTRY_MODEL refers to model '%s' that has not been installed"
-            % model_string
+            % settings.AUDITLOG_LOGENTRY_MODEL
         )

auditlog/management/commands/auditlogmigratejson.py

@@ -126,13 +126,15 @@ class Command(BaseCommand):
 
         def postgres():
             with connection.cursor() as cursor:
-                cursor.execute(f"""
+                cursor.execute(
+                    f"""
                     UPDATE {LogEntry._meta.db_table}
                     SET changes="changes_text"::jsonb
                     WHERE changes_text IS NOT NULL
                         AND changes_text <> ''
                         AND changes IS NULL
-                    """)
+                    """
+                )
                 return cursor.cursor.rowcount
 
         if database == "postgres":

auditlog/models.py

@@ -427,29 +427,21 @@ class AbstractLogEntry(models.Model):
         not satisfying, please use :py:func:`LogEntry.changes_dict` and format the string yourself.
 
         :param colon: The string to place between the field name and the values.
-        :param arrow: The string to place between each old and new value (non-m2m field changes only).
+        :param arrow: The string to place between each old and new value.
         :param separator: The string to place between each field.
         :return: A readable string of the changes in this log entry.
         """
         substrings = []
 
-        for field, value in sorted(self.changes_dict.items()):
-            if isinstance(value, (list, tuple)) and len(value) == 2:
-                # handle regular field change
-                substring = "{field_name:s}{colon:s}{old:s}{arrow:s}{new:s}".format(
-                    field_name=field,
-                    colon=colon,
-                    old=value[0],
-                    arrow=arrow,
-                    new=value[1],
-                )
-                substrings.append(substring)
-            elif isinstance(value, dict) and value.get("type") == "m2m":
-                # handle m2m change
-                substring = (
-                    f"{field}{colon}{value['operation']} {sorted(value['objects'])}"
-                )
-                substrings.append(substring)
+        for field, values in self.changes_dict.items():
+            substring = "{field_name:s}{colon:s}{old:s}{arrow:s}{new:s}".format(
+                field_name=field,
+                colon=colon,
+                old=values[0],
+                arrow=arrow,
+                new=values[1],
+            )
+            substrings.append(substring)
 
         return separator.join(substrings)
 

auditlog_tests/tests.py

@@ -131,11 +131,6 @@ class SimpleModelTest(TestCase):
             {"boolean": ["False", "True"]},
             msg="The change is correctly logged",
         )
-        self.assertEqual(
-            history.changes_str,
-            "boolean: False → True",
-            msg="Changes string is correct",
-        )
 
     def test_update_specific_field_supplied_via_save_method(self):
         obj = self.obj
@@ -154,11 +149,6 @@ class SimpleModelTest(TestCase):
                 "when using the `update_fields`."
             ),
         )
-        self.assertEqual(
-            obj.history.get(action=LogEntry.Action.UPDATE).changes_str,
-            "boolean: False → True",
-            msg="Changes string is correct",
-        )
 
     def test_django_update_fields_edge_cases(self):
         """
@@ -189,11 +179,6 @@ class SimpleModelTest(TestCase):
             {"boolean": ["False", "True"], "integer": ["None", "1"]},
             msg="The 2 fields changed are correctly logged",
         )
-        self.assertEqual(
-            obj.history.get(action=LogEntry.Action.UPDATE).changes_str,
-            "boolean: False → True; integer: None → 1",
-            msg="Changes string is correct",
-        )
 
     def test_delete(self):
         """Deletion is logged correctly."""
@@ -509,13 +494,6 @@ class ManyRelatedModelTest(TestCase):
             },
         )
 
-    def test_changes_str(self):
-        self.obj.related.add(self.related)
-        log_entry = self.obj.history.first()
-        self.assertEqual(
-            log_entry.changes_str, f"related: add {[smart_str(self.related)]}"
-        )
-
     def test_adding_existing_related_obj(self):
         self.obj.related.add(self.related)
         log_entry = self.obj.history.first()
@@ -747,11 +725,6 @@ class SimpleIncludeModelTest(TestCase):
             {"label": ["Initial label", "New label"]},
             msg="Only the label was logged, regardless of multiple entries in `update_fields`",
         )
-        self.assertEqual(
-            obj.history.get(action=LogEntry.Action.UPDATE).changes_str,
-            "label: Initial label → New label",
-            msg="Changes string is correct",
-        )
 
     def test_register_include_fields(self):
         sim = SimpleIncludeModel(label="Include model", text="Looong text")
@@ -2088,11 +2061,6 @@ class JSONModelTest(TestCase):
             {"json": ["{}", '{"quantity": "1"}']},
             msg="The change is correctly logged",
         )
-        self.assertEqual(
-            history.changes_str,
-            'json: {} → {"quantity": "1"}',
-            msg="Changes string is correct",
-        )
 
     def test_update_with_no_changes(self):
         """No changes are logged."""
@@ -2729,7 +2697,6 @@ class TestAccessLog(TestCase):
         )
         self.assertIsNone(log_entry.changes)
         self.assertEqual(log_entry.changes_dict, {})
-        self.assertEqual(log_entry.changes_str, "")
 
 
 class SignalTests(TestCase):
@@ -3153,9 +3120,6 @@ class BaseManagerSettingTest(TestCase):
                 }
             },
         )
-        self.assertEqual(
-            log_entry.changes_str, f"m2m_related: add {[smart_str(obj_two)]}"
-        )
 
 
 class TestMaskStr(TestCase):
@@ -3284,22 +3248,3 @@ class GetLogEntryModelTest(TestCase):
     def test_invalid_appname(self):
         with self.assertRaises(ImproperlyConfigured):
             get_logentry_model()
-
-    def test_logentry_model_default_when_setting_missing(self):
-        """Regression test for issue #788: AttributeError when AUDITLOG_LOGENTRY_MODEL is not set."""
-        # Save and remove the setting to simulate the bug condition
-        original_value = getattr(settings, "AUDITLOG_LOGENTRY_MODEL", None)
-        if hasattr(settings, "AUDITLOG_LOGENTRY_MODEL"):
-            delattr(settings, "AUDITLOG_LOGENTRY_MODEL")
-
-        try:
-            # This should NOT raise AttributeError - it should use the default
-            model = get_logentry_model()
-            self.assertEqual(
-                f"{model._meta.app_label}.{model._meta.object_name}",
-                "auditlog.LogEntry",
-            )
-        finally:
-            # Restore the original setting
-            if original_value is not None:
-                settings.AUDITLOG_LOGENTRY_MODEL = original_value

docs/source/usage.rst

@@ -76,7 +76,7 @@ You can also add log-access to function base views, as the following example ill
 
 Fields that are excluded will not trigger saving a new log entry and will not show up in the recorded changes.
 
-To exclude specific fields from the log you can pass ``include_fields`` or ``exclude_fields`` to the ``register``
+To exclude specific fields from the log you can pass ``include_fields`` resp. ``exclude_fields`` to the ``register``
 method. If ``exclude_fields`` is specified the fields with the given names will not be included in the generated log
 entries. If ``include_fields`` is specified only the fields with the given names will be included in the generated log
 entries. Explicitly excluding fields through ``exclude_fields`` takes precedence over specifying which fields to