Further testing and examples

src/documents/file_handling.py

@@ -279,7 +279,7 @@ def validate_template_and_render(
         Used to provide context to the user, rather than mostly failing silently
 
         """
-        pattern = rf"{INVALID_VARIABLE_STR}: (\w+)"
+        pattern = rf"{INVALID_VARIABLE_STR}: ([\w]+(?:[.\s]+[\w]+)*)"
         matches = re.findall(pattern, rendered_string)
 
         if matches:
@@ -323,8 +323,6 @@ def validate_template_and_render(
         )
         rendered_template = template.render(Context(context))
 
-        logger.info(rendered_template)
-
         # Check for errors
         undefined_vars = detect_undefined_variables(rendered_template)
         if undefined_vars:

src/documents/tests/test_file_handling.py

@@ -1289,3 +1289,22 @@ class TestFilenameGeneration(DirectoriesMixin, TestCase):
             generate_filename(doc_a),
             "not-invoices/Some Title.pdf",
         )
+
+    def test_using_other_filters(self):
+        doc_a = Document.objects.create(
+            title="Some Title",
+            created=timezone.make_aware(datetime.datetime(2020, 6, 25, 7, 36, 51, 153)),
+            added=timezone.make_aware(datetime.datetime(2024, 10, 1, 7, 36, 51, 153)),
+            mime_type="application/pdf",
+            pk=2,
+            checksum="2",
+            archive_serial_number=25,
+        )
+
+        with override_settings(
+            FILENAME_FORMAT="{% if document.correspondent %}{{ document.correspondent.name }}{% else %}No Correspondent{% endif %}/{title}",
+        ):
+            self.assertEqual(
+                generate_filename(doc_a),
+                "No Correspondent/Some Title.pdf",
+            )