Siste finish

navikt · Dec 5, 2024 · d8d969b · d8d969b
1 parent 63ceee0
commit d8d969b
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 6 deletions.
diff --git a/src/main/java/no/nav/veilarboppfolging/service/IservService.java b/src/main/java/no/nav/veilarboppfolging/service/IservService.java
@@ -128,6 +128,7 @@ AvslutteOppfolgingResultat avslutteOppfolging(AktorId aktorId) {
             } else {
                 Fnr fnr = authService.getFnrOrThrow(aktorId);
 
+                log.info("Utgang: Oppfølging avsluttet automatisk grunnet iserv i 28 dager");
                 AvslutningStatusData avslutningStatus = oppfolgingService.avsluttOppfolging(fnr, SYSTEM_USER_NAME, "Oppfølging avsluttet automatisk grunnet iserv i 28 dager");
                 // TODO litt i tvil om denne her. Attributtet sier om du per def er under oppfølging i arena, ikke om du er under oppfølging hos oss.
                 boolean oppfolgingAvsluttet = !avslutningStatus.underOppfolging;

diff --git a/src/main/java/no/nav/veilarboppfolging/service/OppfolgingEndringService.java b/src/main/java/no/nav/veilarboppfolging/service/OppfolgingEndringService.java
@@ -46,17 +46,17 @@ public void oppdaterOppfolgingMedStatusFraArena(EndringPaaOppfoelgingsBrukerV2 b
 
         Optional<OppfolgingEntity> currentLocalOppfolging = oppfolgingsStatusRepository.hentOppfolging(aktorId);
 
-        boolean erBrukerUnderOppfolging = currentLocalOppfolging.map(OppfolgingEntity::isUnderOppfolging).orElse(false);
+        boolean erBrukerUnderOppfolgingLokalt = currentLocalOppfolging.map(OppfolgingEntity::isUnderOppfolging).orElse(false);
         boolean erUnderOppfolgingIArena = erUnderOppfolging(formidlingsgruppe, kvalifiseringsgruppe);
         boolean erInaktivIArena = erIserv(formidlingsgruppe);
-        boolean skalOppfolges = !erBrukerUnderOppfolging && erUnderOppfolgingIArena;
+        boolean skalOppfolges = !erBrukerUnderOppfolgingLokalt && erUnderOppfolgingIArena;
 
         secureLog.info(
                 "Status for automatisk oppdatering av oppfølging."
                         + " aktorId={} erUnderOppfølgingIVeilarboppfolging={}"
                         + " erUnderOppfølgingIArena={} erInaktivIArena={}"
                         + " formidlingsgruppe={} kvalifiseringsgruppe={}",
-                aktorId, erBrukerUnderOppfolging,
+                aktorId, erBrukerUnderOppfolgingLokalt,
                 erUnderOppfolgingIArena, erInaktivIArena,
                 formidlingsgruppe, kvalifiseringsgruppe
         );
@@ -78,7 +78,7 @@ public void oppdaterOppfolgingMedStatusFraArena(EndringPaaOppfoelgingsBrukerV2 b
             secureLog.info("Starter oppfølging på bruker som er under oppfølging i Arena, men ikke i veilarboppfolging. aktorId={}", aktorId);
             oppfolgingService.startOppfolgingHvisIkkeAlleredeStartet(
                     Oppfolgingsbruker.arenaSyncOppfolgingBruker(aktorId, formidlingsgruppe, kvalifiseringsgruppe));
-        } else if (erBrukerUnderOppfolging && erInaktivIArena) {
+        } else if (erBrukerUnderOppfolgingLokalt && erInaktivIArena) {
             Optional<Boolean> kanEnkeltReaktiveresLokalt = kanEnkeltReaktiveresLokalt(currentLocalOppfolging, brukerV2);
             var maybeKanEnkeltReaktiveres = arenaOppfolgingService.kanEnkeltReaktiveres(fnr);
 
@@ -109,6 +109,7 @@ public void oppdaterOppfolgingMedStatusFraArena(EndringPaaOppfoelgingsBrukerV2 b
 
                 if (skalAvsluttes) {
                     secureLog.info("Automatisk avslutting av oppfølging på bruker. aktorId={}", aktorId);
+                    log.info("Utgang: Oppfølging avsluttet automatisk pga. inaktiv bruker som ikke kan reaktiveres");
                     oppfolgingService.avsluttOppfolging(fnr, SYSTEM_USER_NAME, "Oppfølging avsluttet automatisk pga. inaktiv bruker som ikke kan reaktiveres");
                     metricsService.rapporterAutomatiskAvslutningAvOppfolging(true);
                 }

diff --git a/src/main/kotlin/no/nav/veilarboppfolging/kafka/ArbeidssøkerperiodeConsumerService.kt b/src/main/kotlin/no/nav/veilarboppfolging/kafka/ArbeidssøkerperiodeConsumerService.kt
@@ -82,7 +82,7 @@ open class ArbeidssøkerperiodeConsumerService(
         runCatching {
             val oppfolgingsbruker = arenaOppfolgingService.hentIservDatoOgFormidlingsGruppe(fnr) ?: throw IllegalStateException("Fant ikke bruker")
             if (oppfolgingsbruker.iservDato == null || oppfolgingsbruker.formidlingsGruppe == null) return@runCatching null
-            KanskjeIservBrukerMedPresisIserbDato(oppfolgingsbruker.iservDato, fnr.get(), oppfolgingsbruker.formidlingsGruppe)
+            KanskjeIservBrukerMedPresisIservDato(oppfolgingsbruker.iservDato, fnr.get(), oppfolgingsbruker.formidlingsGruppe)
         }.onSuccess { kanskjeIservBruker ->
             if (kanskjeIservBruker == null) return
             if (kanskjeIservBruker.iservFraDato.atStartOfDay(ZoneId.systemDefault()).isAfter(arbeidssøkerperiodeStartet)) {
@@ -93,7 +93,7 @@ open class ArbeidssøkerperiodeConsumerService(
     }
 }
 
-data class KanskjeIservBrukerMedPresisIserbDato(
+data class KanskjeIservBrukerMedPresisIservDato(
     val iservFraDato: LocalDate,
     val fnr: String,
     val formidlingsgruppe: Formidlingsgruppe