From 979b123c1b4d2db5a76b949bfc88eabf5b869ec8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Oliver=20S=C3=B8berg?= Date: Fri, 22 Nov 2024 13:40:10 +0100 Subject: [PATCH 1/2] =?UTF-8?q?Fjern=20fritekst=20lagring=20p=C3=A5=20beha?= =?UTF-8?q?ndling=20og=20flytt=20ut=20lagring=20av=20revurderingsinfo=20fr?= =?UTF-8?q?itekst=C3=A5rsak=20der=20det=20skal=20lagres=20alts=C3=A5=20ved?= =?UTF-8?q?=20manuell?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../main/kotlin/behandling/BehandlingDao.kt | 19 +++++++++---------- .../behandling/domain/OpprettBehandling.kt | 1 - .../revurdering/ManuellRevurderingService.kt | 3 +-- .../OmgjoeringKlageRevurderingService.kt | 1 - .../revurdering/RevurderingService.kt | 4 +--- .../src/test/kotlin/TestHelper.kt | 1 - .../AktivitetspliktAktivitetsgradDaoTest.kt | 1 - .../vurdering/AktivitetspliktUnntakDaoTest.kt | 1 - .../ManuellRevurderingServiceTest.kt | 1 - .../GrunnlagsendringshendelseDaoTest.kt | 1 - .../AarligInntektsjusteringJobbServiceTest.kt | 3 --- 11 files changed, 11 insertions(+), 25 deletions(-) diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingDao.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingDao.kt index f74a908e28d..2be8821f58e 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingDao.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/behandling/BehandlingDao.kt @@ -235,9 +235,9 @@ class BehandlingDao( """ INSERT INTO behandling(id, sak_id, behandling_opprettet, sist_endret, status, behandlingstype, soeknad_mottatt_dato, virkningstidspunkt, utlandstilknytning, bodd_eller_arbeidet_utlandet, - revurdering_aarsak, fritekst_aarsak, prosesstype, kilde, begrunnelse, relatert_behandling, + revurdering_aarsak, prosesstype, kilde, begrunnelse, relatert_behandling, sende_brev, opphoer_fom, tidligere_familiepleier) - VALUES(?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?) + VALUES(?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?) """.trimIndent(), ) @@ -253,14 +253,13 @@ class BehandlingDao( stmt.setJsonb(9, utlandstilknytning) stmt.setString(10, objectMapper.writeValueAsString(boddEllerArbeidetUtlandet)) stmt.setString(11, revurderingsAarsak?.name) - stmt.setString(12, fritekstAarsak) - stmt.setString(13, prosesstype.toString()) - stmt.setString(14, kilde.toString()) - stmt.setString(15, begrunnelse) - stmt.setString(16, relatertBehandlingId) - stmt.setBoolean(17, sendeBrev) - stmt.setString(18, opphoerFraOgMed?.let { fom -> objectMapper.writeValueAsString(fom) }) - stmt.setJsonb(19, tidligereFamiliepleier) + stmt.setString(12, prosesstype.toString()) + stmt.setString(13, kilde.toString()) + stmt.setString(14, begrunnelse) + stmt.setString(15, relatertBehandlingId) + stmt.setBoolean(16, sendeBrev) + stmt.setString(17, opphoerFraOgMed?.let { fom -> objectMapper.writeValueAsString(fom) }) + stmt.setJsonb(18, tidligereFamiliepleier) } checkInternFeil(stmt.executeUpdate() == 1) { "Kunne ikke opprette behandling for ${behandling.id}" diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/domain/OpprettBehandling.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/domain/OpprettBehandling.kt index 96210b39ccb..b3deca38dbc 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/domain/OpprettBehandling.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/behandling/domain/OpprettBehandling.kt @@ -24,7 +24,6 @@ data class OpprettBehandling( val utlandstilknytning: Utlandstilknytning? = null, val boddEllerArbeidetUtlandet: BoddEllerArbeidetUtlandet? = null, val revurderingsAarsak: Revurderingaarsak? = null, - val fritekstAarsak: String? = null, val prosesstype: Prosesstype = Prosesstype.MANUELL, val kilde: Vedtaksloesning, val begrunnelse: String? = null, diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/ManuellRevurderingService.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/ManuellRevurderingService.kt index 3601db1ac14..9836495dbbd 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/ManuellRevurderingService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/ManuellRevurderingService.kt @@ -127,7 +127,6 @@ class ManuellRevurderingService( utlandstilknytning = forrigeBehandling.utlandstilknytning, boddEllerArbeidetUtlandet = forrigeBehandling.boddEllerArbeidetUtlandet, begrunnelse = begrunnelse ?: triggendeOppgave?.merknad, - fritekstAarsak = fritekstAarsak, saksbehandlerIdent = saksbehandler.ident, frist = triggendeOppgave?.frist, paaGrunnAvOppgave = paaGrunnAvOppgave, @@ -136,7 +135,7 @@ class ManuellRevurderingService( .also { revurdering -> // Eller sjekke revurdering annen? if (!fritekstAarsak.isNullOrEmpty()) { - revurderingService.lagreRevurderingsaarsakFritekst( + revurderingService.lagreRevurderingsaarsakFritekstForRevurderingAnnen( fritekstAarsak, revurdering.id, saksbehandler.ident, diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/OmgjoeringKlageRevurderingService.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/OmgjoeringKlageRevurderingService.kt index 6b737e5b53b..ce97c60eb02 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/OmgjoeringKlageRevurderingService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/OmgjoeringKlageRevurderingService.kt @@ -82,7 +82,6 @@ class OmgjoeringKlageRevurderingService( utlandstilknytning = behandlingSomOmgjoeres.utlandstilknytning, boddEllerArbeidetUtlandet = behandlingSomOmgjoeres.boddEllerArbeidetUtlandet, begrunnelse = "Omgjøring på grunn av klage", - fritekstAarsak = omgjoeringsoppgave.merknad, saksbehandlerIdent = saksbehandler.ident, relatertBehandlingId = klagenViOmgjoerPaaGrunnAv.id.toString(), ).oppdater() diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/RevurderingService.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/RevurderingService.kt index 954b30e2361..3c2236d80fa 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/RevurderingService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/RevurderingService.kt @@ -155,7 +155,6 @@ class RevurderingService( utlandstilknytning: Utlandstilknytning?, boddEllerArbeidetUtlandet: BoddEllerArbeidetUtlandet?, begrunnelse: String?, - fritekstAarsak: String? = null, saksbehandlerIdent: String?, relatertBehandlingId: String? = null, frist: Tidspunkt? = null, @@ -175,7 +174,6 @@ class RevurderingService( kilde = kilde, prosesstype = prosessType, begrunnelse = begrunnelse, - fritekstAarsak = fritekstAarsak, relatertBehandlingId = relatertBehandlingId, sendeBrev = revurderingAarsak.skalSendeBrev, opphoerFraOgMed = opphoerFraOgMed, @@ -279,7 +277,7 @@ class RevurderingService( } } - fun lagreRevurderingsaarsakFritekst( + fun lagreRevurderingsaarsakFritekstForRevurderingAnnen( fritekstAarsak: String, behandlingId: UUID, saksbehandlerIdent: String, diff --git a/apps/etterlatte-behandling/src/test/kotlin/TestHelper.kt b/apps/etterlatte-behandling/src/test/kotlin/TestHelper.kt index 112f62cfc58..65d3d5dfaa5 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/TestHelper.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/TestHelper.kt @@ -228,7 +228,6 @@ fun opprettBehandling( soeknadMottattDato = soeknadMottattDato, virkningstidspunkt = virkningstidspunkt, revurderingsAarsak = revurderingAarsak, - fritekstAarsak = fritekstAarsak, prosesstype = prosesstype, kilde = kilde, sendeBrev = true, diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/aktivitetsplikt/vurdering/AktivitetspliktAktivitetsgradDaoTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/aktivitetsplikt/vurdering/AktivitetspliktAktivitetsgradDaoTest.kt index 75a53c91fb2..c1c94d25a3c 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/aktivitetsplikt/vurdering/AktivitetspliktAktivitetsgradDaoTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/aktivitetsplikt/vurdering/AktivitetspliktAktivitetsgradDaoTest.kt @@ -177,7 +177,6 @@ class AktivitetspliktAktivitetsgradDaoTest( utlandstilknytning = null, boddEllerArbeidetUtlandet = null, revurderingsAarsak = null, - fritekstAarsak = null, prosesstype = Prosesstype.MANUELL, kilde = Vedtaksloesning.GJENNY, begrunnelse = null, diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/aktivitetsplikt/vurdering/AktivitetspliktUnntakDaoTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/aktivitetsplikt/vurdering/AktivitetspliktUnntakDaoTest.kt index f74282b342c..af45a4891cc 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/aktivitetsplikt/vurdering/AktivitetspliktUnntakDaoTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/aktivitetsplikt/vurdering/AktivitetspliktUnntakDaoTest.kt @@ -157,7 +157,6 @@ class AktivitetspliktUnntakDaoTest( utlandstilknytning = null, boddEllerArbeidetUtlandet = null, revurderingsAarsak = null, - fritekstAarsak = null, prosesstype = Prosesstype.MANUELL, kilde = Vedtaksloesning.GJENNY, begrunnelse = null, diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/ManuellRevurderingServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/ManuellRevurderingServiceTest.kt index 557add8b7a8..30c3924f23e 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/ManuellRevurderingServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/ManuellRevurderingServiceTest.kt @@ -903,7 +903,6 @@ class ManuellRevurderingServiceTest : BehandlingIntegrationTest() { any(), any(), any(), - any(), opphoerFraOgMed = null, ) } diff --git a/apps/etterlatte-behandling/src/test/kotlin/grunnlagsendring/GrunnlagsendringshendelseDaoTest.kt b/apps/etterlatte-behandling/src/test/kotlin/grunnlagsendring/GrunnlagsendringshendelseDaoTest.kt index 6ed6a93b766..ac4b7b5a52a 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/grunnlagsendring/GrunnlagsendringshendelseDaoTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/grunnlagsendring/GrunnlagsendringshendelseDaoTest.kt @@ -335,7 +335,6 @@ internal class GrunnlagsendringshendelseDaoTest( soeknadMottattDato = null, virkningstidspunkt = null, revurderingsAarsak = Revurderingaarsak.SOESKENJUSTERING, - fritekstAarsak = null, prosesstype = Prosesstype.MANUELL, kilde = Vedtaksloesning.GJENNY, sendeBrev = true, diff --git a/apps/etterlatte-behandling/src/test/kotlin/inntektsjustering/AarligInntektsjusteringJobbServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/inntektsjustering/AarligInntektsjusteringJobbServiceTest.kt index cd7613e3ce7..9f226be9a30 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/inntektsjustering/AarligInntektsjusteringJobbServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/inntektsjustering/AarligInntektsjusteringJobbServiceTest.kt @@ -154,8 +154,6 @@ class AarligInntektsjusteringJobbServiceTest { any(), any(), any(), - any(), - any(), ) } returns mockk { @@ -214,7 +212,6 @@ class AarligInntektsjusteringJobbServiceTest { utlandstilknytning = any(), boddEllerArbeidetUtlandet = any(), begrunnelse = any(), - fritekstAarsak = any(), saksbehandlerIdent = any(), relatertBehandlingId = any(), frist = any(), From a132c9ad2d1919a8871e06cf7bfe66253250f9f7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Oliver=20S=C3=B8berg?= Date: Fri, 22 Nov 2024 14:41:01 +0100 Subject: [PATCH 2/2] Litt bedre sjekk --- .../revurdering/ManuellRevurderingService.kt | 12 +++++------ .../revurdering/RevurderingService.kt | 21 +++++++++++++++---- .../ManuellRevurderingServiceTest.kt | 1 + .../kotlin/behandling/Revurderingaarsak.kt | 2 ++ 4 files changed, 26 insertions(+), 10 deletions(-) diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/ManuellRevurderingService.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/ManuellRevurderingService.kt index 9836495dbbd..734d8dc827e 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/ManuellRevurderingService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/ManuellRevurderingService.kt @@ -45,7 +45,7 @@ class ManuellRevurderingService( val paaGrunnAvHendelseUuid = try { paaGrunnAvHendelseId?.let { UUID.fromString(it) } - } catch (e: Exception) { + } catch (_: Exception) { throw BadRequestException( "$aarsak har en ugyldig hendelse id for sakid" + " $sakId. " + @@ -56,7 +56,7 @@ class ManuellRevurderingService( val paaGrunnAvOppgaveUuid = try { paaGrunnAvOppgaveId?.let { UUID.fromString(it) } - } catch (e: Exception) { + } catch (_: Exception) { throw BadRequestException("Ugyldig oppgaveId $paaGrunnAvOppgaveId (sakid=$sakId).") } @@ -133,14 +133,14 @@ class ManuellRevurderingService( opphoerFraOgMed = opphoerFraOgMed, ).oppdater() .also { revurdering -> - // Eller sjekke revurdering annen? - if (!fritekstAarsak.isNullOrEmpty()) { - revurderingService.lagreRevurderingsaarsakFritekstForRevurderingAnnen( + if (!fritekstAarsak.isNullOrEmpty() && revurdering.revurderingsaarsak!!.kanLagreFritekstFeltForManuellRevurdering()) { + revurderingService.lagreRevurderingsaarsakFritekstForRevurderingAnnenMedEllerUtenBrev( fritekstAarsak, - revurdering.id, + revurdering, saksbehandler.ident, ) } + if (paaGrunnAvHendelse != null) { grunnlagsendringshendelseDao.settBehandlingIdForTattMedIRevurdering( paaGrunnAvHendelse, diff --git a/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/RevurderingService.kt b/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/RevurderingService.kt index 3c2236d80fa..94ca2290f23 100644 --- a/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/RevurderingService.kt +++ b/apps/etterlatte-behandling/src/main/kotlin/behandling/revurdering/RevurderingService.kt @@ -25,6 +25,7 @@ import no.nav.etterlatte.libs.common.behandling.TidligereFamiliepleier import no.nav.etterlatte.libs.common.behandling.Utlandstilknytning import no.nav.etterlatte.libs.common.behandling.Virkningstidspunkt import no.nav.etterlatte.libs.common.feilhaandtering.IkkeTillattException +import no.nav.etterlatte.libs.common.feilhaandtering.InternfeilException import no.nav.etterlatte.libs.common.feilhaandtering.UgyldigForespoerselException import no.nav.etterlatte.libs.common.grunnlag.Grunnlagsopplysning import no.nav.etterlatte.libs.common.oppgave.OppgaveKilde @@ -277,13 +278,19 @@ class RevurderingService( } } - fun lagreRevurderingsaarsakFritekstForRevurderingAnnen( + fun lagreRevurderingsaarsakFritekstForRevurderingAnnenMedEllerUtenBrev( fritekstAarsak: String, - behandlingId: UUID, + revurdering: Revurdering, saksbehandlerIdent: String, ) { - val revurderingInfo = RevurderingInfo.RevurderingAarsakAnnen(fritekstAarsak) - lagreRevurderingInfo(behandlingId, RevurderingInfoMedBegrunnelse(revurderingInfo, null), saksbehandlerIdent) + val revurderingInfo = + when (revurdering.revurderingsaarsak) { + Revurderingaarsak.ANNEN -> RevurderingInfo.RevurderingAarsakAnnen(fritekstAarsak) + Revurderingaarsak.ANNEN_UTEN_BREV -> RevurderingInfo.RevurderingAarsakAnnenUtenBrev(fritekstAarsak) + else -> throw FeilRevurderingAarsakForFritekstLagring(revurdering) + } + + lagreRevurderingInfo(revurdering.id, RevurderingInfoMedBegrunnelse(revurderingInfo, null), saksbehandlerIdent) } } @@ -304,3 +311,9 @@ data class RevurderingOgOppfoelging( fun sakType() = revurdering.sak.sakType } + +class FeilRevurderingAarsakForFritekstLagring( + revurdering: Revurdering, +) : InternfeilException( + detail = "Prøvde å lagre revurdering info annen/annen uten brev med årsak ${revurdering.revurderingsaarsak} id: ${revurdering.id}", + ) diff --git a/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/ManuellRevurderingServiceTest.kt b/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/ManuellRevurderingServiceTest.kt index 30c3924f23e..afc46f57398 100644 --- a/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/ManuellRevurderingServiceTest.kt +++ b/apps/etterlatte-behandling/src/test/kotlin/behandling/revurdering/ManuellRevurderingServiceTest.kt @@ -872,6 +872,7 @@ class ManuellRevurderingServiceTest : BehandlingIntegrationTest() { every { it.id } returns UUID.randomUUID() every { it.utlandstilknytning } returns null every { it.boddEllerArbeidetUtlandet } returns null + every { it.revurderingsaarsak() } returns Revurderingaarsak.REVURDERE_ETTER_OPPHOER } }, grunnlagService = mockk().also { coEvery { it.hentPersongalleri(any()) } returns mockk() }, diff --git a/libs/saksbehandling-common/src/main/kotlin/behandling/Revurderingaarsak.kt b/libs/saksbehandling-common/src/main/kotlin/behandling/Revurderingaarsak.kt index 9cff5ad1405..2b2de03a1c6 100644 --- a/libs/saksbehandling-common/src/main/kotlin/behandling/Revurderingaarsak.kt +++ b/libs/saksbehandling-common/src/main/kotlin/behandling/Revurderingaarsak.kt @@ -109,6 +109,8 @@ enum class Revurderingaarsak( val erIkkeStoetta = listOf(AARLIG_INNTEKTSJUSTERING) return kanBrukesIMiljo() && gyldigForSakType(sakType) && !erIkkeStoetta.contains(this) } + + fun kanLagreFritekstFeltForManuellRevurdering(): Boolean = this in listOf(ANNEN, ANNEN_UTEN_BREV) } enum class GcpEnv(