Merge pull request #80 from navikt/pdl_query

Pdl query
navikt · Dec 17, 2020 · b6528a6 · b6528a6
2 parents 0fc60c1 + 1dbb434
commit b6528a6
Show file tree

Hide file tree

Showing 4 changed files with 75 additions and 48 deletions.
diff --git a/src/main/kotlin/no/nav/klage/oppgave/api/mapper/OppgaveMapper.kt b/src/main/kotlin/no/nav/klage/oppgave/api/mapper/OppgaveMapper.kt
@@ -3,15 +3,23 @@ package no.nav.klage.oppgave.api.mapper
 
 import no.nav.klage.oppgave.api.view.*
 import no.nav.klage.oppgave.clients.gosys.*
-import no.nav.klage.oppgave.clients.pdl.Navn
+import no.nav.klage.oppgave.clients.pdl.HentPersonBolkResult
 import no.nav.klage.oppgave.clients.pdl.PdlClient
+import no.nav.klage.oppgave.util.getLogger
+import no.nav.klage.oppgave.util.getSecureLogger
 import org.springframework.stereotype.Service
 import no.nav.klage.oppgave.api.view.Oppgave as OppgaveView
 import no.nav.klage.oppgave.clients.gosys.Oppgave as OppgaveBackend
 
 @Service
 class OppgaveMapper(val pdlClient: PdlClient) {
 
+    companion object {
+        @Suppress("JAVA_CLASS_ON_COMPANION")
+        private val logger = getLogger(javaClass.enclosingClass)
+        private val secureLogger = getSecureLogger()
+    }
+
     fun mapOppgaveToView(oppgaveBackend: OppgaveBackend, fetchPersoner: Boolean): OppgaveView {
         return mapOppgaverToView(listOf(oppgaveBackend), fetchPersoner).single()
     }
@@ -63,17 +71,43 @@ class OppgaveMapper(val pdlClient: PdlClient) {
         }
 
     private fun getPersoner(fnrList: List<String>): Map<String, OppgaveView.Person> {
-        val people = pdlClient.getPersonInfo(fnrList).data?.hentPersonBolk
-        return people?.map {
-            val fnr = it.folkeregisteridentifikator.first().identifikasjonsnummer
+        logger.debug("getPersoner is called with {} fnr", fnrList.size)
+        secureLogger.debug("getPersoner with fnr: {}", fnrList)
+
+        val people = pdlClient.getPersonInfo(fnrList).data?.hentPersonBolk ?: emptyList()
+
+        logger.debug("pdl returned {} people", people.size)
+        secureLogger.debug("pdl returned {}", people)
+
+        val fnrToPerson: Map<String, no.nav.klage.oppgave.api.view.Oppgave.Person> = people.map {
+            val fnr = it.ident
             fnr to OppgaveView.Person(
                 fnr = fnr,
-                navn = it.navn.firstOrNull()?.toName() ?: "mangler"
+                navn = it.person.navn.firstOrNull()?.toName() ?: "mangler navn"
             )
-        }?.toMap() ?: emptyMap()
+        }.toMap()
+        return fnrList.map {
+            if (fnrToPerson.containsKey(it)) {
+                Pair(it, fnrToPerson.getValue(it))
+            } else {
+                Pair(it, OppgaveView.Person(fnr = it, navn = "Mangler navn"))
+            }
+        }.toMap()
     }
 
-    private fun OppgaveBackend.getFnrForBruker() = identer?.find { i -> i.gruppe == Gruppe.FOLKEREGISTERIDENT }?.ident
+    private fun OppgaveBackend.getFnrForBruker(): String? {
+        logger.debug("getFnrForBruker is called")
+        secureLogger.debug("getFnrForBruker is called with oppgave: {}", this)
+
+        return identer?.find { i -> i.gruppe == Gruppe.FOLKEREGISTERIDENT }?.ident.also {
+            if (it != null) {
+                logger.debug("Returning found fnr")
+                secureLogger.debug("Returning found fnr from oppgave: {}", it)
+            } else {
+                logger.debug("No fnr found in oppgave")
+            }
+        }
+    }
 
-    private fun Navn.toName() = "$fornavn $etternavn"
+    private fun HentPersonBolkResult.Person.Navn.toName() = "$fornavn $etternavn"
 }
