Opprydding i div smårusk i brev (#6491)

* Opprydding i div smårusk i brev * bedre logging / beskrivelse
navikt · Dec 2, 2024 · 7bec34e · 7bec34e
1 parent b755d50
commit 7bec34e
Show file tree

Hide file tree

Showing 4 changed files with 34 additions and 67 deletions.
diff --git a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/Brevoppretter.kt b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/Brevoppretter.kt
@@ -3,7 +3,6 @@ package no.nav.etterlatte.brev
 import no.nav.etterlatte.brev.adresse.AdresseService
 import no.nav.etterlatte.brev.db.BrevRepository
 import no.nav.etterlatte.brev.model.Brev
-import no.nav.etterlatte.brev.model.BrevID
 import no.nav.etterlatte.brev.model.BrevProsessType
 import no.nav.etterlatte.brev.model.BrevkodeRequest
 import no.nav.etterlatte.brev.model.OpprettNyttBrev
@@ -19,38 +18,6 @@ class Brevoppretter(
     private val db: BrevRepository,
     private val innholdTilRedigerbartBrevHenter: InnholdTilRedigerbartBrevHenter,
 ) {
-    suspend fun oppdaterBrevInnhold(
-        sakId: SakId,
-        brevID: BrevID,
-        behandlingId: UUID?,
-        brevKode: Brevkoder,
-        brevData: BrevDataRedigerbar,
-        spraak: Spraak? = null,
-        bruker: BrukerTokenInfo,
-    ): Brev =
-        with(
-            innholdTilRedigerbartBrevHenter.hentInnDataForBrevMedData(
-                sakId = sakId,
-                behandlingId = behandlingId,
-                bruker = bruker,
-                brevKode = brevKode,
-                brevData = brevData,
-                spraak = spraak,
-            ),
-        ) {
-            db.oppdaterPayload(
-                id = brevID,
-                payload = this.innhold.payload!!,
-                bruker = bruker,
-            )
-            db.oppdaterPayloadVedlegg(
-                id = brevID,
-                payload = this.innholdVedlegg!!,
-                bruker = bruker,
-            )
-            db.hentBrev(brevID)
-        }
-
     suspend fun opprettBrevSomHarInnhold(
         sakId: SakId,
         behandlingId: UUID?,

diff --git a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/adresse/RegoppslagKlient.kt b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/adresse/RegoppslagKlient.kt
@@ -46,34 +46,39 @@ class RegoppslagKlient(
                     .post("$url/rest/postadresse") {
                         behandlingsnummer(sakType)
                         contentType(ContentType.Application.Json)
-                        setBody(RegoppslagRequest(ident))
+                        setBody(RegoppslagRequest(ident, sakType.tema))
                     }.body<RegoppslagResponseDTO>()
                     .also {
                         cache.put(ident, it)
                     }
             }
         } catch (re: ResponseException) {
-            if (re.response.status == HttpStatusCode.NotFound) {
-                null
-            } else if (re.response.status == HttpStatusCode.Gone) {
-                logger.warn(re.response.bodyAsText())
-                null
-            } else {
-                logger.error("Uhåndtert feil fra regoppslag: ${re.response.bodyAsText()}")
+            when (re.response.status) {
+                HttpStatusCode.NotFound -> {
+                    logger.info("Bruker har ukjent adresse")
+                    null
+                }
+                HttpStatusCode.Gone -> {
+                    logger.warn("Person er død og har ukjent adresse: \n${re.response.bodyAsText()}")
+                    null
+                }
+
+                else -> {
+                    logger.error("Uhåndtert feil fra regoppslag: ${re.response.bodyAsText()}")
 
-                throw ForespoerselException(
-                    status = re.response.status.value,
-                    code = "UKJENT_FEIL_REGOPPSLAG",
-                    detail = "Ukjent feil oppsto ved uthenting av mottakers adresse fra regoppslag",
-                )
+                    throw ForespoerselException(
+                        status = re.response.status.value,
+                        code = "UKJENT_FEIL_REGOPPSLAG",
+                        detail = "Ukjent feil oppsto ved uthenting av mottakers adresse fra regoppslag",
+                    )
+                }
             }
         }
 }
 
 data class RegoppslagRequest(
     val ident: String,
-    // Todo: mulig bytte tema til et av de nye
-    val tema: String = "PEN",
+    val tema: String,
 )
 
 @JsonIgnoreProperties(ignoreUnknown = true)

diff --git a/...ev-api/src/main/kotlin/no/nav/etterlatte/brev/oversendelsebrev/OversendelseBrevService.kt b/...ev-api/src/main/kotlin/no/nav/etterlatte/brev/oversendelsebrev/OversendelseBrevService.kt
@@ -126,7 +126,7 @@ class OversendelseBrevServiceImpl(
             )
         val verge =
             grunnlagService.hentVergeForSak(
-                sakType = behandlingService.hentSak(klage.sak.id, brukerTokenInfo).sakType,
+                sakType = klage.sak.sakType,
                 brevutfallDto = null,
                 grunnlag = grunnlag,
             )

diff --git a/...etterlatte-brev-model/src/main/kotlin/no/nav/etterlatte/brev/behandling/GrunnlagMapper.kt b/...etterlatte-brev-model/src/main/kotlin/no/nav/etterlatte/brev/behandling/GrunnlagMapper.kt
@@ -24,19 +24,15 @@ import java.time.LocalDate
 private val logger = LoggerFactory.getLogger(Grunnlag::class.java)
 
 fun Grunnlag.mapAvdoede(): List<Avdoed> =
-    with(this.familie) {
-        val avdoede = hentAvdoede()
-
-        return avdoede
-            .filter { it.hentDoedsdato() != null }
-            .map { avdoed ->
-                Avdoed(
-                    fnr = Foedselsnummer(avdoed.hentFoedselsnummer()!!.verdi.value),
-                    navn = avdoed.hentNavn()!!.verdi.fulltNavn(),
-                    doedsdato = avdoed.hentDoedsdato()!!.verdi!!,
-                )
-            }
-    }
+    hentAvdoede()
+        .filter { it.hentDoedsdato() != null }
+        .map { avdoed ->
+            Avdoed(
+                fnr = Foedselsnummer(avdoed.hentFoedselsnummer()!!.verdi.value),
+                navn = avdoed.hentNavn()!!.verdi.fulltNavn(),
+                doedsdato = avdoed.hentDoedsdato()!!.verdi!!,
+            )
+        }
 
 fun Navn.fulltNavn(): String = listOfNotNull(fornavn, mellomnavn, etternavn).joinToString(" ") { it.storForbokstav() }
 
@@ -47,11 +43,11 @@ fun String.storForbokstavEtter(delim: String) =
         it.replaceFirstChar { c -> c.uppercase() }
     }
 
-fun Grunnlag.mapSoeker(aldersgruppe: Aldersgruppe?): no.nav.etterlatte.brev.behandling.Soeker =
+fun Grunnlag.mapSoeker(aldersgruppe: Aldersgruppe?): Soeker =
     with(this.soeker) {
         val navn = hentNavn()!!.verdi
 
-        no.nav.etterlatte.brev.behandling.Soeker(
+        Soeker(
             fornavn = navn.fornavn.storForbokstav(),
             mellomnavn = navn.mellomnavn?.storForbokstav(),
             etternavn = navn.etternavn.storForbokstav(),
@@ -62,7 +58,7 @@ fun Grunnlag.mapSoeker(aldersgruppe: Aldersgruppe?): no.nav.etterlatte.brev.beha
         )
     }
 
-fun Grunnlag.mapInnsender(): no.nav.etterlatte.brev.behandling.Innsender? =
+fun Grunnlag.mapInnsender(): Innsender? =
     with(this.sak) {
         val opplysning = hentKonstantOpplysning<Persongalleri>(Opplysningstype.PERSONGALLERI_V1)
 
@@ -72,8 +68,7 @@ fun Grunnlag.mapInnsender(): no.nav.etterlatte.brev.behandling.Innsender? =
             }
 
         persongalleri.innsender?.let {
-            no.nav.etterlatte.brev.behandling
-                .Innsender(fnr = Foedselsnummer(it))
+            Innsender(fnr = Foedselsnummer(it))
         }
     }