diff --git a/src/openklant/components/contactgegevens/admin.py b/src/openklant/components/contactgegevens/admin.py index 293b6b8a..73dd23dc 100644 --- a/src/openklant/components/contactgegevens/admin.py +++ b/src/openklant/components/contactgegevens/admin.py @@ -19,7 +19,6 @@ class PersoonAdmin(admin.ModelAdmin): "geslachtsnaam", "geboortedatum", "overlijdensdatum", - "land", ] }, ), @@ -55,7 +54,6 @@ class OrganisatieAdmin(admin.ModelAdmin): "handelsnaam", "oprichtingsdatum", "opheffingsdatum", - "land", ] }, ), diff --git a/src/openklant/components/contactgegevens/api/serializers.py b/src/openklant/components/contactgegevens/api/serializers.py index e4050719..163c9534 100644 --- a/src/openklant/components/contactgegevens/api/serializers.py +++ b/src/openklant/components/contactgegevens/api/serializers.py @@ -31,7 +31,6 @@ class Meta: "oprichtingsdatum", "opheffingsdatum", "adres", - "land", ) extra_kwargs = { "uuid": {"read_only": True}, @@ -71,7 +70,6 @@ class Meta: "voorvoegsel", "voornamen", "adres", - "land", ) extra_kwargs = { "uuid": {"read_only": True}, diff --git a/src/openklant/components/contactgegevens/api/tests/test_apis.py b/src/openklant/components/contactgegevens/api/tests/test_apis.py index e8c55a07..21e97ea3 100644 --- a/src/openklant/components/contactgegevens/api/tests/test_apis.py +++ b/src/openklant/components/contactgegevens/api/tests/test_apis.py @@ -27,7 +27,6 @@ def test_persoon_detail(self): adres_adresregel2="adresregel2", adres_adresregel3="adresregel3", adres_land="CA", - land="CA", ) detail_url = reverse( "contactgegevens:persoon-detail", @@ -56,7 +55,6 @@ def test_persoon_detail(self): "voorvoegsel": "", "voornamen": "John", "adres": expected_adres, - "land": "CA", } response = self.client.get(detail_url) @@ -85,7 +83,6 @@ def test_create_persoon(self): "adresregel3": "adresregel3", "land": "CA", }, - "land": "CA", } response = self.client.post(list_url, data) @@ -115,7 +112,6 @@ def test_create_persoon(self): "land": "CA", }, ) - self.assertEqual(data["land"], "CA") def test_update_persoon(self): persoon = PersoonFactory.create( @@ -135,7 +131,6 @@ def test_update_persoon(self): adres_adresregel2="adresregel2", adres_adresregel3="adresregel3", adres_land="CA", - land="CA", ) detail_url = reverse( "contactgegevens:persoon-detail", @@ -169,7 +164,6 @@ def test_update_persoon(self): "land": "CA", }, ) - self.assertEqual(data["land"], "CA") data = { "geboortedatum": "1972-05-06", @@ -190,7 +184,6 @@ def test_update_persoon(self): "adresregel3": "changed", "land": "FR", }, - "land": "FR", } response = self.client.put(detail_url, data) data = response.json() @@ -216,7 +209,6 @@ def test_update_persoon(self): "land": "FR", }, ) - self.assertEqual(data["land"], "FR") def test_update_partial_persoon(self): persoon = PersoonFactory.create( @@ -236,7 +228,6 @@ def test_update_partial_persoon(self): adres_adresregel2="adresregel2", adres_adresregel3="adresregel3", adres_land="CA", - land="CA", ) detail_url = reverse( "contactgegevens:persoon-detail", @@ -270,7 +261,6 @@ def test_update_partial_persoon(self): "land": "CA", }, ) - self.assertEqual(data["land"], "CA") data = { "overlijdensdatum": "2023-11-22", @@ -299,7 +289,6 @@ def test_update_partial_persoon(self): "land": "CA", }, ) - self.assertEqual(data["land"], "CA") def test_list_pagination_pagesize_param(self): list_url = reverse("contactgegevens:persoon-list") @@ -331,7 +320,6 @@ def test_organisatie_detail(self): adres_adresregel2="adresregel2", adres_adresregel3="adresregel3", adres_land="CA", - land="CA", ) detail_url = reverse( "contactgegevens:organisatie-detail", @@ -357,7 +345,6 @@ def test_organisatie_detail(self): "opheffingsdatum": "2020-09-05", "handelsnaam": "Devin Townsend", "adres": expected_adres, - "land": "CA", } response = self.client.get(detail_url) @@ -384,7 +371,6 @@ def test_create_organisatie(self): "adresregel3": "adresregel3", "land": "CA", }, - "land": "CA", } response = self.client.post(list_url, data) @@ -411,7 +397,6 @@ def test_create_organisatie(self): "land": "CA", }, ) - self.assertEqual(data["land"], "CA") def test_update_organisatie(self): organisatie = OrganisatieFactory.create( @@ -428,7 +413,6 @@ def test_update_organisatie(self): adres_adresregel2="adresregel2", adres_adresregel3="adresregel3", adres_land="CA", - land="CA", ) detail_url = reverse( "contactgegevens:organisatie-detail", @@ -459,7 +443,6 @@ def test_update_organisatie(self): "land": "CA", }, ) - self.assertEqual(data["land"], "CA") data = { "handelsnaam": "changed", @@ -477,7 +460,6 @@ def test_update_organisatie(self): "adresregel3": "changed", "land": "FR", }, - "land": "FR", } response = self.client.put(detail_url, data) data = response.json() @@ -499,7 +481,6 @@ def test_update_organisatie(self): "land": "FR", }, ) - self.assertEqual(data["land"], "FR") def test_update_partial_organisatie(self): organisatie = OrganisatieFactory.create( @@ -516,7 +497,6 @@ def test_update_partial_organisatie(self): adres_adresregel2="adresregel2", adres_adresregel3="adresregel3", adres_land="CA", - land="CA", ) detail_url = reverse( "contactgegevens:organisatie-detail", @@ -547,7 +527,6 @@ def test_update_partial_organisatie(self): "land": "CA", }, ) - self.assertEqual(data["land"], "CA") data = { "opheffingsdatum": "2023-11-22", @@ -574,7 +553,6 @@ def test_update_partial_organisatie(self): "land": "CA", }, ) - self.assertEqual(data["land"], "CA") def test_list_pagination_pagesize_param(self): list_url = reverse("contactgegevens:organisatie-list") diff --git a/src/openklant/components/contactgegevens/migrations/0006_organisatie_adres_huisnummer_and_more.py b/src/openklant/components/contactgegevens/migrations/0006_remove_organisatie_land_remove_persoon_land_and_more.py similarity index 94% rename from src/openklant/components/contactgegevens/migrations/0006_organisatie_adres_huisnummer_and_more.py rename to src/openklant/components/contactgegevens/migrations/0006_remove_organisatie_land_remove_persoon_land_and_more.py index 46ae4631..38492726 100644 --- a/src/openklant/components/contactgegevens/migrations/0006_organisatie_adres_huisnummer_and_more.py +++ b/src/openklant/components/contactgegevens/migrations/0006_remove_organisatie_land_remove_persoon_land_and_more.py @@ -1,4 +1,4 @@ -# Generated by Django 4.2.17 on 2025-02-21 16:05 +# Generated by Django 4.2.17 on 2025-02-27 10:22 import django.core.validators from django.db import migrations, models @@ -15,6 +15,14 @@ class Migration(migrations.Migration): ] operations = [ + migrations.RemoveField( + model_name="organisatie", + name="land", + ), + migrations.RemoveField( + model_name="persoon", + name="land", + ), migrations.AddField( model_name="organisatie", name="adres_huisnummer", diff --git a/src/openklant/components/contactgegevens/models.py b/src/openklant/components/contactgegevens/models.py index c6321f84..f334636f 100644 --- a/src/openklant/components/contactgegevens/models.py +++ b/src/openklant/components/contactgegevens/models.py @@ -40,18 +40,6 @@ class Organisatie(AdresMixin): "significante onderdeel. Een voorbeeld: 2022-02-21" ), ) - land = models.CharField( - _("land"), - help_text=_( - "ISO 3166-code die het land (buiten Nederland) aangeeft alwaar de ingeschrevene verblijft." - ), - validators=[ - MinLengthValidator(limit_value=2), - validate_country, - ], - max_length=2, - blank=True, - ) class Meta: verbose_name = _("organisatie") @@ -125,18 +113,6 @@ class Persoon(AdresMixin): max_length=200, blank=True, ) - land = models.CharField( - _("land"), - help_text=_( - "ISO 3166-code die het land (buiten Nederland) aangeeft alwaar de ingeschrevene verblijft." - ), - validators=[ - MinLengthValidator(limit_value=2), - validate_country, - ], - max_length=2, - blank=True, - ) class Meta: verbose_name = _("persoon") diff --git a/src/openklant/components/contactgegevens/openapi.yaml b/src/openklant/components/contactgegevens/openapi.yaml index f46f82d5..612eb768 100644 --- a/src/openklant/components/contactgegevens/openapi.yaml +++ b/src/openklant/components/contactgegevens/openapi.yaml @@ -365,12 +365,6 @@ components: - $ref: '#/components/schemas/OrganisatieAdres' nullable: true description: De adres gegevens van een organisatie. - land: - type: string - description: ISO 3166-code die het land (buiten Nederland) aangeeft alwaar - de ingeschrevene verblijft. - maxLength: 2 - minLength: 2 required: - handelsnaam - url @@ -519,12 +513,6 @@ components: - $ref: '#/components/schemas/OrganisatieAdres' nullable: true description: De adres gegevens van een organisatie. - land: - type: string - description: ISO 3166-code die het land (buiten Nederland) aangeeft alwaar - de ingeschrevene verblijft. - maxLength: 2 - minLength: 2 PatchedPersoon: type: object description: |- @@ -585,12 +573,6 @@ components: - $ref: '#/components/schemas/PersoonAdres' nullable: true description: De adres gegevens van een organisatie. - land: - type: string - description: ISO 3166-code die het land (buiten Nederland) aangeeft alwaar - de ingeschrevene verblijft. - maxLength: 2 - minLength: 2 Persoon: type: object description: |- @@ -651,12 +633,6 @@ components: - $ref: '#/components/schemas/PersoonAdres' nullable: true description: De adres gegevens van een organisatie. - land: - type: string - description: ISO 3166-code die het land (buiten Nederland) aangeeft alwaar - de ingeschrevene verblijft. - maxLength: 2 - minLength: 2 required: - geboortedatum - geslachtsnaam