[#310] Add validate_bag_id + migrations

maykinmedia · Jan 8, 2025 · b9c2d53 · b9c2d53
1 parent 2d1abfa
commit b9c2d53
Show file tree

Hide file tree

Showing 5 changed files with 132 additions and 5 deletions.
diff --git a/...s/contactgegevens/migrations/0004_alter_organisatie_adres_nummeraanduiding_id_and_more.py b/...s/contactgegevens/migrations/0004_alter_organisatie_adres_nummeraanduiding_id_and_more.py
@@ -0,0 +1,44 @@
+# Generated by Django 4.2.15 on 2025-01-08 11:12
+
+from django.db import migrations, models
+import openklant.utils.validators
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("contactgegevens", "0003_alter_persoon_overlijdensdatum"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="organisatie",
+            name="adres_nummeraanduiding_id",
+            field=models.CharField(
+                blank=True,
+                help_text="Identificatie van het adres bij de Basisregistratie Adressen en Gebouwen.",
+                max_length=16,
+                validators=[
+                    openklant.utils.validators.CustomRegexValidator(
+                        message="Ongeldige nummeraanduiding BAG-ID", regex="^[0-9]{16}$"
+                    )
+                ],
+                verbose_name="nummeraanduiding ID",
+            ),
+        ),
+        migrations.AlterField(
+            model_name="persoon",
+            name="adres_nummeraanduiding_id",
+            field=models.CharField(
+                blank=True,
+                help_text="Identificatie van het adres bij de Basisregistratie Adressen en Gebouwen.",
+                max_length=16,
+                validators=[
+                    openklant.utils.validators.CustomRegexValidator(
+                        message="Ongeldige nummeraanduiding BAG-ID", regex="^[0-9]{16}$"
+                    )
+                ],
+                verbose_name="nummeraanduiding ID",
+            ),
+        ),
+    ]
diff --git a/src/openklant/components/contactgegevens/mixins.py b/src/openklant/components/contactgegevens/mixins.py
@@ -4,14 +4,17 @@
 
 from vng_api_common.descriptors import GegevensGroepType
 
+from openklant.utils.validators import validate_bag_id
+
 
 class AdresMixin(models.Model):
     adres_nummeraanduiding_id = models.CharField(
         _("nummeraanduiding ID"),
         help_text=_(
             "Identificatie van het adres bij de Basisregistratie Adressen en Gebouwen."
         ),
-        max_length=255,
+        max_length=16,
+        validators=[validate_bag_id],
         blank=True,
     )
     adres_adresregel1 = models.CharField(

diff --git a/...tinteracties/migrations/0024_alter_betrokkene_bezoekadres_nummeraanduiding_id_and_more.py b/...tinteracties/migrations/0024_alter_betrokkene_bezoekadres_nummeraanduiding_id_and_more.py
@@ -0,0 +1,74 @@
+# Generated by Django 4.2.15 on 2025-01-08 11:12
+
+from django.db import migrations, models
+import openklant.utils.validators
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("klantinteracties", "0023_alter_digitaaladres_omschrijving"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="betrokkene",
+            name="bezoekadres_nummeraanduiding_id",
+            field=models.CharField(
+                blank=True,
+                help_text="Identificatie van het adres bij de Basisregistratie Adressen en Gebouwen.",
+                max_length=16,
+                validators=[
+                    openklant.utils.validators.CustomRegexValidator(
+                        message="Ongeldige nummeraanduiding BAG-ID", regex="^[0-9]{16}$"
+                    )
+                ],
+                verbose_name="nummeraanduiding ID",
+            ),
+        ),
+        migrations.AlterField(
+            model_name="betrokkene",
+            name="correspondentieadres_nummeraanduiding_id",
+            field=models.CharField(
+                blank=True,
+                help_text="Identificatie van het adres bij de Basisregistratie Adressen en Gebouwen.",
+                max_length=16,
+                validators=[
+                    openklant.utils.validators.CustomRegexValidator(
+                        message="Ongeldige nummeraanduiding BAG-ID", regex="^[0-9]{16}$"
+                    )
+                ],
+                verbose_name="nummeraanduiding ID",
+            ),
+        ),
+        migrations.AlterField(
+            model_name="partij",
+            name="bezoekadres_nummeraanduiding_id",
+            field=models.CharField(
+                blank=True,
+                help_text="Identificatie van het adres bij de Basisregistratie Adressen en Gebouwen.",
+                max_length=16,
+                validators=[
+                    openklant.utils.validators.CustomRegexValidator(
+                        message="Ongeldige nummeraanduiding BAG-ID", regex="^[0-9]{16}$"
+                    )
+                ],
+                verbose_name="nummeraanduiding ID",
+            ),
+        ),
+        migrations.AlterField(
+            model_name="partij",
+            name="correspondentieadres_nummeraanduiding_id",
+            field=models.CharField(
+                blank=True,
+                help_text="Identificatie van het adres bij de Basisregistratie Adressen en Gebouwen.",
+                max_length=16,
+                validators=[
+                    openklant.utils.validators.CustomRegexValidator(
+                        message="Ongeldige nummeraanduiding BAG-ID", regex="^[0-9]{16}$"
+                    )
+                ],
+                verbose_name="nummeraanduiding ID",
+            ),
+        ),
+    ]
diff --git a/src/openklant/components/klantinteracties/models/mixins.py b/src/openklant/components/klantinteracties/models/mixins.py
@@ -4,15 +4,17 @@
 
 from vng_api_common.descriptors import GegevensGroepType
 
+from openklant.utils.validators import validate_bag_id
+
 
 class BezoekadresMixin(models.Model):
-    # TODO: Check if this is correct.
     bezoekadres_nummeraanduiding_id = models.CharField(
         _("nummeraanduiding ID"),
         help_text=_(
             "Identificatie van het adres bij de Basisregistratie Adressen en Gebouwen."
         ),
-        max_length=255,
+        max_length=16,
+        validators=[validate_bag_id],
         blank=True,
     )
     bezoekadres_adresregel1 = models.CharField(
@@ -75,13 +77,13 @@ class Meta:
 
 
 class CorrespondentieadresMixin(models.Model):
-    # TODO: Check if this is correct.
     correspondentieadres_nummeraanduiding_id = models.CharField(
         _("nummeraanduiding ID"),
         help_text=_(
             "Identificatie van het adres bij de Basisregistratie Adressen en Gebouwen."
         ),
-        max_length=255,
+        max_length=16,
+        validators=[validate_bag_id],
         blank=True,
     )
     correspondentieadres_adresregel1 = models.CharField(

diff --git a/src/openklant/utils/validators.py b/src/openklant/utils/validators.py
@@ -55,3 +55,7 @@ def __call__(self, value):
 validate_no_space = CustomRegexValidator(
     regex="^[\S]+$", message=_("Geen spaties toegestaan")  # noqa
 )
+
+validate_bag_id = CustomRegexValidator(
+    regex="^[0-9]{16}$", message=_("Ongeldige nummeraanduiding BAG-ID")
+)