diff --git a/src/main/java/no/nav/veilarbaktivitet/aktivitet/AktivitetAppService.java b/src/main/java/no/nav/veilarbaktivitet/aktivitet/AktivitetAppService.java index 2601e6174..8700a7495 100644 --- 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 hentEventTypePåSamtalereferat(AktivitetData originalAktivitet, AktivitetData oppdatertAktivitet) { - var forrigeReferat = Optional.ofNullable(originalAktivitet.getMoteData()).map(it -> it.getReferat()).orElse(""); + private Optional 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 index 8d3742a9c..0636d34ef 100644 --- 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, diff --git a/src/test/kotlin/no/nav/veilarbaktivitet/aktivitet/AktivitetsplanControllerTest.kt b/src/test/kotlin/no/nav/veilarbaktivitet/aktivitet/AktivitetsplanControllerTest.kt index 042b36074..c2e8ed93c 100644 --- a/src/test/kotlin/no/nav/veilarbaktivitet/aktivitet/AktivitetsplanControllerTest.kt +++ b/src/test/kotlin/no/nav/veilarbaktivitet/aktivitet/AktivitetsplanControllerTest.kt @@ -196,7 +196,7 @@ internal class AktivitetsplanControllerTest: SpringBootTestBase() { val aktivitet = aktivitetTestService.opprettAktivitet( happyBruker, veileder, - AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.MOTE).setErReferatPublisert(false).setReferat("") + AktivitetDtoTestBuilder.nyAktivitet(AktivitetTypeDTO.MOTE).setErReferatPublisert(false).setReferat(null) ) val oppdatertAktivitet = aktivitet oppdatertAktivitet.setReferat("Et referat")