🔥 Fjern unødvendig enum - bruk typesystemet i stedet

Co-authored-by: Elias Andreassen Thøgersen <[email protected]>
navikt · Oct 22, 2024 · 7a69812 · 7a69812
1 parent 5501d18
commit 7a69812
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 15 deletions.
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 type: Innslagstype
     val notat: NotatDto?
     val saksbehandler: SaksbehandlerDto?
     val tidspunkt: LocalDateTime
@@ -24,7 +23,6 @@ sealed interface HistorikkinnslagDto {
             frist: LocalDate,
         ): LagtPåVent =
             LagtPåVent(
-                type = Innslagstype.LAGT_PA_VENT,
                 notat = notat,
                 saksbehandler = saksbehandler,
                 årsak = årsaker,
@@ -34,7 +32,6 @@ sealed interface HistorikkinnslagDto {
 
         fun fjernetFraPåVentInnslag(saksbehandler: SaksbehandlerDto): FjernetFraPåVent =
             FjernetFraPåVent(
-                type = Innslagstype.FJERNET_FRA_PA_VENT,
                 saksbehandler = saksbehandler,
                 tidspunkt = LocalDateTime.now(),
             )
@@ -47,7 +44,6 @@ data class NotatDto(
 )
 
 data class LagtPåVent(
-    override val type: Innslagstype,
     override val notat: NotatDto?,
     override val saksbehandler: SaksbehandlerDto,
     override val tidspunkt: LocalDateTime,
@@ -67,14 +63,8 @@ private val objectMapper =
         .registerModule(JavaTimeModule())
 
 data class FjernetFraPåVent(
-    override val type: Innslagstype,
     override val saksbehandler: SaksbehandlerDto,
     override val tidspunkt: LocalDateTime,
 ) : HistorikkinnslagDto {
     override val notat: NotatDto? = null
 }
-
-enum class Innslagstype {
-    LAGT_PA_VENT,
-    FJERNET_FRA_PA_VENT,
-}
diff --git a/spesialist-selve/src/main/kotlin/no/nav/helse/db/TransactionalPeriodehistorikkDao.kt b/spesialist-selve/src/main/kotlin/no/nav/helse/db/TransactionalPeriodehistorikkDao.kt
@@ -2,8 +2,9 @@ package no.nav.helse.db
 
 import kotliquery.Session
 import kotliquery.queryOf
+import no.nav.helse.modell.periodehistorikk.FjernetFraPåVent
 import no.nav.helse.modell.periodehistorikk.HistorikkinnslagDto
-import no.nav.helse.modell.periodehistorikk.Innslagstype
+import no.nav.helse.modell.periodehistorikk.LagtPåVent
 import no.nav.helse.spesialist.api.periodehistorikk.PeriodehistorikkType
 import org.intellij.lang.annotations.Language
 import java.util.UUID
@@ -30,7 +31,7 @@ class TransactionalPeriodehistorikkDao(private val session: Session) : Historikk
             queryOf(
                 statement,
                 mapOf(
-                    "type" to historikkinnslag.type.toDb(),
+                    "type" to historikkinnslag.type(),
                     "saksbehandler_oid" to historikkinnslag.saksbehandler?.oid,
                     "generasjon_id" to generasjonId,
                     "notat_id" to notatId,
@@ -40,10 +41,10 @@ class TransactionalPeriodehistorikkDao(private val session: Session) : Historikk
         )
     }
 
-    private fun Innslagstype.toDb() =
+    private fun HistorikkinnslagDto.type() =
         when (this) {
-            Innslagstype.LAGT_PA_VENT -> "LEGG_PA_VENT"
-            Innslagstype.FJERNET_FRA_PA_VENT -> "FJERN_FRA_PA_VENT" // TODO: Mangler å migrere typen i databasen
+            is LagtPåVent -> "LEGG_PA_VENT"
+            is FjernetFraPåVent -> "FJERN_FRA_PA_VENT" // TODO: Mangler å migrere typen i databasen
         }
 
     override fun lagre(