diff --git a/src/main/java/no/nav/bidrag/dokument/arkiv/controller/AvvikController.kt b/src/main/java/no/nav/bidrag/dokument/arkiv/controller/AvvikController.kt index 9aa3171a..a9ca6bf5 100644 --- a/src/main/java/no/nav/bidrag/dokument/arkiv/controller/AvvikController.kt +++ b/src/main/java/no/nav/bidrag/dokument/arkiv/controller/AvvikController.kt @@ -38,8 +38,8 @@ class AvvikController(private val avvikService: AvvikService) : BaseController() @Operation( security = [SecurityRequirement(name = "bearer-key")], summary = "Henter mulige avvik for en journalpost, id på formatet '" + - KildesystemIdenfikator.PREFIX_JOARK + - "'" + KildesystemIdenfikator.PREFIX_JOARK + + "'" ) @ApiResponses( value = [ @@ -93,8 +93,8 @@ class AvvikController(private val avvikService: AvvikService) : BaseController() @Operation( security = [SecurityRequirement(name = "bearer-key")], summary = "Behandler et avvik for en journalpost, id på formatet '" + - KildesystemIdenfikator.PREFIX_JOARK_COMPLETE + - "'" + KildesystemIdenfikator.PREFIX_JOARK_COMPLETE + + "'" ) @Transactional @ApiResponses( diff --git a/src/main/java/no/nav/bidrag/dokument/arkiv/kafka/BehandleOppgaveHendelseService.kt b/src/main/java/no/nav/bidrag/dokument/arkiv/kafka/BehandleOppgaveHendelseService.kt index 09ecfaff..a418ee08 100644 --- a/src/main/java/no/nav/bidrag/dokument/arkiv/kafka/BehandleOppgaveHendelseService.kt +++ b/src/main/java/no/nav/bidrag/dokument/arkiv/kafka/BehandleOppgaveHendelseService.kt @@ -138,9 +138,11 @@ class BehandleOppgaveHendelseService( private fun Journalpost.harReturKommetFraNavNo() = distribuertTilAdresse() == null private fun opprettKommentarSomLeggesTilOppgave(journalpost: Journalpost): String? { - SECURE_LOGGER.info("Journalpost kommet retur med følgende detaljer ${journalpost.journalpostId} ${journalpost.journalstatus} ${journalpost.distribuertTilAdresse()} ${ + SECURE_LOGGER.info( + "Journalpost kommet retur med følgende detaljer ${journalpost.journalpostId} ${journalpost.journalstatus} ${journalpost.distribuertTilAdresse()} ${ journalpost.relevanteDatoer.joinToString(",") { "${it.datotype}:${it.dato}" } - }") + }" + ) return if (journalpost.harReturKommetFraNavNo()) "Mottaker har ikke åpnet forsendelsen via www.nav.no innen 40 timer. Ingen postadresse er registrert. Vurder om mottaker har adresse forsendelsen kan sendes til." else null } diff --git a/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/Journalpost.kt b/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/Journalpost.kt index dd91af70..3ecc6329 100644 --- a/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/Journalpost.kt +++ b/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/Journalpost.kt @@ -227,9 +227,13 @@ data class Journalpost( val postnummerPoststed = postadresseSplit.getOrNull(1)?.split(" ") ?: emptyList() val postnummer = if (postnummerPoststed.size == 2) postnummerPoststed.getOrNull(0) else null val poststed = - if (postnummerPoststed.size == 1) postnummerPoststed.getOrNull(0) else postnummerPoststed.getOrNull( - 1 - ) + if (postnummerPoststed.size == 1) { + postnummerPoststed.getOrNull(0) + } else { + postnummerPoststed.getOrNull( + 1 + ) + } val adresselinje1 = when (postadresseSplit.size) { 3 -> postadresseSplit.getOrNull(2) 4 -> postadresseSplit.getOrNull(3) @@ -893,9 +897,9 @@ data class ReturDetaljerLogDO( fun toMap(): List> = beskrivelse.chunked(100).mapIndexed { index, it -> mapOf( "nokkel" to "${if (locked == true) "L" else ""}$RETUR_DETALJER_KEY${index}_${ - DateUtils.formatDate( - dato - ) + DateUtils.formatDate( + dato + ) }", "verdi" to it ) diff --git a/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/Oppgave.kt b/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/Oppgave.kt index f818786c..45f2ec03 100644 --- a/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/Oppgave.kt +++ b/src/main/kotlin/no/nav/bidrag/dokument/arkiv/dto/Oppgave.kt @@ -99,8 +99,8 @@ data class LeggTilKommentarPaaOppgave( versjon = oppgaveData.versjon, endretAvEnhetsnr = _endretAvEnhetsnr, beskrivelse = beskrivelseHeader(saksbehandlersInfo) + - "$kommentar\r\n\r\n" + - "${oppgaveData.beskrivelse}" + "$kommentar\r\n\r\n" + + "${oppgaveData.beskrivelse}" ) data class FerdigstillOppgaveRequest( @@ -155,7 +155,7 @@ data class BestillSplittingoppgaveRequest( ) { init { beskrivelse = "${beskrivelseHeader(saksbehandlerMedEnhet.hentSaksbehandlerInfo())}\n${ - bestillSplittingKommentar(beskrivSplitting) + bestillSplittingKommentar(beskrivSplitting) }" } } @@ -177,7 +177,7 @@ data class BestillReskanningOppgaveRequest( ) { init { beskrivelse = "${beskrivelseHeader(saksbehandlerMedEnhet.hentSaksbehandlerInfo())}\n${ - bestillReskanningKommentar(kommentar) + bestillReskanningKommentar(kommentar) }" } } @@ -299,7 +299,7 @@ internal fun lagVurderDokumentOppgaveBeskrivelse( regDato: LocalDate ): String { var description = "--- ${ - LocalDate.now().format(NORSK_DATO_FORMAT) + LocalDate.now().format(NORSK_DATO_FORMAT) } ${saksbehandlerMedEnhet.hentSaksbehandlerInfo()} ---\n $brevKode $dokumentTittel" if (kommentar != null) { description += "\n\n $kommentar" diff --git a/src/test/java/no/nav/bidrag/dokument/arkiv/controller/DistribuerControllerTest.kt b/src/test/java/no/nav/bidrag/dokument/arkiv/controller/DistribuerControllerTest.kt index 9551ac83..37171789 100644 --- a/src/test/java/no/nav/bidrag/dokument/arkiv/controller/DistribuerControllerTest.kt +++ b/src/test/java/no/nav/bidrag/dokument/arkiv/controller/DistribuerControllerTest.kt @@ -162,11 +162,11 @@ internal class DistribuerControllerTest : AbstractControllerTest() { stubs.verifyStub.dokarkivOppdaterKalt( JOURNALPOST_ID, "{\"tilleggsopplysninger\":[" + - "{\"nokkel\":\"dokdistBestillingsId\",\"verdi\":\"asdsadasdsadasdasd\"}," + - "{\"nokkel\":\"journalfortAvIdent\",\"verdi\":\"Z99999\"}," + - "{\"nokkel\":\"distAdresse0\",\"verdi\":\"{\\\"adresselinje1\\\":\\\"Adresselinje1\\\",\\\"adresselinje2\\\":\\\"Adresselinje2\\\",\\\"adresselinje3\\\":\\\"Adresselinje3\\\",\\\"la\"}," + - "{\"nokkel\":\"distAdresse1\",\"verdi\":\"nd\\\":\\\"NO\\\",\\\"postnummer\\\":\\\"3000\\\",\\\"poststed\\\":\\\"Ingen\\\"}\"}," + - "{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"},{\"nokkel\":\"distribuertAvIdent\",\"verdi\":\"aud-localhost\"}],\"dokumenter\":[]}" + "{\"nokkel\":\"dokdistBestillingsId\",\"verdi\":\"asdsadasdsadasdasd\"}," + + "{\"nokkel\":\"journalfortAvIdent\",\"verdi\":\"Z99999\"}," + + "{\"nokkel\":\"distAdresse0\",\"verdi\":\"{\\\"adresselinje1\\\":\\\"Adresselinje1\\\",\\\"adresselinje2\\\":\\\"Adresselinje2\\\",\\\"adresselinje3\\\":\\\"Adresselinje3\\\",\\\"la\"}," + + "{\"nokkel\":\"distAdresse1\",\"verdi\":\"nd\\\":\\\"NO\\\",\\\"postnummer\\\":\\\"3000\\\",\\\"poststed\\\":\\\"Ingen\\\"}\"}," + + "{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"},{\"nokkel\":\"distribuertAvIdent\",\"verdi\":\"aud-localhost\"}],\"dokumenter\":[]}" ) } } @@ -258,10 +258,10 @@ internal class DistribuerControllerTest : AbstractControllerTest() { stubs.verifyStub.dokarkivOppdaterKalt( JOURNALPOST_ID, "{\"tilleggsopplysninger\":[" + - "{\"nokkel\":\"dokdistBestillingsId\",\"verdi\":\"asdsadasdsadasdasd\"}," + - "{\"nokkel\":\"journalfortAvIdent\",\"verdi\":\"Z99999\"},{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"}," + - "{\"nokkel\":\"origDistDigitalt\",\"verdi\":\"true\"}," + - "{\"nokkel\":\"distribuertAvIdent\",\"verdi\":\"aud-localhost\"}],\"dokumenter\":[]}" + "{\"nokkel\":\"dokdistBestillingsId\",\"verdi\":\"asdsadasdsadasdasd\"}," + + "{\"nokkel\":\"journalfortAvIdent\",\"verdi\":\"Z99999\"},{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"}," + + "{\"nokkel\":\"origDistDigitalt\",\"verdi\":\"true\"}," + + "{\"nokkel\":\"distribuertAvIdent\",\"verdi\":\"aud-localhost\"}],\"dokumenter\":[]}" ) } } @@ -351,11 +351,11 @@ internal class DistribuerControllerTest : AbstractControllerTest() { stubs.verifyStub.dokarkivOppdaterKalt( JOURNALPOST_ID, "{\"tilleggsopplysninger\":[" + - "{\"nokkel\":\"dokdistBestillingsId\",\"verdi\":\"asdsadasdsadasdasd\"}," + - "{\"nokkel\":\"journalfortAvIdent\",\"verdi\":\"Z99999\"}," + - "{\"nokkel\":\"distAdresse0\",\"verdi\":\"{\\\"adresselinje1\\\":\\\"Adresselinje1\\\",\\\"adresselinje2\\\":\\\"Adresselinje2\\\",\\\"adresselinje3\\\":\\\"Adresselinje3\\\",\\\"la\"}," + - "{\"nokkel\":\"distAdresse1\",\"verdi\":\"nd\\\":\\\"NO\\\",\\\"postnummer\\\":\\\"3000\\\",\\\"poststed\\\":\\\"Ingen\\\"}\"}," + - "{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"},{\"nokkel\":\"distribuertAvIdent\",\"verdi\":\"aud-localhost\"}],\"dokumenter\":[]}" + "{\"nokkel\":\"dokdistBestillingsId\",\"verdi\":\"asdsadasdsadasdasd\"}," + + "{\"nokkel\":\"journalfortAvIdent\",\"verdi\":\"Z99999\"}," + + "{\"nokkel\":\"distAdresse0\",\"verdi\":\"{\\\"adresselinje1\\\":\\\"Adresselinje1\\\",\\\"adresselinje2\\\":\\\"Adresselinje2\\\",\\\"adresselinje3\\\":\\\"Adresselinje3\\\",\\\"la\"}," + + "{\"nokkel\":\"distAdresse1\",\"verdi\":\"nd\\\":\\\"NO\\\",\\\"postnummer\\\":\\\"3000\\\",\\\"poststed\\\":\\\"Ingen\\\"}\"}," + + "{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"},{\"nokkel\":\"distribuertAvIdent\",\"verdi\":\"aud-localhost\"}],\"dokumenter\":[]}" ) } } diff --git a/src/test/java/no/nav/bidrag/dokument/arkiv/hendelser/OppgaveHendelseListenerTest.kt b/src/test/java/no/nav/bidrag/dokument/arkiv/hendelser/OppgaveHendelseListenerTest.kt index e93b4f61..9354091c 100644 --- a/src/test/java/no/nav/bidrag/dokument/arkiv/hendelser/OppgaveHendelseListenerTest.kt +++ b/src/test/java/no/nav/bidrag/dokument/arkiv/hendelser/OppgaveHendelseListenerTest.kt @@ -6,10 +6,12 @@ import io.kotest.assertions.assertSoftly import no.nav.bidrag.dokument.arkiv.BidragDokumentArkivConfig import no.nav.bidrag.dokument.arkiv.BidragDokumentArkivTest import no.nav.bidrag.dokument.arkiv.dto.DatoType +import no.nav.bidrag.dokument.arkiv.dto.FysiskpostSendt import no.nav.bidrag.dokument.arkiv.dto.OppgaveData import no.nav.bidrag.dokument.arkiv.dto.ReturDetaljerLogDO import no.nav.bidrag.dokument.arkiv.dto.Sak import no.nav.bidrag.dokument.arkiv.dto.TilleggsOpplysninger +import no.nav.bidrag.dokument.arkiv.dto.UtsendingsInfo import no.nav.bidrag.dokument.arkiv.kafka.HendelseListener import no.nav.bidrag.dokument.arkiv.kafka.dto.OppgaveKafkaHendelse import no.nav.bidrag.dokument.arkiv.model.OppgaveStatus @@ -85,7 +87,7 @@ class OppgaveHendelseListenerTest { relevanteDatoer = listOf( DatoType("2021-08-18T13:20:33", "DATO_DOKUMENT") ) - ) + ).copy(utsendingsinfo = UtsendingsInfo(fysiskpostSendt = FysiskpostSendt("Adresselinje1\n3033 Drammen\nNO"))) safResponse.antallRetur = 1 stubs.mockSafResponseHentJournalpost(safResponse) @@ -188,7 +190,8 @@ class OppgaveHendelseListenerTest { relevanteDatoer = listOf( DatoType("2021-08-18T13:20:33", "DATO_DOKUMENT") ) - ) + ).copy(utsendingsinfo = UtsendingsInfo(fysiskpostSendt = FysiskpostSendt("Adresselinje1\n3033 Drammen\nNO"))) + safResponse.antallRetur = 1 safResponse.sak = Sak("5276661") @@ -236,7 +239,8 @@ class OppgaveHendelseListenerTest { relevanteDatoer = listOf( DatoType("2021-08-18T13:20:33", "DATO_DOKUMENT") ) - ) + ).copy(utsendingsinfo = UtsendingsInfo(fysiskpostSendt = FysiskpostSendt("Adresselinje1\n3033 Drammen\nNO"))) + safResponse.antallRetur = 1 stubs.mockSafResponseHentJournalpost(opprettUtgaendeSafResponse(), null, "NO_RETUR") @@ -365,6 +369,12 @@ class OppgaveHendelseListenerTest { "\"versjon\":22", "Mottaker har ikke åpnet forsendelsen via www.nav.no innen 40 timer. Ingen postadresse er registrert. Vurder om mottaker har adresse forsendelsen kan sendes til" ) + stubs.verifyStub.dokarkivOppdaterKalt( + journalpostId, + "\"tilleggsopplysninger\":" + + "[{\"nokkel\":\"distribusjonBestilt\",\"verdi\":\"true\"}," + + "{\"nokkel\":\"retur0_${DateUtils.formatDate(LocalDate.now())}\",\"verdi\":\"Distribusjon feilet, mottaker mangler postadresse\"}]" + ) } }