Fixes #870 -- Delete all ZaakTypenRelaties when doing a PUT call

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

@@ -126,12 +126,7 @@ class ZaakTypeViewSet(
     def perform_update(self, serializer):
 
         if not serializer.partial:
-            for zaaktypenrelaties in serializer.validated_data["zaaktypenrelaties"]:
-                # Delete any ZaakTypenRelatie so they can be recreated
-                ZaakTypenRelatie.objects.filter(
-                    zaaktype=serializer.instance,
-                    gerelateerd_zaaktype=zaaktypenrelaties["gerelateerd_zaaktype"],
-                ).delete()
+            serializer.instance.zaaktypenrelaties.all().delete()
 
         super().perform_update(serializer)
 

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

@@ -22,7 +22,7 @@
     M2MConceptUpdateValidator,
 )
 from ..constants import AardRelatieChoices, InternExtern
-from ..models import ZaakType
+from ..models import ZaakType, ZaakTypenRelatie
 from .base import APITestCase
 from .factories import (
     BesluitTypeFactory,
@@ -555,6 +555,58 @@ def test_update_zaaktype_with_existing_zaaktyperelatie(self):
         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)