🔊 forbedring av logging ved leting etter overstyringer

- logger ikke hvis verken ny eller gammel metode finner overstyringer Co-authored-by: Jakob Havstein Eriksen <[email protected]>
navikt · Jun 24, 2024 · 5e3cf3b · 5e3cf3b
1 parent 6185c2f
commit 5e3cf3b
Showing 1 changed file with 20 additions and 16 deletions.
diff --git a/...list-selve/src/main/kotlin/no/nav/helse/modell/kommando/VurderBehovForTotrinnskontroll.kt b/...list-selve/src/main/kotlin/no/nav/helse/modell/kommando/VurderBehovForTotrinnskontroll.kt
@@ -28,22 +28,21 @@ internal class VurderBehovForTotrinnskontroll(
         val overstyringer = finnOverstyringerMedType()
         finnOverstyringer(overstyringer)
 
-        if ((kreverTotrinnsvurdering && !vedtaksperiodeHarFerdigstiltOppgave) || overstyringer.isNotEmpty())
-            {
-                logg.info("Vedtaksperioden: $vedtaksperiodeId trenger totrinnsvurdering")
-                val totrinnsvurdering = totrinnsvurderingMediator.opprett(vedtaksperiodeId)
+        if ((kreverTotrinnsvurdering && !vedtaksperiodeHarFerdigstiltOppgave) || overstyringer.isNotEmpty()) {
+            logg.info("Vedtaksperioden: $vedtaksperiodeId trenger totrinnsvurdering")
+            val totrinnsvurdering = totrinnsvurderingMediator.opprett(vedtaksperiodeId)
 
-                if (totrinnsvurdering.erBeslutteroppgave()) {
-                    totrinnsvurderingMediator.settAutomatiskRetur(vedtaksperiodeId)
-                }
-                val behandlendeSaksbehandlerOid = totrinnsvurdering.saksbehandler
-                if (behandlendeSaksbehandlerOid != null) {
-                    oppgaveService.reserverOppgave(
-                        saksbehandleroid = behandlendeSaksbehandlerOid,
-                        fødselsnummer = fødselsnummer,
-                    )
-                }
+            if (totrinnsvurdering.erBeslutteroppgave()) {
+                totrinnsvurderingMediator.settAutomatiskRetur(vedtaksperiodeId)
             }
+            val behandlendeSaksbehandlerOid = totrinnsvurdering.saksbehandler
+            if (behandlendeSaksbehandlerOid != null) {
+                oppgaveService.reserverOppgave(
+                    saksbehandleroid = behandlendeSaksbehandlerOid,
+                    fødselsnummer = fødselsnummer,
+                )
+            }
+        }
 
         return true
     }
@@ -52,6 +51,8 @@ internal class VurderBehovForTotrinnskontroll(
         val spleisVedtaksperiodeIder = spleisVedtaksperioder.map { it.vedtaksperiodeId }
         val vedtaksperiodeOverstyringtyper = overstyringDao.finnOverstyringerMedTypeForVedtaksperioder(spleisVedtaksperiodeIder)
 
+        if (overstyringer.isEmpty() && vedtaksperiodeOverstyringtyper.isEmpty()) return vedtaksperiodeOverstyringtyper
+
         if (overstyringer.toSet() == vedtaksperiodeOverstyringtyper.toSet()) {
             logg.info("Finne overstyringer. Gammel og ny metode fant samme overstyring(er)")
         } else {
@@ -63,8 +64,11 @@ internal class VurderBehovForTotrinnskontroll(
     // Overstyringer og Revurderinger
     private fun finnOverstyringerMedType(): List<OverstyringType> {
         val vedtaksperiodeOverstyringtyper = overstyringDao.finnOverstyringerMedTypeForVedtaksperiode(vedtaksperiodeId)
-
-        logg.info("Vedtaksperioden: $vedtaksperiodeId har blitt overstyrt eller revurdert med typer: $vedtaksperiodeOverstyringtyper")
+        if (vedtaksperiodeOverstyringtyper.isNotEmpty()) {
+            logg.info(
+                "Vedtaksperioden: $vedtaksperiodeId har blitt overstyrt eller revurdert med typer: $vedtaksperiodeOverstyringtyper",
+            )
+        }
 
         return vedtaksperiodeOverstyringtyper
     }