Merge pull request #1241 from navikt/deprecate_skal_motta_kopi

Deprecate "Skal motta kopi", remove traces.
navikt · Oct 10, 2024 · 803d1c9 · 803d1c9
2 parents 6f898db + 74559bb
commit 803d1c9
Show file tree

Hide file tree

Showing 18 changed files with 48 additions and 44 deletions.
diff --git a/src/main/kotlin/no/nav/klage/oppgave/api/view/OversendtKlageCommon.kt b/src/main/kotlin/no/nav/klage/oppgave/api/view/OversendtKlageCommon.kt
@@ -11,14 +11,15 @@ data class OversendtSakenGjelder(
     )
     val id: OversendtPartId,
     @Schema(
-        required = true,
-        example = "true"
+        description = "Ikke lenger i bruk",
+        required = false,
+        example = "true",
+        deprecated = true,
     )
     val skalMottaKopi: Boolean
 ) {
     fun toSakenGjelder() = SakenGjelder(
         partId = id.toPartId(),
-        skalMottaKopi = skalMottaKopi
     )
 }
 
@@ -46,14 +47,15 @@ data class OversendtProsessfullmektig(
     )
     val id: OversendtPartId,
     @Schema(
-        required = true,
-        example = "true"
+        description = "Ikke lenger i bruk",
+        required = false,
+        example = "true",
+        deprecated = true,
     )
     val skalKlagerMottaKopi: Boolean
 ) {
     fun toProsessfullmektig() = Prosessfullmektig(
         partId = id.toPartId(),
-        skalPartenMottaKopi = skalKlagerMottaKopi
     )
 }
 

diff --git a/src/main/kotlin/no/nav/klage/oppgave/domain/klage/BehandlingSetters.kt b/src/main/kotlin/no/nav/klage/oppgave/domain/klage/BehandlingSetters.kt
@@ -482,7 +482,9 @@ object BehandlingSetters {
         if (nyVerdi == null) {
             klager.prosessfullmektig = null
         } else {
-            klager.prosessfullmektig = Prosessfullmektig(partId = nyVerdi, skalPartenMottaKopi = false)
+            klager.prosessfullmektig = Prosessfullmektig(
+                partId = nyVerdi,
+            )
         }
         modified = tidspunkt
 

diff --git a/src/main/kotlin/no/nav/klage/oppgave/domain/klage/Klager.kt b/src/main/kotlin/no/nav/klage/oppgave/domain/klage/Klager.kt
@@ -17,14 +17,11 @@ data class Klager(
         value = [
             AttributeOverride(name = "partId.type", column = Column(name = "klager_prosessfullmektig_type")),
             AttributeOverride(name = "partId.value", column = Column(name = "klager_prosessfullmektig_value")),
-            AttributeOverride(name = "skalPartenMottaKopi", column = Column(name = "klager_skal_motta_kopi"))
-
         ]
     )
     var prosessfullmektig: Prosessfullmektig? = null
 ) {
     fun toSakenGjelder() = SakenGjelder(
         partId = this.partId.copy(),
-        skalMottaKopi = false // Siden denne nå peker på samme som klager trenger ikke brev sendes
     )
 }
diff --git a/src/main/kotlin/no/nav/klage/oppgave/domain/klage/Prosessfullmektig.kt b/src/main/kotlin/no/nav/klage/oppgave/domain/klage/Prosessfullmektig.kt
@@ -12,6 +12,4 @@ data class Prosessfullmektig(
         ]
     )
     val partId: PartId,
-    @Column(name = "skal_parten_motta_kopi")
-    val skalPartenMottaKopi: Boolean
 )
diff --git a/src/main/kotlin/no/nav/klage/oppgave/domain/klage/SakenGjelder.kt b/src/main/kotlin/no/nav/klage/oppgave/domain/klage/SakenGjelder.kt
@@ -13,8 +13,6 @@ data class SakenGjelder(
         ]
     )
     val partId: PartId,
