Merge pull request #331 from nl-portal/feature/fix-lastname-brp

fix for official lastname based on AanduidingNaamGebruik, which was n…

Author: flinden68

gradle.properties

@@ -14,4 +14,4 @@ dokkaVersion=1.9.20
 
 org.gradle.jvmargs=-Xmx6g -XX:MaxMetaspaceSize=2048m
 org.gradle.workers.max=10
-version=1.4.30-SNAPSHOT
+version=1.4.31-SNAPSHOT

haalcentraal/haalcentraal-all/src/test/kotlin/nl/nlportal/haalcentraal/all/graphql/GemachtigdeQueryTest.kt

@@ -22,6 +22,7 @@ import kotlinx.coroutines.test.runTest
 import nl.nlportal.haalcentraal.hr.domain.MaterieleRegistratie
 import nl.nlportal.commonground.authentication.CommonGroundAuthentication
 import nl.nlportal.graphql.security.SecurityConstants.AUTHENTICATION_KEY
+import nl.nlportal.haalcentraal.brp.domain.persoon.AanduidingNaamGebruik
 import nl.nlportal.haalcentraal.brp.domain.persoon.PersoonNaam
 import nl.nlportal.haalcentraal.brp.service.HaalCentraalBrpService
 import nl.nlportal.haalcentraal.hr.domain.MaatschappelijkeActiviteit
@@ -59,6 +60,7 @@ internal class GemachtigdeQueryTest {
                     "test",
                     "test",
                     "test",
+                    AanduidingNaamGebruik.EIGEN,
                 ),
             )
             whenever(handelsregisterService.getGemachtigde(authentication)).thenReturn(

haalcentraal/haalcentraal-all/src/test/resources/data/get-ingeschreven-persoon.json

@@ -46,7 +46,7 @@
         "aanhef": "Geachte mevrouw Kooyman",
         "aanschrijfwijze": "M. Kooyman",
         "gebruikInLopendeTekst": "mevrouw Kooyman",
-        "aanduidingNaamgebruik": "eigen",
+        "aanduidingNaamgebruik": "partner",
         "voornamen": "Merel",
         "voorletters": "M.",
         "voorvoegsel": "de",
@@ -208,6 +208,7 @@
             "naam": {
                 "voornamen": "Martin Gerard",
                 "voorletters": "M.G.",
+                "voorvoegsel": "van der",
                 "geslachtsnaam": "Maassen"
             },
             "aangaanHuwelijkPartnerschap": {
@@ -246,4 +247,4 @@
             "dag": 10
         }
     }
-}
+}

haalcentraal/haalcentraal-brp/src/main/kotlin/nl/nlportal/haalcentraal/brp/domain/persoon/Persoon.kt

@@ -25,7 +25,7 @@ data class Persoon(
     val burgerservicenummer: String? = null,
     val geslachtsaanduiding: String? = null,
     val geheimhoudingPersoonsgegevens: Boolean? = false,
-    val naam: PersoonNaam? = null,
+    val naam: PersoonNaam,
     val geboorte: PersoonDatumLandPlaats? = null,
     val nationaliteiten: List<PersoonNationaliteiten>? = null,
     val verblijfplaats: PersoonVerblijfplaats? = null,
@@ -45,4 +45,25 @@ data class Persoon(
             haalCentraalBrpService.getBewonersAantal(dfe.graphQlContext[AUTHENTICATION_KEY], it)
         }
     }
+
+    fun officialLastName(): String {
+        if (naam.aanduidingNaamgebruik == null) {
+            return naam.lastName()
+        }
+        val lastNamePartner = partners?.firstOrNull { it.naam != null }?.naam?.lastName() ?: ""
+        return when (naam.aanduidingNaamgebruik) {
+            AanduidingNaamGebruik.PARTNER -> {
+                lastNamePartner
+            }
+            AanduidingNaamGebruik.PARTNER_EIGEN -> {
+                "$lastNamePartner - ${naam.lastName()}"
+            }
+            AanduidingNaamGebruik.EIGEN_PARTNER -> {
+                "${naam.lastName()} - $lastNamePartner"
+            }
+            else -> {
+                naam.lastName()
+            }
+        }
+    }
 }

