Flytter fraværskorrigering IM inn under ytelse for OmsorgspengerUtbetaling (#208)

Gjenbruker patternet for delvis søknadsinnhold som også anvendes av PSB

Author: espennav

soknad/src/main/java/no/nav/k9/søknad/felles/fravær/FraværPeriode.java

@@ -6,7 +6,6 @@
 import java.util.stream.Collectors;
 
 import javax.validation.Valid;
-import javax.validation.constraints.NotNull;
 import javax.validation.constraints.Size;
 
 import com.fasterxml.jackson.annotation.JsonAutoDetect;
@@ -27,15 +26,15 @@ public class FraværPeriode implements Comparable<FraværPeriode> {
     @JsonProperty("duration")
     private final Duration duration;
 
-    @NotNull
+    @Valid
     @JsonProperty(value = "årsak", required = true)
     private final FraværÅrsak årsak;
 
     @Valid
     @JsonProperty(value = "søknadÅrsak")
     private SøknadÅrsak søknadÅrsak;
 
-    @NotNull
+    @Valid
     @Size(min = 1, max = 2)
     @JsonProperty(value = "aktivitetFravær", required = true)
     private final List<AktivitetFravær> aktivitetFravær;

soknad/src/main/java/no/nav/k9/søknad/ytelse/omsorgspenger/fraværskorrigering/v1/OmsorgspengerFraværskorrigeringInntektsmelding.java

@@ -1,28 +1,25 @@
 package no.nav.k9.søknad.ytelse.omsorgspenger.v1;
 
+import java.time.LocalDate;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
 
 import javax.validation.Valid;
-import javax.validation.constraints.NotNull;
-import javax.validation.constraints.Size;
 
 import com.fasterxml.jackson.annotation.JsonAutoDetect;
 import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.annotation.JsonTypeName;
 
-import no.nav.k9.søknad.PeriodeValidator;
-import no.nav.k9.søknad.felles.Feil;
-import no.nav.k9.søknad.felles.opptjening.OpptjeningAktivitet;
 import no.nav.k9.søknad.felles.fravær.FraværPeriode;
+import no.nav.k9.søknad.felles.opptjening.OpptjeningAktivitet;
 import no.nav.k9.søknad.felles.personopplysninger.Barn;
 import no.nav.k9.søknad.felles.personopplysninger.Bosteder;
 import no.nav.k9.søknad.felles.personopplysninger.Utenlandsopphold;
-import no.nav.k9.søknad.felles.type.Person;
 import no.nav.k9.søknad.felles.type.Periode;
+import no.nav.k9.søknad.felles.type.Person;
 import no.nav.k9.søknad.ytelse.Ytelse;
 import no.nav.k9.søknad.ytelse.YtelseValidator;
 
@@ -36,8 +33,7 @@ public class OmsorgspengerUtbetaling implements Ytelse {
     private List<Barn> fosterbarn;
 
     @Valid
-    @NotNull
-    @JsonProperty(value = "aktivitet", required = true)
+    @JsonProperty(value = "aktivitet")
     private OpptjeningAktivitet aktivitet;
 
     @Valid
@@ -49,24 +45,28 @@ public class OmsorgspengerUtbetaling implements Ytelse {
     private Utenlandsopphold utenlandsopphold;
 
     @Valid
-    @NotNull
-    @Size(min = 1, message = "Minst 1 fraværsperiode må oppgis")
-    @JsonProperty(value = "fraværsperioder", required = true)
+    @JsonProperty(value = "fraværsperioder")
     private List<FraværPeriode> fraværsperioder;
 
+    @Valid
+    @JsonProperty(value = "fraværsperioderKorrigeringIm")
+    private List<FraværPeriode> fraværsperioderKorrigeringIm;
+
     public OmsorgspengerUtbetaling() {
         //
     }
 
     @JsonCreator
     public OmsorgspengerUtbetaling(@JsonProperty("fosterbarn") @Valid List<Barn> fosterbarn,
-                                   @JsonProperty(value = "aktivitet", required = true) @Valid @NotNull OpptjeningAktivitet opptjening,
-                                   @JsonProperty(value = "fraværsperioder", required = true) @Valid @NotNull @Size(min = 1) List<FraværPeriode> fraværsperioder,
-                                   @JsonProperty(value = "bosteder") @Valid @NotNull Bosteder bosteder,
-                                   @JsonProperty(value = "utenlandsopphold") @Valid @NotNull Utenlandsopphold utenlandsopphold) {
+                                   @JsonProperty(value = "aktivitet") @Valid OpptjeningAktivitet opptjening,
+                                   @JsonProperty(value = "fraværsperioder") @Valid List<FraværPeriode> fraværsperioder,
+                                   @JsonProperty(value = "fraværsperioderKorrigeringIm") @Valid List<FraværPeriode> fraværsperioderKorrigeringIm,
+                                   @JsonProperty(value = "bosteder") @Valid Bosteder bosteder,
+                                   @JsonProperty(value = "utenlandsopphold") @Valid Utenlandsopphold utenlandsopphold) {
         this.fosterbarn = fosterbarn;
         this.aktivitet = opptjening;
         this.fraværsperioder = fraværsperioder;
+        this.fraværsperioderKorrigeringIm = fraværsperioderKorrigeringIm;
         this.bosteder = bosteder;
         this.utenlandsopphold = utenlandsopphold;
     }
@@ -83,6 +83,10 @@ public OpptjeningAktivitet getAktivitet() {
         return fraværsperioder == null? null: Collections.unmodifiableList(fraværsperioder);
     }
 
+    public List<FraværPeriode> getFraværsperioderKorrigeringIm() {
+        return fraværsperioderKorrigeringIm;
+    }
+
     public Bosteder getBosteder() {
         return bosteder;
     }
@@ -104,8 +108,18 @@ public Person getAnnenPart() {
     /** har ikke egen søknadsperiode for omsorgspenger, men angir på bakggrunn av angitte fraværsperioder. */
     @Override
     public Periode getSøknadsperiode() {
-        var fom = fraværsperioder.stream().min(FraværPeriode::compareTo).orElseThrow().getPeriode().getFraOgMed();
-        var tom = fraværsperioder.stream().max(FraværPeriode::compareTo).orElseThrow().getPeriode().getTilOgMed();
+        LocalDate fom;
+        LocalDate tom;
+        if (fraværsperioder != null && !fraværsperioder.isEmpty()) {
+            fom = fraværsperioder.stream().min(FraværPeriode::compareTo).orElseThrow().getPeriode().getFraOgMed();
+            tom = fraværsperioder.stream().max(FraværPeriode::compareTo).orElseThrow().getPeriode().getTilOgMed();
+        } else if (fraværsperioderKorrigeringIm != null && !fraværsperioderKorrigeringIm.isEmpty()) {
+            fom = fraværsperioderKorrigeringIm.stream().min(FraværPeriode::compareTo).orElseThrow().getPeriode().getFraOgMed();
+            tom = fraværsperioderKorrigeringIm.stream().max(FraværPeriode::compareTo).orElseThrow().getPeriode().getTilOgMed();
+        } else {
+            // Skal fanges opp av validator
+            throw new IllegalArgumentException("Må ha fravær fra enten søker eller fra fraværskorrigering av inntektsmelding");
+        }
         return new Periode(fom, tom);
     }
 
@@ -152,38 +166,4 @@ public OmsorgspengerUtbetaling medUtenlandsopphold(Utenlandsopphold utenlandsopp
         this.utenlandsopphold = utenlandsopphold;
         return this;
     }
-
-    @Size(max = 0, message = "${validatedValue}")
-    private List<Feil> validerAngittUtenlandsopphold() {
-        if (utenlandsopphold == null)
-            List.of();
-        return new PeriodeValidator().validerIkkeTillattOverlapp(utenlandsopphold.getPerioder(), "utenlandsopphold.perioder");
-    }
-
-    @Size(max = 0, message = "${validatedValue}")
-    private List<Feil> validerAngittBosteder() {
-        if (bosteder == null)
-            return List.of();
-        return new PeriodeValidator().validerIkkeTillattOverlapp(bosteder.getPerioder(), "bosteder.perioder");
-    }
-
-    @Size(max = 0, message = "${validatedValue}")
-    private List<Feil> getValiderAngittFosterbarn() {
-        var barn = this.fosterbarn;
-        if (barn == null || barn.isEmpty())
-            return List.of();
-        var index = 0;
-        List<Feil> feil = new ArrayList<>();
-        for (Barn b : barn) {
-            if (b.getPersonIdent() == null && b.getFødselsdato() == null) {
-                feil.add(new Feil("fosterbarn[" + index + "]", "norskIdentitetsnummerEllerFødselsdatoPåkrevd",
-                    "Må sette enten Personnummer/D-nummer på fosterbarn, eller fødselsdato."));
-            } else if (b.getPersonIdent() != null && b.getFødselsdato() != null) {
-                feil.add(
-                    new Feil("fosterbarn[" + index + "]", "ikkeEntydigIdPåBarnet", "Må sette enten Personnummer/D-nummer på fosterbarn, eller fødselsdato."));
-            }
-            index++;
-        }
-        return feil;
-    }
 }