diff --git a/src/main/kotlin/no/nav/klage/oppgave/clients/pdl/HentPersonResponse.kt b/src/main/kotlin/no/nav/klage/oppgave/clients/pdl/HentPersonResponse.kt
@@ -3,24 +3,21 @@ package no.nav.klage.oppgave.clients.pdl
 
 data class HentPersonResponse(val data: HentPersonBolk?, val errors: List<PdlError>? = null)
 
-data class HentPersonBolk(val hentPersonBolk: List<Person>?)
-
-data class Person(
-    val navn: List<Navn>,
-    val folkeregisteridentifikator: List<Folkeregisteridentifikator>
-)
-
-data class Navn(
-    val fornavn: String,
-    val mellomnavn: String? = null,
-    val etternavn: String
-)
-
-data class Folkeregisteridentifikator(
-    val identifikasjonsnummer: String,
-    val type: String,
-    val status: String
-)
+data class HentPersonBolk(val hentPersonBolk: List<HentPersonBolkResult>?)
+
+data class HentPersonBolkResult(
+    val person: Person,
+    val ident: String
+) {
+    data class Person(
+        val navn: List<Navn>
+    ) {
+        data class Navn(
+            val fornavn: String,
+            val etternavn: String
+        )
+    }
+}
 
 data class PdlError(
     val message: String,

diff --git a/src/main/resources/pdl/hentPerson.graphql b/src/main/resources/pdl/hentPerson.graphql
@@ -1,14 +1,11 @@
 query($identer: [ID!]!){
     hentPersonBolk(identer: $identer) {
-        navn {
-            fornavn,
-            mellomnavn,
-            etternavn
+        person {
+            navn {
+                fornavn,
+                etternavn
+            }
         },
-        folkeregisteridentifikator {
-            identifikasjonsnummer,
-            type,
-            status
-        }
+        ident
     }
 }
diff --git a/src/test/kotlin/no/nav/klage/oppgave/service/OppgaveFacadeTest.kt b/src/test/kotlin/no/nav/klage/oppgave/service/OppgaveFacadeTest.kt
@@ -8,7 +8,10 @@ import no.nav.klage.oppgave.api.view.HJEMMEL
 import no.nav.klage.oppgave.api.view.TYPE_ANKE
 import no.nav.klage.oppgave.api.view.TYPE_KLAGE
 import no.nav.klage.oppgave.clients.gosys.*
-import no.nav.klage.oppgave.clients.pdl.*
+import no.nav.klage.oppgave.clients.pdl.HentPersonBolk
+import no.nav.klage.oppgave.clients.pdl.HentPersonBolkResult
+import no.nav.klage.oppgave.clients.pdl.HentPersonResponse
+import no.nav.klage.oppgave.clients.pdl.PdlClient
 import no.nav.klage.oppgave.domain.OppgaverSearchCriteria
 import org.assertj.core.api.Assertions.assertThat
 import org.junit.jupiter.api.Test
@@ -78,20 +81,16 @@ internal class OppgaveFacadeTest {
         return HentPersonResponse(
             data = HentPersonBolk(
                 listOf(
-                    Person(
-                        navn = listOf(
-                            Navn(
-                                fornavn = "Test",
-                                etternavn = "Person"
+                    HentPersonBolkResult(
+                        HentPersonBolkResult.Person(
+                            navn = listOf(
+                                HentPersonBolkResult.Person.Navn(
+                                    fornavn = "Test",
+                                    etternavn = "Person"
+                                )
                             )
                         ),
-                        folkeregisteridentifikator = listOf(
-                            Folkeregisteridentifikator(
-                                identifikasjonsnummer = "12345678910",
-                                type = "FNR",
-                                status = ""
-                            )
-                        )
+                        ident = "12345678910"
                     )
                 )
             )