Merge pull request #1487 from open-zaak/feature/1486-zaaktype-overlap

change unique check for ZaakType
open-zaak · Nov 1, 2023 · 59f9bca · 59f9bca
2 parents 93c2c4c + a66ddac
commit 59f9bca
Show file tree

Hide file tree

Showing 8 changed files with 9 additions and 7 deletions.
diff --git a/bin/postman_tests.sh b/bin/postman_tests.sh
@@ -2,7 +2,7 @@
 
 set -x
 
-POSTMAN_TESTS_REF=e8d1b827ebc7927255f8f462bcd8a2987ef8a99e
+POSTMAN_TESTS_REF=d13c71796be9360a22f86c98a39fa0352f08dafe
 
 # 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/api/serializers/zaaktype.py b/src/openzaak/components/catalogi/api/serializers/zaaktype.py
@@ -258,7 +258,7 @@ class Meta:
         }
 
         validators = [
-            GeldigheidValidator("zaaktype_omschrijving"),
+            GeldigheidValidator("identificatie"),
             RelationCatalogValidator("besluittypen"),
             ConceptUpdateValidator(),
             M2MConceptCreateValidator(["besluittypen", "informatieobjecttypen"]),

diff --git a/src/openzaak/components/catalogi/api/validators.py b/src/openzaak/components/catalogi/api/validators.py
@@ -48,7 +48,7 @@ def __call__(self, attrs, serializer):
             "einde_geldigheid", attrs, serializer
         )
         omschrijving = get_from_serializer_data_or_instance(
-            "omschrijving", attrs, serializer
+            self.omschrijving_field, attrs, serializer
         )
 
         if has_overlapping_objects(

diff --git a/src/openzaak/components/catalogi/api/viewsets/zaaktype.py b/src/openzaak/components/catalogi/api/viewsets/zaaktype.py
@@ -57,7 +57,7 @@ class ZaakTypeViewSet(
 
     Er wordt gevalideerd op:
     - geldigheid `catalogus` URL, dit moet een catalogus binnen dezelfde API zijn
-    - Uniciteit `catalogus` en `omschrijving`. Dezelfde omeschrijving mag enkel
+    - Uniciteit `catalogus` en `identificatie`. Dezelfde identificatie mag enkel
       opnieuw gebruikt worden als het zaaktype een andere geldigheidsperiode
       kent dan bestaande zaaktypen.
     - `deelzaaktypen` moeten tot dezelfde catalogus behoren als het ZAAKTYPE.

diff --git a/src/openzaak/components/catalogi/models/zaaktype.py b/src/openzaak/components/catalogi/models/zaaktype.py
@@ -337,7 +337,7 @@ class ZaakType(ETagMixin, APIMixin, ConceptMixin, GeldigheidMixin, models.Model)
     objects = SyncAutorisatieManager.from_queryset(GeldigheidQuerySet)()
 
     IDENTIFICATIE_PREFIX = "ZAAKTYPE"
-    omschrijving_field = "zaaktype_omschrijving"
+    omschrijving_field = "identificatie"
 
     class Meta:
         verbose_name = _("Zaaktype")

diff --git a/src/openzaak/components/catalogi/openapi.yaml b/src/openzaak/components/catalogi/openapi.yaml
@@ -3976,7 +3976,7 @@ paths:
       summary: Maak een ZAAKTYPE aan.
       description: "Maak een ZAAKTYPE aan.\n\nEr wordt gevalideerd op:\n- geldigheid\
         \ `catalogus` URL, dit moet een catalogus binnen dezelfde API zijn\n- Uniciteit\
-        \ `catalogus` en `omschrijving`. Dezelfde omeschrijving mag enkel\n  opnieuw\
+        \ `catalogus` en `identificatie`. Dezelfde identificatie mag enkel\n  opnieuw\
         \ gebruikt worden als het zaaktype een andere geldigheidsperiode\n  kent dan\
         \ bestaande zaaktypen.\n- `deelzaaktypen` moeten tot dezelfde catalogus behoren\
         \ als het ZAAKTYPE."

diff --git a/src/openzaak/components/catalogi/swagger2.0.json b/src/openzaak/components/catalogi/swagger2.0.json
@@ -5459,7 +5459,7 @@
             "post": {
                 "operationId": "zaaktype_create",
                 "summary": "Maak een ZAAKTYPE aan.",
-                "description": "Maak een ZAAKTYPE aan.\n\nEr wordt gevalideerd op:\n- geldigheid `catalogus` URL, dit moet een catalogus binnen dezelfde API zijn\n- Uniciteit `catalogus` en `omschrijving`. Dezelfde omeschrijving mag enkel\n  opnieuw gebruikt worden als het zaaktype een andere geldigheidsperiode\n  kent dan bestaande zaaktypen.\n- `deelzaaktypen` moeten tot dezelfde catalogus behoren als het ZAAKTYPE.",
+                "description": "Maak een ZAAKTYPE aan.\n\nEr wordt gevalideerd op:\n- geldigheid `catalogus` URL, dit moet een catalogus binnen dezelfde API zijn\n- Uniciteit `catalogus` en `identificatie`. Dezelfde identificatie mag enkel\n  opnieuw gebruikt worden als het zaaktype een andere geldigheidsperiode\n  kent dan bestaande zaaktypen.\n- `deelzaaktypen` moeten tot dezelfde catalogus behoren als het ZAAKTYPE.",
                 "parameters": [
                     {
                         "name": "Content-Type",

diff --git a/src/openzaak/components/catalogi/tests/test_zaaktype.py b/src/openzaak/components/catalogi/tests/test_zaaktype.py
@@ -1320,6 +1320,7 @@ def test_partial_update_non_concept_zaaktype_reset_einde_geldigheid(self):
         zaaktype = ZaakTypeFactory.create(
             concept=False,
             zaaktype_omschrijving="GH-981",
+            identificatie="ZAAKTYPE1",
             datum_begin_geldigheid=date(2021, 1, 1),
             datum_einde_geldigheid=date(2022, 1, 1),
         )
@@ -1337,6 +1338,7 @@ def test_partial_update_non_concept_zaaktype_reset_einde_geldigheid(self):
                 concept=False,
                 catalogus=zaaktype.catalogus,
                 zaaktype_omschrijving="GH-981",
+                identificatie="ZAAKTYPE1",
                 datum_begin_geldigheid=date(2020, 1, 1),
                 datum_einde_geldigheid=date(2020, 12, 31),
             )