diff --git a/src/openzaak/components/catalogi/api/viewsets/zaaktype.py b/src/openzaak/components/catalogi/api/viewsets/zaaktype.py index 6dcdf8654..08e2268b9 100644 --- a/src/openzaak/components/catalogi/api/viewsets/zaaktype.py +++ b/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) diff --git a/src/openzaak/components/catalogi/tests/test_zaaktype.py b/src/openzaak/components/catalogi/tests/test_zaaktype.py index 4366a5409..6cb226f22 100644 --- a/src/openzaak/components/catalogi/tests/test_zaaktype.py +++ b/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, @@ -597,6 +597,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)