♻️ Sletter notat_id kolonne fra periodehistorikk

* og rydder opp Co-authored-by: Øydis Kind Refsum <[email protected]>
navikt · Nov 20, 2024 · 4fc8eaa · 4fc8eaa
1 parent 1ef9831
commit 4fc8eaa
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 50 deletions.
diff --git a/...pi/src/main/kotlin/no/nav/helse/spesialist/api/periodehistorikk/PeriodehistorikkApiDao.kt b/...pi/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/...es/src/main/resources/db/migration/V556__fjerne_notat_id_kolonne_fra_periodehistorikk.sql b/...es/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/...ialist-modell/src/main/kotlin/no/nav/helse/modell/periodehistorikk/HistorikkinnslagDto.kt b/...ialist-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,28 +90,24 @@ data class FjernetFraPåVent(
     override val saksbehandler: SaksbehandlerDto,
     override val tidspunkt: LocalDateTime,
 ) : HistorikkinnslagDto {
-    override val notat: NotatDto? = null
     override val dialogRef: Long? = null
 }
 
 data class TotrinnsvurderingFerdigbehandlet(
     override val saksbehandler: SaksbehandlerDto,
     override val tidspunkt: LocalDateTime,
 ) : HistorikkinnslagDto {
-    override val notat: NotatDto? = null
     override val dialogRef: Long? = null
 }
 
 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,23 +122,20 @@ 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
 }
 
 data class AutomatiskBehandlingStanset(
     override val tidspunkt: LocalDateTime,
 ) : HistorikkinnslagDto {
-    override val notat: NotatDto? = null
     override val dialogRef: Long? = null
     override val saksbehandler: SaksbehandlerDto? = null
 }
 
 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
@@ -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
@@ -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/...t-selve/src/main/kotlin/no/nav/helse/modell/totrinnsvurdering/TotrinnsvurderingService.kt b/...t-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,
             )