Bd 1958: Maskere identitet på beskyttede husstandsmedlemmer (#509)

* bd-1958: Bønnifisere mapping av BehandlingDtoV2 * bd-1958: Refaktorisering * ktlint * Update BoforholdServiceTest.kt * ktlint * Maskere notat * ktlinting * bd-1958: Bønnifisere mapping av BehandlingDtoV2 * bd-1958: Refaktorisering * ktlint * Update BoforholdServiceTest.kt * ktlint * Maskere notat * ktlinting * Update initEnv.sh * Update BehandlingControllerV2.kt * Endre generisk tekst * Update BoforholdControllerTest.kt * Fiks feil vedtak lesemodus * Oppdater notat boforhold og andre voksne i husstanden beskyttet parameter * Oppdater notat boforhold og andre voksne i husstanden beskyttet parameter * Oppdater bidrag-felles * Oppdater bidrag-felles * Oppdater bidrag-felles --------- Co-authored-by: Ugur Alpay Cenar <[email protected]>
navikt · Oct 2, 2024 · ac1eab0 · ac1eab0
1 parent 82274e6
commit ac1eab0
Show file tree

Hide file tree

Showing 25 changed files with 1,078 additions and 906 deletions.
diff --git a/initEnv.sh b/initEnv.sh
@@ -1,2 +1,2 @@
 kubectx dev-gcp
-kubectl exec --tty deployment/bidrag-behandling-feature printenv | grep -E 'AZURE_|_URL|SCOPE|UNLEASH' | grep -v -e 'BIDRAG_FORSENDELSE_URL' -e 'BIDRAG_TILGANGSKONTROLL_URL' -e 'BIDRAG_GRUNNLAG_URL' -e 'BIDRAG_VEDTAK_SCOPE' -e 'BIDRAG_VEDTAK_URL' > src/test/resources/application-lokal-nais-secrets.properties
+kubectl exec --tty deployment/bidrag-behandling-feature printenv | grep -E 'AZURE_|_URL|SCOPE|UNLEASH' | grep -v -e 'BIDRAG_FORSENDELSE_URL' -e 'BIDRAG_TILGANGSKONTROLL_URL' -e 'BIDRAG_GRUNNLAG_URL' -e 'BIDRAG_VEDTAK_SCOPE' -e 'BIDRAG_VEDTAK_URL' -e 'BIDRAG_BBM_URL' > src/test/resources/application-lokal-nais-secrets.properties
diff --git a/pom.xml b/pom.xml
@@ -16,8 +16,8 @@
     <properties>
         <java.version>21</java.version>
         <kotlin.version>2.0.20</kotlin.version>
-        <bidrag-felles.version>2024.09.24.192807</bidrag-felles.version>
-        <bidrag-beregn-felles.version>2024.09.30.124555</bidrag-beregn-felles.version>
+        <bidrag-felles.version>2024.10.02.070120</bidrag-felles.version>
+        <bidrag-beregn-felles.version>2024.10.02.074523</bidrag-beregn-felles.version>
         <h2.version>2.3.232</h2.version>
         <logback-encoder.version>8.0</logback-encoder.version>
         <token-support.version>3.2.0</token-support.version>

diff --git a/src/main/kotlin/no/nav/bidrag/behandling/controller/v2/BehandlingControllerV2.kt b/src/main/kotlin/no/nav/bidrag/behandling/controller/v2/BehandlingControllerV2.kt
@@ -7,7 +7,6 @@ import io.swagger.v3.oas.annotations.responses.ApiResponses
 import io.swagger.v3.oas.annotations.security.SecurityRequirement
 import jakarta.validation.Valid
 import no.nav.bidrag.behandling.Ressurstype
-import no.nav.bidrag.behandling.database.datamodell.hentSisteAktiv
 import no.nav.bidrag.behandling.dto.v1.behandling.OppdaterRollerRequest
 import no.nav.bidrag.behandling.dto.v1.behandling.OppdatereVirkningstidspunkt
 import no.nav.bidrag.behandling.dto.v1.behandling.OpprettBehandlingFraVedtakRequest
@@ -26,11 +25,10 @@ import no.nav.bidrag.behandling.dto.v2.utgift.OppdatereUtgiftResponse
 import no.nav.bidrag.behandling.requestManglerDataException
 import no.nav.bidrag.behandling.service.BehandlingService
 import no.nav.bidrag.behandling.service.BoforholdService
-import no.nav.bidrag.behandling.service.GrunnlagService
 import no.nav.bidrag.behandling.service.InntektService
 import no.nav.bidrag.behandling.service.UtgiftService
 import no.nav.bidrag.behandling.service.VedtakService
-import no.nav.bidrag.behandling.transformers.behandling.tilBehandlingDtoV2
+import no.nav.bidrag.behandling.transformers.Dtomapper
 import no.nav.bidrag.commons.util.secureLogger
 import org.springframework.web.bind.annotation.DeleteMapping
 import org.springframework.web.bind.annotation.GetMapping
@@ -47,9 +45,9 @@ class BehandlingControllerV2(
     private val vedtakService: VedtakService,
     private val behandlingService: BehandlingService,
     private val boforholdService: BoforholdService,
-    private val grunnlagService: GrunnlagService,
     private val inntektService: InntektService,
     private val utgiftService: UtgiftService,
+    private val dtomapper: Dtomapper,
 ) {
     @Suppress("unused")
     @GetMapping("/behandling/vedtak/{vedtakId}")
@@ -73,7 +71,7 @@ class BehandlingControllerV2(
         val resultat =
             vedtakService.konverterVedtakTilBehandlingForLesemodus(vedtakId)
                 ?: throw RuntimeException("Fant ikke vedtak for vedtakid $vedtakId")
-        return resultat.tilBehandlingDtoV2(resultat.grunnlagListe, inkluderHistoriskeInntekter = inkluderHistoriskeInntekter)
+        return dtomapper.tilDto(resultat, inkluderHistoriskeInntekter)
     }
 
     @PutMapping("/behandling/{behandlingsid}/inntekt")
@@ -164,10 +162,7 @@ class BehandlingControllerV2(
 
         val behandling = behandlingService.oppdatereVirkningstidspunkt(behandlingsid, request)
 
-        return behandling.tilBehandlingDtoV2(
-            behandling.grunnlag.hentSisteAktiv(),
-            grunnlagService.henteNyeGrunnlagsdataMedEndringsdiff(behandling),
-        )
+        return dtomapper.tilDto(behandling)
     }
 
     @PutMapping("/behandling/{behandlingsid}/boforhold")

diff --git a/src/main/kotlin/no/nav/bidrag/behandling/dto/v2/behandling/BehandlingDtoV2.kt b/src/main/kotlin/no/nav/bidrag/behandling/dto/v2/behandling/BehandlingDtoV2.kt
@@ -294,6 +294,7 @@ data class SivilstandIkkeAktivGrunnlagDto(
 
 data class HusstandsmedlemGrunnlagDto(
     val perioder: Set<BostatusperiodeGrunnlagDto>,
+    // TODO: bd-1958
     val ident: String? = null,
     val innhentetTidspunkt: LocalDateTime,
 ) {
@@ -332,12 +333,14 @@ data class PeriodeAndreVoksneIHusstanden(
     val husstandsmedlemmer: List<AndreVoksneIHusstandenDetaljerDto> = emptyList(),
 )
 
+// TODO: bd-1958
 data class AndreVoksneIHusstandenDetaljerDto(
     val navn: String,
     val fødselsdato: LocalDate?,
     val harRelasjonTilBp: Boolean,
     @Schema(description = "Relasjon til BP. Brukes for debugging", deprecated = true)
     val relasjon: Familierelasjon,
+    val erBeskyttet: Boolean = false,
 )
 
 @Schema(enumAsRef = true, name = "OpplysningerType")

diff --git a/src/main/kotlin/no/nav/bidrag/behandling/dto/v2/boforhold/BoforholdDtoV2.kt b/src/main/kotlin/no/nav/bidrag/behandling/dto/v2/boforhold/BoforholdDtoV2.kt
@@ -44,6 +44,7 @@ data class BoforholdDtoV2(
     val notat: BegrunnelseDto = begrunnelse
 }
 
+// TODO: bd-1958
 data class HusstandsmedlemDtoV2(
     val id: Long?,
     @Schema(required = true)
@@ -55,5 +56,5 @@ data class HusstandsmedlemDtoV2(
     val navn: String? = null,
     @Schema(type = "string", format = "date", example = "2025-01-25")
     @JsonFormat(pattern = "yyyy-MM-dd")
-    val fødselsdato: LocalDate,
+    val fødselsdato: LocalDate?,
 )
diff --git a/src/main/kotlin/no/nav/bidrag/behandling/service/BehandlingService.kt b/src/main/kotlin/no/nav/bidrag/behandling/service/BehandlingService.kt
@@ -4,7 +4,6 @@ import io.github.oshai.kotlinlogging.KotlinLogging
 import no.nav.bidrag.behandling.behandlingNotFoundException
 import no.nav.bidrag.behandling.database.datamodell.Behandling
 import no.nav.bidrag.behandling.database.datamodell.Utgift
-import no.nav.bidrag.behandling.database.datamodell.hentSisteAktiv
 import no.nav.bidrag.behandling.database.datamodell.tilBehandlingstype
 import no.nav.bidrag.behandling.database.repository.BehandlingRepository
 import no.nav.bidrag.behandling.dto.v1.behandling.OppdaterRollerResponse
@@ -19,11 +18,8 @@ import no.nav.bidrag.behandling.dto.v2.behandling.AktivereGrunnlagRequestV2
 import no.nav.bidrag.behandling.dto.v2.behandling.AktivereGrunnlagResponseV2
 import no.nav.bidrag.behandling.dto.v2.behandling.BehandlingDetaljerDtoV2
 import no.nav.bidrag.behandling.dto.v2.behandling.BehandlingDtoV2
-import no.nav.bidrag.behandling.transformers.behandling.tilAktiveGrunnlagsdata
+import no.nav.bidrag.behandling.transformers.Dtomapper
 import no.nav.bidrag.behandling.transformers.behandling.tilBehandlingDetaljerDtoV2
-import no.nav.bidrag.behandling.transformers.behandling.tilBehandlingDtoV2
-import no.nav.bidrag.behandling.transformers.behandling.tilBoforholdV2
-import no.nav.bidrag.behandling.transformers.behandling.tilInntektDtoV2
 import no.nav.bidrag.behandling.transformers.tilForsendelseRolleDto
 import no.nav.bidrag.behandling.transformers.tilType
 import no.nav.bidrag.behandling.transformers.toHusstandsmedlem
@@ -58,6 +54,7 @@ class BehandlingService(
     private val tilgangskontrollService: TilgangskontrollService,
     private val grunnlagService: GrunnlagService,
     private val inntektService: InntektService,
+    private val mapper: Dtomapper,
 ) {
     @Transactional
     fun slettBehandling(behandlingId: Long) {
@@ -221,14 +218,7 @@ class BehandlingService(
                         "for person ${request.personident}: $request"
                 }
                 grunnlagService.aktivereGrunnlag(it, request)
-                val gjeldendeAktiveGrunnlagsdata = it.grunnlagListe.toSet().hentSisteAktiv()
-                val ikkeAktiverteEndringerIGrunnlagsdata = grunnlagService.henteNyeGrunnlagsdataMedEndringsdiff(it)
-                return AktivereGrunnlagResponseV2(
-                    boforhold = it.tilBoforholdV2(),
-                    inntekter = it.tilInntektDtoV2(gjeldendeAktiveGrunnlagsdata),
-                    aktiveGrunnlagsdata = gjeldendeAktiveGrunnlagsdata.tilAktiveGrunnlagsdata(),
-                    ikkeAktiverteEndringerIGrunnlagsdata = ikkeAktiverteEndringerIGrunnlagsdata,
-                )
+                return mapper.tilAktivereGrunnlagResponseV2(it)
             }
     }
 
@@ -363,14 +353,8 @@ class BehandlingService(
 
         grunnlagService.oppdatereGrunnlagForBehandling(behandling)
         behandling.oppdatereVirkningstidspunktSærbidrag()
-        val grunnlagsdataEndretEtterAktivering =
-            grunnlagService.henteNyeGrunnlagsdataMedEndringsdiff(behandling)
 
-        return behandling.tilBehandlingDtoV2(
-            behandling.grunnlagListe.toSet().hentSisteAktiv(),
-            grunnlagsdataEndretEtterAktivering,
-            inkluderHistoriskeInntekter,
-        )
+        return mapper.tilDto(behandling, true)
     }
 
     @Transactional

diff --git a/src/main/kotlin/no/nav/bidrag/behandling/service/BoforholdService.kt b/src/main/kotlin/no/nav/bidrag/behandling/service/BoforholdService.kt
@@ -35,6 +35,7 @@ import no.nav.bidrag.behandling.dto.v2.boforhold.OppdatereSivilstand
 import no.nav.bidrag.behandling.dto.v2.boforhold.Sivilstandsperiode
 import no.nav.bidrag.behandling.oppdateringAvBoforholdFeilet
 import no.nav.bidrag.behandling.oppdateringAvBoforholdFeiletException
+import no.nav.bidrag.behandling.transformers.Dtomapper
 import no.nav.bidrag.behandling.transformers.Jsonoperasjoner.Companion.jsonListeTilObjekt
 import no.nav.bidrag.behandling.transformers.Jsonoperasjoner.Companion.tilJson
 import no.nav.bidrag.behandling.transformers.boforhold.overskriveAndreVoksneIHusstandMedBearbeidaPerioder
@@ -90,6 +91,7 @@ class BoforholdService(
     private val husstandsmedlemRepository: HusstandsmedlemRepository,
     private val notatService: NotatService,
     private val sivilstandRepository: SivilstandRepository,
+    private val dtomapper: Dtomapper,
 ) {
     @Transactional
     fun oppdatereNotat(
@@ -285,7 +287,7 @@ class BoforholdService(
             if (Kilde.MANUELL == husstandsmedlemSomSkalSlettes?.kilde) {
                 behandling.husstandsmedlem.remove(husstandsmedlemSomSkalSlettes)
                 loggeEndringHusstandsmedlem(behandling, oppdatereHusstandsmedlem, husstandsmedlemSomSkalSlettes)
-                return husstandsmedlemSomSkalSlettes.tilOppdatereBoforholdResponse(behandling)
+                return dtomapper.tilOppdatereBoforholdResponse(husstandsmedlemSomSkalSlettes)
             }
         }
 
@@ -331,7 +333,7 @@ class BoforholdService(
             }
             behandling.husstandsmedlem.add(husstandsmedlem)
             loggeEndringHusstandsmedlem(behandling, oppdatereHusstandsmedlem, husstandsmedlem)
-            return husstandsmedlemRepository.save(husstandsmedlem).tilOppdatereBoforholdResponse(behandling)
+            return dtomapper.tilOppdatereBoforholdResponse(husstandsmedlemRepository.save(husstandsmedlem))
         }
 
         oppdatereHusstandsmedlem.slettPeriode?.let { idHusstansmedlemsperiode ->
@@ -342,7 +344,7 @@ class BoforholdService(
             husstandsmedlem.oppdaterePerioder(sletteHusstandsmedlemsperiode = idHusstansmedlemsperiode)
 
             loggeEndringHusstandsmedlem(behandling, oppdatereHusstandsmedlem, husstandsmedlem)
-            return husstandsmedlemRepository.save(husstandsmedlem).tilOppdatereBoforholdResponse(behandling)
+            return dtomapper.tilOppdatereBoforholdResponse(husstandsmedlemRepository.save(husstandsmedlem))
         }
 
         oppdatereHusstandsmedlem.oppdaterPeriode?.let { bostatusperiode ->
@@ -366,7 +368,9 @@ class BoforholdService(
             )
 
             loggeEndringHusstandsmedlem(behandling, oppdatereHusstandsmedlem, eksisterendeHusstandsmedlem)
-            return husstandsmedlemRepository.save(eksisterendeHusstandsmedlem).tilOppdatereBoforholdResponse(behandling)
+            return dtomapper.tilOppdatereBoforholdResponse(
+                husstandsmedlemRepository.save(eksisterendeHusstandsmedlem),
+            )
         }
 
         oppdatereHusstandsmedlem.tilbakestillPerioderForHusstandsmedlem?.let { husstandsmedlemId ->
@@ -376,7 +380,7 @@ class BoforholdService(
             husstandsmedlem.lagreEksisterendePerioder()
             husstandsmedlem.resetTilOffentligePerioder()
             loggeEndringHusstandsmedlem(behandling, oppdatereHusstandsmedlem, husstandsmedlem)
-            return husstandsmedlemRepository.save(husstandsmedlem).tilOppdatereBoforholdResponse(behandling)
+            return dtomapper.tilOppdatereBoforholdResponse(husstandsmedlemRepository.save(husstandsmedlem))
         }
 
         oppdatereHusstandsmedlem.angreSisteStegForHusstandsmedlem?.let { husstandsmedlemId ->
@@ -385,7 +389,7 @@ class BoforholdService(
                     ?: oppdateringAvBoforholdFeiletException(behandlingsid)
             husstandsmedlem.oppdaterTilForrigeLagredePerioder()
             loggeEndringHusstandsmedlem(behandling, oppdatereHusstandsmedlem, husstandsmedlem)
-            return husstandsmedlemRepository.save(husstandsmedlem).tilOppdatereBoforholdResponse(behandling)
+            return dtomapper.tilOppdatereBoforholdResponse(husstandsmedlemRepository.save(husstandsmedlem))
         }
         oppdateringAvBoforholdFeilet("Oppdatering av boforhold feilet. Forespørsel mangler informasjon om hva som skal oppdateres")
     }
@@ -419,9 +423,9 @@ class BoforholdService(
                 oppdatereAndreVoksneIHusstanden,
                 husstandsmedlemSomSkalOppdateres,
             )
-            return husstandsmedlemRepository
-                .save(husstandsmedlemSomSkalOppdateres)
-                .tilOppdatereBoforholdResponse(behandling)
+            return dtomapper.tilOppdatereBoforholdResponse(
+                husstandsmedlemRepository.save(husstandsmedlemSomSkalOppdateres),
+            )
         }
 
         oppdatereAndreVoksneIHusstanden.oppdaterePeriode?.let { oppdatereStatus ->
@@ -453,9 +457,9 @@ class BoforholdService(
                 oppdatereAndreVoksneIHusstanden,
                 husstandsmedlemSomSkalOppdateres,
             )
-            return husstandsmedlemRepository
-                .save(husstandsmedlemSomSkalOppdateres)
-                .tilOppdatereBoforholdResponse(behandling)
+            return dtomapper.tilOppdatereBoforholdResponse(
+                husstandsmedlemRepository.save(husstandsmedlemSomSkalOppdateres),
+            )
         }
 
         if (oppdatereAndreVoksneIHusstanden.angreSisteEndring) {
@@ -465,9 +469,9 @@ class BoforholdService(
                 oppdatereAndreVoksneIHusstanden,
                 husstandsmedlemSomSkalOppdateres,
             )
-            return husstandsmedlemRepository
-                .save(husstandsmedlemSomSkalOppdateres)
-                .tilOppdatereBoforholdResponse(behandling)
+            return dtomapper.tilOppdatereBoforholdResponse(
+                husstandsmedlemRepository.save(husstandsmedlemSomSkalOppdateres),
+            )
         }
 
         if (oppdatereAndreVoksneIHusstanden.tilbakestilleHistorikk) {
@@ -479,9 +483,9 @@ class BoforholdService(
                 husstandsmedlemSomSkalOppdateres,
             )
 
-            return husstandsmedlemRepository
-                .save(husstandsmedlemSomSkalOppdateres)
-                .tilOppdatereBoforholdResponse(behandling)
+            return dtomapper.tilOppdatereBoforholdResponse(
+                husstandsmedlemRepository.save(husstandsmedlemSomSkalOppdateres),
+            )
         }
 
         oppdateringAvBoforholdFeilet("Oppdatering av boforhold andre-voksne-i-husstanden feilet.")