From c6f602325731b4da6b56e3e7aef3d86024cc218e Mon Sep 17 00:00:00 2001 From: John Andre Hestad Date: Tue, 3 Oct 2023 17:00:31 +0200 Subject: [PATCH] =?UTF-8?q?Bytt=20s=C3=A5=20mange=20sealed=20class->interf?= =?UTF-8?q?ace=20p=C3=A5=2010=20min?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../client/dokarkiv/JournalpostRequest.kt | 12 ++-- .../DokDistFordelingClient.kt | 2 +- .../journalpost/JournalpostHttpClient.kt | 16 +++--- .../oppdrag/avstemming/AvstemmingRequest.kt | 3 + .../bakover/client/person/PdlAdresseformat.kt | 55 +++++++++++++++---- .../se/bakover/common/domain/Attestering.kt | 10 ++-- .../su/se/bakover/common/domain/Kopierbar.kt | 10 ++-- .../common/domain/nais/LeaderPodLookup.kt | 6 +- ...TilbakekrevingsbehandlingForRevurdering.kt | 6 +- .../TilbakekrevingsbehandlingTest.kt | 2 +- .../src/main/kotlin/TilbakekrevingTestData.kt | 6 +- .../kravgrunnlag/KravgrunnlagTestData.kt | 4 +- .../domain/kravgrunnlag/Kravgrunnlag.kt | 12 ++-- .../infrastructure/KravgrunnlagDbJson.kt | 8 +-- .../api/common/KravgrunnlagJson.kt | 4 +- .../consumer/TilbakekrevingsmeldingMapper.kt | 4 +- .../LokalMottaKravgrunnlagJob.kt | 12 ++-- .../TilbakekrevingsmeldingMapperTest.kt | 10 ++-- 18 files changed, 108 insertions(+), 74 deletions(-) diff --git a/client/src/main/kotlin/no/nav/su/se/bakover/client/dokarkiv/JournalpostRequest.kt b/client/src/main/kotlin/no/nav/su/se/bakover/client/dokarkiv/JournalpostRequest.kt index e5cafd5855..42b8045b6e 100644 --- a/client/src/main/kotlin/no/nav/su/se/bakover/client/dokarkiv/JournalpostRequest.kt +++ b/client/src/main/kotlin/no/nav/su/se/bakover/client/dokarkiv/JournalpostRequest.kt @@ -163,21 +163,21 @@ data class Fagsak( val sakstype: String = "FAGSAK", ) -sealed class DokumentVariant { - abstract val filtype: String - abstract val fysiskDokument: String - abstract val variantformat: String +sealed interface DokumentVariant { + val filtype: String + val fysiskDokument: String + val variantformat: String data class ArkivPDF( override val fysiskDokument: String, - ) : DokumentVariant() { + ) : DokumentVariant { override val filtype: String = "PDFA" override val variantformat: String = "ARKIV" } data class OriginalJson( override val fysiskDokument: String, - ) : DokumentVariant() { + ) : DokumentVariant { override val filtype: String = "JSON" override val variantformat: String = "ORIGINAL" } diff --git a/client/src/main/kotlin/no/nav/su/se/bakover/client/dokdistfordeling/DokDistFordelingClient.kt b/client/src/main/kotlin/no/nav/su/se/bakover/client/dokdistfordeling/DokDistFordelingClient.kt index 88b5108abd..dca243d91f 100644 --- a/client/src/main/kotlin/no/nav/su/se/bakover/client/dokdistfordeling/DokDistFordelingClient.kt +++ b/client/src/main/kotlin/no/nav/su/se/bakover/client/dokdistfordeling/DokDistFordelingClient.kt @@ -87,7 +87,7 @@ class DokDistFordelingClient(val baseUrl: String, val tokenOppslag: TokenOppslag } } -private sealed class PayloadTyper { +private sealed interface PayloadTyper { enum class Distribusjonstype(val value: String) { VEDTAK("VEDTAK"), VIKTIG("VIKTIG"), diff --git a/client/src/main/kotlin/no/nav/su/se/bakover/client/journalpost/JournalpostHttpClient.kt b/client/src/main/kotlin/no/nav/su/se/bakover/client/journalpost/JournalpostHttpClient.kt index c0eab178fa..4485e9944e 100644 --- a/client/src/main/kotlin/no/nav/su/se/bakover/client/journalpost/JournalpostHttpClient.kt +++ b/client/src/main/kotlin/no/nav/su/se/bakover/client/journalpost/JournalpostHttpClient.kt @@ -219,16 +219,16 @@ internal class JournalpostHttpClient( } } - sealed class GraphQLApiFeil { + sealed interface GraphQLApiFeil { - data class TekniskFeil(val request: Any, val msg: String) : GraphQLApiFeil() - sealed class HttpFeil : GraphQLApiFeil() { - data class NotFound(val request: Any, val msg: String) : HttpFeil() - data class ServerError(val request: Any, val msg: String) : HttpFeil() - data class BadRequest(val request: Any, val msg: String) : HttpFeil() - data class Forbidden(val request: Any, val msg: String) : HttpFeil() + data class TekniskFeil(val request: Any, val msg: String) : GraphQLApiFeil + sealed interface HttpFeil : GraphQLApiFeil { + data class NotFound(val request: Any, val msg: String) : HttpFeil + data class ServerError(val request: Any, val msg: String) : HttpFeil + data class BadRequest(val request: Any, val msg: String) : HttpFeil + data class Forbidden(val request: Any, val msg: String) : HttpFeil - data class Ukjent(val request: Any, val msg: String) : HttpFeil() + data class Ukjent(val request: Any, val msg: String) : HttpFeil } } } diff --git a/client/src/main/kotlin/no/nav/su/se/bakover/client/oppdrag/avstemming/AvstemmingRequest.kt b/client/src/main/kotlin/no/nav/su/se/bakover/client/oppdrag/avstemming/AvstemmingRequest.kt index b0c7946c4f..77430c93cb 100644 --- a/client/src/main/kotlin/no/nav/su/se/bakover/client/oppdrag/avstemming/AvstemmingRequest.kt +++ b/client/src/main/kotlin/no/nav/su/se/bakover/client/oppdrag/avstemming/AvstemmingRequest.kt @@ -168,6 +168,9 @@ internal data class KonsistensavstemmingData( ) } +/** + * TODO jah: Bytt til sealed interface? Må løse protected + */ sealed class Aksjonsdata { @JacksonXmlProperty @Suppress("unused") diff --git a/client/src/main/kotlin/no/nav/su/se/bakover/client/person/PdlAdresseformat.kt b/client/src/main/kotlin/no/nav/su/se/bakover/client/person/PdlAdresseformat.kt index 7742930a5c..a6cf2a35e9 100644 --- a/client/src/main/kotlin/no/nav/su/se/bakover/client/person/PdlAdresseformat.kt +++ b/client/src/main/kotlin/no/nav/su/se/bakover/client/person/PdlAdresseformat.kt @@ -11,6 +11,7 @@ internal fun List.finnRiktigAdresseformatOgMapTilPdlAdresse(): List adresseformat = format.type, adressetype = adressetype, ) + is PostadresseIFrittFormat -> PdlData.Adresse( adresselinje = listOfNotNull( format.adresselinje1, @@ -21,12 +22,14 @@ internal fun List.finnRiktigAdresseformatOgMapTilPdlAdresse(): List adresseformat = format.type, adressetype = adressetype, ) + is Postboksadresse -> PdlData.Adresse( adresselinje = "${format.postbokseier}, ${format.postboks}", postnummer = format.postnummer, adresseformat = format.type, adressetype = adressetype, ) + is UkjentBosted -> null is Matrikkeladresse -> PdlData.Adresse( adresselinje = format.tilleggsnavn ?: "", @@ -36,6 +39,7 @@ internal fun List.finnRiktigAdresseformatOgMapTilPdlAdresse(): List adresseformat = format.type, adressetype = adressetype, ) + is UtenlandskAdresse -> PdlData.Adresse( adresselinje = listOfNotNull( format.adressenavnNummer, @@ -48,6 +52,7 @@ internal fun List.finnRiktigAdresseformatOgMapTilPdlAdresse(): List adresseformat = format.type, adressetype = adressetype, ) + is UtenlandskAdresseIFrittFormat -> PdlData.Adresse( adresselinje = listOfNotNull( format.adresselinje1, @@ -93,12 +98,17 @@ internal fun List.finnRiktigAdresseformatOgMapTilPdlAdresse(): List }.distinct() } -sealed class Adressetype(val type: String) +sealed interface Adressetype { + val type: String +} + data class Bostedsadresse( val vegadresse: Vegadresse?, val ukjentBosted: UkjentBosted?, val matrikkeladresse: Matrikkeladresse?, -) : Adressetype(type = "Bostedsadresse") +) : Adressetype { + override val type: String = "Bostedsadresse" +} data class Kontaktadresse( val vegadresse: Vegadresse?, @@ -106,15 +116,22 @@ data class Kontaktadresse( val postboksadresse: Postboksadresse?, val utenlandskAdresse: UtenlandskAdresse?, val utenlandskAdresseIFrittFormat: UtenlandskAdresseIFrittFormat?, -) : Adressetype(type = "Kontaktadresse") +) : Adressetype { + override val type: String = "Kontaktadresse" +} data class Oppholdsadresse( val vegadresse: Vegadresse?, val matrikkeladresse: Matrikkeladresse?, val utenlandskAdresse: UtenlandskAdresse?, -) : Adressetype(type = "Oppholdsadresse") +) : Adressetype { + override val type: String = "Oppholdsadresse" +} + +sealed interface Adresseformat { + val type: String +} -sealed class Adresseformat(val type: String) data class Vegadresse( val husnummer: String?, val husbokstav: String?, @@ -122,24 +139,32 @@ data class Vegadresse( val kommunenummer: String?, val postnummer: String?, val bruksenhetsnummer: String?, -) : Adresseformat(type = "Vegadresse") +) : Adresseformat { + override val type = "Vegadresse" +} data class PostadresseIFrittFormat( val adresselinje1: String?, val adresselinje2: String?, val adresselinje3: String?, val postnummer: String?, -) : Adresseformat(type = "PostadresseIFrittFormat") +) : Adresseformat { + override val type = "PostadresseIFrittFormat" +} data class Postboksadresse( val postbokseier: String?, val postboks: String?, val postnummer: String?, -) : Adresseformat(type = "Postboksadresse") +) : Adresseformat { + override val type = "Postboksadresse" +} data class UkjentBosted( val bostedskommune: String, -) : Adresseformat(type = "UkjentBosted") +) : Adresseformat { + override val type = "UkjentBosted" +} data class Matrikkeladresse( val matrikkelId: Long?, @@ -147,7 +172,9 @@ data class Matrikkeladresse( val tilleggsnavn: String?, val postnummer: String?, val kommunenummer: String?, -) : Adresseformat(type = "Matrikkeladresse") +) : Adresseformat { + override val type = "PostadresseIFrittFormat" +} data class UtenlandskAdresse( val adressenavnNummer: String?, @@ -157,7 +184,9 @@ data class UtenlandskAdresse( val bySted: String?, val regionDistriktOmraade: String?, val landkode: String, -) : Adresseformat(type = "UtenlandskAdresse") +) : Adresseformat { + override val type = "UtenlandskAdresse" +} data class UtenlandskAdresseIFrittFormat( val adresselinje1: String?, @@ -166,4 +195,6 @@ data class UtenlandskAdresseIFrittFormat( val postkode: String?, val byEllerStedsnavn: String?, val landkode: String, -) : Adresseformat(type = "UtenlandskAdresseIFrittFormat") +) : Adresseformat { + override val type = "UtenlandskAdresseIFrittFormat" +} diff --git a/common/domain/src/main/kotlin/no/nav/su/se/bakover/common/domain/Attestering.kt b/common/domain/src/main/kotlin/no/nav/su/se/bakover/common/domain/Attestering.kt index 375f1a63fb..e49c3a9b90 100644 --- a/common/domain/src/main/kotlin/no/nav/su/se/bakover/common/domain/Attestering.kt +++ b/common/domain/src/main/kotlin/no/nav/su/se/bakover/common/domain/Attestering.kt @@ -66,19 +66,19 @@ data class Attesteringshistorikk private constructor( JsonSubTypes.Type(value = Attestering.Iverksatt::class, name = "Iverksatt"), JsonSubTypes.Type(value = Attestering.Underkjent::class, name = "Underkjent"), ) -sealed class Attestering { - abstract val attestant: NavIdentBruker.Attestant - abstract val opprettet: Tidspunkt +sealed interface Attestering { + val attestant: NavIdentBruker.Attestant + val opprettet: Tidspunkt data class Iverksatt(override val attestant: NavIdentBruker.Attestant, override val opprettet: Tidspunkt) : - Attestering() + Attestering data class Underkjent( override val attestant: NavIdentBruker.Attestant, override val opprettet: Tidspunkt, val grunn: Grunn, val kommentar: String, - ) : Attestering() { + ) : Attestering { enum class Grunn { INNGANGSVILKÅRENE_ER_FEILVURDERT, BEREGNINGEN_ER_FEIL, diff --git a/common/domain/src/main/kotlin/no/nav/su/se/bakover/common/domain/Kopierbar.kt b/common/domain/src/main/kotlin/no/nav/su/se/bakover/common/domain/Kopierbar.kt index 6834bf1f07..7a11b7116d 100644 --- a/common/domain/src/main/kotlin/no/nav/su/se/bakover/common/domain/Kopierbar.kt +++ b/common/domain/src/main/kotlin/no/nav/su/se/bakover/common/domain/Kopierbar.kt @@ -19,23 +19,23 @@ interface KopierbarForSnitt { fun copy(args: CopyArgs.Snitt): Type? } -sealed class CopyArgs { - sealed class Tidslinje : CopyArgs() { +sealed interface CopyArgs { + sealed interface Tidslinje : CopyArgs { /** * Kopier aktuelt objekt og sett perioden til å være [periode]. */ - data class NyPeriode(val periode: Periode) : Tidslinje() + data class NyPeriode(val periode: Periode) : Tidslinje /** * Kopier aktuelt objekt som det er. */ - data object Full : Tidslinje() + data object Full : Tidslinje } /** * Lager kopi hvor perioden settes til snittet av periodene. */ - data class Snitt(val periode: Periode) : CopyArgs() { + data class Snitt(val periode: Periode) : CopyArgs { fun snittFor(original: Periode): Periode? { return original.snitt(periode) } diff --git a/common/domain/src/main/kotlin/no/nav/su/se/bakover/common/domain/nais/LeaderPodLookup.kt b/common/domain/src/main/kotlin/no/nav/su/se/bakover/common/domain/nais/LeaderPodLookup.kt index 20158837c3..ed919fc993 100644 --- a/common/domain/src/main/kotlin/no/nav/su/se/bakover/common/domain/nais/LeaderPodLookup.kt +++ b/common/domain/src/main/kotlin/no/nav/su/se/bakover/common/domain/nais/LeaderPodLookup.kt @@ -6,7 +6,7 @@ interface LeaderPodLookup { fun amITheLeader(localHostName: String): Either } -sealed class LeaderPodLookupFeil { - data object KunneIkkeKontakteLeaderElectorContainer : LeaderPodLookupFeil() - data object UkjentSvarFraLeaderElectorContainer : LeaderPodLookupFeil() +sealed interface LeaderPodLookupFeil { + data object KunneIkkeKontakteLeaderElectorContainer : LeaderPodLookupFeil + data object UkjentSvarFraLeaderElectorContainer : LeaderPodLookupFeil } diff --git a/domain/src/main/kotlin/no/nav/su/se/bakover/domain/oppdrag/tilbakekreving/TilbakekrevingsbehandlingForRevurdering.kt b/domain/src/main/kotlin/no/nav/su/se/bakover/domain/oppdrag/tilbakekreving/TilbakekrevingsbehandlingForRevurdering.kt index b212196e91..52a60ed22f 100644 --- a/domain/src/main/kotlin/no/nav/su/se/bakover/domain/oppdrag/tilbakekreving/TilbakekrevingsbehandlingForRevurdering.kt +++ b/domain/src/main/kotlin/no/nav/su/se/bakover/domain/oppdrag/tilbakekreving/TilbakekrevingsbehandlingForRevurdering.kt @@ -147,7 +147,7 @@ data class MottattKravgrunnlag( ansvarligEnhet = "8020", kontrollFelt = kravgrunnlag.eksternKontrollfelt, behandler = kravgrunnlag.behandler, - tilbakekrevingsperioder = kravgrunnlag.grunnlagsperioder.map { grunnlagsperiode -> + tilbakekrevingsperioder = kravgrunnlag.grunnlagsmåneder.map { grunnlagsperiode -> Tilbakekrevingsvedtak.Tilbakekrevingsperiode( periode = grunnlagsperiode.måned, renterBeregnes = false, @@ -165,7 +165,7 @@ data class MottattKravgrunnlag( .multiply(it.skatteProsent) .divide(BigDecimal("100")) .setScale(0, RoundingMode.DOWN) - .min(grunnlagsperiode.beløpSkattMnd), + .min(grunnlagsperiode.betaltSkattForYtelsesgruppen), tilbakekrevingsresultat = Tilbakekrevingsvedtak.Tilbakekrevingsresultat.FULL_TILBAKEKREVING, skyld = Tilbakekrevingsvedtak.Skyld.BRUKER, ) @@ -189,7 +189,7 @@ data class MottattKravgrunnlag( ansvarligEnhet = "8020", kontrollFelt = kravgrunnlag.eksternKontrollfelt, behandler = kravgrunnlag.behandler, // TODO behandler bør sannsynligvis være fra tilbakekrevingsbehandling/revurdering og ikke kravgrunnlaget - tilbakekrevingsperioder = kravgrunnlag.grunnlagsperioder.map { grunnlagsperiode -> + tilbakekrevingsperioder = kravgrunnlag.grunnlagsmåneder.map { grunnlagsperiode -> Tilbakekrevingsvedtak.Tilbakekrevingsperiode( periode = grunnlagsperiode.måned, renterBeregnes = false, diff --git a/domain/src/test/kotlin/no/nav/su/se/bakover/domain/oppdrag/tilbakekreving/TilbakekrevingsbehandlingTest.kt b/domain/src/test/kotlin/no/nav/su/se/bakover/domain/oppdrag/tilbakekreving/TilbakekrevingsbehandlingTest.kt index b03543a595..598cecafa2 100644 --- a/domain/src/test/kotlin/no/nav/su/se/bakover/domain/oppdrag/tilbakekreving/TilbakekrevingsbehandlingTest.kt +++ b/domain/src/test/kotlin/no/nav/su/se/bakover/domain/oppdrag/tilbakekreving/TilbakekrevingsbehandlingTest.kt @@ -128,7 +128,7 @@ internal class TilbakekrevingsbehandlingTest { tilbakekrevingsvedtak.tilbakekrevingsperioder .flatMap { it.tilbakekrevingsbeløp } .filterIsInstance() - .all { it.beløpSkatt == BigDecimal("1759") && it.beløpSkatt < kravgrunnlag.grunnlagsperioder[0].beløpSkattMnd } shouldBe true + .all { it.beløpSkatt == BigDecimal("1759") && it.beløpSkatt < kravgrunnlag.grunnlagsmåneder[0].betaltSkattForYtelsesgruppen } shouldBe true } } diff --git a/test-common/src/main/kotlin/TilbakekrevingTestData.kt b/test-common/src/main/kotlin/TilbakekrevingTestData.kt index 2b872ed56c..b33a9abf92 100644 --- a/test-common/src/main/kotlin/TilbakekrevingTestData.kt +++ b/test-common/src/main/kotlin/TilbakekrevingTestData.kt @@ -33,11 +33,11 @@ fun matchendeKravgrunnlag( status = Kravgrunnlag.KravgrunnlagStatus.Nytt, behandler = "K231B433", utbetalingId = utbetalingId, - grunnlagsperioder = it.hentFeilutbetalteBeløp() + grunnlagsmåneder = it.hentFeilutbetalteBeløp() .map { (periode, feilutbetaling) -> Kravgrunnlag.Grunnlagsmåned( måned = periode.tilMåned(), - beløpSkattMnd = BigDecimal(4395), + betaltSkattForYtelsesgruppen = BigDecimal(4395), grunnlagsbeløp = listOf( Kravgrunnlag.Grunnlagsmåned.Grunnlagsbeløp( kode = KlasseKode.KL_KODE_FEIL_INNT, @@ -86,6 +86,6 @@ fun nyKravgrunnlag( status = status, behandler = behandler, utbetalingId = utbetalingId, - grunnlagsperioder = grunnlagsperioder, + grunnlagsmåneder = grunnlagsperioder, ) } diff --git a/test-common/src/main/kotlin/kravgrunnlag/KravgrunnlagTestData.kt b/test-common/src/main/kotlin/kravgrunnlag/KravgrunnlagTestData.kt index bb6c3b720c..4f47c5b354 100644 --- a/test-common/src/main/kotlin/kravgrunnlag/KravgrunnlagTestData.kt +++ b/test-common/src/main/kotlin/kravgrunnlag/KravgrunnlagTestData.kt @@ -85,7 +85,7 @@ fun kravgrunnlagPåSakHendelse( status = status, behandler = behandler, utbetalingId = utbetalingId, - grunnlagsperioder = grunnlagsmåneder, + grunnlagsmåneder = grunnlagsmåneder, saksnummer = saksnummer, ), revurderingId = revurderingId, @@ -102,7 +102,7 @@ fun grunnlagsmåned( ): Kravgrunnlag.Grunnlagsmåned { return Kravgrunnlag.Grunnlagsmåned( måned = måned, - beløpSkattMnd = betaltSkattForYtelsesgruppen, + betaltSkattForYtelsesgruppen = betaltSkattForYtelsesgruppen, grunnlagsbeløp = grunnlagsbeløps, ) } diff --git a/tilbakekreving/domain/src/main/kotlin/tilbakekreving/domain/kravgrunnlag/Kravgrunnlag.kt b/tilbakekreving/domain/src/main/kotlin/tilbakekreving/domain/kravgrunnlag/Kravgrunnlag.kt index 2f7e0984da..5a5d533670 100644 --- a/tilbakekreving/domain/src/main/kotlin/tilbakekreving/domain/kravgrunnlag/Kravgrunnlag.kt +++ b/tilbakekreving/domain/src/main/kotlin/tilbakekreving/domain/kravgrunnlag/Kravgrunnlag.kt @@ -6,7 +6,6 @@ import no.nav.su.se.bakover.common.Månedsbeløp import no.nav.su.se.bakover.common.UUID30 import no.nav.su.se.bakover.common.domain.Saksnummer import no.nav.su.se.bakover.common.tid.periode.Måned -import no.nav.su.se.bakover.common.tid.periode.tilMåned import økonomi.domain.KlasseKode import økonomi.domain.KlasseType import java.math.BigDecimal @@ -30,12 +29,12 @@ data class Kravgrunnlag( */ val behandler: String, val utbetalingId: UUID30, - val grunnlagsperioder: List, + val grunnlagsmåneder: List, ) { fun hentBeløpSkalTilbakekreves(): Månedsbeløp { return Månedsbeløp( - grunnlagsperioder + grunnlagsmåneder .map { it.hentBeløpSkalTilbakekreves() } .filter { it.sum() > 0 }, ) @@ -43,14 +42,15 @@ data class Kravgrunnlag( data class Grunnlagsmåned( val måned: Måned, - val beløpSkattMnd: BigDecimal, + /** Kravgrunnlaget oppgir kun total skatt som er betalt for hele ytelsesgruppen til SU. Så denne kan bare brukes som en øvre grense. */ + val betaltSkattForYtelsesgruppen: BigDecimal, val grunnlagsbeløp: List, ) { fun hentBeløpSkalTilbakekreves(): MånedBeløp { return MånedBeløp( - måned.tilMåned(), - Beløp(grunnlagsbeløp.sumOf { it.beløpSkalTilbakekreves.intValueExact() }), + periode = måned, + beløp = Beløp(grunnlagsbeløp.sumOf { it.beløpSkalTilbakekreves.intValueExact() }), ) } diff --git a/tilbakekreving/infrastructure/src/main/kotlin/tilbakekreving/infrastructure/KravgrunnlagDbJson.kt b/tilbakekreving/infrastructure/src/main/kotlin/tilbakekreving/infrastructure/KravgrunnlagDbJson.kt index 399531abe0..e546ce3a43 100644 --- a/tilbakekreving/infrastructure/src/main/kotlin/tilbakekreving/infrastructure/KravgrunnlagDbJson.kt +++ b/tilbakekreving/infrastructure/src/main/kotlin/tilbakekreving/infrastructure/KravgrunnlagDbJson.kt @@ -97,7 +97,7 @@ internal data class KravgrunnlagDbJson( fun toDomain(): Kravgrunnlag.Grunnlagsmåned { return Kravgrunnlag.Grunnlagsmåned( måned = Måned.fra(YearMonth.parse(this.måned)), - beløpSkattMnd = BigDecimal(this.betaltSkattForYtelsesgruppen), + betaltSkattForYtelsesgruppen = BigDecimal(this.betaltSkattForYtelsesgruppen), grunnlagsbeløp = this.grunnlagsbeløp.map { it.toDomain() }, ) } @@ -106,7 +106,7 @@ internal data class KravgrunnlagDbJson( fun Kravgrunnlag.Grunnlagsmåned.toDbJson(): Grunnlagsmåned { return Grunnlagsmåned( måned = this.måned.tilMåned().toString(), // uuuu-MM - betaltSkattForYtelsesgruppen = this.beløpSkattMnd.toString(), + betaltSkattForYtelsesgruppen = this.betaltSkattForYtelsesgruppen.toString(), grunnlagsbeløp = this.grunnlagsbeløp.map { it.toDbJson() }, ) } @@ -132,7 +132,7 @@ internal data class KravgrunnlagDbJson( }, behandler = this.behandler, utbetalingId = UUID30.fromString(this.utbetalingId), - grunnlagsperioder = this.grunnlagsmåneder.map { it.toDomain() }, + grunnlagsmåneder = this.grunnlagsmåneder.map { it.toDomain() }, saksnummer = Saksnummer.parse(this.saksnummer), ) } @@ -157,7 +157,7 @@ internal data class KravgrunnlagDbJson( behandler = this.behandler, utbetalingId = this.utbetalingId.value, saksnummer = this.saksnummer.toString(), - grunnlagsmåneder = this.grunnlagsperioder.map { it.toDbJson() }, + grunnlagsmåneder = this.grunnlagsmåneder.map { it.toDbJson() }, ) } } diff --git a/tilbakekreving/presentation/src/main/kotlin/tilbakekreving/presentation/api/common/KravgrunnlagJson.kt b/tilbakekreving/presentation/src/main/kotlin/tilbakekreving/presentation/api/common/KravgrunnlagJson.kt index af4779da93..f40e20d158 100644 --- a/tilbakekreving/presentation/src/main/kotlin/tilbakekreving/presentation/api/common/KravgrunnlagJson.kt +++ b/tilbakekreving/presentation/src/main/kotlin/tilbakekreving/presentation/api/common/KravgrunnlagJson.kt @@ -26,7 +26,7 @@ data class KravgrunnlagJson( eksternVedtakId = this.eksternVedtakId, kontrollfelt = this.eksternKontrollfelt, status = this.status.toJson(), - grunnlagsperiode = this.grunnlagsperioder.toJson(), + grunnlagsperiode = this.grunnlagsmåneder.toJson(), ) fun Kravgrunnlag.toStringifiedJson(): String = serialize(this.toJson()) @@ -43,7 +43,7 @@ data class GrunnlagsperiodeJson( fun List.toJson(): List = this.map { GrunnlagsperiodeJson( periode = it.måned.toJson(), - beløpSkattMnd = it.beløpSkattMnd.toString(), + beløpSkattMnd = it.betaltSkattForYtelsesgruppen.toString(), grunnlagsbeløp = it.grunnlagsbeløp.toJson(), ) } diff --git a/tilbakekreving/presentation/src/main/kotlin/tilbakekreving/presentation/consumer/TilbakekrevingsmeldingMapper.kt b/tilbakekreving/presentation/src/main/kotlin/tilbakekreving/presentation/consumer/TilbakekrevingsmeldingMapper.kt index 7a9c08e3ea..81cfaafb40 100644 --- a/tilbakekreving/presentation/src/main/kotlin/tilbakekreving/presentation/consumer/TilbakekrevingsmeldingMapper.kt +++ b/tilbakekreving/presentation/src/main/kotlin/tilbakekreving/presentation/consumer/TilbakekrevingsmeldingMapper.kt @@ -75,13 +75,13 @@ data object TilbakekrevingsmeldingMapper { eksternKontrollfelt = kravgrunnlagDto.kontrollfelt, behandler = kravgrunnlagDto.saksbehId, utbetalingId = UUID30.fromString(kravgrunnlagDto.utbetalingId), - grunnlagsperioder = kravgrunnlagDto.tilbakekrevingsperioder.map { tilbakekrevingsperiode -> + grunnlagsmåneder = kravgrunnlagDto.tilbakekrevingsperioder.map { tilbakekrevingsperiode -> Kravgrunnlag.Grunnlagsmåned( måned = Måned.Companion.fra( LocalDate.parse(tilbakekrevingsperiode.periode.fraOgMed), LocalDate.parse(tilbakekrevingsperiode.periode.tilOgMed), ), - beløpSkattMnd = BigDecimal(tilbakekrevingsperiode.skattebeløpPerMåned), + betaltSkattForYtelsesgruppen = BigDecimal(tilbakekrevingsperiode.skattebeløpPerMåned), grunnlagsbeløp = tilbakekrevingsperiode.tilbakekrevingsbeløp.map { Kravgrunnlag.Grunnlagsmåned.Grunnlagsbeløp( kode = KlasseKode.valueOf(it.kodeKlasse), diff --git a/web/src/main/kotlin/no/nav/su/se/bakover/web/services/tilbakekreving/LokalMottaKravgrunnlagJob.kt b/web/src/main/kotlin/no/nav/su/se/bakover/web/services/tilbakekreving/LokalMottaKravgrunnlagJob.kt index 46439d5a8b..c742c40f27 100644 --- a/web/src/main/kotlin/no/nav/su/se/bakover/web/services/tilbakekreving/LokalMottaKravgrunnlagJob.kt +++ b/web/src/main/kotlin/no/nav/su/se/bakover/web/services/tilbakekreving/LokalMottaKravgrunnlagJob.kt @@ -116,13 +116,13 @@ internal class LokalMottaKravgrunnlagJob( kontrollfelt = kravgrunnlag.eksternKontrollfelt, saksbehId = kravgrunnlag.behandler, utbetalingId = kravgrunnlag.utbetalingId.toString(), - tilbakekrevingsperioder = kravgrunnlag.grunnlagsperioder.map { + tilbakekrevingsperioder = kravgrunnlag.grunnlagsmåneder.map { KravgrunnlagDto.Tilbakekrevingsperiode( periode = KravgrunnlagDto.Tilbakekrevingsperiode.Periode( fraOgMed = it.måned.fraOgMed.toString(), tilOgMed = it.måned.tilOgMed.toString(), ), - skattebeløpPerMåned = it.beløpSkattMnd.toString(), + skattebeløpPerMåned = it.betaltSkattForYtelsesgruppen.toString(), tilbakekrevingsbeløp = it.grunnlagsbeløp.map { KravgrunnlagDto.Tilbakekrevingsperiode.Tilbakekrevingsbeløp( kodeKlasse = it.kode.toString(), @@ -161,11 +161,11 @@ fun matchendeKravgrunnlag( status = Kravgrunnlag.KravgrunnlagStatus.Nytt, behandler = "K231B433", utbetalingId = utbetalingId, - grunnlagsperioder = it.hentFeilutbetalteBeløp() + grunnlagsmåneder = it.hentFeilutbetalteBeløp() .map { (måned, feilutbetaling) -> Kravgrunnlag.Grunnlagsmåned( måned = måned, - beløpSkattMnd = BigDecimal(4395), + betaltSkattForYtelsesgruppen = BigDecimal(4395), grunnlagsbeløp = listOf( Kravgrunnlag.Grunnlagsmåned.Grunnlagsbeløp( kode = KlasseKode.KL_KODE_FEIL_INNT, @@ -225,13 +225,13 @@ internal fun matchendeKravgrunnlagDto( kontrollfelt = kravgrunnlag.eksternKontrollfelt, saksbehId = kravgrunnlag.behandler, utbetalingId = kravgrunnlag.utbetalingId.toString(), - tilbakekrevingsperioder = kravgrunnlag.grunnlagsperioder.map { + tilbakekrevingsperioder = kravgrunnlag.grunnlagsmåneder.map { KravgrunnlagDto.Tilbakekrevingsperiode( periode = KravgrunnlagDto.Tilbakekrevingsperiode.Periode( fraOgMed = it.måned.fraOgMed.toString(), tilOgMed = it.måned.tilOgMed.toString(), ), - skattebeløpPerMåned = it.beløpSkattMnd.toString(), + skattebeløpPerMåned = it.betaltSkattForYtelsesgruppen.toString(), tilbakekrevingsbeløp = it.grunnlagsbeløp.map { KravgrunnlagDto.Tilbakekrevingsperiode.Tilbakekrevingsbeløp( kodeKlasse = it.kode.toString(), diff --git a/web/src/test/kotlin/no/nav/su/se/bakover/web/services/tilbakekreving/TilbakekrevingsmeldingMapperTest.kt b/web/src/test/kotlin/no/nav/su/se/bakover/web/services/tilbakekreving/TilbakekrevingsmeldingMapperTest.kt index dab45be601..633ffa07b8 100644 --- a/web/src/test/kotlin/no/nav/su/se/bakover/web/services/tilbakekreving/TilbakekrevingsmeldingMapperTest.kt +++ b/web/src/test/kotlin/no/nav/su/se/bakover/web/services/tilbakekreving/TilbakekrevingsmeldingMapperTest.kt @@ -86,10 +86,10 @@ internal class TilbakekrevingsmeldingMapperTest { status = Kravgrunnlag.KravgrunnlagStatus.Nytt, behandler = "K231B433", utbetalingId = UUID30.fromString("268e62fb-3079-4e8d-ab32-ff9fb9"), - grunnlagsperioder = listOf( + grunnlagsmåneder = listOf( Kravgrunnlag.Grunnlagsmåned( måned = oktober(2021), - beløpSkattMnd = BigDecimal(4395).setScale(2), + betaltSkattForYtelsesgruppen = BigDecimal(4395).setScale(2), grunnlagsbeløp = listOf( Kravgrunnlag.Grunnlagsmåned.Grunnlagsbeløp( kode = KlasseKode.KL_KODE_FEIL_INNT, @@ -208,10 +208,10 @@ internal class TilbakekrevingsmeldingMapperTest { status = Kravgrunnlag.KravgrunnlagStatus.Nytt, behandler = "K231B433", utbetalingId = UUID30.fromString("268e62fb-3079-4e8d-ab32-ff9fb9"), - grunnlagsperioder = listOf( + grunnlagsmåneder = listOf( Kravgrunnlag.Grunnlagsmåned( måned = oktober(2021), - beløpSkattMnd = BigDecimal(5280).setScale(2), + betaltSkattForYtelsesgruppen = BigDecimal(5280).setScale(2), grunnlagsbeløp = listOf( Kravgrunnlag.Grunnlagsmåned.Grunnlagsbeløp( kode = KlasseKode.KL_KODE_FEIL_INNT, @@ -236,7 +236,7 @@ internal class TilbakekrevingsmeldingMapperTest { ), Kravgrunnlag.Grunnlagsmåned( måned = november(2021), - beløpSkattMnd = BigDecimal(5280).setScale(2), + betaltSkattForYtelsesgruppen = BigDecimal(5280).setScale(2), grunnlagsbeløp = listOf( Kravgrunnlag.Grunnlagsmåned.Grunnlagsbeløp( kode = KlasseKode.KL_KODE_FEIL_INNT,