Hardere validering på meldekortkjeder. log feil ved opprett meldekortbehandling

Author: hestad

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

@@ -39,7 +39,11 @@ data class MeldeperiodeKjeder(
         }
 
         meldeperiodeKjeder.zipWithNext { a, b ->
-            require(a.periode.fraOgMed <= b.periode.fraOgMed) {
+            require(a.kjedeId != b.kjedeId) {
+                "Meldeperiodekjedene kan ikke ha samme kjedeId - ${a.kjedeId} og ${b.kjedeId} (sak ${a.sakId})"
+            }
+            // 2 kjeder kan ikke ha samme meldeperiode. Og de må være sortert på periode.
+            require(a.periode.tilOgMed < b.periode.fraOgMed) {
                 "Meldeperiodekjedene må være sortert på periode - ${a.kjedeId} og ${b.kjedeId} var i feil rekkefølge (sak ${a.sakId})"
             }
             require(ChronoUnit.DAYS.between(a.periode.fraOgMed, b.periode.fraOgMed) % 14 == 0L) {

app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/meldekort/service/OpprettMeldekortBehandlingService.kt

@@ -56,6 +56,7 @@ class OpprettMeldekortBehandlingService(
                 clock = clock,
             )
         }.getOrElse {
+            logger.error(it) { "Kunne ikke opprette meldekort behandling på kjede $kjedeId for sak $sakId" }
             return KanIkkeOppretteMeldekortBehandling.KanIkkeOpprettePåKjede.left()
         }
 

app/src/main/kotlin/no/nav/tiltakspenger/saksbehandling/repository/meldekort/MeldeperiodePostgresRepo.kt

@@ -78,7 +78,7 @@ internal class MeldeperiodePostgresRepo(
         sessionContext: SessionContext?,
     ): MeldeperiodeKjeder {
         return sessionFactory.withSession(sessionContext) { session ->
-            hentForSakId(sakId, session)
+            hentMeldeperiodekjederForSakId(sakId, session)
         }
     }
 
@@ -154,7 +154,7 @@ internal class MeldeperiodePostgresRepo(
             )
         }
 
-        internal fun hentForSakId(
+        internal fun hentMeldeperiodekjederForSakId(
             sakId: SakId,
             session: Session,
         ): MeldeperiodeKjeder {

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

@@ -264,7 +264,7 @@ internal class SakPostgresRepo(
                 val vedtaksliste: Vedtaksliste = RammevedtakPostgresRepo.hentForSakId(id, session)
                 val meldekortBehandlinger =
                     MeldekortBehandlingPostgresRepo.hentForSakId(id, session) ?: MeldekortBehandlinger.empty()
-                val meldeperioder = MeldeperiodePostgresRepo.hentForSakId(id, session)
+                val meldeperiodekjeder = MeldeperiodePostgresRepo.hentMeldeperiodekjederForSakId(id, session)
                 val soknader = SøknadDAO.hentForSakId(id, session)
                 Sak(
                     id = id,
@@ -274,7 +274,7 @@ internal class SakPostgresRepo(
                     vedtaksliste = vedtaksliste,
                     meldekortBehandlinger = meldekortBehandlinger,
                     utbetalinger = UtbetalingsvedtakPostgresRepo.hentForSakId(id, session),
-                    meldeperiodeKjeder = meldeperioder,
+                    meldeperiodeKjeder = meldeperiodekjeder,
                     brukersMeldekort = BrukersMeldekortPostgresRepo.hentForSakId(id, session),
                     soknader = soknader,
                 ).also { sak ->

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

@@ -25,7 +25,7 @@ fun Sak.toMeldeperiodeKjedeDTO(kjedeId: MeldeperiodeKjedeId, clock: Clock): Meld
 }
 
 fun Sak.toMeldeperiodeKjederDTO(clock: Clock): List<MeldeperiodeKjedeDTO> {
-    return this.meldeperiodeKjeder.map { meldeperiodeKjede ->
+    return this.meldeperiodeKjeder.sortedByDescending { it.periode.fraOgMed }.map { meldeperiodeKjede ->
         MeldeperiodeKjedeDTO(
             kjedeId = meldeperiodeKjede.kjedeId.toString(),
             periode = meldeperiodeKjede.periode.toDTO(),