diff --git a/spesialist-api/src/main/kotlin/no/nav/helse/spesialist/api/periodehistorikk/PeriodehistorikkApiDao.kt b/spesialist-api/src/main/kotlin/no/nav/helse/spesialist/api/periodehistorikk/PeriodehistorikkApiDao.kt index e9bf8c2b3..914151d45 100644 --- a/spesialist-api/src/main/kotlin/no/nav/helse/spesialist/api/periodehistorikk/PeriodehistorikkApiDao.kt +++ b/spesialist-api/src/main/kotlin/no/nav/helse/spesialist/api/periodehistorikk/PeriodehistorikkApiDao.kt @@ -19,7 +19,7 @@ class PeriodehistorikkApiDao( join person p on v.person_ref = p.id where p.fødselsnummer = :foedselsnummer ) - select id, type, timestamp, notat_id, json, ident, dialog_ref, coalesce(utvalg.utbetaling_id, ph.utbetaling_id) as utbetaling_id, utvalg.generasjon_id + select id, type, timestamp, json, ident, dialog_ref, coalesce(utvalg.utbetaling_id, ph.utbetaling_id) as utbetaling_id, utvalg.generasjon_id from generasjon_id_og_utbetaling_id utvalg left join periodehistorikk ph on utvalg.utbetaling_id = ph.utbetaling_id or utvalg.generasjon_id = ph.generasjon_id left join saksbehandler s on saksbehandler_oid = s.oid diff --git a/spesialist-felles/src/main/resources/db/migration/V556__fjerne_notat_id_kolonne_fra_periodehistorikk.sql b/spesialist-felles/src/main/resources/db/migration/V556__fjerne_notat_id_kolonne_fra_periodehistorikk.sql new file mode 100644 index 000000000..c2b2f976f --- /dev/null +++ b/spesialist-felles/src/main/resources/db/migration/V556__fjerne_notat_id_kolonne_fra_periodehistorikk.sql @@ -0,0 +1 @@ +ALTER TABLE periodehistorikk DROP COLUMN notat_id; diff --git a/spesialist-modell/src/main/kotlin/no/nav/helse/modell/periodehistorikk/HistorikkinnslagDto.kt b/spesialist-modell/src/main/kotlin/no/nav/helse/modell/periodehistorikk/HistorikkinnslagDto.kt index 42c26b356..03b0f92d6 100644 --- a/spesialist-modell/src/main/kotlin/no/nav/helse/modell/periodehistorikk/HistorikkinnslagDto.kt +++ b/spesialist-modell/src/main/kotlin/no/nav/helse/modell/periodehistorikk/HistorikkinnslagDto.kt @@ -9,7 +9,6 @@ import java.time.LocalDate import java.time.LocalDateTime sealed interface HistorikkinnslagDto { - val notat: NotatDto? val dialogRef: Long? val saksbehandler: SaksbehandlerDto? val tidspunkt: LocalDateTime @@ -47,13 +46,11 @@ sealed interface HistorikkinnslagDto { fun totrinnsvurderingRetur( notattekst: String, - notat: NotatDto, saksbehandler: SaksbehandlerDto, dialogRef: Long, ): TotrinnsvurderingRetur = TotrinnsvurderingRetur( notattekst = notattekst, - notat = notat, saksbehandler = saksbehandler, tidspunkt = LocalDateTime.now(), dialogRef = dialogRef, @@ -68,11 +65,6 @@ sealed interface HistorikkinnslagDto { } } -data class NotatDto( - val oppgaveId: Long, - val tekst: String, -) - data class LagtPåVent( override val saksbehandler: SaksbehandlerDto, override val tidspunkt: LocalDateTime, @@ -81,8 +73,6 @@ data class LagtPåVent( val notattekst: String?, val frist: LocalDate, ) : HistorikkinnslagDto { - override val notat: NotatDto? = null - override fun toJson(): String = mapOf( "årsaker" to årsaker.map { it }, @@ -100,7 +90,6 @@ data class FjernetFraPåVent( override val saksbehandler: SaksbehandlerDto, override val tidspunkt: LocalDateTime, ) : HistorikkinnslagDto { - override val notat: NotatDto? = null override val dialogRef: Long? = null } @@ -108,7 +97,6 @@ data class TotrinnsvurderingFerdigbehandlet( override val saksbehandler: SaksbehandlerDto, override val tidspunkt: LocalDateTime, ) : HistorikkinnslagDto { - override val notat: NotatDto? = null override val dialogRef: Long? = null } @@ -116,12 +104,10 @@ data class AvventerTotrinnsvurdering( override val saksbehandler: SaksbehandlerDto, override val tidspunkt: LocalDateTime, ) : HistorikkinnslagDto { - override val notat: NotatDto? = null override val dialogRef: Long? = null } data class TotrinnsvurderingRetur( - override val notat: NotatDto, override val saksbehandler: SaksbehandlerDto, override val tidspunkt: LocalDateTime, override val dialogRef: Long, @@ -136,7 +122,6 @@ data class TotrinnsvurderingRetur( data class TotrinnsvurderingAutomatiskRetur( override val tidspunkt: LocalDateTime, ) : HistorikkinnslagDto { - override val notat: NotatDto? = null override val dialogRef: Long? = null override val saksbehandler: SaksbehandlerDto? = null } @@ -144,7 +129,6 @@ data class TotrinnsvurderingAutomatiskRetur( data class AutomatiskBehandlingStanset( override val tidspunkt: LocalDateTime, ) : HistorikkinnslagDto { - override val notat: NotatDto? = null override val dialogRef: Long? = null override val saksbehandler: SaksbehandlerDto? = null } @@ -152,7 +136,6 @@ data class AutomatiskBehandlingStanset( data class VedtaksperiodeReberegnet( override val tidspunkt: LocalDateTime, ) : HistorikkinnslagDto { - override val notat: NotatDto? = null override val dialogRef: Long? = null override val saksbehandler: SaksbehandlerDto? = null } diff --git a/spesialist-opprydding-dev/src/test/resources/db/testperson/R__testperson.sql b/spesialist-opprydding-dev/src/test/resources/db/testperson/R__testperson.sql index fa618466e..c5b2847d8 100644 --- a/spesialist-opprydding-dev/src/test/resources/db/testperson/R__testperson.sql +++ b/spesialist-opprydding-dev/src/test/resources/db/testperson/R__testperson.sql @@ -136,9 +136,9 @@ VALUES (${sequence_number}, 'UTBETALT', now(), '{}'::json, ${sequence_number}, $ INSERT INTO totrinnsvurdering(id, vedtaksperiode_id, er_retur, saksbehandler, beslutter, utbetaling_id_ref, opprettet, oppdatert) VALUES (${sequence_number}, '${vedtaksperiode_id}', false, '${saksbehandler_oid}', '${saksbehandler_oid}', ${sequence_number}, now(), null); -INSERT INTO periodehistorikk(id, type, timestamp, generasjon_id, saksbehandler_oid, notat_id, dialog_ref) +INSERT INTO periodehistorikk(id, type, timestamp, generasjon_id, saksbehandler_oid, dialog_ref) VALUES (${sequence_number}, 'TOTRINNSVURDERING_RETUR', now(), '${generasjon_id}', '${saksbehandler_oid}', - ${sequence_number}, ${sequence_number}); + ${sequence_number}); INSERT INTO feilende_meldinger(id, event_name, opprettet, blob) VALUES (gen_random_uuid(), 'FEILENDE_TESTHENDELSE', now(), '{}'::json); diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/db/PgPeriodehistorikkDao.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/db/PgPeriodehistorikkDao.kt index 3b69c7467..d375a4859 100644 --- a/spesialist-selve/src/main/kotlin/no/nav/helse/db/PgPeriodehistorikkDao.kt +++ b/spesialist-selve/src/main/kotlin/no/nav/helse/db/PgPeriodehistorikkDao.kt @@ -11,7 +11,6 @@ import no.nav.helse.modell.periodehistorikk.TotrinnsvurderingAutomatiskRetur import no.nav.helse.modell.periodehistorikk.TotrinnsvurderingFerdigbehandlet import no.nav.helse.modell.periodehistorikk.TotrinnsvurderingRetur import no.nav.helse.modell.periodehistorikk.VedtaksperiodeReberegnet -import no.nav.helse.spesialist.api.graphql.schema.NotatType import java.util.UUID import javax.sql.DataSource @@ -33,45 +32,29 @@ class PgPeriodehistorikkDao( override fun lagreMedGenerasjonId( historikkinnslag: HistorikkinnslagDto, generasjonId: UUID, - ) { - when (historikkinnslag) { - is FjernetFraPåVent -> lagre(historikkinnslag, generasjonId) - is LagtPåVent -> lagre(historikkinnslag, generasjonId) - is TotrinnsvurderingFerdigbehandlet -> lagre(historikkinnslag, generasjonId) - is AvventerTotrinnsvurdering -> lagre(historikkinnslag, generasjonId) - is TotrinnsvurderingAutomatiskRetur -> lagre(historikkinnslag, generasjonId) - is TotrinnsvurderingRetur -> { - val notatId = - PgNotatDao(queryRunner) - .lagreForOppgaveId( - oppgaveId = historikkinnslag.notat.oppgaveId, - tekst = historikkinnslag.notat.tekst, - saksbehandlerOid = historikkinnslag.saksbehandler.oid, - notatType = NotatType.Retur, - dialogRef = historikkinnslag.dialogRef, - )?.toInt() - lagre(historikkinnslag, generasjonId, notatId) - } - - is AutomatiskBehandlingStanset -> lagre(historikkinnslag, generasjonId) - is VedtaksperiodeReberegnet -> lagre(historikkinnslag, generasjonId) - } + ) = when (historikkinnslag) { + is FjernetFraPåVent -> lagre(historikkinnslag, generasjonId) + is LagtPåVent -> lagre(historikkinnslag, generasjonId) + is TotrinnsvurderingFerdigbehandlet -> lagre(historikkinnslag, generasjonId) + is AvventerTotrinnsvurdering -> lagre(historikkinnslag, generasjonId) + is TotrinnsvurderingAutomatiskRetur -> lagre(historikkinnslag, generasjonId) + is TotrinnsvurderingRetur -> lagre(historikkinnslag, generasjonId) + is AutomatiskBehandlingStanset -> lagre(historikkinnslag, generasjonId) + is VedtaksperiodeReberegnet -> lagre(historikkinnslag, generasjonId) } private fun lagre( historikkinnslag: HistorikkinnslagDto, generasjonId: UUID, - notatId: Int? = null, ) { asSQL( """ - INSERT INTO periodehistorikk (type, saksbehandler_oid, generasjon_id, utbetaling_id, notat_id, dialog_ref, json) - VALUES (:type, :saksbehandler_oid, :generasjon_id, null, :notat_id, :dialog_ref, :json::json) + INSERT INTO periodehistorikk (type, saksbehandler_oid, generasjon_id, utbetaling_id, dialog_ref, json) + VALUES (:type, :saksbehandler_oid, :generasjon_id, null, :dialog_ref, :json::json) """, "type" to historikkinnslag.type(), "saksbehandler_oid" to historikkinnslag.saksbehandler?.oid, "generasjon_id" to generasjonId, - "notat_id" to notatId, "dialog_ref" to historikkinnslag.dialogRef, "json" to historikkinnslag.toJson(), ).update() diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/modell/totrinnsvurdering/TotrinnsvurderingService.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/modell/totrinnsvurdering/TotrinnsvurderingService.kt index 7ba4eacad..df2082b84 100644 --- a/spesialist-selve/src/main/kotlin/no/nav/helse/modell/totrinnsvurdering/TotrinnsvurderingService.kt +++ b/spesialist-selve/src/main/kotlin/no/nav/helse/modell/totrinnsvurdering/TotrinnsvurderingService.kt @@ -5,7 +5,6 @@ import no.nav.helse.db.OppgaveDao import no.nav.helse.db.PeriodehistorikkDao import no.nav.helse.db.TotrinnsvurderingDao import no.nav.helse.modell.periodehistorikk.HistorikkinnslagDto -import no.nav.helse.modell.periodehistorikk.NotatDto import no.nav.helse.modell.saksbehandler.SaksbehandlerDto import no.nav.helse.spesialist.api.Totrinnsvurderinghåndterer import no.nav.helse.spesialist.api.saksbehandler.SaksbehandlerFraApi @@ -41,7 +40,6 @@ class TotrinnsvurderingService( val innslag = HistorikkinnslagDto.totrinnsvurderingRetur( notattekst = notattekst, - notat = NotatDto(oppgaveId, notattekst), saksbehandler = saksbehandlerFraApi.toDto(), dialogRef = dialogRef, )