TFP-5763: Rydder opp etter oppgitte opptjeninger. (#2558)

Author: mrsladek

domenetjenester/iay/src/main/java/no/nav/foreldrepenger/abakus/iay/tjeneste/dto/iay/MapOppgittOpptjening.java

@@ -1,7 +1,6 @@
 package no.nav.foreldrepenger.abakus.iay.tjeneste.dto.iay;
 
 import java.time.ZoneId;
-import java.util.Collection;
 import java.util.Collections;
 import java.util.Comparator;
 import java.util.List;
@@ -20,7 +19,6 @@
 import no.nav.abakus.iaygrunnlag.oppgittopptjening.v1.OppgittFrilansDto;
 import no.nav.abakus.iaygrunnlag.oppgittopptjening.v1.OppgittFrilansoppdragDto;
 import no.nav.abakus.iaygrunnlag.oppgittopptjening.v1.OppgittOpptjeningDto;
-import no.nav.abakus.iaygrunnlag.oppgittopptjening.v1.OppgitteOpptjeningerDto;
 import no.nav.foreldrepenger.abakus.domene.iay.søknad.OppgittAnnenAktivitet;
 import no.nav.foreldrepenger.abakus.domene.iay.søknad.OppgittArbeidsforhold;
 import no.nav.foreldrepenger.abakus.domene.iay.søknad.OppgittEgenNæring;
@@ -64,10 +62,6 @@ public class MapOppgittOpptjening {
         .thenComparing(dto -> dto.getLandkode() == null ? null : dto.getLandkode().getKode(), Comparator.nullsLast(Comparator.naturalOrder()))
         .thenComparing(OppgittEgenNæringDto::getVirksomhetNavn, Comparator.nullsLast(Comparator.naturalOrder()));
 
-    public OppgitteOpptjeningerDto mapTilDto(Collection<OppgittOpptjening> oppgittOpptjeninger) {
-        return new OppgitteOpptjeningerDto().medOppgitteOpptjeninger(oppgittOpptjeninger.stream().map(this::mapTilDto).collect(Collectors.toList()));
-    }
-
     public OppgittOpptjeningDto mapTilDto(OppgittOpptjening oppgittOpptjening) {
         return new MapTilDto().map(oppgittOpptjening);
     }

kontrakt/src/main/java/no/nav/abakus/iaygrunnlag/oppgittopptjening/v1/OppgitteOpptjeningerDto.java

@@ -1,25 +1,27 @@
 package no.nav.abakus.iaygrunnlag.v1;
 
-import com.fasterxml.jackson.annotation.*;
+import java.time.LocalDateTime;
+import java.time.OffsetDateTime;
+import java.time.ZoneId;
+import java.util.Objects;
+import java.util.Optional;
+import java.util.UUID;
+
+import com.fasterxml.jackson.annotation.JsonAutoDetect;
+import com.fasterxml.jackson.annotation.JsonCreator;
+import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
+import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.annotation.JsonInclude.Include;
+import com.fasterxml.jackson.annotation.JsonProperty;
 
+import jakarta.validation.Valid;
+import jakarta.validation.constraints.NotNull;
 import no.nav.abakus.iaygrunnlag.PersonIdent;
 import no.nav.abakus.iaygrunnlag.UuidDto;
 import no.nav.abakus.iaygrunnlag.arbeidsforhold.v1.ArbeidsforholdInformasjon;
 import no.nav.abakus.iaygrunnlag.inntektsmelding.v1.InntektsmeldingerDto;
 import no.nav.abakus.iaygrunnlag.kodeverk.YtelseType;
 import no.nav.abakus.iaygrunnlag.oppgittopptjening.v1.OppgittOpptjeningDto;
-import no.nav.abakus.iaygrunnlag.oppgittopptjening.v1.OppgitteOpptjeningerDto;
-
-import jakarta.validation.Valid;
-import jakarta.validation.constraints.NotNull;
-
-import java.time.LocalDateTime;
-import java.time.OffsetDateTime;
-import java.time.ZoneId;
-import java.util.Objects;
-import java.util.Optional;
-import java.util.UUID;
 
 @JsonIgnoreProperties(ignoreUnknown = true)
 @JsonInclude(value = Include.NON_ABSENT, content = Include.NON_EMPTY)
@@ -86,13 +88,6 @@ public class InntektArbeidYtelseGrunnlagDto {
     @Valid
     private OppgittOpptjeningDto overstyrtOppgittOpptjening;
 
-    /**
-     * Variant som støtter mer enn en oppgitt opptjening. Den støtter oppgitt opptjening pr. journalpost
-     */
-    @JsonProperty(value = "oppgitteOpptjeninger")
-    @Valid
-    private OppgitteOpptjeningerDto oppgitteOpptjeninger;
-
     @JsonProperty(value = "arbeidsforholdInformasjon")
     @Valid
     private ArbeidsforholdInformasjon arbeidsforholdInformasjon;
@@ -175,10 +170,6 @@ public OppgittOpptjeningDto getOppgittOpptjening() {
     }
 
     public void setOppgittOpptjening(OppgittOpptjeningDto oppgittOpptjening) {
-        if (oppgitteOpptjeninger != null) {
-            throw new IllegalArgumentException(
-                "Skal ikke bruke både ny (oppgitt opptjening pr journalpostId) og gammel (en oppgitt opptjening) i samme sak.");
-        }
         this.oppgittOpptjening = oppgittOpptjening;
     }
 
@@ -187,21 +178,13 @@ public OppgittOpptjeningDto getOverstyrtOppgittOpptjening() {
     }
 
     public void setOverstyrtOppgittOpptjening(OppgittOpptjeningDto overstyrtOppgittOpptjening) {
-        if (oppgitteOpptjeninger != null) {
-            throw new IllegalArgumentException(
-                "Skal ikke bruke både ny (oppgitt opptjening pr journalpostId) og gammel (en oppgitt opptjening) i samme sak.");
-        }
         this.overstyrtOppgittOpptjening = overstyrtOppgittOpptjening;
     }
 
     public OppgittOpptjeningDto getGjeldendeOppgittOpptjening() {
         return Optional.ofNullable(getOverstyrtOppgittOpptjening()).orElseGet(this::getOppgittOpptjening);
     }
 
-    public OppgitteOpptjeningerDto getOppgitteOpptjeninger() {
-        return oppgitteOpptjeninger;
-    }
-
     public InntektArbeidYtelseAggregatOverstyrtDto getOverstyrt() {
         return overstyrt;
     }
@@ -246,11 +229,6 @@ public InntektArbeidYtelseGrunnlagDto medOppgittOpptjening(OppgittOpptjeningDto
         return this;
     }
 
-    public InntektArbeidYtelseGrunnlagDto medOppgittOpptjeninger(OppgitteOpptjeningerDto oppgitteOpptjeninger) {
-        setOppgittOpptjeningPrDokument(oppgitteOpptjeninger);
-        return this;
-    }
-
     public InntektArbeidYtelseGrunnlagDto medOverstyrtOppgittOpptjening(OppgittOpptjeningDto overstyrtOppgittOpptjening) {
         setOverstyrtOppgittOpptjening(overstyrtOppgittOpptjening);
         return this;
@@ -266,12 +244,4 @@ public InntektArbeidYtelseGrunnlagDto medRegister(InntektArbeidYtelseAggregatReg
         return this;
     }
 
-    public void setOppgittOpptjeningPrDokument(OppgitteOpptjeningerDto oppgitteOpptjeninger) {
-        if (oppgittOpptjening != null || overstyrtOppgittOpptjening != null) {
-            throw new IllegalArgumentException(
-                "Skal ikke bruke både ny (oppgitt opptjening pr journalpostId) og gammel (en oppgitt opptjening) i samme sak.");
-        }
-        this.oppgitteOpptjeninger = oppgitteOpptjeninger;
-    }
-
 }

kontrakt/src/main/java/no/nav/abakus/iaygrunnlag/v1/InntektArbeidYtelseGrunnlagDto.java

@@ -0,0 +1,4 @@
+alter table gr_arbeid_inntekt drop column oppgitte_opptjeninger_id;
+alter table iay_oppgitt_opptjening drop column oppgitte_opptjeninger_id;
+drop table iay_oppgitte_opptjeninger;
+drop sequence seq_iay_oppgitte_opptjeninger;

migreringer/src/main/resources/db/migration/defaultDS/2.2/V2.2_00__TFP-5763_rydder_opp_oppgitte_opptjeninger.sql

@@ -30,9 +30,6 @@ public JacksonJsonConfig() {
         objectMapper.registerModule(new JavaTimeModule());
         objectMapper.disable(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS);
         objectMapper.configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false);
-        // TODO (u139158): PK-44270 Diskutere med Front-end, ønsker i utgangpunktet å fjerne null, men hva med Javascript
-        // KodelisteSerializer og KodeverkSerializer bør i tilfelle også støtte JsonInclude.Include.*
-        // objectMapper.setSerializationInclusion(JsonInclude.Include.NON_EMPTY);
 
         registerSubTypesDynamically();
     }