From 4345db4891f8869806e6c54b6f479d6e52e47c78 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Geir=20Andr=C3=A9=20Lund?= Date: Fri, 22 Nov 2024 08:06:31 +0100 Subject: [PATCH] ktlinting. --- build.gradle.kts | 2 +- .../JournalpostBehovL\303\270ser.kt" | 3 +- .../behov/journalpost/JournalpostDokarkiv.kt | 19 ++- .../nav/dagpenger/mottak/behov/person/Pdl.kt | 144 +++++++++--------- .../no/nav/dagpenger/mottak/Innsending.kt | 17 ++- 5 files changed, 101 insertions(+), 84 deletions(-) diff --git a/build.gradle.kts b/build.gradle.kts index 3caf56ef..84d63069 100644 --- a/build.gradle.kts +++ b/build.gradle.kts @@ -1 +1 @@ -buildscript { repositories { mavenCentral() } } \ No newline at end of file +buildscript { repositories { mavenCentral() } } diff --git "a/mediator/src/main/kotlin/no/nav/dagpenger/mottak/behov/journalpost/JournalpostBehovL\303\270ser.kt" "b/mediator/src/main/kotlin/no/nav/dagpenger/mottak/behov/journalpost/JournalpostBehovL\303\270ser.kt" index a4da8159..2c35d918 100644 --- "a/mediator/src/main/kotlin/no/nav/dagpenger/mottak/behov/journalpost/JournalpostBehovL\303\270ser.kt" +++ "b/mediator/src/main/kotlin/no/nav/dagpenger/mottak/behov/journalpost/JournalpostBehovL\303\270ser.kt" @@ -60,6 +60,5 @@ internal class JournalpostBehovLøser( } } - private fun SafGraphQL.Journalpost.harDokumentitlerLengreEnn(lengde: Int) = - dokumenter.mapNotNull { dokument -> dokument.tittel }.any { tittel -> tittel.length > lengde } + private fun SafGraphQL.Journalpost.harDokumentitlerLengreEnn(lengde: Int) = dokumenter.mapNotNull { dokument -> dokument.tittel }.any { tittel -> tittel.length > lengde } } diff --git a/mediator/src/main/kotlin/no/nav/dagpenger/mottak/behov/journalpost/JournalpostDokarkiv.kt b/mediator/src/main/kotlin/no/nav/dagpenger/mottak/behov/journalpost/JournalpostDokarkiv.kt index 037757c4..8573d44b 100644 --- a/mediator/src/main/kotlin/no/nav/dagpenger/mottak/behov/journalpost/JournalpostDokarkiv.kt +++ b/mediator/src/main/kotlin/no/nav/dagpenger/mottak/behov/journalpost/JournalpostDokarkiv.kt @@ -25,7 +25,9 @@ internal class JournalpostApi { val journalfoerendeEnhet: String = "9999", ) - internal data class Sak(val fagsakId: String?) { + internal data class Sak( + val fagsakId: String?, + ) { val saksType: SaksType val fagsaksystem: String? @@ -40,11 +42,20 @@ internal class JournalpostApi { } } - internal data class Avsender(val id: String, val idType: String = "FNR") + internal data class Avsender( + val id: String, + val idType: String = "FNR", + ) - internal data class Bruker(val id: String, val idType: String = "FNR") + internal data class Bruker( + val id: String, + val idType: String = "FNR", + ) - internal data class Dokument(val dokumentInfoId: String, val tittel: String) + internal data class Dokument( + val dokumentInfoId: String, + val tittel: String, + ) internal enum class SaksType { GENERELL_SAK, diff --git a/mediator/src/main/kotlin/no/nav/dagpenger/mottak/behov/person/Pdl.kt b/mediator/src/main/kotlin/no/nav/dagpenger/mottak/behov/person/Pdl.kt index 8eb05f36..3483f85b 100644 --- a/mediator/src/main/kotlin/no/nav/dagpenger/mottak/behov/person/Pdl.kt +++ b/mediator/src/main/kotlin/no/nav/dagpenger/mottak/behov/person/Pdl.kt @@ -26,7 +26,9 @@ import no.nav.dagpenger.mottak.behov.JsonMapper.jacksonJsonAdapter private val logg = KotlinLogging.logger {} private val sikkerlogg = KotlinLogging.logger("tjenestekall.Pdl") -internal class PdlPersondataOppslag(config: Configuration) { +internal class PdlPersondataOppslag( + config: Configuration, +) { private val tokenProvider = config.pdlApiTokenProvider private val pdlClient = HttpClient { @@ -44,23 +46,27 @@ internal class PdlPersondataOppslag(config: Configuration) { } suspend fun hentPerson(id: String): Pdl.Person? = - pdlClient.request { - header(HttpHeaders.Authorization, "Bearer ${tokenProvider.invoke()}") - method = HttpMethod.Post - setBody(PersonQuery(id).toJson().also { sikkerlogg.info { "Forsøker å hente person med id $id fra PDL" } }) - }.bodyAsText().let { body -> - getWarnings(body).takeIf { it.isNotEmpty() }?.map { warnings -> - logg.warn { "Warnings ved henting av person fra PDL: $warnings" } - } - if (hasError(body)) { - throw PdlPersondataOppslagException(body) - } else { - Pdl.Person.fromGraphQlJson(body) + pdlClient + .request { + header(HttpHeaders.Authorization, "Bearer ${tokenProvider.invoke()}") + method = HttpMethod.Post + setBody(PersonQuery(id).toJson().also { sikkerlogg.info { "Forsøker å hente person med id $id fra PDL" } }) + }.bodyAsText() + .let { body -> + getWarnings(body).takeIf { it.isNotEmpty() }?.map { warnings -> + logg.warn { "Warnings ved henting av person fra PDL: $warnings" } + } + if (hasError(body)) { + throw PdlPersondataOppslagException(body) + } else { + Pdl.Person.fromGraphQlJson(body) + } } - } } -internal class PdlPersondataOppslagException(s: String) : RuntimeException(s) +internal class PdlPersondataOppslagException( + s: String, +) : RuntimeException(s) internal fun hasError(json: String): Boolean { val j = jacksonObjectMapper().readTree(json) @@ -97,34 +103,36 @@ private fun harGraphqlErrors(json: JsonNode) = json["errors"] != null && !json[" private fun ukjentPersonIdent(node: JsonNode) = node["errors"]?.any { it["message"].asText() == "Fant ikke person" } ?: false -internal data class PersonQuery(val id: String) : GraphqlQuery( - //language=Graphql - query = - """ - query(${'$'}ident: ID!) { - hentPerson(ident: ${'$'}ident) { - navn { - fornavn, - mellomnavn, - etternavn - }, - adressebeskyttelse{ - gradering +internal data class PersonQuery( + val id: String, +) : GraphqlQuery( + //language=Graphql + query = + """ + query(${'$'}ident: ID!) { + hentPerson(ident: ${'$'}ident) { + navn { + fornavn, + mellomnavn, + etternavn + }, + adressebeskyttelse{ + gradering + } } - } - hentGeografiskTilknytning(ident: ${'$'}ident){ - gtLand - } - hentIdenter(ident: ${'$'}ident, grupper: [AKTORID,FOLKEREGISTERIDENT]) { - identer { - ident, - gruppe + hentGeografiskTilknytning(ident: ${'$'}ident){ + gtLand } - } - } - """.trimIndent(), - variables = mapOf("ident" to id), -) + hentIdenter(ident: ${'$'}ident, grupper: [AKTORID,FOLKEREGISTERIDENT]) { + identer { + ident, + gruppe + } + } + } + """.trimIndent(), + variables = mapOf("ident" to id), + ) internal class Pdl { @JsonDeserialize(using = PersonDeserializer::class) @@ -147,12 +155,10 @@ internal class Pdl { internal fun JsonNode.norskTilknyting(): Boolean = findValue("gtLand")?.isNull ?: false - internal fun JsonNode.diskresjonsKode(): String? { - return findValue("adressebeskyttelse").firstOrNull()?.path("gradering")?.asText(null) - } + internal fun JsonNode.diskresjonsKode(): String? = findValue("adressebeskyttelse").firstOrNull()?.path("gradering")?.asText(null) - internal fun JsonNode.personNavn(): String { - return findValue("navn").first().let { node -> + internal fun JsonNode.personNavn(): String = + findValue("navn").first().let { node -> val fornavn = node.path("fornavn").asText() val mellomnavn = node.path("mellomnavn").asText("") val etternavn = node.path("etternavn").asText() @@ -162,11 +168,8 @@ internal class Pdl { else -> "$fornavn $mellomnavn $etternavn" } } - } - private fun JsonNode.ident(type: String): String { - return findValue("identer").first { it.path("gruppe").asText() == type }.get("ident").asText() - } + private fun JsonNode.ident(type: String): String = findValue("identer").first { it.path("gruppe").asText() == type }.get("ident").asText() private fun JsonNode.harIdent(type: String): Boolean = findValue("identer").map { it["gruppe"].asText() }.contains(type) @@ -176,27 +179,28 @@ internal class Pdl { ): Person? { val node: JsonNode = p.readValueAsTree() - return kotlin.runCatching { - Person( - navn = node.personNavn(), - aktørId = node.aktørId(), - fødselsnummer = node.fødselsnummer(), - norskTilknytning = node.norskTilknyting(), - diskresjonskode = node.diskresjonsKode(), + return kotlin + .runCatching { + Person( + navn = node.personNavn(), + aktørId = node.aktørId(), + fødselsnummer = node.fødselsnummer(), + norskTilknytning = node.norskTilknyting(), + diskresjonskode = node.diskresjonsKode(), + ) + }.fold( + onSuccess = { + it + }, + onFailure = { + if (ukjentPersonIdent(node) || !node.harIdent("FOLKEREGISTERIDENT")) { + return null + } else { + sikkerlogg.error(it) { "Feil ved deserialisering av PDL response: $node" } + throw it + } + }, ) - }.fold( - onSuccess = { - it - }, - onFailure = { - if (ukjentPersonIdent(node) || !node.harIdent("FOLKEREGISTERIDENT")) { - return null - } else { - sikkerlogg.error(it) { "Feil ved deserialisering av PDL response: $node" } - throw it - } - }, - ) } } } diff --git a/modell/src/main/kotlin/no/nav/dagpenger/mottak/Innsending.kt b/modell/src/main/kotlin/no/nav/dagpenger/mottak/Innsending.kt index 345abfce..c5df054d 100644 --- a/modell/src/main/kotlin/no/nav/dagpenger/mottak/Innsending.kt +++ b/modell/src/main/kotlin/no/nav/dagpenger/mottak/Innsending.kt @@ -203,14 +203,13 @@ class Innsending private constructor( hendelse: Hendelse, ) {} - override fun toSpesifikkKontekst(): SpesifikkKontekst { - return SpesifikkKontekst( + override fun toSpesifikkKontekst(): SpesifikkKontekst = + SpesifikkKontekst( "Tilstand", mapOf( "tilstand" to type.name, ), ) - } } internal object Mottatt : Tilstand { @@ -694,8 +693,10 @@ class Innsending private constructor( datoRegistrert = jp.datoRegistrert(), søknadsData = rutingOppslag?.data(), behandlendeEnhet = - jp.kategorisertJournalpost() - .oppgaveBenk(person, rutingOppslag).id, + jp + .kategorisertJournalpost() + .oppgaveBenk(person, rutingOppslag) + .id, tittel = jp.hovedDokument().tittel, ).also { ferdig -> observers.forEach { it.innsendingFerdigstilt(ferdig) } @@ -714,8 +715,10 @@ class Innsending private constructor( datoRegistrert = jp.datoRegistrert(), søknadsData = rutingOppslag?.data(), behandlendeEnhet = - jp.kategorisertJournalpost() - .oppgaveBenk(person, rutingOppslag).id, + jp + .kategorisertJournalpost() + .oppgaveBenk(person, rutingOppslag) + .id, tittel = jp.hovedDokument().tittel, ).also { mottatt -> observers.forEach { it.innsendingMottatt(mottatt) }