Prøv å håndtere utnulling av samtalereferat

navikt · Dec 23, 2024 · 568128c · 568128c
1 parent 8f8c866
commit 568128c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/src/main/java/no/nav/veilarbaktivitet/aktivitet/AktivitetAppService.java b/src/main/java/no/nav/veilarbaktivitet/aktivitet/AktivitetAppService.java
@@ -249,7 +249,7 @@ public AktivitetData oppdaterReferat(AktivitetData aktivitet) {
 
         var oppdatertAktivitet = aktivitetService.oppdaterReferat(originalAktivitet, aktivitet);
 
-        var maybeEventType = hentEventTypePåSamtalereferat(originalAktivitet, aktivitet);
+        var maybeEventType = hentEventType(originalAktivitet, aktivitet);
         maybeEventType.ifPresent(eventType -> {
             bigQueryClient.logEvent(oppdatertAktivitet, eventType);
             sendMeldingTilOversikten(oppdatertAktivitet, eventType);
@@ -265,11 +265,13 @@ private void sendMeldingTilOversikten(AktivitetData aktivitet, EventType eventTy
         }
     }
 
-    private Optional<EventType> hentEventTypePåSamtalereferat(AktivitetData originalAktivitet, AktivitetData oppdatertAktivitet) {
-        var forrigeReferat = Optional.ofNullable(originalAktivitet.getMoteData()).map(it -> it.getReferat()).orElse("");
+    private Optional<EventType> hentEventType(AktivitetData originalAktivitet, AktivitetData oppdatertAktivitet) {
+        var forrigeReferat = Optional.ofNullable(originalAktivitet.getMoteData()).map(it -> it.getReferat()).orElse(null);
         var nesteReferat = Optional.ofNullable(oppdatertAktivitet.getMoteData()).map(it -> it.getReferat()).orElse("");
 
-        var referatHarNåFåttInnhold = forrigeReferat.isEmpty() && !nesteReferat.isEmpty();
+        /* Hvis samtalereferatet har fått innhold men det hviskes bort så vil forreigeReferat være "",
+        * i samtalereferat er feltet referat obligatoprisk, dvs ikke sjekk mot null ikke isEmpty her */
+        var referatHarNåFåttInnhold = forrigeReferat == null && !nesteReferat.isEmpty();
         var referatHarNåBlittDeltMedBruker = !originalAktivitet.getMoteData().isReferatPublisert() && oppdatertAktivitet.getMoteData().isReferatPublisert();
 
         if (referatHarNåFåttInnhold && !referatHarNåBlittDeltMedBruker) {

diff --git a/src/main/kotlin/no/nav/veilarbaktivitet/eventsLogger/BigQueryClient.kt b/src/main/kotlin/no/nav/veilarbaktivitet/eventsLogger/BigQueryClient.kt
@@ -11,7 +11,7 @@ import java.time.ZoneOffset
 import java.time.ZonedDateTime
 
 enum class EventType {
-    SAMTALEREFERAT_OPPRETTET,
+    SAMTALEREFERAT_OPPRETTET, // TODO: Endre navn, dette er referat opprettet, ikke aktivitet med type samtalereferat
     SAMTALEREFERAT_FIKK_INNHOLD,
     SAMTALEREFERAT_DELT_MED_BRUKER,
     SAMTALEREFERAT_OPPRETTET_OG_DELT_MED_BRUKER,