Merge pull request #875 from open-zaak/issue/fix-put-exception

Prevent exception when doing PUT and ZaakTypenRelatie already exists

src/openzaak/components/catalogi/api/viewsets/zaaktype.py

@@ -123,6 +123,13 @@ class ZaakTypeViewSet(
     notifications_kanaal = KANAAL_ZAAKTYPEN
     concept_related_fields = ["besluittypen", "informatieobjecttypen"]
 
+    def perform_update(self, serializer):
+
+        if not serializer.partial:
+            serializer.instance.zaaktypenrelaties.all().delete()
+
+        super().perform_update(serializer)
+
     @swagger_auto_schema(
         request_body=no_body,
         responses={

src/openzaak/components/catalogi/tests/test_zaaktype.py

@@ -23,7 +23,7 @@
     VerlengingsValidator,
 )
 from ..constants import AardRelatieChoices, InternExtern
-from ..models import ZaakType
+from ..models import ZaakType, ZaakTypenRelatie
 from .base import APITestCase
 from .factories import (
     BesluitTypeFactory,
@@ -549,6 +549,106 @@ def test_update_zaaktype(self):
         zaaktype.refresh_from_db()
         self.assertEqual(zaaktype.aanleiding, "aangepast")
 
+    def test_update_zaaktype_with_existing_zaaktyperelatie(self):
+        zaaktype = ZaakTypeFactory.create()
+        zaaktype_url = reverse(zaaktype)
+
+        ZaakTypenRelatieFactory.create(
+            zaaktype=zaaktype, gerelateerd_zaaktype="http://example.com/zaaktype/1"
+        )
+
+        data = {
+            "identificatie": 0,
+            "doel": "some test",
+            "aanleiding": "aangepast",
+            "indicatieInternOfExtern": InternExtern.extern,
+            "handelingInitiator": "indienen",
+            "onderwerp": "Klacht",
+            "handelingBehandelaar": "uitvoeren",
+            "doorlooptijd": "P30D",
+            "opschortingEnAanhoudingMogelijk": False,
+            "verlengingMogelijk": True,
+            "verlengingstermijn": "P30D",
+            "publicatieIndicatie": True,
+            "verantwoordingsrelatie": [],
+            "productenOfDiensten": ["https://example.com/product/123"],
+            "vertrouwelijkheidaanduiding": VertrouwelijkheidsAanduiding.openbaar,
+            "omschrijving": "some test",
+            "gerelateerdeZaaktypen": [
+                {
+                    "zaaktype": "http://example.com/zaaktype/1",
+                    "aard_relatie": AardRelatieChoices.bijdrage,
+                    "toelichting": "test relations",
+                }
+            ],
+            "referentieproces": {"naam": "ReferentieProces 0", "link": ""},
+            "catalogus": f"http://testserver{self.catalogus_detail_url}",
+            # 'informatieobjecttypen': [f'http://testserver{informatieobjecttype_url}'],
+            "besluittypen": [],
+            "beginGeldigheid": "2018-01-01",
+            "versiedatum": "2018-01-01",
+        }
+
+        response = self.client.put(zaaktype_url, data)
+
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+        self.assertEqual(response.data["aanleiding"], "aangepast")
+
+        zaaktype.refresh_from_db()
+        self.assertEqual(zaaktype.aanleiding, "aangepast")
+
+    def test_update_zaaktype_with_two_existing_zaaktyperelatie(self):
+        zaaktype = ZaakTypeFactory.create()
+        zaaktype_url = reverse(zaaktype)
+
+        ZaakTypenRelatieFactory.create(
+            zaaktype=zaaktype, gerelateerd_zaaktype="http://example.com/zaaktype/1"
+        )
+        ZaakTypenRelatieFactory.create(
+            zaaktype=zaaktype, gerelateerd_zaaktype="http://example.com/zaaktype/2"
+        )
+
+        data = {
+            "identificatie": 0,
+            "doel": "some test",
+            "aanleiding": "aangepast",
+            "indicatieInternOfExtern": InternExtern.extern,
+            "handelingInitiator": "indienen",
+            "onderwerp": "Klacht",
+            "handelingBehandelaar": "uitvoeren",
+            "doorlooptijd": "P30D",
+            "opschortingEnAanhoudingMogelijk": False,
+            "verlengingMogelijk": True,
+            "verlengingstermijn": "P30D",
+            "publicatieIndicatie": True,
+            "verantwoordingsrelatie": [],
+            "productenOfDiensten": ["https://example.com/product/123"],
+            "vertrouwelijkheidaanduiding": VertrouwelijkheidsAanduiding.openbaar,
+            "omschrijving": "some test",
+            "gerelateerdeZaaktypen": [
+                {
+                    "zaaktype": "http://example.com/zaaktype/1",
+                    "aard_relatie": AardRelatieChoices.bijdrage,
+                    "toelichting": "test relations",
+                }
+            ],
+            "referentieproces": {"naam": "ReferentieProces 0", "link": ""},
+            "catalogus": f"http://testserver{self.catalogus_detail_url}",
+            # 'informatieobjecttypen': [f'http://testserver{informatieobjecttype_url}'],
+            "besluittypen": [],
+            "beginGeldigheid": "2018-01-01",
+            "versiedatum": "2018-01-01",
+        }
+
+        response = self.client.put(zaaktype_url, data)
+
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+        self.assertEqual(response.data["aanleiding"], "aangepast")
+
+        zaaktype.refresh_from_db()
+        self.assertEqual(ZaakTypenRelatie.objects.filter(zaaktype=zaaktype).count(), 1)
+        self.assertEqual(zaaktype.aanleiding, "aangepast")
+
     def test_update_zaaktype_fail_not_concept(self):
         zaaktype = ZaakTypeFactory.create(concept=False)
         zaaktype_url = reverse(zaaktype)