haalcentraal/haalcentraal-brp/src/main/kotlin/nl/nlportal/haalcentraal/brp/domain/persoon/PersoonKind.kt

@@ -18,6 +18,6 @@ package nl.nlportal.haalcentraal.brp.domain.persoon
 data class PersoonKind(
     val burgerservicenummer: String? = null,
     val leeftijd: Int? = null,
-    val naam: PersoonNaam? = null,
+    val naam: Naam? = null,
     val geboorte: PersoonDatumLandPlaats? = null,
 )

haalcentraal/haalcentraal-brp/src/main/kotlin/nl/nlportal/haalcentraal/brp/domain/persoon/PersoonNaam.kt

@@ -15,10 +15,30 @@
  */
 package nl.nlportal.haalcentraal.brp.domain.persoon
 
+import com.fasterxml.jackson.annotation.JsonValue
+
 data class PersoonNaam(
     val aanhef: String? = null,
     val voornamen: String? = null,
     val voorletters: String? = null,
     val voorvoegsel: String? = null,
     val geslachtsnaam: String? = null,
-)
+    val aanduidingNaamgebruik: AanduidingNaamGebruik?,
+) {
+    fun lastName(): String {
+        return if (voorvoegsel != null && geslachtsnaam != null) {
+            "$voorvoegsel $geslachtsnaam"
+        } else {
+            geslachtsnaam ?: ""
+        }
+    }
+}
+
+enum class AanduidingNaamGebruik(
+    @JsonValue val value: String,
+) {
+    EIGEN("eigen"),
+    EIGEN_PARTNER("eigen_partner"),
+    PARTNER("partner"),
+    PARTNER_EIGEN("partner_eigen"),
+}

haalcentraal/haalcentraal-brp/src/main/kotlin/nl/nlportal/haalcentraal/brp/domain/persoon/PersoonOuder.kt

@@ -20,6 +20,6 @@ data class PersoonOuder(
     val geslachtsaanduiding: String? = null,
     val ouderAanduiding: String? = null,
     val datumIngangFamilierechtelijkeBetrekking: PersoonDatum? = null,
-    val naam: PersoonNaam? = null,
+    val naam: Naam? = null,
     val geboorte: PersoonDatumLandPlaats? = null,
 )

haalcentraal/haalcentraal-brp/src/main/kotlin/nl/nlportal/haalcentraal/brp/domain/persoon/PersoonPartner.kt

@@ -20,7 +20,21 @@ data class PersoonPartner(
     val geslachtsaanduiding: String? = null,
     val soortVerbintenis: String? = null,
     val datumIngangFamilierechtelijkeBetrekking: PersoonDatum? = null,
-    val naam: PersoonNaam? = null,
+    val naam: Naam? = null,
     val geboorte: PersoonDatumLandPlaats? = null,
     val aangaanHuwelijkPartnerschap: PersoonDatumLandPlaats? = null,
-)
+)
+
+data class Naam(
+    val voorletters: String? = null,
+    val voorvoegsel: String? = null,
+    val geslachtsnaam: String? = null,
+) {
+    fun lastName(): String {
+        return if (voorvoegsel != null && geslachtsnaam != null) {
+            "$voorvoegsel $geslachtsnaam"
+        } else {
+            geslachtsnaam ?: ""
+        }
+    }
+}

haalcentraal/haalcentraal-brp/src/test/kotlin/nl/nlportal/haalcentraal/brp/client/HaalCentraalBrpClientTest.kt

@@ -21,6 +21,7 @@ import nl.nlportal.commonground.authentication.JwtBuilder
 import nl.nlportal.core.ssl.ClientKey
 import nl.nlportal.core.ssl.Ssl
 import nl.nlportal.core.ssl.StringClientSslContextResolver
+import nl.nlportal.haalcentraal.brp.domain.persoon.AanduidingNaamGebruik
 import nl.nlportal.haalcentraal.brp.domain.persoon.Persoon
 import nl.nlportal.haalcentraal.brp.domain.persoon.PersoonNaam
 import nl.nlportal.haalcentraal.client.HaalCentraalClientConfig
