Merge pull request #1559 from open-zaak/feature/1558-besluittype-zaak…

…typen ✨ [#1558] make besluittype.zaaktypen read-only
open-zaak · Feb 1, 2024 · 8e68fa6 · 8e68fa6
2 parents 4ea3205 + bf5d561
commit 8e68fa6
Show file tree

Hide file tree

Showing 7 changed files with 119 additions and 246 deletions.
diff --git a/bin/postman_tests.sh b/bin/postman_tests.sh
@@ -2,7 +2,7 @@
 
 set -x
 
-POSTMAN_TESTS_REF=d13c71796be9360a22f86c98a39fa0352f08dafe
+POSTMAN_TESTS_REF=bd2785bd611cd65bd180caf5dbc0b71034ae4653
 
 # These client IDs and secrets are dummy variables that are only used by
 # the Docker build in Travis, so they can be public

diff --git a/src/openzaak/components/catalogi/admin/utils.py b/src/openzaak/components/catalogi/admin/utils.py
@@ -140,7 +140,6 @@ def construct_besluittypen(
                 data=imported, context={"request": REQUEST}
             )
             if deserialized.is_valid():
-                deserialized.validated_data.pop("zaaktypen")
                 deserialized.validated_data.pop("informatieobjecttypen")
 
                 instance = BesluitType(**deserialized.validated_data)

diff --git a/src/openzaak/components/catalogi/api/serializers/besluittype.py b/src/openzaak/components/catalogi/api/serializers/besluittype.py
@@ -5,7 +5,7 @@
 from rest_framework import serializers
 from vng_api_common.utils import get_help_text
 
-from ...models import BesluitType, InformatieObjectType, ZaakType
+from ...models import BesluitType, InformatieObjectType
 from ..validators import (
     ConceptUpdateValidator,
     GeldigheidValidator,
@@ -28,7 +28,7 @@ class BesluitTypeSerializer(serializers.HyperlinkedModelSerializer):
         many=True,
         view_name="zaaktype-detail",
         lookup_field="uuid",
-        queryset=ZaakType.objects.all(),
+        read_only=True,
         help_text=get_help_text("catalogi.BesluitType", "zaaktypen"),
     )
 
@@ -103,8 +103,7 @@ class Meta:
         validators = [
             GeldigheidValidator(),
             RelationCatalogValidator("informatieobjecttypen"),
-            RelationCatalogValidator("zaaktypen"),
             ConceptUpdateValidator(),
-            M2MConceptCreateValidator(["zaaktypen", "informatieobjecttypen"]),
-            M2MConceptUpdateValidator(["zaaktypen", "informatieobjecttypen"]),
+            M2MConceptCreateValidator(["informatieobjecttypen"]),
+            M2MConceptUpdateValidator(["informatieobjecttypen"]),
         ]
diff --git a/src/openzaak/components/catalogi/openapi.yaml b/src/openzaak/components/catalogi/openapi.yaml
@@ -4715,7 +4715,6 @@ components:
     BesluitType:
       required:
       - catalogus
-      - zaaktypen
       - publicatieIndicatie
       - informatieobjecttypen
       - beginGeldigheid
@@ -4742,6 +4741,7 @@ components:
             description: ZAAKTYPE met ZAAKen die relevant kunnen zijn voor dit BESLUITTYPE
             type: string
             format: uri
+          readOnly: true
           uniqueItems: true
         omschrijving:
           title: Omschrijving

diff --git a/src/openzaak/components/catalogi/resources.md b/src/openzaak/components/catalogi/resources.md
@@ -12,7 +12,7 @@ Objecttype op [GEMMA Online](https://www.gemmaonline.nl/index.php/Rgbz_1.0/doc/o
 | --- | --- | --- | --- | --- |
 | url | URL-referentie naar dit object. Dit is de unieke identificatie en locatie van dit object. | string | nee | ~~C~~R~~U~~~~D~~ |
 | catalogus | URL-referentie naar de CATALOGUS waartoe dit BESLUITTYPE behoort. | string | ja | CRUD |
-| zaaktypen | ZAAKTYPE met ZAAKen die relevant kunnen zijn voor dit BESLUITTYPE | array | ja | CRUD |
+| zaaktypen | ZAAKTYPE met ZAAKen die relevant kunnen zijn voor dit BESLUITTYPE | array | nee | ~~C~~R~~U~~~~D~~ |
 | omschrijving | Omschrijving van de aard van BESLUITen van het BESLUITTYPE. | string | nee | CRUD |
 | omschrijvingGeneriek | Algemeen gehanteerde omschrijving van de aard van BESLUITen van het BESLUITTYPE | string | nee | CRUD |
 | besluitcategorie | Typering van de aard van BESLUITen van het BESLUITTYPE. | string | nee | CRUD |

diff --git a/src/openzaak/components/catalogi/swagger2.0.json b/src/openzaak/components/catalogi/swagger2.0.json
@@ -6125,7 +6125,6 @@
         "BesluitType": {
             "required": [
                 "catalogus",
-                "zaaktypen",
                 "publicatieIndicatie",
                 "informatieobjecttypen",
                 "beginGeldigheid"
@@ -6155,6 +6154,7 @@
                         "type": "string",
                         "format": "uri"
                     },
+                    "readOnly": true,
                     "uniqueItems": true
                 },
                 "omschrijving": {