src/__init__.py tests/: added test for #4047.

Also added assert that will catch the problem in future.
pymupdf · Nov 14, 2024 · fb905b1 · fb905b1
1 parent 69891d3
commit fb905b1
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 0 deletions.
diff --git a/src/__init__.py b/src/__init__.py
@@ -7750,6 +7750,7 @@ def _add_redact_annot(self, quad, text=None, da_str=None, align=0, fill=None, te
                 mupdf.pdf_array_push_real(arr, fcol[i])
             mupdf.pdf_dict_put(mupdf.pdf_annot_obj(annot), PDF_NAME('IC'), arr)
         if text:
+            assert da_str
             mupdf.pdf_dict_puts(
                     mupdf.pdf_annot_obj(annot),
                     "OverlayText",

diff --git a/tests/resources/test_4047.pdf b/tests/resources/test_4047.pdf
diff --git a/tests/test_annots.py b/tests/test_annots.py
@@ -458,3 +458,17 @@ def test_parent():
             assert str(e) == 'weakly-referenced object no longer exists'
     else:
         assert 0, f'Failed to get expected exception.'
+
+def test_4047():
+    path = os.path.normpath(f'{__file__}/../../tests/resources/test_4047.pdf')
+    with pymupdf.open(path) as document:
+        page = document[0]
+        fontname = page.get_fonts()[0][3]
+        if fontname not in pymupdf.Base14_fontnames:
+            fontname = "Courier"
+        hits = page.search_for("|")
+        for rect in hits:
+            page.add_redact_annot(
+                rect, " ", fontname=fontname, align=pymupdf.TEXT_ALIGN_CENTER, fontsize=10
+            )  # Segmentation Fault...
+        page.apply_redactions()