@@ -40,7 +41,11 @@ internal class HaalCentraalBrpClientTest {
     private lateinit var haalCentraalClientConfig: HaalCentraalClientConfig
     private lateinit var server: MockWebServer
     private val bsn = "0123456789"
-    private val persoon = Persoon(burgerservicenummer = bsn, naam = PersoonNaam(voornamen = "John", geslachtsnaam = "Doe"))
+    private val persoon =
+        Persoon(
+            burgerservicenummer = bsn,
+            naam = PersoonNaam(voornamen = "John", geslachtsnaam = "Doe", aanduidingNaamgebruik = AanduidingNaamGebruik.EIGEN),
+        )
 
     @BeforeEach
     internal fun setUp() {

haalcentraal/haalcentraal-brp/src/test/kotlin/nl/nlportal/haalcentraal/brp/graphql/PersoonQueryIT.kt

@@ -70,8 +70,11 @@ internal class PersoonQueryIT(
                         voorletters,
                         voornamen,
                         voorvoegsel,
-                        geslachtsnaam
-                    }
+                        geslachtsnaam,
+                        lastName,
+                        aanduidingNaamgebruik,
+                    },
+                    officialLastName
                 }
             }
             """.trimIndent()
@@ -214,7 +217,6 @@ internal class PersoonQueryIT(
             .jsonPath("$basePath.kinderen[0].leeftijd").isEqualTo(18)
             .jsonPath("$basePath.geheimhoudingPersoonsgegevens").isEqualTo(false)
             .jsonPath("$basePath.nationaliteiten").isArray()
-            .jsonPath("$basePath.nationaliteiten[0].nationaliteit.omschrijving", contains("Nederlandse", "Franse"))
     }
 
     private fun setupMockServer() {

haalcentraal/haalcentraal-brp/src/test/kotlin/nl/nlportal/haalcentraal/brp/impl/HaalCentraalBrpServiceTest.kt

@@ -21,6 +21,7 @@ import nl.nlportal.haalcentraal.brp.domain.persoon.PersoonNaam
 import nl.nlportal.haalcentraal.brp.client.HaalCentraalBrpClient
 import kotlinx.coroutines.ExperimentalCoroutinesApi
 import kotlinx.coroutines.test.runTest
+import nl.nlportal.haalcentraal.brp.domain.persoon.AanduidingNaamGebruik
 import nl.nlportal.haalcentraal.brp.service.impl.HaalCentraalBrpServiceImpl
 import org.junit.jupiter.api.Assertions.assertEquals
 import org.junit.jupiter.api.Assertions.assertNull
@@ -49,6 +50,7 @@ class HaalCentraalBrpServiceTest {
                         "V.",
                         "van",
                         "Achternaam",
+                        AanduidingNaamGebruik.EIGEN,
                     ),
                     null,
                     null,
@@ -89,6 +91,7 @@ class HaalCentraalBrpServiceTest {
                     "V.",
                     "van",
                     "Achternaam",
+                    AanduidingNaamGebruik.EIGEN,
                 ),
             )
 

haalcentraal/haalcentraal-brp/src/test/resources/data/get-ingeschreven-persoon.json

@@ -46,7 +46,7 @@
         "aanhef": "Geachte mevrouw Kooyman",
         "aanschrijfwijze": "M. Kooyman",
         "gebruikInLopendeTekst": "mevrouw Kooyman",
-        "aanduidingNaamgebruik": "eigen",
+        "aanduidingNaamgebruik": "partner",
         "voornamen": "Merel",
         "voorletters": "M.",
         "voorvoegsel": "de",
@@ -208,6 +208,7 @@
             "naam": {
                 "voornamen": "Martin Gerard",
                 "voorletters": "M.G.",
+                "voorvoegsel": "van der",
                 "geslachtsnaam": "Maassen"
             },
             "aangaanHuwelijkPartnerschap": {
@@ -246,4 +247,4 @@
             "dag": 10
         }
     }
-}
+}

settings.gradle.kts

@@ -26,7 +26,7 @@ pluginManagement {
     }
 }
 plugins {
-    id("org.gradle.toolchains.foojay-resolver-convention") version "0.8.0"
+    id("org.gradle.toolchains.foojay-resolver-convention") version "0.9.0"
 }
 
 include(