-    @Column(name = "saken_gjelder_skal_motta_kopi")
-    val skalMottaKopi: Boolean
 ) {
     fun erPerson() = partId.type == PartIdType.PERSON
 

diff --git a/src/main/kotlin/no/nav/klage/oppgave/service/MottakService.kt b/src/main/kotlin/no/nav/klage/oppgave/service/MottakService.kt
@@ -540,7 +540,6 @@ class MottakService(
                     type = PartIdType.of(input.fullmektig.type.name),
                     value = input.fullmektig.value
                 ),
-                skalPartenMottaKopi = true
             )
         } else {
             null
@@ -605,7 +604,6 @@ class MottakService(
         val prosessfullmektig = if (fullmektig != null) {
             Prosessfullmektig(
                 partId = fullmektig.toPartId(),
-                skalPartenMottaKopi = true
             )
         } else {
             null
@@ -628,8 +626,6 @@ class MottakService(
             klager = klager,
             sakenGjelder = SakenGjelder(
                 partId = sakenGjelder.toPartId(),
-                //TODO ever used?
-                skalMottaKopi = false
             ),
             innsynUrl = null,
             fagsystem = Fagsystem.of(fagsystemId),
@@ -659,7 +655,6 @@ class MottakService(
         val prosessfullmektig = if (fullmektig != null) {
             Prosessfullmektig(
                 partId = fullmektig.toPartId(),
-                skalPartenMottaKopi = true
             )
         } else {
             null
@@ -682,8 +677,6 @@ class MottakService(
             klager = klager,
             sakenGjelder = SakenGjelder(
                 partId = sakenGjelder.toPartId(),
-                //TODO ever used?
-                skalMottaKopi = false
             ),
             innsynUrl = null,
             fagsystem = Fagsystem.of(fagsystemId),

diff --git a/src/main/resources/db/migration/V152__remove_skal_parten_motta_kopi.sql b/src/main/resources/db/migration/V152__remove_skal_parten_motta_kopi.sql
@@ -0,0 +1,12 @@
+ALTER TABLE klage.mottak
+    DROP COLUMN saken_gjelder_skal_motta_kopi;
+
+ALTER TABLE klage.mottak
+    DROP COLUMN klager_skal_motta_kopi;
+
+ALTER TABLE klage.behandling
+    DROP COLUMN saken_gjelder_skal_motta_kopi;
+
+ALTER TABLE klage.behandling
+    DROP COLUMN klager_skal_motta_kopi;
+
diff --git a/src/test/kotlin/no/nav/klage/oppgave/api/BehandlingMedunderskriverControllerTest.kt b/src/test/kotlin/no/nav/klage/oppgave/api/BehandlingMedunderskriverControllerTest.kt
@@ -47,7 +47,6 @@ class BehandlingMedunderskriverControllerTest {
         klager = Klager(partId = PartId(type = PartIdType.PERSON, value = "23452354")),
         sakenGjelder = SakenGjelder(
             partId = PartId(type = PartIdType.PERSON, value = "23452354"),
-            skalMottaKopi = false
         ),
         ytelse = Ytelse.OMS_OMP,
         type = Type.KLAGE,

diff --git a/src/test/kotlin/no/nav/klage/oppgave/api/mapper/BehandlingMapperTest.kt b/src/test/kotlin/no/nav/klage/oppgave/api/mapper/BehandlingMapperTest.kt
@@ -91,7 +91,9 @@ class BehandlingMapperTest {
             fagsakId = "123",
             kildeReferanse = "abc",
             klager = Klager(PartId(PartIdType.PERSON, FNR)),
-            sakenGjelder = SakenGjelder(PartId(PartIdType.PERSON, FNR), false),
+            sakenGjelder = SakenGjelder(
+                partId = PartId(PartIdType.PERSON, FNR),
+            ),
             mottakId = UUID.randomUUID(),
             mottattKlageinstans = LocalDateTime.now(),
             ytelse = Ytelse.OMS_OMP,

diff --git a/src/test/kotlin/no/nav/klage/oppgave/domain/klage/KlagebehandlingTest.kt b/src/test/kotlin/no/nav/klage/oppgave/domain/klage/KlagebehandlingTest.kt
@@ -18,7 +18,9 @@ internal class KlagebehandlingTest {
             fagsakId = "123",
             kildeReferanse = "abc",
             klager = Klager(PartId(PartIdType.PERSON, fnr)),
-            sakenGjelder = SakenGjelder(PartId(PartIdType.PERSON, fnr), false),
+            sakenGjelder = SakenGjelder(
+                partId = PartId(PartIdType.PERSON, fnr),
+            ),
             mottakId = UUID.randomUUID(),
             mottattKlageinstans = LocalDateTime.now(),
             ytelse = Ytelse.OMS_OMP,

diff --git a/src/test/kotlin/no/nav/klage/oppgave/eventlisteners/StatistikkTilDVHServiceTest.kt b/src/test/kotlin/no/nav/klage/oppgave/eventlisteners/StatistikkTilDVHServiceTest.kt
@@ -171,7 +171,7 @@ class StatistikkTilDVHServiceTest {
             partId = PartId(
                 type = PartIdType.PERSON,
                 value = ""
-            ), skalMottaKopi = false
+            ),
         ),
         ytelse = Ytelse.OMS_OMP,
         type = Type.KLAGE,
@@ -210,7 +210,7 @@ class StatistikkTilDVHServiceTest {
             partId = PartId(
                 type = PartIdType.PERSON,
                 value = ""
-            ), skalMottaKopi = false
+            ),
         ),
         ytelse = Ytelse.HJE_HJE,
         type = Type.KLAGE,
@@ -245,7 +245,7 @@ class StatistikkTilDVHServiceTest {
             partId = PartId(
                 type = PartIdType.PERSON,
                 value = ""
-            ), skalMottaKopi = false
+            ),
         ),
         ytelse = Ytelse.OMS_OMP,
         type = Type.ANKE,
@@ -282,7 +282,7 @@ class StatistikkTilDVHServiceTest {
             partId = PartId(
                 type = PartIdType.PERSON,
                 value = ""
-            ), skalMottaKopi = false
+            ),
         ),
         ytelse = Ytelse.OMS_OMP,
         type = Type.ANKE_I_TRYGDERETTEN,

diff --git a/src/test/kotlin/no/nav/klage/oppgave/repositories/BehandlingRepositoryTest.kt b/src/test/kotlin/no/nav/klage/oppgave/repositories/BehandlingRepositoryTest.kt
@@ -111,7 +111,7 @@ class BehandlingRepositoryTest {
             saksbehandlerident = "1", enhet = ENHET_1, tidspunkt = LocalDateTime.now()
         )
         fullfoertKlage.ferdigstilling = Ferdigstilling(
-                        avsluttetAvSaksbehandler = LocalDateTime.now(),
+            avsluttetAvSaksbehandler = LocalDateTime.now(),
             navIdent = "navIdent",
             navn = "navn",
         )
@@ -143,7 +143,6 @@ class BehandlingRepositoryTest {
         klager = Klager(partId = PartId(type = PartIdType.PERSON, value = "23452354")),
         sakenGjelder = SakenGjelder(
             partId = PartId(type = PartIdType.PERSON, value = "23452354"),
-            skalMottaKopi = false
         ),
         ytelse = Ytelse.OMS_OMP,
         type = Type.KLAGE,

diff --git a/src/test/kotlin/no/nav/klage/oppgave/repositories/KlagebehandlingRepositoryTest.kt b/src/test/kotlin/no/nav/klage/oppgave/repositories/KlagebehandlingRepositoryTest.kt
@@ -159,7 +159,6 @@ class KlagebehandlingRepositoryTest {
             klager = Klager(partId = PartId(type = PartIdType.PERSON, value = "23452354")),
             sakenGjelder = SakenGjelder(
                 partId = PartId(type = PartIdType.PERSON, value = "23452354"),
-                skalMottaKopi = false
             ),
             ytelse = Ytelse.OMS_OMP,
             type = Type.KLAGE,
@@ -211,7 +210,6 @@ class KlagebehandlingRepositoryTest {
         klager = Klager(partId = PartId(type = PartIdType.PERSON, value = "23452354")),
         sakenGjelder = SakenGjelder(
             partId = PartId(type = PartIdType.PERSON, value = "23452354"),
-            skalMottaKopi = false
         ),
         ytelse = Ytelse.OMS_OMP,
         type = Type.KLAGE,

diff --git a/src/test/kotlin/no/nav/klage/oppgave/repositories/MeldingRepositoryTest.kt b/src/test/kotlin/no/nav/klage/oppgave/repositories/MeldingRepositoryTest.kt
@@ -67,7 +67,6 @@ class MeldingRepositoryTest {
             klager = Klager(partId = PartId(type = PartIdType.PERSON, value = "23452354")),
             sakenGjelder = SakenGjelder(
                 partId = PartId(type = PartIdType.PERSON, value = "23452354"),
-                skalMottaKopi = false
             ),
             ytelse = Ytelse.OMS_OMP,
             type = Type.KLAGE,

diff --git a/src/test/kotlin/no/nav/klage/oppgave/service/BehandlingServiceTest.kt b/src/test/kotlin/no/nav/klage/oppgave/service/BehandlingServiceTest.kt
@@ -431,7 +431,6 @@ class BehandlingServiceTest {
             klager = Klager(partId = PartId(type = PartIdType.PERSON, value = "23452354")),
             sakenGjelder = SakenGjelder(
                 partId = PartId(type = PartIdType.PERSON, value = "23452354"),
-                skalMottaKopi = false
             ),
             ytelse = Ytelse.OMS_OMP,
             type = Type.KLAGE,

diff --git a/src/test/kotlin/no/nav/klage/oppgave/service/OppgaveServiceTest.kt b/src/test/kotlin/no/nav/klage/oppgave/service/OppgaveServiceTest.kt
@@ -473,7 +473,6 @@ class OppgaveServiceTest {
                     klager = Klager(partId = PartId(type = PartIdType.PERSON, value = fnr)),
                     sakenGjelder = SakenGjelder(
                         partId = PartId(type = PartIdType.PERSON, value = fnr),
-                        skalMottaKopi = false
                     ),
                     ytelse = ytelse,
                     type = type,
@@ -509,7 +508,6 @@ class OppgaveServiceTest {
                     klager = Klager(partId = PartId(type = PartIdType.PERSON, value = fnr)),
                     sakenGjelder = SakenGjelder(
                         partId = PartId(type = PartIdType.PERSON, value = fnr),
-                        skalMottaKopi = false
                     ),
                     ytelse = ytelse,
                     type = type,
@@ -545,7 +543,6 @@ class OppgaveServiceTest {
                     klager = Klager(partId = PartId(type = PartIdType.PERSON, value = fnr)),
                     sakenGjelder = SakenGjelder(
                         partId = PartId(type = PartIdType.PERSON, value = fnr),
-                        skalMottaKopi = false
                     ),
                     ytelse = ytelse,
                     type = type,
@@ -576,7 +573,6 @@ class OppgaveServiceTest {
                     klager = Klager(partId = PartId(type = PartIdType.PERSON, value = "23452354")),
                     sakenGjelder = SakenGjelder(
                         partId = PartId(type = PartIdType.PERSON, value = "23452354"),
-                        skalMottaKopi = false
                     ),
                     ytelse = ytelse,
                     type = type,

diff --git a/src/test/kotlin/no/nav/klage/oppgave/service/TilgangServiceTest.kt b/src/test/kotlin/no/nav/klage/oppgave/service/TilgangServiceTest.kt
@@ -77,11 +77,16 @@ class TilgangServiceTest {
     @Test
     fun `verifySaksbehandlersSkrivetilgang gir feil ved annen tildelt saksbehandler`() {
         val klagebehandling = getKlagebehandling()
-        klagebehandling.tildeling = Tildeling(saksbehandlerident = "Z123456", enhet = "", tidspunkt = LocalDateTime.now())
+        klagebehandling.tildeling =
+            Tildeling(saksbehandlerident = "Z123456", enhet = "", tidspunkt = LocalDateTime.now())
 
         every { innloggetSaksbehandlerService.getInnloggetIdent() }.returns("Z654321")
 
-        assertThrows<MissingTilgangException> { tilgangService.verifyInnloggetSaksbehandlersSkrivetilgang(klagebehandling) }
+        assertThrows<MissingTilgangException> {
+            tilgangService.verifyInnloggetSaksbehandlersSkrivetilgang(
+                klagebehandling
+            )
+        }
     }
 
     @Test
@@ -90,13 +95,18 @@ class TilgangServiceTest {
 
         every { innloggetSaksbehandlerService.getInnloggetIdent() }.returns("Z654321")
 
-        assertThrows<MissingTilgangException> { tilgangService.verifyInnloggetSaksbehandlersSkrivetilgang(klagebehandling) }
+        assertThrows<MissingTilgangException> {
+            tilgangService.verifyInnloggetSaksbehandlersSkrivetilgang(
+                klagebehandling
+            )
+        }
     }
 
     @Test
     fun `verifySaksbehandlersSkrivetilgang gir ok ved samme ident`() {
         val klagebehandling = getKlagebehandling()
-        klagebehandling.tildeling = Tildeling(saksbehandlerident = "Z123456", enhet = "", tidspunkt = LocalDateTime.now())
+        klagebehandling.tildeling =
+            Tildeling(saksbehandlerident = "Z123456", enhet = "", tidspunkt = LocalDateTime.now())
 
         every { innloggetSaksbehandlerService.getInnloggetIdent() }.returns("Z123456")
 
@@ -210,7 +220,6 @@ fun getKlagebehandling(): Klagebehandling = Klagebehandling(
     klager = Klager(partId = PartId(type = PartIdType.PERSON, value = "23452354")),
     sakenGjelder = SakenGjelder(
         partId = PartId(type = PartIdType.PERSON, value = "23452354"),
-        skalMottaKopi = false
     ),
     ytelse = Ytelse.OMS_OMP,
     type = Type.KLAGE,

diff --git a/src/test/kotlin/no/nav/klage/oppgave/service/distribusjon/BehandlingAvslutningServiceTest.kt b/src/test/kotlin/no/nav/klage/oppgave/service/distribusjon/BehandlingAvslutningServiceTest.kt
@@ -175,7 +175,6 @@ internal class BehandlingAvslutningServiceTest {
         klager = Klager(partId = PartId(type = PartIdType.PERSON, value = fnr)),
         sakenGjelder = SakenGjelder(
             partId = PartId(type = PartIdType.PERSON, value = fnr),
-            skalMottaKopi = false
         ),
         ytelse = Ytelse.OMS_OMP,
         type = Type.KLAGE,