Merge remote-tracking branch 'origin/main'

navikt · Oct 4, 2024 · d0741ed · d0741ed
2 parents 5262ada + efe3233
commit d0741ed
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 14 deletions.
diff --git a/src/main/kotlin/no/nav/klage/dokument/service/DokumentUnderArbeidService.kt b/src/main/kotlin/no/nav/klage/dokument/service/DokumentUnderArbeidService.kt
@@ -813,13 +813,15 @@ class DokumentUnderArbeidService(
 
         dokumentUnderArbeid.modified = LocalDateTime.now()
 
-        behandling.publishEndringsloggEvent(
-            saksbehandlerident = innloggetIdent,
-            felt = Felt.DOKUMENT_UNDER_ARBEID_AVSENDER_MOTTAKER,
-            fraVerdi = previousValue.toString(),
-            tilVerdi = dokumentUnderArbeid.avsenderMottakerInfoSet.toString(),
-            tidspunkt = dokumentUnderArbeid.modified,
-        )
+        if (previousValue.toString() != dokumentUnderArbeid.avsenderMottakerInfoSet.toString()) {
+            behandling.publishEndringsloggEvent(
+                saksbehandlerident = innloggetIdent,
+                felt = Felt.DOKUMENT_UNDER_ARBEID_AVSENDER_MOTTAKER,
+                fraVerdi = previousValue.toString(),
+                tilVerdi = dokumentUnderArbeid.avsenderMottakerInfoSet.toString(),
+                tidspunkt = dokumentUnderArbeid.modified,
+            )
+        }
 
         publishInternalEvent(
             data = objectMapper.writeValueAsString(
@@ -854,6 +856,7 @@ class DokumentUnderArbeidService(
         utfoerendeIdent: String,
         systemContext: Boolean,
     ): DokumentUnderArbeidAsHoveddokument {
+        //TODO: Undersøk om vi gjør dette kallet unødvendig når vi sender ut svarbrev fra mottak.
         //Validate parts
         mottakerInput.mottakerList.forEach { mottaker ->
             val part = partSearchService.searchPart(
@@ -954,13 +957,15 @@ class DokumentUnderArbeidService(
 
         dokumentUnderArbeid.modified = LocalDateTime.now()
 
-        behandling.publishEndringsloggEvent(
-            saksbehandlerident = utfoerendeIdent,
-            felt = Felt.DOKUMENT_UNDER_ARBEID_AVSENDER_MOTTAKER,
-            fraVerdi = previousValue.toString(),
-            tilVerdi = dokumentUnderArbeid.avsenderMottakerInfoSet.toString(),
-            tidspunkt = dokumentUnderArbeid.modified,
-        )
+        if (previousValue.toString() != dokumentUnderArbeid.avsenderMottakerInfoSet.toString()) {
+            behandling.publishEndringsloggEvent(
+                saksbehandlerident = utfoerendeIdent,
+                felt = Felt.DOKUMENT_UNDER_ARBEID_AVSENDER_MOTTAKER,
+                fraVerdi = previousValue.toString(),
+                tilVerdi = dokumentUnderArbeid.avsenderMottakerInfoSet.toString(),
+                tidspunkt = dokumentUnderArbeid.modified,
+            )
+        }
 
         publishInternalEvent(
             data = objectMapper.writeValueAsString(

diff --git a/src/main/kotlin/no/nav/klage/oppgave/domain/klage/Endringslogginnslag.kt b/src/main/kotlin/no/nav/klage/oppgave/domain/klage/Endringslogginnslag.kt
@@ -1,6 +1,7 @@
 package no.nav.klage.oppgave.domain.klage
 
 import jakarta.persistence.*
+import no.nav.klage.oppgave.util.getLogger
 import java.time.LocalDateTime
 import java.util.*
 
@@ -30,6 +31,8 @@ class Endringslogginnslag(
     val tidspunkt: LocalDateTime = LocalDateTime.now()
 ) {
     companion object {
+        @Suppress("JAVA_CLASS_ON_COMPANION")
+        private val logger = getLogger(javaClass.enclosingClass)
 
         fun endringslogg(
             saksbehandlerident: String?,
@@ -40,6 +43,13 @@ class Endringslogginnslag(
             tidspunkt: LocalDateTime
         ): Endringslogginnslag? {
             if ((fraVerdi == null && tilVerdi == null) || fraVerdi == tilVerdi) {
+                logger.debug(
+                    "Returning null from endringslogg. Felt: {}, fraVerdi: {}, tilVerdi: {}, behandlingId: {}",
+                    felt.name,
+                    fraVerdi,
+                    tilVerdi,
+                    behandlingId
+                )
                 return null
             } else {
                 val handling = when {