opprydning av sak, behandling, og meldeperiodekjeder

Author: ramnav990

Diff for: app/build.gradle.kts

@@ -1,6 +1,6 @@
 val kotlinxCoroutinesVersion = "1.10.1"
 val kotestVersion = "5.9.1"
-val felleslibVersion = "0.0.403"
+val felleslibVersion = "0.0.406"
 val mockkVersion = "1.13.17"
 val ktorVersion = "3.1.1"
 val testContainersVersion = "1.20.6"

Diff for: app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/meldekort/domene/MeldeperiodeKjeder.kt

@@ -9,6 +9,7 @@ import no.nav.tiltakspenger.libs.common.MeldeperiodeKjedeId
 import no.nav.tiltakspenger.libs.common.SakId
 import no.nav.tiltakspenger.libs.common.nonDistinctBy
 import no.nav.tiltakspenger.libs.periodisering.Periode
+import no.nav.tiltakspenger.libs.periodisering.overlapperIkke
 import no.nav.tiltakspenger.saksbehandling.felles.singleOrNullOrThrow
 import no.nav.tiltakspenger.saksbehandling.saksbehandling.domene.sak.Saksnummer
 import no.nav.tiltakspenger.saksbehandling.saksbehandling.domene.vedtak.Vedtaksliste
