Fjern gammel kvittering #822
Conversation
.../db/src/main/kotlin/no/nav/helsearbeidsgiver/inntektsmelding/db/InntektsmeldingRepository.kt
Outdated
Show resolved
Hide resolved
| @@ -16,7 +16,6 @@ import java.util.UUID | |||
| data class ForespoerselFraBro( | |||
| val orgnr: Orgnr, | |||
| val fnr: Fnr, | |||
| /** Ikke bruk ved henting av én forespørsel (Storebror lekker feil id). */ | |||
There was a problem hiding this comment.
Denne er ikke lenger aktuell.
|
Testet OK. |
|
Testet OK. |
| refusjon = inntektsmelding.refusjon.convert(), | ||
| ) | ||
|
|
||
| LagretInntektsmelding.Skjema(inntektsmelding.innsenderNavn, bakoverkompatibeltSkjema, mottatt) |
There was a problem hiding this comment.
Navn på avsender lagres nå kun i inntektsmelding.innsenderNavn. Dette bør nok lagres i egen kolonne i databasen på sikt.
| ).where { InntektsmeldingEntitet.forespoerselId eq forespoerselId.toString() } | ||
| .orderBy(InntektsmeldingEntitet.innsendt, SortOrder.DESC) | ||
| .limit(1) | ||
| .map { | ||
| Triple( | ||
| it[InntektsmeldingEntitet.skjema], | ||
| Quadruple( |
There was a problem hiding this comment.
Kan man ikke droppe map og bare gjøre
val inntektsmelding = result[InntektsmeldingEntitet.dokument]
val skjema = result[InntektsmeldingEntitet.skjema]
val eksternInntektsmelding = result[InntektsmeldingEntitet.eksternInntektsmelding]
val mottatt = result[InntektsmeldingEntitet.innsendt]
Synes ikke at Quadruple-objektet hjelper noe særlig på lesbarhet i hvert fall
There was a problem hiding this comment.
Jeg brukte Quadruple for å holde transaksjonen mot databasen kortest mulig, siden det som kommer etter ikke er en del av spørringen, men det er kanskje ikke så farlig 🤔
apps/db/src/main/kotlin/no/nav/helsearbeidsgiver/inntektsmelding/db/river/HentLagretImRiver.kt
Show resolved
Hide resolved
.../db/src/main/kotlin/no/nav/helsearbeidsgiver/inntektsmelding/db/InntektsmeldingRepository.kt
Outdated
Show resolved
Hide resolved
...src/test/kotlin/no/nav/helsearbeidsgiver/inntektsmelding/db/InntektsmeldingRepositoryTest.kt
Outdated
Show resolved
Hide resolved
...st/src/test/kotlin/no/nav/helsearbeidsgiver/inntektsmelding/integrasjonstest/KvitteringIT.kt
Outdated
Show resolved
Hide resolved
# Conflicts: # apps/db/src/test/kotlin/no/nav/helsearbeidsgiver/inntektsmelding/db/river/HentLagretImRiverTest.kt # apps/innsending/src/test/kotlin/no.nav.helsearbeidsgiver.inntektsmelding.innsending/KvitteringServiceTest.kt # apps/integrasjonstest/src/test/kotlin/no/nav/helsearbeidsgiver/inntektsmelding/integrasjonstest/InnsendingServiceIT.kt
Må vente med denne til frontend har byttet fra
kvitteringDokumenttilkvitteringNavNo.Fjerner feltet som inneholder kvitteringsdata på det gamle formatet.