From 5e3cf3bfe0b0d6941a48a8d3fa541b8c73152c37 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=98ydis=20Kind=20Refsum?= <oydis.kind.refsum@nav.no>
Date: Mon, 24 Jun 2024 16:12:33 +0200
Subject: [PATCH] =?UTF-8?q?=F0=9F=94=8A=20forbedring=20av=20logging=20ved?=
 =?UTF-8?q?=20leting=20etter=20overstyringer?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- logger ikke hvis verken ny eller gammel metode finner overstyringer

Co-authored-by: Jakob Havstein Eriksen <jakob.havstein.eriksen@nav.no>
---
 .../VurderBehovForTotrinnskontroll.kt         | 36 ++++++++++---------
 1 file changed, 20 insertions(+), 16 deletions(-)

diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/modell/kommando/VurderBehovForTotrinnskontroll.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/modell/kommando/VurderBehovForTotrinnskontroll.kt
index 2cc266abc..23acb25cd 100644
--- a/spesialist-selve/src/main/kotlin/no/nav/helse/modell/kommando/VurderBehovForTotrinnskontroll.kt
+++ b/spesialist-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
     }