@@ -203,9 +204,6 @@ data class MeldeperiodeKjeder(
     }
 
     companion object {
-        fun List<Periode>.overlapper(periode: Periode): Boolean = this.any { it.overlapperMed(periode) }
-        fun List<Periode>.overlapperIkke(periode: Periode): Boolean = !this.any { it.overlapperMed(periode) }
-
         /**
          * Skal kun kalles/brukes på en sak som aldri har hatt en meldeperiode før
          */

Diff for: app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/repository/sak/SakPostgresRepo.kt

@@ -201,9 +201,7 @@ internal class SakPostgresRepo(
             return sessionContext.withSession { session ->
                 val behandlinger = BehandlingPostgresRepo.hentForSakId(id, session)
                 val vedtaksliste: Vedtaksliste = RammevedtakPostgresRepo.hentForSakId(id, session)
-                val meldekortBehandlinger = vedtaksliste.førstegangsvedtak?.let {
-                    MeldekortBehandlingPostgresRepo.hentForSakId(id, session)
-                } ?: MeldekortBehandlinger.empty()
+                val meldekortBehandlinger = MeldekortBehandlingPostgresRepo.hentForSakId(id, session) ?: MeldekortBehandlinger.empty()
                 val meldeperioder = MeldeperiodePostgresRepo.hentForSakId(id, session)
                 val soknader = SøknadDAO.hentForSakId(id, session)
                 Sak(

Diff for: app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/saksbehandling/domene/behandling/Behandling.kt

@@ -27,6 +27,7 @@ import no.nav.tiltakspenger.saksbehandling.saksbehandling.domene.tiltak.ValgteTi
 import no.nav.tiltakspenger.saksbehandling.saksbehandling.domene.vilkår.Utfallsperiode
 import no.nav.tiltakspenger.saksbehandling.saksbehandling.domene.vilkår.Utfallsperiode.IKKE_RETT_TIL_TILTAKSPENGER
 import no.nav.tiltakspenger.saksbehandling.saksbehandling.domene.vilkår.Utfallsperiode.RETT_TIL_TILTAKSPENGER
+import java.time.LocalDate
 import java.time.LocalDateTime
 
 /**
@@ -253,7 +254,7 @@ data class Behandling(
 
     fun sendRevurderingTilBeslutning(
         kommando: SendRevurderingTilBeslutningKommando,
-        vedtaksperiode: Periode,
+        sisteDagSomGirRett: LocalDate,
     ): Behandling {
         check(status == UNDER_BEHANDLING) {
             "Behandlingen må være under behandling, det innebærer også at en saksbehandler må ta saken før den kan sendes til beslutter. Behandlingsstatus: ${this.status}. Utøvende saksbehandler: $saksbehandler. Saksbehandler på behandling: ${this.saksbehandler}"
@@ -264,7 +265,7 @@ data class Behandling(
             status = if (beslutter == null) KLAR_TIL_BESLUTNING else UNDER_BESLUTNING,
             sendtTilBeslutning = nå(),
             begrunnelseVilkårsvurdering = kommando.begrunnelse,
-            virkningsperiode = Periode(kommando.stansDato, vedtaksperiode.tilOgMed),
+            virkningsperiode = Periode(kommando.stansDato, sisteDagSomGirRett),
         )
     }
 

Diff for: app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/saksbehandling/domene/behandling/SakBehandlingEx.kt

@@ -47,7 +47,7 @@ fun Sak.sendRevurderingTilBeslutning(
 
     val behandling: Behandling = this.hentBehandling(kommando.behandlingId)!!
     require(behandling.erRevurdering) { "Finnes egen funksjon for å sende til førstegangbehandling til beslutning" }
-    val oppdatertBehandling = behandling.sendRevurderingTilBeslutning(kommando, this.vedtaksperiode!!)
+    val oppdatertBehandling = behandling.sendRevurderingTilBeslutning(kommando, this.vedtaksliste.sisteDagSomGirRett!!)
 
     return oppdatertBehandling.right()
 }

Diff for: app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/saksbehandling/domene/behandling/SakRevurderingEx.kt

@@ -32,9 +32,9 @@ suspend fun Sak.startRevurdering(
         "Kan kun opprette en stansrevurdering dersom vi har en sammenhengende innvilgelsesperiode. sakId=${this.id}"
     }
 
-    requireNotNull(this.ikkeAvbruttFørstegangsbehandlinger) { "Kan ikke opprette revurdering uten en førstegangsbehandling" }
-
-    val saksopplysningsperiode = this.førsteSaksopplysningsperiode!!
+    // TODO - dette gjelder bare så lenge dette er en stans revurdering
+    // Her har vi ikke valgt revurderingsperioden, men revurderingsperioden kan aldri være større enn innvilgelseprioden
+    val saksopplysningsperiode = this.vedtaksliste.innvilgelsesperiode!!
 
     val revurdering = Behandling.opprettRevurdering(
         sakId = this.id,

Diff for: app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/saksbehandling/domene/sak/Sak.kt

@@ -11,7 +11,6 @@ import no.nav.tiltakspenger.libs.periodisering.Periodisering
 import no.nav.tiltakspenger.libs.tiltak.TiltakstypeSomGirRett
 import no.nav.tiltakspenger.saksbehandling.barnetillegg.AntallBarn
 import no.nav.tiltakspenger.saksbehandling.felles.Navkontor
-import no.nav.tiltakspenger.saksbehandling.felles.min
 import no.nav.tiltakspenger.saksbehandling.meldekort.domene.BrukersMeldekort
 import no.nav.tiltakspenger.saksbehandling.meldekort.domene.MeldekortBehandling
 import no.nav.tiltakspenger.saksbehandling.meldekort.domene.MeldekortBehandlinger
@@ -40,26 +39,15 @@ data class Sak(
     val utbetalinger: Utbetalinger,
     val soknader: List<Søknad>,
 ) {
-    /** Nåtilstand. Dette er sakens totale vedtaksperiode. Vær veldig obs når du bruker denne, fordi den sier ikke noe om antall perioder, om de gir rett eller ikke. */
-    val vedtaksperiode: Periode? = vedtaksliste.vedtaksperiode
-
     /** Nåtilstand. Tar utgangspunkt i tidslinja på saken og henter den siste innvilget dagen. */
     val førsteDagSomGirRett = vedtaksliste.førsteDagSomGirRett
 
     /** Nåtilstand. Tar utgangspunkt i tidslinja på saken og henter den siste innvilget dagen. */
     @Suppress("unused")
     val sisteDagSomGirRett = vedtaksliste.sisteDagSomGirRett
 
-    /**
-     * En sak kan kun ha en førstegangsbehandling, dersom perioden til den vedtatte førstegangsbehandlingen skal utvides eller minskes (den må fortsatt være sammenhengende) må vi revurdere/omgjøre, ikke førstegangsbehandle på nytt.
-     * Dersom den nye søknaden ikke overlapper eller tilstøter den gamle perioden, må vi opprette en ny sak som får en ny førstegangsbehandling.
-     */
-    val ikkeAvbruttFørstegangsbehandlinger: List<Behandling> = behandlinger.ikkeAvbrutteFørstegangsbehandlinger
     val revurderinger = behandlinger.revurderinger
 
-    val førsteSaksopplysningsperiode: Periode? =
-        ikkeAvbruttFørstegangsbehandlinger.firstOrNull()?.saksopplysningsperiode
-
     /** Henter fra siste godkjente meldekort */
     @Suppress("unused")
     val sisteNavkontor: Navkontor? by lazy {
@@ -107,12 +95,18 @@ data class Sak(
 
     fun sisteUtbetalteMeldekortDag(): LocalDate? = meldekortBehandlinger.sisteUtbetalteMeldekortDag
 
-    /**
-     * Vil være innenfor [vedtaksperiode]. Dersom vi ikke har et vedtak, vil den være null.
-     */
-    fun førsteLovligeStansdato(): LocalDate? = sisteUtbetalteMeldekortDag()?.plusDays(1)?.let {
-        min(it, vedtaksperiode!!.tilOgMed)
-    } ?: vedtaksperiode?.fraOgMed
+    fun førsteLovligeStansdato(): LocalDate? {
+        val innvilgelsesperioder = this.vedtaksliste.innvilgelsesperioder
+        if (innvilgelsesperioder.isEmpty()) return null
+        val førsteDagSomIkkeErUtbetalt =
+            sisteUtbetalteMeldekortDag()?.plusDays(1) ?: innvilgelsesperioder.first().fraOgMed
+
+        return innvilgelsesperioder.firstOrNull {
+            it.tilOgMed >= førsteDagSomIkkeErUtbetalt
+        }?.tilDager()?.firstOrNull {
+            it >= førsteDagSomIkkeErUtbetalt
+        }
+    }
 
     fun erSisteVersjonAvMeldeperiode(meldeperiode: Meldeperiode): Boolean {
         return meldeperiodeKjeder.erSisteVersjonAvMeldeperiode(meldeperiode)

Diff for: app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/saksbehandling/domene/vedtak/Rammevedtak.kt

@@ -114,7 +114,7 @@ fun Sak.utledVedtakstype(behandling: Behandling): Vedtakstype {
         Behandlingstype.REVURDERING -> {
             // Kommentar jah: Dette er en førsteimplementasjon for å avgjøre om dette er et stansvedtak. Ved andre typer revurderinger må vi utvide denne.
             if (behandling.virkningsperiode!!.tilOgMed != this.utfallsperioder().totalePeriode.tilOgMed) {
-                throw IllegalStateException("Kan ikke lage stansvedtak for revurdering - revurderingens tilOgMed (${behandling.virkningsperiode.tilOgMed}) må være lik sakens tilOgMed (${this.vedtaksperiode!!.tilOgMed})")
+                throw IllegalStateException("Kan ikke lage stansvedtak for revurdering - revurderingens tilOgMed (${behandling.virkningsperiode.tilOgMed}) må være lik sakens tilOgMed (${this.utfallsperioder().totalePeriode.tilOgMed})")
             }
 
             if (this.sisteUtbetalteMeldekortDag() == null || this.sisteUtbetalteMeldekortDag()!! < behandling.virkningsperiode.fraOgMed) {

Diff for: app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/saksbehandling/domene/vedtak/Vedtaksliste.kt

@@ -23,17 +23,11 @@ data class Vedtaksliste(
 
     val tidslinje: Periodisering<Rammevedtak?> by lazy { value.toTidslinjeMedHull() }
 
-    val førstegangsvedtak: List<Rammevedtak> by lazy { value.filter { it.erFørstegangsvedtak } }
-
-    /** Nåtilstand. Dette er sakens totale vedtaksperiode. Vær veldig obs når du bruker denne, fordi den sier ikke noe om antall perioder, om de gir rett eller ikke. */
-    val vedtaksperiode: Periode? by lazy { tidslinje.ifEmpty { null }?.totalePeriode }
-
     val innvilgelsesperiode: Periode? by lazy {
         innvilgelsesperioder.ifEmpty { null }?.let { Periode(it.minOf { it.fraOgMed }, it.maxOf { it.tilOgMed }) }
     }
 
     /** Nåtilstand. Sakens totale vedtaksperioder. Vil kunne ha hull dersom det f.eks. er opphold mellom 2 tiltaksdeltagelsesperioder. Avslag og delvis avslag vil ikke være med her. */
-    @Suppress("unused")
     val vedtaksperioder: List<Periode> by lazy { tidslinje.perioder }
 
     /** Nåtilstand. De periodene som gir rett til tiltakspenger. Vil kunne være hull. */

Diff for: app/src/test/kotlin/no/nav/tiltakspenger/saksbehandling/db/TestDataHelperBehandlingEx.kt

@@ -67,7 +67,7 @@ internal fun TestDataHelper.persisterOpprettetFørstegangsbehandling(
             saksnummer = sak.saksnummer,
         ),
     barnetillegg: Barnetillegg? = null,
-): Pair<Sak, Søknad> {
+): Triple<Sak, Behandling, Søknad> {
     this.persisterSakOgSøknad(
         fnr = sak.fnr,
         søknad = søknad,
@@ -83,10 +83,11 @@ internal fun TestDataHelper.persisterOpprettetFørstegangsbehandling(
             sakId = sak.id,
             barnetillegg = barnetillegg,
         )
-    behandlingRepo.lagre(sakMedBehandling.ikkeAvbruttFørstegangsbehandlinger.singleOrNullOrThrow()!!)
+    behandlingRepo.lagre(sakMedBehandling.behandlinger.singleOrNullOrThrow()!!)
 
-    return Pair(
+    return Triple(
         sakRepo.hentForSakId(sakId)!!,
+        sakMedBehandling.behandlinger.singleOrNullOrThrow()!!,
         søknadRepo.hentForSøknadId(søknad.id)!!,
     )
 }
@@ -137,7 +138,7 @@ internal fun TestDataHelper.persisterAvbruttFørstegangsbehandling(
         søknad = søknad,
         sak = sak,
     )
-    val førstegangsbehandling = sakMedFørstegangsbehandling.ikkeAvbruttFørstegangsbehandlinger.singleOrNullOrThrow()!!
+    val førstegangsbehandling = sakMedFørstegangsbehandling.behandlinger.singleOrNullOrThrow()!!
     val avbruttBehandling = førstegangsbehandling.avbryt(
         saksbehandler,
         "begrunnelse",
@@ -184,8 +185,8 @@ internal fun TestDataHelper.persisterIverksattFørstegangsbehandling(
             saksnummer = sak.saksnummer,
         ),
     correlationId: CorrelationId = CorrelationId.generate(),
-): Pair<Sak, Rammevedtak> {
-    val (sak, _) = persisterOpprettetFørstegangsbehandling(
+): Triple<Sak, Rammevedtak, Behandling> {
+    val (sak, førstegangsbehandling) = persisterOpprettetFørstegangsbehandling(
         sakId = sak.id,
         fnr = sak.fnr,
         deltakelseFom = deltakelseFom,
@@ -197,7 +198,6 @@ internal fun TestDataHelper.persisterIverksattFørstegangsbehandling(
         søknad = søknad,
         sak = sak,
     )
-    val førstegangsbehandling = sak.ikkeAvbruttFørstegangsbehandlinger.singleOrNullOrThrow()!!
     val oppdatertFørstegangsbehandling =
         førstegangsbehandling
             .tilBeslutning(
@@ -210,15 +210,20 @@ internal fun TestDataHelper.persisterIverksattFørstegangsbehandling(
                     begrunnelseVilkårsvurdering = BegrunnelseVilkårsvurdering("begrunnelseVilkårsvurdering"),
                     innvilgelsesperiode = tiltaksOgVurderingsperiode,
                     barnetillegg = null,
-                    tiltaksdeltakelser = listOf(Pair(tiltaksOgVurderingsperiode, førstegangsbehandling.saksopplysninger.tiltaksdeltagelse.first().eksternDeltagelseId)),
+                    tiltaksdeltakelser = listOf(
+                        Pair(
+                            tiltaksOgVurderingsperiode,
+                            førstegangsbehandling.saksopplysninger.tiltaksdeltagelse.first().eksternDeltagelseId,
+                        ),
+                    ),
                 ),
             )
             .taBehandling(beslutter)
             .iverksett(beslutter, ObjectMother.godkjentAttestering(beslutter))
     behandlingRepo.lagre(oppdatertFørstegangsbehandling)
     val vedtak = sak.opprettVedtak(oppdatertFørstegangsbehandling).second
     vedtakRepo.lagre(vedtak)
-    return sakRepo.hentForSakId(sakId)!! to vedtak
+    return Triple(sakRepo.hentForSakId(sakId)!!, vedtak, oppdatertFørstegangsbehandling)
 }
 
 /**

Diff for: app/src/test/kotlin/no/nav/tiltakspenger/saksbehandling/objectmothers/BehandlingMother.kt

@@ -297,7 +297,7 @@ suspend fun TestApplicationContext.førstegangsbehandlingTilBeslutter(
     this.behandlingContext.sendBehandlingTilBeslutningService.sendFørstegangsbehandlingTilBeslutning(
         SendSøknadsbehandlingTilBeslutningKommando(
             sakId = sakMedFørstegangsbehandling.id,
-            behandlingId = sakMedFørstegangsbehandling.ikkeAvbruttFørstegangsbehandlinger.singleOrNullOrThrow()!!.id,
+            behandlingId = sakMedFørstegangsbehandling.behandlinger.singleOrNullOrThrow()!!.id,
             saksbehandler = saksbehandler,
             correlationId = correlationId,
             fritekstTilVedtaksbrev = fritekstTilVedtaksbrev,
@@ -307,7 +307,7 @@ suspend fun TestApplicationContext.førstegangsbehandlingTilBeslutter(
             tiltaksdeltakelser = listOf(
                 Pair(
                     periode,
-                    sakMedFørstegangsbehandling.ikkeAvbruttFørstegangsbehandlinger.singleOrNullOrThrow()!!.saksopplysninger.tiltaksdeltagelse.first().eksternDeltagelseId,
+                    sakMedFørstegangsbehandling.behandlinger.singleOrNullOrThrow()!!.saksopplysninger.tiltaksdeltagelse.first().eksternDeltagelseId,
                 ),
             ),
         ),
@@ -332,7 +332,7 @@ suspend fun TestApplicationContext.førstegangsbehandlingUnderBeslutning(
         saksbehandler = saksbehandler,
     )
     this.behandlingContext.behandlingService.taBehandling(
-        vilkårsvurdert.ikkeAvbruttFørstegangsbehandlinger.singleOrNullOrThrow()!!.id,
+        vilkårsvurdert.behandlinger.singleOrNullOrThrow()!!.id,
         beslutter,
         correlationId = correlationId,
     )
@@ -359,7 +359,7 @@ suspend fun TestApplicationContext.førstegangsbehandlingIverksatt(
     )
     runBlocking {
         tac.behandlingContext.iverksettBehandlingService.iverksett(
-            behandlingId = underBeslutning.ikkeAvbruttFørstegangsbehandlinger.singleOrNullOrThrow()!!.id,
+            behandlingId = underBeslutning.behandlinger.singleOrNullOrThrow()!!.id,
             beslutter = beslutter,
             correlationId = correlationId,
             sakId = underBeslutning.id,

Diff for: app/src/test/kotlin/no/nav/tiltakspenger/saksbehandling/repository/behandling/BehandlingRepoTest.kt

@@ -30,7 +30,7 @@ internal class BehandlingRepoTest {
 
             val (sak, _) = testDataHelper.persisterOpprettetFørstegangsbehandling()
             sakRepo.hentForSakId(sak.id) shouldBe sak
-            behandlingRepo.hent(sak.ikkeAvbruttFørstegangsbehandlinger.singleOrNullOrThrow()!!.id) shouldBe sak.ikkeAvbruttFørstegangsbehandlinger.singleOrNullOrThrow()
+            behandlingRepo.hent(sak.behandlinger.singleOrNullOrThrow()!!.id) shouldBe sak.behandlinger.singleOrNullOrThrow()
         }
     }
 
@@ -53,8 +53,8 @@ internal class BehandlingRepoTest {
                 ),
             )
             sakRepo.hentForSakId(sak.id) shouldBe sak
-            behandlingRepo.hent(sak.ikkeAvbruttFørstegangsbehandlinger.singleOrNullOrThrow()!!.id).also {
-                it shouldBe sak.ikkeAvbruttFørstegangsbehandlinger.singleOrNullOrThrow()
+            behandlingRepo.hent(sak.behandlinger.singleOrNullOrThrow()!!.id).also {
+                it shouldBe sak.behandlinger.singleOrNullOrThrow()
                 it.barnetillegg shouldNotBe null
             }
         }
@@ -80,8 +80,8 @@ internal class BehandlingRepoTest {
             val (sak1, _) = testDataHelper.persisterOpprettetFørstegangsbehandling()
             val (sak2, _) = testDataHelper.persisterOpprettetFørstegangsbehandling()
 
-            behandlingRepo.hentAlleForFnr(sak1.fnr) shouldBe sak1.ikkeAvbruttFørstegangsbehandlinger
-            behandlingRepo.hentAlleForFnr(sak2.fnr) shouldBe sak2.ikkeAvbruttFørstegangsbehandlinger
+            behandlingRepo.hentAlleForFnr(sak1.fnr) shouldBe sak1.behandlinger
+            behandlingRepo.hentAlleForFnr(sak2.fnr) shouldBe sak2.behandlinger
         }
     }
 }