Brev flere tiltak (#852)

* fjerner tiltaksnavnet fra innvilgelse- og revurdering-pdfer

* sender liste med tiltak til utbetalingsvedtak-pdf, rydder opp litt

* finner overlappende perioder for meldekort

* retter opp i logikk rundt tiltaksdeltakelser og periodisering

Author: tiafiring

app/src/main/kotlin/no/nav/tiltakspenger/vedtak/clients/pdfgen/BrevPersonaliaDTO.kt

@@ -4,5 +4,4 @@ internal data class BrevPersonaliaDTO(
     val ident: String,
     val fornavn: String,
     val etternavn: String,
-    val antallBarn: Int,
 )

app/src/main/kotlin/no/nav/tiltakspenger/vedtak/clients/pdfgen/BrevRevurderingStansDTO.kt

@@ -14,7 +14,6 @@ private class BrevRevurderingStansDTO(
     val personalia: BrevPersonaliaDTO,
     val saksnummer: String,
     val datoForUtsending: String,
-    val tiltaksnavn: String,
     val barnetillegg: Boolean = false,
     val rammevedtakFraDato: String,
     val rammevedtakTilDato: String,
@@ -36,7 +35,6 @@ internal suspend fun Rammevedtak.toRevurderingStans(
         fnr = fnr,
         saksbehandlerNavIdent = saksbehandlerNavIdent,
         beslutterNavIdent = beslutterNavIdent,
-        tiltaksnavn = this.behandling.tiltaksnavn,
         stansperiode = this.periode,
         saksnummer = saksnummer,
         // finnes ikke noe forhåndsvisning for Rammevedtak
@@ -51,7 +49,6 @@ internal suspend fun genererStansbrev(
     fnr: Fnr,
     saksbehandlerNavIdent: String,
     beslutterNavIdent: String?,
-    tiltaksnavn: String,
     stansperiode: Periode,
     saksnummer: Saksnummer,
     forhåndsvisning: Boolean,
@@ -65,9 +62,7 @@ internal suspend fun genererStansbrev(
             ident = fnr.verdi,
             fornavn = brukersNavn.fornavn,
             etternavn = brukersNavn.mellomnavnOgEtternavn,
-            antallBarn = 0,
         ),
-        tiltaksnavn = tiltaksnavn,
         rammevedtakFraDato = stansperiode.fraOgMed.format(norskDatoFormatter),
         rammevedtakTilDato = stansperiode.tilOgMed.format(norskDatoFormatter),
         saksnummer = saksnummer.verdi,

app/src/main/kotlin/no/nav/tiltakspenger/vedtak/clients/pdfgen/BrevSøknadInnvilgetDTO.kt

@@ -16,7 +16,6 @@ import java.time.LocalDate
 @Suppress("unused")
 private data class BrevFørstegangsvedtakInnvilgelseDTO(
     val personalia: BrevPersonaliaDTO,
-    val tiltaksnavn: String,
     val rammevedtakFraDato: String,
     val rammevedtakTilDato: String,
     val saksnummer: String,
@@ -54,7 +53,6 @@ internal suspend fun Rammevedtak.toInnvilgetSøknadsbrev(
         fnr = fnr,
         saksbehandlerNavIdent = saksbehandlerNavIdent,
         beslutterNavIdent = beslutterNavIdent,
-        tiltaksnavn = this.behandling.tiltaksnavn,
         innvilgelsesperiode = this.periode,
         saksnummer = saksnummer,
         // finnes ikke noe forhåndsvisning for rammevedtak
@@ -71,7 +69,6 @@ internal suspend fun genererInnvilgetSøknadsbrev(
     fnr: Fnr,
     saksbehandlerNavIdent: String,
     beslutterNavIdent: String?,
-    tiltaksnavn: String,
     innvilgelsesperiode: Periode,
     saksnummer: Saksnummer,
     forhåndsvisning: Boolean,
@@ -86,9 +83,7 @@ internal suspend fun genererInnvilgetSøknadsbrev(
             ident = fnr.verdi,
             fornavn = brukersNavn.fornavn,
             etternavn = brukersNavn.mellomnavnOgEtternavn,
-            antallBarn = 0,
         ),
-        tiltaksnavn = tiltaksnavn,
         rammevedtakFraDato = innvilgelsesperiode.fraOgMed.format(norskDatoFormatter),
         rammevedtakTilDato = innvilgelsesperiode.tilOgMed.format(norskDatoFormatter),
         saksnummer = saksnummer.verdi,

app/src/main/kotlin/no/nav/tiltakspenger/vedtak/clients/pdfgen/PdfgenHttpClient.kt

@@ -19,6 +19,7 @@ import no.nav.tiltakspenger.meldekort.ports.GenererUtbetalingsvedtakGateway
 import no.nav.tiltakspenger.saksbehandling.domene.behandling.FritekstTilVedtaksbrev
 import no.nav.tiltakspenger.saksbehandling.domene.personopplysninger.Navn
 import no.nav.tiltakspenger.saksbehandling.domene.sak.Saksnummer
+import no.nav.tiltakspenger.saksbehandling.domene.tiltak.Tiltaksdeltagelse
 import no.nav.tiltakspenger.saksbehandling.domene.vedtak.Rammevedtak
 import no.nav.tiltakspenger.saksbehandling.ports.GenererInnvilgelsesvedtaksbrevGateway
 import no.nav.tiltakspenger.saksbehandling.ports.GenererStansvedtaksbrevGateway
@@ -102,7 +103,6 @@ internal class PdfgenHttpClient(
         fnr: Fnr,
         saksbehandlerNavIdent: String,
         beslutterNavIdent: String?,
-        tiltaksnavn: String,
         innvilgelsesperiode: Periode,
         saksnummer: Saksnummer,
         sakId: SakId,
@@ -119,7 +119,6 @@ internal class PdfgenHttpClient(
                     fnr = fnr,
                     saksbehandlerNavIdent = saksbehandlerNavIdent,
                     beslutterNavIdent = beslutterNavIdent,
-                    tiltaksnavn = tiltaksnavn,
                     innvilgelsesperiode = innvilgelsesperiode,
                     saksnummer = saksnummer,
                     forhåndsvisning = forhåndsvisning,
@@ -133,20 +132,14 @@ internal class PdfgenHttpClient(
 
     override suspend fun genererUtbetalingsvedtak(
         utbetalingsvedtak: Utbetalingsvedtak,
-        tiltakstype: String,
-        tiltaksnavn: String,
-        eksternGjennomføringId: String?,
-        eksternDeltagelseId: String,
+        tiltaksdeltagelser: List<Tiltaksdeltagelse>,
         hentSaksbehandlersNavn: suspend (String) -> String,
     ): Either<KunneIkkeGenererePdf, PdfOgJson> {
         return pdfgenRequest(
             jsonPayload = {
                 utbetalingsvedtak.toJsonRequest(
                     hentSaksbehandlersNavn,
-                    tiltakstype,
-                    tiltaksnavn,
-                    eksternGjennomføringId,
-                    eksternDeltagelseId,
+                    tiltaksdeltagelser,
                 )
             },
             errorContext = "SakId: ${utbetalingsvedtak.sakId}, saksnummer: ${utbetalingsvedtak.saksnummer}, vedtakId: ${utbetalingsvedtak.id}",
@@ -174,7 +167,6 @@ internal class PdfgenHttpClient(
         fnr: Fnr,
         saksbehandlerNavIdent: String,
         beslutterNavIdent: String?,
-        tiltaksnavn: String,
         stansperiode: Periode,
         saksnummer: Saksnummer,
         sakId: SakId,
@@ -189,7 +181,6 @@ internal class PdfgenHttpClient(
                     fnr = fnr,
                     saksbehandlerNavIdent = saksbehandlerNavIdent,
                     beslutterNavIdent = beslutterNavIdent,
-                    tiltaksnavn = tiltaksnavn,
                     stansperiode = stansperiode,
                     saksnummer = saksnummer,
                     forhåndsvisning = forhåndsvisning,

app/src/main/kotlin/no/nav/tiltakspenger/vedtak/clients/pdfgen/UtbetalingsvedtakDTO.kt

@@ -4,6 +4,7 @@ import com.fasterxml.jackson.annotation.JsonInclude
 import no.nav.tiltakspenger.libs.json.serialize
 import no.nav.tiltakspenger.meldekort.domene.MeldeperiodeBeregningDag
 import no.nav.tiltakspenger.meldekort.domene.ReduksjonAvYtelsePåGrunnAvFravær
+import no.nav.tiltakspenger.saksbehandling.domene.tiltak.Tiltaksdeltagelse
 import no.nav.tiltakspenger.utbetaling.domene.Utbetalingsvedtak
 import no.nav.tiltakspenger.vedtak.clients.pdfgen.formattering.norskDatoFormatter
 import no.nav.tiltakspenger.vedtak.clients.pdfgen.formattering.norskTidspunktFormatter
@@ -15,10 +16,7 @@ private data class UtbetalingsvedtakDTO(
     val saksbehandler: SaksbehandlerDTO,
     val beslutter: SaksbehandlerDTO,
     val meldekortDager: List<MeldekortDagDTO>,
-    val tiltakstype: String,
-    val eksternDeltagelseId: String,
-    val eksternGjennomføringId: String?,
-    val tiltaksnavn: String,
+    val tiltak: List<TiltakDTO>,
     val iverksattTidspunkt: String,
     val fødselsnummer: String,
 ) {
@@ -28,7 +26,6 @@ private data class UtbetalingsvedtakDTO(
 
     data class SaksbehandlerDTO(
         val navn: String,
-        val navIdent: String,
     )
 
     data class PeriodeDTO(
@@ -38,21 +35,24 @@ private data class UtbetalingsvedtakDTO(
 
     data class MeldekortDagDTO(
         val dato: String,
-        val tiltakType: String,
         val status: String,
         val beløp: Int,
         val beløpBarnetillegg: Int,
         val prosent: Int,
         val reduksjon: String?,
     )
+
+    data class TiltakDTO(
+        val tiltakstypenavn: String,
+        val tiltakstype: String,
+        val eksternDeltagelseId: String,
+        val eksternGjennomføringId: String?,
+    )
 }
 
 suspend fun Utbetalingsvedtak.toJsonRequest(
     hentSaksbehandlersNavn: suspend (String) -> String,
-    tiltakstype: String,
-    tiltaksnavn: String,
-    eksternGjennomføringId: String?,
-    eksternDeltagelseId: String,
+    tiltaksdeltagelser: List<Tiltaksdeltagelse>,
 ): String {
     return UtbetalingsvedtakDTO(
         fødselsnummer = fnr.verdi,
@@ -67,24 +67,28 @@ suspend fun Utbetalingsvedtak.toJsonRequest(
         meldekortDager = meldekortbehandling.beregning.dager.map { dag ->
             UtbetalingsvedtakDTO.MeldekortDagDTO(
                 dato = dag.dato.format(norskDatoFormatter),
-                tiltakType = dag.tiltakstype.toString(),
                 status = dag.toStatus(),
                 beløp = dag.beløp,
                 beløpBarnetillegg = dag.beløpBarnetillegg,
                 prosent = dag.prosent,
                 reduksjon = dag.toReduksjon(),
             )
         },
-        eksternGjennomføringId = eksternGjennomføringId,
-        tiltakstype = tiltakstype,
-        tiltaksnavn = tiltaksnavn,
-        eksternDeltagelseId = eksternDeltagelseId,
+        tiltak = tiltaksdeltagelser.map { it.toTiltakDTO() },
         iverksattTidspunkt = opprettet.format(norskTidspunktFormatter),
     ).let { serialize(it) }
 }
 
+private fun Tiltaksdeltagelse.toTiltakDTO() =
+    UtbetalingsvedtakDTO.TiltakDTO(
+        tiltakstypenavn = typeNavn,
+        tiltakstype = typeKode.name,
+        eksternDeltagelseId = eksternDeltagelseId,
+        eksternGjennomføringId = gjennomføringId,
+    )
+
 private suspend fun tilSaksbehadlerDto(navIdent: String, hentSaksbehandlersNavn: suspend (String) -> String): UtbetalingsvedtakDTO.SaksbehandlerDTO {
-    return UtbetalingsvedtakDTO.SaksbehandlerDTO(navn = hentSaksbehandlersNavn(navIdent), navIdent = navIdent)
+    return UtbetalingsvedtakDTO.SaksbehandlerDTO(navn = hentSaksbehandlersNavn(navIdent))
 }
 
 private fun MeldeperiodeBeregningDag.toStatus(): String {

app/src/main/kotlin/no/nav/tiltakspenger/vedtak/routes/meldekort/dto/MeldeperiodeKjedeDTO.kt

@@ -5,6 +5,7 @@ import no.nav.tiltakspenger.libs.periodisering.PeriodeDTO
 import no.nav.tiltakspenger.libs.periodisering.toDTO
 import no.nav.tiltakspenger.saksbehandling.domene.sak.Sak
 
+// TODO: fjerne tiltaksnavn fra responsen, eller la det være en liste
 data class MeldeperiodeKjedeDTO(
     val kjedeId: String,
     val periode: PeriodeDTO,
@@ -18,7 +19,7 @@ fun Sak.toMeldeperiodeKjedeDTO(meldeperiodeKjedeId: MeldeperiodeKjedeId): Meldep
     return MeldeperiodeKjedeDTO(
         kjedeId = meldeperiodeKjede.kjedeId.toString(),
         periode = meldeperiodeKjede.periode.toDTO(),
-        tiltaksnavn = this.hentTiltaksnavn(),
+        tiltaksnavn = this.vedtaksliste.valgteTiltaksdeltakelserForPeriode(meldeperiodeKjede.periode).map { it.typeNavn }.joinToString { it.verdi },
         meldeperioder = meldeperiodeKjede.map { toMeldeperiodeDTO(it) },
     )
 }
@@ -28,7 +29,7 @@ fun Sak.toMeldeperiodeKjederDTO(): List<MeldeperiodeKjedeDTO> {
         MeldeperiodeKjedeDTO(
             kjedeId = meldeperiodeKjede.kjedeId.toString(),
             periode = meldeperiodeKjede.periode.toDTO(),
-            tiltaksnavn = this.hentTiltaksnavn(),
+            tiltaksnavn = this.vedtaksliste.valgteTiltaksdeltakelserForPeriode(meldeperiodeKjede.periode).map { it.typeNavn }.joinToString { it.verdi },
             meldeperioder = meldeperiodeKjede.map { toMeldeperiodeDTO(it) },
         )
     }

app/src/test/kotlin/no/nav/tiltakspenger/clients/pdfgen/PdfgenHttpClientTest.kt

@@ -13,10 +13,7 @@ class PdfgenHttpClientTest {
             val utbetalingsvedtak = ObjectMother.utbetalingsvedtak()
             PdfgenHttpClient("unused").genererUtbetalingsvedtak(
                 utbetalingsvedtak,
-                tiltakstype = "tiltakstype",
-                tiltaksnavn = "tiltaksnavn",
-                eksternDeltagelseId = "213-1232-2133-123",
-                eksternGjennomføringId = null,
+                tiltaksdeltagelser = listOf(ObjectMother.tiltaksdeltagelse()),
             ) { ObjectMother.saksbehandler().brukernavn }
         }
     }

domene/src/main/kotlin/no/nav/tiltakspenger/meldekort/ports/GenererUtbetalingsvedtakGateway.kt

@@ -3,15 +3,13 @@ package no.nav.tiltakspenger.meldekort.ports
 import arrow.core.Either
 import no.nav.tiltakspenger.felles.KunneIkkeGenererePdf
 import no.nav.tiltakspenger.felles.journalføring.PdfOgJson
+import no.nav.tiltakspenger.saksbehandling.domene.tiltak.Tiltaksdeltagelse
 import no.nav.tiltakspenger.utbetaling.domene.Utbetalingsvedtak
 
 interface GenererUtbetalingsvedtakGateway {
     suspend fun genererUtbetalingsvedtak(
         utbetalingsvedtak: Utbetalingsvedtak,
-        tiltakstype: String,
-        tiltaksnavn: String,
-        eksternGjennomføringId: String?,
-        eksternDeltagelseId: String,
+        tiltaksdeltagelser: List<Tiltaksdeltagelse>,
         hentSaksbehandlersNavn: suspend (String) -> String,
     ): Either<KunneIkkeGenererePdf, PdfOgJson>
 }

domene/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/domene/behandling/Behandling.kt

@@ -12,7 +12,6 @@ import no.nav.tiltakspenger.libs.common.SakId
 import no.nav.tiltakspenger.libs.common.Saksbehandler
 import no.nav.tiltakspenger.libs.periodisering.Periode
 import no.nav.tiltakspenger.libs.periodisering.Periodisering
-import no.nav.tiltakspenger.libs.tiltak.TiltakstypeSomGirRett
 import no.nav.tiltakspenger.saksbehandling.domene.behandling.Behandlingsstatus.AVBRUTT
 import no.nav.tiltakspenger.saksbehandling.domene.behandling.Behandlingsstatus.KLAR_TIL_BEHANDLING
 import no.nav.tiltakspenger.saksbehandling.domene.behandling.Behandlingsstatus.KLAR_TIL_BESLUTNING
@@ -75,12 +74,8 @@ data class Behandling(
     /** John og Agnethe har kommet fram til at vi setter denne til 14 dager for meldeperiode i førsteomgang. Hvis det fører til mye feilutbetaling eller lignende må vi la saksbehandler periodisere dette selv, litt på samme måte som barnetillegg. */
     val maksDagerMedTiltakspengerForPeriode: Int = 14
 
-    // TODO: Når saksbehandler kan velge hvilken deltakelse som gjelder på hvilke dager må vi bruke det som er valgt, ikke bare den første
+    // TODO: Brukes bare for å angi deltakelse frem til saksbehandler setter den selv via input. Skal fjernes når det er på plass.
     private val tiltaksdeltakelse = saksopplysninger.tiltaksdeltagelse.first()
-    val tiltaksnavn = tiltaksdeltakelse.typeNavn
-    val tiltakstype: TiltakstypeSomGirRett = tiltaksdeltakelse.typeKode
-    val tiltaksid: String = tiltaksdeltakelse.eksternDeltagelseId
-    val gjennomføringId: String? = tiltaksdeltakelse.gjennomføringId
 
     fun inneholderEksternDeltagelseId(eksternDeltagelseId: String): Boolean =
         saksopplysninger.tiltaksdeltagelse.find { it.eksternDeltagelseId == eksternDeltagelseId } != null
@@ -458,6 +453,9 @@ data class Behandling(
                     val barnetilleggsperiode = barnetillegg.periodisering.totalePeriode
                     require(barnetilleggsperiode == virkningsperiode) { "Barnetilleggsperioden ($barnetilleggsperiode) må ha samme periode som virkningsperioden($virkningsperiode)" }
                 }
+                if (behandlingstype == FØRSTEGANGSBEHANDLING) {
+                    require(valgteTiltaksdeltakelser != null) { "Valgte tiltaksdeltakelser må være satt for førstegangsbehandling" }
+                }
             }
 
             UNDER_BESLUTNING -> {
@@ -470,6 +468,9 @@ data class Behandling(
                     val barnetilleggsperiode = barnetillegg.periodisering.totalePeriode
                     require(barnetilleggsperiode == virkningsperiode) { "Barnetilleggsperioden ($barnetilleggsperiode) må ha samme periode som virkningsperioden($virkningsperiode)" }
                 }
+                if (behandlingstype == FØRSTEGANGSBEHANDLING) {
+                    require(valgteTiltaksdeltakelser != null) { "Valgte tiltaksdeltakelser må være satt for førstegangsbehandling" }
+                }
             }
 
             VEDTATT -> {
@@ -483,6 +484,9 @@ data class Behandling(
                     val barnetilleggsperiode = barnetillegg.periodisering.totalePeriode
                     require(barnetilleggsperiode == virkningsperiode) { "Barnetilleggsperioden ($barnetilleggsperiode) må ha samme periode som virkningsperioden($virkningsperiode)" }
                 }
+                if (behandlingstype == FØRSTEGANGSBEHANDLING) {
+                    require(valgteTiltaksdeltakelser != null) { "Valgte tiltaksdeltakelser må være satt for førstegangsbehandling" }
+                }
             }
 
             AVBRUTT -> {

domene/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/domene/sak/Sak.kt

@@ -20,7 +20,6 @@ import no.nav.tiltakspenger.meldekort.domene.MeldeperiodeKjeder
 import no.nav.tiltakspenger.saksbehandling.domene.behandling.Behandling
 import no.nav.tiltakspenger.saksbehandling.domene.behandling.Behandlinger
 import no.nav.tiltakspenger.saksbehandling.domene.behandling.Søknad
-import no.nav.tiltakspenger.saksbehandling.domene.tiltak.Tiltaksdeltagelse
 import no.nav.tiltakspenger.saksbehandling.domene.vedtak.Vedtaksliste
 import no.nav.tiltakspenger.saksbehandling.service.avslutt.AvbrytSøknadOgBehandlingCommand
 import no.nav.tiltakspenger.utbetaling.domene.Utbetalinger
@@ -66,8 +65,6 @@ data class Sak(
 
     val barnetilleggsperioder: Periodisering<AntallBarn> by lazy { vedtaksliste.barnetilleggsperioder }
 
-    val tiltaksdeltagelseperioder: Periodisering<Tiltaksdeltagelse> by lazy { vedtaksliste.tiltaksdeltagelseperioder }
-
     val tiltakstypeperioder: Periodisering<TiltakstypeSomGirRett> by lazy { vedtaksliste.tiltakstypeperioder }
 
     fun antallBarnForDag(dag: LocalDate): AntallBarn {
@@ -103,9 +100,6 @@ data class Sak(
     fun hentAntallDager(): Int? = vedtaksliste.førstegangsvedtak?.behandling?.maksDagerMedTiltakspengerForPeriode
     fun hentTynnSak(): TynnSak = TynnSak(this.id, this.fnr, this.saksnummer)
 
-    /** Den er kun trygg inntil vi støtter mer enn ett tiltak på én sak. */
-    fun hentTiltaksnavn(): String? = vedtaksliste.førstegangsvedtak?.behandling?.tiltaksnavn
-
     fun hentBehandling(behandlingId: BehandlingId): Behandling? = behandlinger.hentBehandling(behandlingId)
 
     fun sisteUtbetalteMeldekortDag(): LocalDate? = meldekortBehandlinger.sisteUtbetalteMeldekortDag

domene/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/domene/tiltak/ValgteTiltaksdeltakelser.kt

@@ -12,7 +12,7 @@ data class ValgteTiltaksdeltakelser(
     val periodisering: Periodisering<Tiltaksdeltagelse>,
 ) {
     companion object {
-        // TODO: Sett inn det saksbehandler faktisk valgte og sjekk at det ikke er huller eller overlapp
+        // TODO: Sett inn det saksbehandler faktisk valgte
         fun periodiser(
             tiltaksdeltagelse: Tiltaksdeltagelse,
             periode: Periode,