Oppdater ktlint. Krever returnverdi på klienter (postForEntity - Klag…

…eClient: opprettKlage og TilbakekrevingClient: opprettManuellTilbakekreving). (#2622)
navikt · Jun 19, 2024 · 3199b48 · 3199b48
1 parent 57906ce
commit 3199b48
Show file tree

Hide file tree

Showing 451 changed files with 4,001 additions and 3,402 deletions.
diff --git a/pom.xml b/pom.xml
@@ -406,7 +406,7 @@
                     <dependency>
                         <groupId>com.pinterest.ktlint</groupId>
                         <artifactId>ktlint-cli</artifactId>
-                        <version>1.2.1</version>
+                        <version>1.3.0</version>
                     </dependency>
                     <!-- additional 3rd party ruleset(s) can be specified here -->
                 </dependencies>

diff --git a/src/main/kotlin/no/nav/familie/ef/sak/AuditLogger.kt b/src/main/kotlin/no/nav/familie/ef/sak/AuditLogger.kt
@@ -25,14 +25,19 @@ data class Sporingsdata(
     val custom3: CustomKeyValue? = null,
 )
 
-enum class AuditLoggerEvent(val type: String) {
+enum class AuditLoggerEvent(
+    val type: String,
+) {
     CREATE("create"),
     UPDATE("update"),
     DELETE("delete"),
     ACCESS("access"),
 }
 
-data class CustomKeyValue(val key: String, val value: String)
+data class CustomKeyValue(
+    val key: String,
+    val value: String,
+)
 
 @Component
 class AuditLogger(
@@ -53,12 +58,12 @@ class AuditLogger(
         }
     }
 
-    private fun getRequest(): HttpServletRequest? {
-        return RequestContextHolder.getRequestAttributes()
+    private fun getRequest(): HttpServletRequest? =
+        RequestContextHolder
+            .getRequestAttributes()
             ?.takeIf { it is ServletRequestAttributes }
             ?.let { it as ServletRequestAttributes }
             ?.request
-    }
 
     private fun createAuditLogString(
         data: Sporingsdata,
@@ -89,16 +94,12 @@ class AuditLogger(
         }
     }
 
-    private fun createCustomString(data: Sporingsdata): String {
-        return listOfNotNull(
+    private fun createCustomString(data: Sporingsdata): String =
+        listOfNotNull(
             data.custom1?.let { "cs3Label=${it.key} cs3=${it.value}" },
             data.custom2?.let { "cs5Label=${it.key} cs5=${it.value}" },
             data.custom3?.let { "cs6Label=${it.key} cs6=${it.value}" },
-        )
-            .joinToString(" ")
-    }
+        ).joinToString(" ")
 
-    private fun getCallId(): String {
-        return MDC.get(MDCConstants.MDC_CALL_ID) ?: throw IllegalStateException("Mangler callId")
-    }
+    private fun getCallId(): String = MDC.get(MDCConstants.MDC_CALL_ID) ?: throw IllegalStateException("Mangler callId")
 }
diff --git a/src/main/kotlin/no/nav/familie/ef/sak/amelding/AMeldingInntektDto.kt b/src/main/kotlin/no/nav/familie/ef/sak/amelding/AMeldingInntektDto.kt
@@ -47,7 +47,9 @@ enum class InntektType {
     YTELSE_FRA_OFFENTLIGE,
 }
 
-enum class Fordel(val verdi: String) {
+enum class Fordel(
+    val verdi: String,
+) {
     KONTANTYTELSE("kontantytelse"),
     NATURALYTELSE("naturalytelse"),
     UTGIFTSGODTGJØRELSE("utgiftsgodtgjoerelse"),

diff --git a/src/main/kotlin/no/nav/familie/ef/sak/amelding/InntektMapper.kt b/src/main/kotlin/no/nav/familie/ef/sak/amelding/InntektMapper.kt
@@ -16,12 +16,11 @@ class InntektMapper(
     private val kodeverkService: CachedKodeverkService,
     private val eregService: EregService,
 ) {
-    fun mapInntekt(response: HentInntektListeResponse): AMeldingInntektDto {
-        return AMeldingInntektDto(
+    fun mapInntekt(response: HentInntektListeResponse): AMeldingInntektDto =
+        AMeldingInntektDto(
             inntektPerVirksomhet = mapOrganisasjoner(response),
             avvik = mapAvvik(response),
         )
-    }
 
     private fun mapOrganisasjoner(response: HentInntektListeResponse): List<InntektForVirksomhetDto> {
         val inntektPerMånedOgAktør = mapInntektresponseTilInntektPerVirksomhetOgPeriode(response)
@@ -47,7 +46,8 @@ class InntektMapper(
         val map: MutableMap<Aktør, MutableMap<YearMonth, MutableList<AMeldingInntekt>>> = mutableMapOf()
         response.arbeidsinntektMåned?.forEach { arbeidsInntektMaaned ->
             arbeidsInntektMaaned.arbeidsInntektInformasjon?.inntektListe?.forEach { inntekt ->
-                map.getOrPut(inntekt.virksomhet) { mutableMapOf() }
+                map
+                    .getOrPut(inntekt.virksomhet) { mutableMapOf() }
                     .getOrPut(arbeidsInntektMaaned.årMåned) { mutableListOf() }
                     .add(inntekt)
             }
@@ -60,7 +60,8 @@ class InntektMapper(
             aktører
                 .filter { it.aktørType == AktørType.ORGANISASJON }
                 .map { it.identifikator }
-        return eregService.hentOrganisasjoner(organisasjonsnumre)
+        return eregService
+            .hentOrganisasjoner(organisasjonsnumre)
             .associate { it.organisasjonsnummer to it.navn }
     }
 
@@ -87,23 +88,21 @@ class InntektMapper(
     ) =
         kodeverkService.hentInntekt()[type]?.get(verdi) ?: "$verdi (mangler verdi i kodeverk)"
 
-    private fun mapInntektType(type: EksternInntektType): InntektType {
-        return when (type) {
+    private fun mapInntektType(type: EksternInntektType): InntektType =
+        when (type) {
             EksternInntektType.LOENNSINNTEKT -> InntektType.LØNNSINNTEKT
             EksternInntektType.NAERINGSINNTEKT -> InntektType.NÆRINGSINNTEKT
             EksternInntektType.PENSJON_ELLER_TRYGD -> InntektType.PENSJON_ELLER_TRYGD
             EksternInntektType.YTELSE_FRA_OFFENTLIGE -> InntektType.YTELSE_FRA_OFFENTLIGE
         }
-    }
 
-    private fun mapInntektTypeTilKodeverkType(type: EksternInntektType): InntektKodeverkType {
-        return when (type) {
+    private fun mapInntektTypeTilKodeverkType(type: EksternInntektType): InntektKodeverkType =
+        when (type) {
             EksternInntektType.LOENNSINNTEKT -> InntektKodeverkType.LOENNSINNTEKT
             EksternInntektType.NAERINGSINNTEKT -> InntektKodeverkType.NAERINGSINNTEKT
             EksternInntektType.PENSJON_ELLER_TRYGD -> InntektKodeverkType.PENSJON_ELLER_TRYGD
             EksternInntektType.YTELSE_FRA_OFFENTLIGE -> InntektKodeverkType.YTELSE_FRA_OFFENTLIGE
         }
-    }
 
     private fun mapAvvik(response: HentInntektListeResponse) =
         response.arbeidsinntektMåned

diff --git a/src/main/kotlin/no/nav/familie/ef/sak/amelding/ekstern/AMeldingInntektClient.kt b/src/main/kotlin/no/nav/familie/ef/sak/amelding/ekstern/AMeldingInntektClient.kt
@@ -21,39 +21,48 @@ class AMeldingInntektClient(
         fom: YearMonth,
         tom: YearMonth,
     ) =
-        UriComponentsBuilder.fromUri(uri).pathSegment("api/inntekt")
+        UriComponentsBuilder
+            .fromUri(uri)
+            .pathSegment("api/inntekt")
             .queryParam("fom", fom)
             .queryParam("tom", tom)
-            .build().toUri()
+            .build()
+            .toUri()
 
-    private val genererUrlUri = UriComponentsBuilder.fromUri(uri).pathSegment("api/ainntekt/generer-url").build().toUri()
-    private val genererUrlUriArbeidsforhold = UriComponentsBuilder.fromUri(uri).pathSegment("api/ainntekt/generer-url-arbeidsforhold").build().toUri()
+    private val genererUrlUri =
+        UriComponentsBuilder
+            .fromUri(uri)
+            .pathSegment("api/ainntekt/generer-url")
+            .build()
+            .toUri()
+    private val genererUrlUriArbeidsforhold =
+        UriComponentsBuilder
+            .fromUri(uri)
+            .pathSegment("api/ainntekt/generer-url-arbeidsforhold")
+            .build()
+            .toUri()
 
     fun hentInntekt(
         personIdent: String,
         fom: YearMonth,
         tom: YearMonth,
-    ): HentInntektListeResponse {
-        return postForEntity(lagInntektUri(fom, tom), PersonIdent(personIdent))
-    }
+    ): HentInntektListeResponse = postForEntity(lagInntektUri(fom, tom), PersonIdent(personIdent))
 
-    fun genererAInntektUrl(personIdent: String): String {
-        return postForEntity(
+    fun genererAInntektUrl(personIdent: String): String =
+        postForEntity(
             genererUrlUri,
             PersonIdent(personIdent),
             HttpHeaders().apply {
                 accept = listOf(MediaType.TEXT_PLAIN)
             },
         )
-    }
 
-    fun genererAInntektArbeidsforholdUrl(personIdent: String): String {
-        return postForEntity(
+    fun genererAInntektArbeidsforholdUrl(personIdent: String): String =
+        postForEntity(
             genererUrlUriArbeidsforhold,
             PersonIdent(personIdent),
             HttpHeaders().apply {
                 accept = listOf(MediaType.TEXT_PLAIN)
             },
         )
-    }
 }
diff --git a/src/main/kotlin/no/nav/familie/ef/sak/arbeidsfordeling/ArbeidsfordelingService.kt b/src/main/kotlin/no/nav/familie/ef/sak/arbeidsfordeling/ArbeidsfordelingService.kt
@@ -27,11 +27,10 @@ class ArbeidsfordelingService(
             else -> hentNavEnhet(ident)?.enhetId
         }
 
-    fun hentNavEnhet(ident: String): Arbeidsfordelingsenhet? {
-        return cacheManager.getNullable("navEnhet", ident) {
+    fun hentNavEnhet(ident: String): Arbeidsfordelingsenhet? =
+        cacheManager.getNullable("navEnhet", ident) {
             personopplysningerIntegrasjonerClient.hentNavEnhetForPersonMedRelasjoner(ident).firstOrNull()
         }
-    }
 
     fun hentNavEnhetForOppfølging(
         ident: String,
@@ -42,7 +41,5 @@ class ArbeidsfordelingService(
                 ?: error("Fant ikke NAV-enhet for oppgave av type $oppgavetype")
         }
 
-    fun hentNavEnhetIdEllerBrukMaskinellEnhetHvisNull(personIdent: String): String {
-        return hentNavEnhet(personIdent)?.enhetId ?: MASKINELL_JOURNALFOERENDE_ENHET
-    }
+    fun hentNavEnhetIdEllerBrukMaskinellEnhetHvisNull(personIdent: String): String = hentNavEnhet(personIdent)?.enhetId ?: MASKINELL_JOURNALFOERENDE_ENHET
 }
diff --git a/src/main/kotlin/no/nav/familie/ef/sak/arbeidsfordeling/Arbeidsfordelingsenhet.kt b/src/main/kotlin/no/nav/familie/ef/sak/arbeidsfordeling/Arbeidsfordelingsenhet.kt
@@ -1,3 +1,6 @@
 package no.nav.familie.ef.sak.arbeidsfordeling
 
-data class Arbeidsfordelingsenhet(val enhetId: String, val enhetNavn: String)
+data class Arbeidsfordelingsenhet(
+    val enhetId: String,
+    val enhetNavn: String,
+)
diff --git a/src/main/kotlin/no/nav/familie/ef/sak/arbeidsforhold/ArbeidsforholdService.kt b/src/main/kotlin/no/nav/familie/ef/sak/arbeidsforhold/ArbeidsforholdService.kt
@@ -30,8 +30,14 @@ class ArbeidsforholdService(
         val arbeidsforhold = arbeidsforholdClient.hentArbeidsforhold(aktivIdent, ansettelsesdato).data
 
         return arbeidsforhold?.any {
-            it.ansettelsesperiode?.periode?.fom?.isEqualOrBefore(ansettelsesdato) == true &&
-                it.ansettelsesperiode?.periode?.tom?.isEqualOrAfter(ansettelsesdato) == true
+            it.ansettelsesperiode
+                ?.periode
+                ?.fom
+                ?.isEqualOrBefore(ansettelsesdato) == true &&
+                it.ansettelsesperiode
+                    ?.periode
+                    ?.tom
+                    ?.isEqualOrAfter(ansettelsesdato) == true
         } == true
     }
 }
diff --git a/src/main/kotlin/no/nav/familie/ef/sak/arbeidsforhold/ekstern/ArbeidsforholdClient.kt b/src/main/kotlin/no/nav/familie/ef/sak/arbeidsforhold/ekstern/ArbeidsforholdClient.kt
@@ -17,14 +17,16 @@ class ArbeidsforholdClient(
     @Qualifier("azure") restOperations: RestOperations,
 ) : AbstractRestClient(restOperations, "arbeidsforhold") {
     private fun lagArbeidsforholdUri() =
-        UriComponentsBuilder.fromUri(uri).pathSegment("api/aareg/arbeidsforhold").build().toUri()
+        UriComponentsBuilder
+            .fromUri(uri)
+            .pathSegment("api/aareg/arbeidsforhold")
+            .build()
+            .toUri()
 
     fun hentArbeidsforhold(
         personIdent: String,
         ansettelsesperiodeFom: LocalDate,
-    ): Ressurs<List<Arbeidsforhold>> {
-        return postForEntity(lagArbeidsforholdUri(), ArbeidsforholdRequest(personIdent, ansettelsesperiodeFom))
-    }
+    ): Ressurs<List<Arbeidsforhold>> = postForEntity(lagArbeidsforholdUri(), ArbeidsforholdRequest(personIdent, ansettelsesperiodeFom))
 }
 
 class ArbeidsforholdRequest(

diff --git a/src/main/kotlin/no/nav/familie/ef/sak/avstemming/KonsistensavstemmingScheduler.kt b/src/main/kotlin/no/nav/familie/ef/sak/avstemming/KonsistensavstemmingScheduler.kt
@@ -20,7 +20,9 @@ import kotlin.random.Random
 
 @Profile("!integrasjonstest")
 @Service
-class KonsistensavstemmingScheduler(private val konsistensavstemmingService: KonsistensavstemmingService) {
+class KonsistensavstemmingScheduler(
+    private val konsistensavstemmingService: KonsistensavstemmingService,
+) {
     private val logger = LoggerFactory.getLogger(javaClass)
 
     @Scheduled(cron = "0 0 0/12 * * *")

diff --git a/src/main/kotlin/no/nav/familie/ef/sak/barn/BarnRepository.kt b/src/main/kotlin/no/nav/familie/ef/sak/barn/BarnRepository.kt
@@ -6,6 +6,8 @@ import org.springframework.stereotype.Repository
 import java.util.UUID
 
 @Repository
-interface BarnRepository : RepositoryInterface<BehandlingBarn, UUID>, InsertUpdateRepository<BehandlingBarn> {
+interface BarnRepository :
+    RepositoryInterface<BehandlingBarn, UUID>,
+    InsertUpdateRepository<BehandlingBarn> {
     fun findByBehandlingId(behandlingId: UUID): List<BehandlingBarn>
 }
diff --git a/src/main/kotlin/no/nav/familie/ef/sak/barn/BarnService.kt b/src/main/kotlin/no/nav/familie/ef/sak/barn/BarnService.kt
@@ -185,8 +185,9 @@ class BarnService(
         barnFraSøknad: List<BehandlingBarn>,
         grunnlagsdataBarn: List<BarnMedIdent>,
         behandlingId: UUID,
-    ): List<BehandlingBarn> {
-        return BarnMatcher.kobleBehandlingBarnOgRegisterBarn(barnFraSøknad, grunnlagsdataBarn)
+    ): List<BehandlingBarn> =
+        BarnMatcher
+            .kobleBehandlingBarnOgRegisterBarn(barnFraSøknad, grunnlagsdataBarn)
             .map {
                 BehandlingBarn(
                     id = it.behandlingBarn.id,
@@ -197,7 +198,6 @@ class BarnService(
                     fødselTermindato = it.behandlingBarn.fødselTermindato,
                 )
             }
-    }
 
     /**
      * Legger sammen koblede barn plus de fra registeret som mangler
@@ -308,9 +308,7 @@ class BarnService(
         }
     }
 
-    fun hentBehandlingBarnForBarnIder(barnId: List<UUID>): List<BehandlingBarn> {
-        return barnRepository.findAllByIdOrThrow(barnId.toSet()) { it.id }
-    }
+    fun hentBehandlingBarnForBarnIder(barnId: List<UUID>): List<BehandlingBarn> = barnRepository.findAllByIdOrThrow(barnId.toSet()) { it.id }
 
     /**
      * Kan strengt tatt kun brukes for barnetilsyn då den er avhengig av at personIdent finnes

diff --git a/src/main/kotlin/no/nav/familie/ef/sak/behandling/BehandlingPåVentService.kt b/src/main/kotlin/no/nav/familie/ef/sak/behandling/BehandlingPåVentService.kt
@@ -158,13 +158,12 @@ class BehandlingPåVentService(
     private fun utledNyBeskrivelse(
         harEndringer: Boolean,
         settPåVentRequest: SettPåVentRequest,
-    ): String {
-        return when {
+    ): String =
+        when {
             settPåVentRequest.beskrivelse.isBlank() -> ""
             harEndringer -> "${settPåVentRequest.beskrivelse}\n"
             else -> "${settPåVentRequest.beskrivelse}\n"
         }
-    }
 
     private fun utledBeskrivelsePrefix(): String {
         val innloggetSaksbehandlerIdent = SikkerhetContext.hentSaksbehandlerEllerSystembruker()

diff --git a/src/main/kotlin/no/nav/familie/ef/sak/behandling/BehandlingRepository.kt b/src/main/kotlin/no/nav/familie/ef/sak/behandling/BehandlingRepository.kt
@@ -13,7 +13,9 @@ import java.time.LocalDateTime
 import java.util.UUID
 
 @Repository
-interface BehandlingRepository : RepositoryInterface<Behandling, UUID>, InsertUpdateRepository<Behandling> {
+interface BehandlingRepository :
+    RepositoryInterface<Behandling, UUID>,
+    InsertUpdateRepository<Behandling> {
     fun findByFagsakId(fagsakId: UUID): List<Behandling>
 
     fun findByFagsakIdAndStatus(