fix: [#2803] make OpenZaakConfig.document_visible_statuses emptyable

Backport-Of: #2805

Author: swrichards

src/open_inwoner/openzaak/migrations/0086_alter_openzaakconfig_document_visible_statuses.py

@@ -0,0 +1,36 @@
+# Generated by Django 5.2.17 on 2026-08-13 11:53
+
+from django.db import migrations, models
+
+import django_jsonform.models.fields
+
+import open_inwoner.openzaak.models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("openzaak", "0085_zaaktypeconfig_zaken_visible_from"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="openzaakconfig",
+            name="document_visible_statuses",
+            field=django_jsonform.models.fields.ArrayField(
+                base_field=models.CharField(
+                    choices=[
+                        ("in_bewerking", "In bewerking"),
+                        ("ter_vaststelling", "Ter vaststelling"),
+                        ("definitief", "Definitief"),
+                        ("gearchiveerd", "Gearchiveerd"),
+                    ],
+                    max_length=32,
+                ),
+                blank=True,
+                default=open_inwoner.openzaak.models.default_document_visible_statuses,
+                help_text="Select which document statuses are visible to users. Documents with other statuses will be hidden. Warning: selecting no statuses means documents with any status will be visible.",
+                size=None,
+                verbose_name="Visible document statuses",
+            ),
+        ),
+    ]

src/open_inwoner/openzaak/models.py

@@ -425,6 +425,7 @@ def form_services(self):
             choices=InformatieObjectStatus.choices,
         ),
         default=default_document_visible_statuses,
+        blank=True,
         verbose_name=_("Visible document statuses"),
         help_text=_(
             "Select which document statuses are visible to users. "

src/open_inwoner/openzaak/tests/test_admin.py

@@ -17,6 +17,7 @@
 
 from open_inwoner.accounts.tests.factories import UserFactory
 from open_inwoner.openzaak.api_models import ResultaatType
+from open_inwoner.openzaak.models import OpenZaakConfig
 
 from .factories import (
     CatalogusConfigFactory,
@@ -408,3 +409,33 @@ def test_import_view_requires_change_permission(self):
         )
 
         self.assertEqual(response.status_code, 403)
+
+
+@disable_admin_mfa()
+class TestOpenZaakConfigAdmin(WebTest):
+    def setUp(self):
+        self.user = UserFactory(is_superuser=True, is_staff=True)
+        self.form = self.app.get(
+            reverse("admin:openzaak_openzaakconfig_change"), user=self.user
+        ).forms["openzaakconfig_form"]
+        # django-jsonform requires JS to work properly and with Webtest the default
+        # value for ArrayFields is an empty string, causing it to crash when trying to
+        # parse that value as JSON
+        self.form["allowed_file_extensions"] = json.dumps(
+            OpenZaakConfig.get_solo().allowed_file_extensions
+        )
+
+    def test_document_visible_statuses_can_be_saved_as_empty_list(self):
+        """An empty `document_visible_statuses` is valid and means all statuses
+        are visible (see `test_utils.test_is_info_object_visible_custom_statuses`),
+        so the admin form must allow saving it as such."""
+        config = OpenZaakConfig.get_solo()
+        self.assertNotEqual(config.document_visible_statuses, [])
+
+        self.form["document_visible_statuses"] = "[]"
+        response = self.form.submit()
+
+        self.assertEqual(response.status_code, 302)
+
+        config.refresh_from_db()
+        self.assertEqual(config.document_visible_statuses, [])