From 36a1d8bdd331c2ed80676297eeba691639d95d43 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=98yvind=20N=2E=20Wed=C3=B8e?= Date: Thu, 10 Oct 2024 10:02:13 +0200 Subject: [PATCH 1/2] Deprecate "Skal motta kopi", remove traces. --- .../oppgave/api/view/OversendtKlageCommon.kt | 14 ++++++++------ .../oppgave/domain/klage/BehandlingSetters.kt | 4 +++- .../nav/klage/oppgave/domain/klage/Klager.kt | 3 --- .../oppgave/domain/klage/Prosessfullmektig.kt | 2 -- .../oppgave/domain/klage/SakenGjelder.kt | 2 -- .../klage/oppgave/service/MottakService.kt | 7 ------- .../V152__remove_skal_parten_motta_kopi.sql | 5 +++++ ...BehandlingMedunderskriverControllerTest.kt | 1 - .../api/mapper/BehandlingMapperTest.kt | 4 +++- .../domain/klage/KlagebehandlingTest.kt | 4 +++- .../StatistikkTilDVHServiceTest.kt | 8 ++++---- .../repositories/BehandlingRepositoryTest.kt | 3 +-- .../KlagebehandlingRepositoryTest.kt | 3 +-- .../repositories/MeldingRepositoryTest.kt | 1 - .../oppgave/service/BehandlingServiceTest.kt | 1 - .../oppgave/service/OppgaveServiceTest.kt | 4 ---- .../oppgave/service/TilgangServiceTest.kt | 19 ++++++++++++++----- .../BehandlingAvslutningServiceTest.kt | 1 - 18 files changed, 42 insertions(+), 44 deletions(-) create mode 100644 src/main/resources/db/migration/V152__remove_skal_parten_motta_kopi.sql 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 index a7c3d5bc9..58baab994 100644 --- 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 index cad252969..13005df09 100644 --- 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 index 5a8396dcc..129b77e31 100644 --- 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 index 7dae300fa..1531037cc 100644 --- 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 ) \ No newline at end of file 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 index a6d8d35cc..4c7b40b53 100644 --- 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 index 30c48c4af..87f2d883d 100644 --- 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 new file mode 100644 index 000000000..dc26752db --- /dev/null +++ b/src/main/resources/db/migration/V152__remove_skal_parten_motta_kopi.sql @@ -0,0 +1,5 @@ +ALTER TABLE klage.mottak + DROP COLUMN saken_gjelder_skal_motta_kopi; + +ALTER TABLE klage.mottak + DROP COLUMN klager_skal_motta_kopi; \ No newline at end of file diff --git a/src/test/kotlin/no/nav/klage/oppgave/api/BehandlingMedunderskriverControllerTest.kt b/src/test/kotlin/no/nav/klage/oppgave/api/BehandlingMedunderskriverControllerTest.kt index 6afc4402b..2f4caa2b9 100644 --- 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 index 806639734..d0033fd1a 100644 --- 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 index ce1beb560..e55c38b48 100644 --- 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 index c72a6edf0..5e6750373 100644 --- 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 index b9e6717e4..fe34f48d7 100644 --- 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 index 6efd30dfa..cee3b97c5 100644 --- 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,7 @@ class KlagebehandlingRepositoryTest { klager = Klager(partId = PartId(type = PartIdType.PERSON, value = "23452354")), sakenGjelder = SakenGjelder( partId = PartId(type = PartIdType.PERSON, value = "23452354"), - skalMottaKopi = false +// skalMottaKopi = false ), ytelse = Ytelse.OMS_OMP, type = Type.KLAGE, @@ -211,7 +211,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 index bdeb3a48a..4e5f273ae 100644 --- 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 index 7d07bcf08..be1c07762 100644 --- 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 index e933a8e1b..799320e5a 100644 --- 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 index c79e7c866..149912ef9 100644 --- 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 { tilgangService.verifyInnloggetSaksbehandlersSkrivetilgang(klagebehandling) } + assertThrows { + tilgangService.verifyInnloggetSaksbehandlersSkrivetilgang( + klagebehandling + ) + } } @Test @@ -90,13 +95,18 @@ class TilgangServiceTest { every { innloggetSaksbehandlerService.getInnloggetIdent() }.returns("Z654321") - assertThrows { tilgangService.verifyInnloggetSaksbehandlersSkrivetilgang(klagebehandling) } + assertThrows { + 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 index 889de9ab1..1851b4e98 100644 --- 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, From 74559bba60b420b873db462dafe1e49d7247d55a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=98yvind=20N=2E=20Wed=C3=B8e?= Date: Thu, 10 Oct 2024 10:13:08 +0200 Subject: [PATCH 2/2] Missed a few points. --- .../db/migration/V152__remove_skal_parten_motta_kopi.sql | 9 ++++++++- .../repositories/KlagebehandlingRepositoryTest.kt | 1 - 2 files changed, 8 insertions(+), 2 deletions(-) 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 index dc26752db..5d27d77ae 100644 --- 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 @@ -2,4 +2,11 @@ ALTER TABLE klage.mottak DROP COLUMN saken_gjelder_skal_motta_kopi; ALTER TABLE klage.mottak - DROP COLUMN klager_skal_motta_kopi; \ No newline at end of file + 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/repositories/KlagebehandlingRepositoryTest.kt b/src/test/kotlin/no/nav/klage/oppgave/repositories/KlagebehandlingRepositoryTest.kt index cee3b97c5..ef1b29b47 100644 --- 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,