From b3a05e64a9772caf0626e8a0801c2d476e3749a7 Mon Sep 17 00:00:00 2001 From: bart-maykin Date: Wed, 31 Jul 2024 14:47:10 +0200 Subject: [PATCH] :sparkles: [#207] changed 'indicatie_geheimhouding' from required to optional --- .../api/tests/test_partijen.py | 9 ++++---- ...16_alter_partij_indicatie_geheimhouding.py | 23 +++++++++++++++++++ .../klantinteracties/models/partijen.py | 5 +++- .../components/klantinteracties/openapi.yaml | 9 +++++--- src/openklant/conf/dev.py | 1 + 5 files changed, 38 insertions(+), 9 deletions(-) create mode 100644 src/openklant/components/klantinteracties/migrations/0016_alter_partij_indicatie_geheimhouding.py diff --git a/src/openklant/components/klantinteracties/api/tests/test_partijen.py b/src/openklant/components/klantinteracties/api/tests/test_partijen.py index 238e8083..e2e07a1f 100644 --- a/src/openklant/components/klantinteracties/api/tests/test_partijen.py +++ b/src/openklant/components/klantinteracties/api/tests/test_partijen.py @@ -88,7 +88,6 @@ def test_create_partij(self): "rekeningnummers": [{"uuid": str(rekeningnummer.uuid)}], "voorkeursRekeningnummer": {"uuid": str(rekeningnummer.uuid)}, "soortPartij": "persoon", - "indicatieGeheimhouding": True, "voorkeurstaal": "ndl", "indicatieActief": True, "bezoekadres": { @@ -132,7 +131,7 @@ def test_create_partij(self): data["voorkeursRekeningnummer"]["uuid"], str(rekeningnummer.uuid) ) self.assertEqual(data["soortPartij"], "persoon") - self.assertTrue(data["indicatieGeheimhouding"]) + self.assertIsNone(data["indicatieGeheimhouding"]) self.assertEqual(data["voorkeurstaal"], "ndl") self.assertTrue(data["indicatieActief"]) self.assertEqual( @@ -188,7 +187,7 @@ def test_create_partij(self): self.assertEqual(response_data["rekeningnummers"], []) self.assertIsNone(response_data["voorkeursRekeningnummer"]) self.assertEqual(response_data["soortPartij"], "persoon") - self.assertTrue(response_data["indicatieGeheimhouding"]) + self.assertIsNone(data["indicatieGeheimhouding"]) self.assertEqual(response_data["voorkeurstaal"], "ndl") self.assertTrue(response_data["indicatieActief"]) self.assertEqual( @@ -663,7 +662,7 @@ def test_update_partij(self): "rekeningnummers": [{"uuid": str(rekeningnummer2.uuid)}], "voorkeursRekeningnummer": {"uuid": str(rekeningnummer2.uuid)}, "soortPartij": "persoon", - "indicatieGeheimhouding": False, + "indicatieGeheimhouding": None, "voorkeurstaal": "ger", "indicatieActief": False, "bezoekadres": { @@ -721,7 +720,7 @@ def test_update_partij(self): data["voorkeursRekeningnummer"]["uuid"], str(rekeningnummer2.uuid) ) self.assertEqual(data["soortPartij"], "persoon") - self.assertFalse(data["indicatieGeheimhouding"]) + self.assertIsNone(data["indicatieGeheimhouding"]) self.assertEqual(data["voorkeurstaal"], "ger") self.assertFalse(data["indicatieActief"]) self.assertEqual( diff --git a/src/openklant/components/klantinteracties/migrations/0016_alter_partij_indicatie_geheimhouding.py b/src/openklant/components/klantinteracties/migrations/0016_alter_partij_indicatie_geheimhouding.py new file mode 100644 index 00000000..78023860 --- /dev/null +++ b/src/openklant/components/klantinteracties/migrations/0016_alter_partij_indicatie_geheimhouding.py @@ -0,0 +1,23 @@ +# Generated by Django 4.2.11 on 2024-07-31 11:37 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ("klantinteracties", "0015_internetaak_afgehandeld_op"), + ] + + operations = [ + migrations.AlterField( + model_name="partij", + name="indicatie_geheimhouding", + field=models.BooleanField( + default=None, + help_text="Geeft aan of de verstrekker van partijgegevens heeft aangegeven dat deze gegevens als geheim beschouwd moeten worden. Als dit niet aangegeven is dan wordt dit ingevuld als `null`.", + null=True, + verbose_name="indicatie geheimhouding", + ), + ), + ] diff --git a/src/openklant/components/klantinteracties/models/partijen.py b/src/openklant/components/klantinteracties/models/partijen.py index 2cb5c782..790bd632 100644 --- a/src/openklant/components/klantinteracties/models/partijen.py +++ b/src/openklant/components/klantinteracties/models/partijen.py @@ -67,8 +67,11 @@ class Partij(APIMixin, BezoekadresMixin, CorrespondentieadresMixin): _("indicatie geheimhouding"), help_text=_( "Geeft aan of de verstrekker van partijgegevens heeft aangegeven dat " - "deze gegevens als geheim beschouwd moeten worden." + "deze gegevens als geheim beschouwd moeten worden. Als dit niet aangegeven is " + "dan wordt dit ingevuld als `null`." ), + null=True, + default=None, ) voorkeurstaal = models.CharField( _("voorkeurstaal"), diff --git a/src/openklant/components/klantinteracties/openapi.yaml b/src/openklant/components/klantinteracties/openapi.yaml index 56e6055b..af6fad1e 100644 --- a/src/openklant/components/klantinteracties/openapi.yaml +++ b/src/openklant/components/klantinteracties/openapi.yaml @@ -4006,8 +4006,10 @@ components: description: Geeft aan van welke specifieke soort partij sprake is. indicatieGeheimhouding: type: boolean + nullable: true description: Geeft aan of de verstrekker van partijgegevens heeft aangegeven - dat deze gegevens als geheim beschouwd moeten worden. + dat deze gegevens als geheim beschouwd moeten worden. Als dit niet aangegeven + is dan wordt dit ingevuld als `null`. voorkeurstaal: type: string description: 'Taal, in ISO 639-2/B formaat, waarin de partij bij voorkeur @@ -4037,7 +4039,6 @@ components: - categorieRelaties - digitaleAdressen - indicatieActief - - indicatieGeheimhouding - partijIdentificatoren - rekeningnummers - soortPartij @@ -4752,8 +4753,10 @@ components: description: Geeft aan van welke specifieke soort partij sprake is. indicatieGeheimhouding: type: boolean + nullable: true description: Geeft aan of de verstrekker van partijgegevens heeft aangegeven - dat deze gegevens als geheim beschouwd moeten worden. + dat deze gegevens als geheim beschouwd moeten worden. Als dit niet aangegeven + is dan wordt dit ingevuld als `null`. voorkeurstaal: type: string description: 'Taal, in ISO 639-2/B formaat, waarin de partij bij voorkeur diff --git a/src/openklant/conf/dev.py b/src/openklant/conf/dev.py index d95bc0e9..b2bdd7c8 100644 --- a/src/openklant/conf/dev.py +++ b/src/openklant/conf/dev.py @@ -15,6 +15,7 @@ os.environ.setdefault("DB_PASSWORD", "openklant"), os.environ.setdefault("ENVIRONMENT", "development") +os.environ.setdefault("DISABLE_2FA", "yes") os.environ.setdefault("RELEASE", "dev") os.environ.setdefault("LOG_REQUESTS", "no")