[#309] Deleted 'land' field at root level

maykinmedia · Feb 27, 2025 · 684545c · 684545c
1 parent 4073a7d
commit 684545c
Show file tree

Hide file tree

Showing 6 changed files with 9 additions and 75 deletions.
diff --git 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
@@ -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
@@ -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/..._organisatie_adres_huisnummer_and_more.py → ...atie_land_remove_persoon_land_and_more.py b/..._organisatie_adres_huisnummer_and_more.py → ...atie_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
@@ -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
@@ -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