From 44981a74a809d2158c3cc865fb816a1cc4b724e2 Mon Sep 17 00:00:00 2001 From: espenwaaga <36693523+espenwaaga@users.noreply.github.com> Date: Tue, 12 Aug 2025 14:38:52 +0200 Subject: [PATCH 1/3] TFP-6008: Litt opprydding og spesifisering --- .../dokumentbestiller/DokumentBestillerTjeneste.java | 7 ++++--- .../foreldrepenger/dokumentbestiller/DokumentMalType.java | 8 ++++---- .../web/app/tjenester/brev/BrevRestTjeneste.java | 2 +- 3 files changed, 9 insertions(+), 8 deletions(-) diff --git a/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentBestillerTjeneste.java b/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentBestillerTjeneste.java index 4a2a83f5750..5063ebc427f 100644 --- a/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentBestillerTjeneste.java +++ b/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentBestillerTjeneste.java @@ -49,12 +49,13 @@ public void produserVedtaksbrev(BehandlingVedtak behandlingVedtak) { DokumentMalType journalførSom = null; // settes kun ved fritekst if (Vedtaksbrev.FRITEKST.equals(behandlingResultat.getVedtaksbrev())) { + if (dokumentBehandlingTjeneste.hentMellomlagretOverstyring(behandling.getId()).isPresent()) { + throw new IllegalStateException("Utviklerfeil: Overstyring av vedtakbrev mangler!"); + } journalførSom = endretVedtakOgKunEndringIFordeling(behandlingResultatType, behandlingResultat.getKonsekvenserForYtelsen()) ? DokumentMalType.ENDRING_UTBETALING : dokumentMal; - dokumentMal = dokumentBehandlingTjeneste.hentMellomlagretOverstyring(behandling.getId()).isPresent() - ? DokumentMalType.VEDTAKSBREV_FRITEKST_HTML - : DokumentMalType.FRITEKSTBREV; + dokumentMal = DokumentMalType.VEDTAKSBREV_FRITEKST_HTML; } bestillDokument(DokumentBestilling.builder() diff --git a/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentMalType.java b/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentMalType.java index f1ae2b5c4ae..e2d1d48c46d 100644 --- a/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentMalType.java +++ b/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentMalType.java @@ -11,8 +11,8 @@ public enum DokumentMalType implements Kodeverdi { - FRITEKSTBREV("FRITEK"), - VEDTAKSBREV_FRITEKST_HTML("FRIHTM"), + FRITEKSTBREV("FRITEK"), // Tidligere overstyring av vedtaksbrev (bare forhåndsvisning av eldre vedtak) + VEDTAKSBREV_FRITEKST_HTML("FRIHTM"), // Ny og oppdatert overstyring av vedtaksbrev, med HTML ENGANGSSTØNAD_INNVILGELSE("INNVES"), ENGANGSSTØNAD_AVSLAG("AVSLES"), FORELDREPENGER_INNVILGELSE("INVFOR"), @@ -59,7 +59,7 @@ public enum DokumentMalType implements Kodeverdi { @Deprecated KLAGE_STADFESTET("KGESTA"), @Deprecated KLAGE_HJEMSENDT("KGEHJE"); - public static final Set VEDTAK_FRITEKTBREV_TYPER = Set.of(FRITEKSTBREV, VEDTAKSBREV_FRITEKST_HTML); + public static final Set VEDTAKBREV_FRITEKT_TYPER = Set.of(FRITEKSTBREV, VEDTAKSBREV_FRITEKST_HTML); public static final Set VEDTAKSBREV = Set.of(ENGANGSSTØNAD_INNVILGELSE, ENGANGSSTØNAD_AVSLAG, FORELDREPENGER_INNVILGELSE, FORELDREPENGER_AVSLAG, FORELDREPENGER_OPPHØR, FORELDREPENGER_ANNULLERT, SVANGERSKAPSPENGER_INNVILGELSE, SVANGERSKAPSPENGER_AVSLAG, @@ -113,7 +113,7 @@ public static DokumentMalType fraKode(String kode) { } public static boolean erVedtakFritektsBrev(DokumentMalType brev) { - return VEDTAK_FRITEKTBREV_TYPER.contains(brev); + return VEDTAKBREV_FRITEKT_TYPER.contains(brev); } public static boolean erVedtaksBrev(DokumentMalType brev) { diff --git a/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/brev/BrevRestTjeneste.java b/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/brev/BrevRestTjeneste.java index 4640800d0ec..1ecebb81a7e 100644 --- a/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/brev/BrevRestTjeneste.java +++ b/web/src/main/java/no/nav/foreldrepenger/web/app/tjenester/brev/BrevRestTjeneste.java @@ -114,7 +114,7 @@ public void bestillDokument(@TilpassetAbacAttributt(supplierClass = BrevAbacData .medSaksnummer(behandling.getSaksnummer()) .medDokumentMal(bestillBrevDto.brevmalkode()) .medRevurderingÅrsak(bestillBrevDto.arsakskode()) - .medFritekst(bestillBrevDto.fritekst() != null ? bestillBrevDto.fritekst() : null) + .medFritekst(bestillBrevDto.fritekst()) .build(); if (DokumentMalType.ETTERLYS_INNTEKTSMELDING.equals(bestillBrevDto.brevmalkode())) { From 09b14fd5385cf8b8e9f41ba708c752d724a1f782 Mon Sep 17 00:00:00 2001 From: espenwaaga <36693523+espenwaaga@users.noreply.github.com> Date: Tue, 12 Aug 2025 14:39:54 +0200 Subject: [PATCH 2/3] rename variabel --- .../nav/foreldrepenger/dokumentbestiller/DokumentMalType.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentMalType.java b/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentMalType.java index e2d1d48c46d..fa1c106d852 100644 --- a/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentMalType.java +++ b/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentMalType.java @@ -59,7 +59,7 @@ public enum DokumentMalType implements Kodeverdi { @Deprecated KLAGE_STADFESTET("KGESTA"), @Deprecated KLAGE_HJEMSENDT("KGEHJE"); - public static final Set VEDTAKBREV_FRITEKT_TYPER = Set.of(FRITEKSTBREV, VEDTAKSBREV_FRITEKST_HTML); + public static final Set VEDTAKBREV_FRITEKST_TYPER = Set.of(FRITEKSTBREV, VEDTAKSBREV_FRITEKST_HTML); public static final Set VEDTAKSBREV = Set.of(ENGANGSSTØNAD_INNVILGELSE, ENGANGSSTØNAD_AVSLAG, FORELDREPENGER_INNVILGELSE, FORELDREPENGER_AVSLAG, FORELDREPENGER_OPPHØR, FORELDREPENGER_ANNULLERT, SVANGERSKAPSPENGER_INNVILGELSE, SVANGERSKAPSPENGER_AVSLAG, @@ -113,7 +113,7 @@ public static DokumentMalType fraKode(String kode) { } public static boolean erVedtakFritektsBrev(DokumentMalType brev) { - return VEDTAKBREV_FRITEKT_TYPER.contains(brev); + return VEDTAKBREV_FRITEKST_TYPER.contains(brev); } public static boolean erVedtaksBrev(DokumentMalType brev) { From 299d37fd06b876c6e8f9a816a54d2992ba576d0d Mon Sep 17 00:00:00 2001 From: espenwaaga <36693523+espenwaaga@users.noreply.github.com> Date: Wed, 17 Sep 2025 09:32:29 +0200 Subject: [PATCH 3/3] Fiks test og feil i opprydding --- .../dokumentbestiller/DokumentBestillerTjeneste.java | 2 +- .../dokumentbestiller/DokumentBestillerTjenesteTest.java | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentBestillerTjeneste.java b/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentBestillerTjeneste.java index 5063ebc427f..9f9b387b305 100644 --- a/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentBestillerTjeneste.java +++ b/domenetjenester/dokumentbestiller/src/main/java/no/nav/foreldrepenger/dokumentbestiller/DokumentBestillerTjeneste.java @@ -49,7 +49,7 @@ public void produserVedtaksbrev(BehandlingVedtak behandlingVedtak) { DokumentMalType journalførSom = null; // settes kun ved fritekst if (Vedtaksbrev.FRITEKST.equals(behandlingResultat.getVedtaksbrev())) { - if (dokumentBehandlingTjeneste.hentMellomlagretOverstyring(behandling.getId()).isPresent()) { + if (dokumentBehandlingTjeneste.hentMellomlagretOverstyring(behandling.getId()).isEmpty()) { throw new IllegalStateException("Utviklerfeil: Overstyring av vedtakbrev mangler!"); } journalførSom = endretVedtakOgKunEndringIFordeling(behandlingResultatType, behandlingResultat.getKonsekvenserForYtelsen()) diff --git a/domenetjenester/dokumentbestiller/src/test/java/no/nav/foreldrepenger/dokumentbestiller/DokumentBestillerTjenesteTest.java b/domenetjenester/dokumentbestiller/src/test/java/no/nav/foreldrepenger/dokumentbestiller/DokumentBestillerTjenesteTest.java index 758d4bf0404..0923fbe6a11 100644 --- a/domenetjenester/dokumentbestiller/src/test/java/no/nav/foreldrepenger/dokumentbestiller/DokumentBestillerTjenesteTest.java +++ b/domenetjenester/dokumentbestiller/src/test/java/no/nav/foreldrepenger/dokumentbestiller/DokumentBestillerTjenesteTest.java @@ -76,7 +76,7 @@ void skal_bestille_vedtak_brev_fritekst() { when(behandlingResultatMock.getVedtaksbrev()).thenReturn(Vedtaksbrev.FRITEKST); when(behandlingResultatMock.getKonsekvenserForYtelsen()).thenReturn(List.of(KonsekvensForYtelsen.ENDRING_I_BEREGNING)); - when(dokumentBehandlingTjeneste.hentMellomlagretOverstyring(anyLong())).thenReturn(Optional.empty()); + when(dokumentBehandlingTjeneste.hentMellomlagretOverstyring(anyLong())).thenReturn(Optional.ofNullable("Fritekst")); when(behandlingVedtakMock.getBehandlingsresultat()).thenReturn(behandlingResultatMock); when(behandlingVedtakMock.getVedtakResultatType()).thenReturn(VedtakResultatType.INNVILGET); @@ -93,7 +93,7 @@ void skal_bestille_vedtak_brev_fritekst() { var bestilling = bestillingCaptor.getValue(); assertThat(bestilling.behandlingUuid()).isEqualTo(behandling.getUuid()); - assertThat(bestilling.dokumentMal()).isEqualTo(DokumentMalType.FRITEKSTBREV); + assertThat(bestilling.dokumentMal()).isEqualTo(DokumentMalType.VEDTAKSBREV_FRITEKST_HTML); assertThat(bestilling.bestillingUuid()).isNotNull(); assertThat(bestilling.journalførSom()).isEqualTo(DokumentMalType.FORELDREPENGER_INNVILGELSE); } @@ -108,7 +108,7 @@ void skal_bestille_vedtak_brev_fritekst_endring_i_utbetaling() { when(behandlingResultatMock.getBehandlingResultatType()).thenReturn(BehandlingResultatType.FORELDREPENGER_ENDRET); when(behandlingResultatMock.getVedtaksbrev()).thenReturn(Vedtaksbrev.FRITEKST); when(behandlingResultatMock.getKonsekvenserForYtelsen()).thenReturn(List.of(KonsekvensForYtelsen.ENDRING_I_FORDELING_AV_YTELSEN)); - when(dokumentBehandlingTjeneste.hentMellomlagretOverstyring(anyLong())).thenReturn(Optional.empty()); + when(dokumentBehandlingTjeneste.hentMellomlagretOverstyring(anyLong())).thenReturn(Optional.of("Fritekst")); when(behandlingVedtakMock.getBehandlingsresultat()).thenReturn(behandlingResultatMock); when(behandlingVedtakMock.getVedtakResultatType()).thenReturn(VedtakResultatType.INNVILGET); @@ -124,7 +124,7 @@ void skal_bestille_vedtak_brev_fritekst_endring_i_utbetaling() { var bestilling = bestillingCaptor.getValue(); assertThat(bestilling.behandlingUuid()).isEqualTo(behandling.getUuid()); - assertThat(bestilling.dokumentMal()).isEqualTo(DokumentMalType.FRITEKSTBREV); + assertThat(bestilling.dokumentMal()).isEqualTo(DokumentMalType.VEDTAKSBREV_FRITEKST_HTML); assertThat(bestilling.bestillingUuid()).isNotNull(); assertThat(bestilling.journalførSom()).isEqualTo(DokumentMalType.ENDRING_UTBETALING); }