OLP - flere perioder per kurs - med avreise og hjemkomst (#301)

* OLP - flere perioder per kurs - med avreise og hjemkomst

* OLP - Validering av kursperioder med reisetid

* nullsjekk i validering

Author: oyvemb

soknad/src/main/java/no/nav/k9/søknad/ytelse/olp/v1/OpplæringspengerYtelseValidator.java

@@ -18,7 +18,7 @@
 import no.nav.k9.søknad.felles.type.Periode;
 import no.nav.k9.søknad.ytelse.Ytelse;
 import no.nav.k9.søknad.ytelse.YtelseValidator;
-import no.nav.k9.søknad.ytelse.omsorgspenger.v1.OmsorgspengerUtbetalingSøknadValidator;
+import no.nav.k9.søknad.ytelse.olp.v1.kurs.KursPeriodeMedReisetid;
 
 class OpplæringspengerYtelseValidator extends YtelseValidator {
 
@@ -96,6 +96,8 @@ List<Feil> validerOgLeggTilFeilene(Opplæringspenger olp,
 
         validerAtYtelsePeriodenErKomplettMedSøknad(søknadsperiodeTidslinje, olp.getUttak().getPerioder(), "uttak", feilene);
 
+        validerReisetidMotKursperioden(olp.getKurs().getKursperioder(), "kurs", feilene);
+
         return feilene;
     }
 
@@ -127,6 +129,28 @@ private List<Feil> validerAtYtelsePerioderErInnenforIntervalForEndring(LocalDate
                 .collect(Collectors.toCollection(ArrayList::new)));
     }
 
+    private void validerReisetidMotKursperioden(List<KursPeriodeMedReisetid> kursperioder, String felt, List<Feil> feil) {
+        for (KursPeriodeMedReisetid kursPeriode : kursperioder) {
+            if (kursPeriode != null) {
+                LocalDate avreise = kursPeriode.getAvreise();
+                LocalDate hjemkomst = kursPeriode.getHjemkomst();
+                Periode periode = kursPeriode.getPeriode();
+
+                if (avreise != null && hjemkomst != null && periode != null) {
+                    if (hjemkomst.isBefore(avreise)) {
+                        feil.add(toFeil(periode, felt, "ugyldigKursPeriode", "hjemkomst er før avreise: "));
+                    }
+                    if (avreise.isAfter(periode.getFraOgMed())) {
+                        feil.add(toFeil(periode, felt, "ugyldigKursPeriode", "avreise er etter kursstart: "));
+                    }
+                    if (hjemkomst.isBefore(periode.getTilOgMed())) {
+                        feil.add(toFeil(periode, felt, "ugyldigKursPeriode", "hjemkomst er før kursslutt: "));
+                    }
+                }
+            }
+        }
+    }
+
     private List<Feil> validerAtIngenPerioderOverlapperMedTrekkKravPerioder(LocalDateTimeline<Boolean> trekkKravPerioder,
                                                                             LocalDateTimeline<Boolean> test,
                                                                             String felt) {

soknad/src/main/java/no/nav/k9/søknad/ytelse/olp/v1/kurs/Kurs.java

@@ -1,14 +1,16 @@
 package no.nav.k9.søknad.ytelse.olp.v1.kurs;
 
+import java.util.ArrayList;
+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.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonProperty;
 
-import no.nav.k9.søknad.felles.type.Periode;
-
 @JsonIgnoreProperties(ignoreUnknown = true)
 @JsonAutoDetect(fieldVisibility = JsonAutoDetect.Visibility.NONE, getterVisibility = JsonAutoDetect.Visibility.NONE, setterVisibility = JsonAutoDetect.Visibility.NONE, isGetterVisibility = JsonAutoDetect.Visibility.NONE, creatorVisibility = JsonAutoDetect.Visibility.NONE)
 public class Kurs {
@@ -23,18 +25,19 @@ public class Kurs {
     @Valid
     private String formål;
 
-    @JsonProperty(value = "periode", required = true)
+    @JsonProperty(value = "kursperioder", required = true)
     @NotNull
+    @Size(min = 1)
     @Valid
-    private Periode periode;
+    private List<KursPeriodeMedReisetid> kursperioder;
 
     public Kurs() {
     }
 
-    public Kurs(String kursholder, String formålMedKurset, Periode periode) {
+    public Kurs(String kursholder, String formålMedKurset, List<KursPeriodeMedReisetid> kursperioder) {
         this.holder = kursholder;
         this.formål = formålMedKurset;
-        this.periode = periode;
+        this.kursperioder = kursperioder;
     }
 
     public String getHolder() {
@@ -45,7 +48,7 @@ public String getHolder() {
         return formål;
     }
 
-    public Periode getPeriode() {
-        return periode;
+    public List<KursPeriodeMedReisetid> getKursperioder() {
+        return new ArrayList<>(kursperioder);
     }
 }

soknad/src/main/java/no/nav/k9/søknad/ytelse/olp/v1/kurs/KursPeriodeMedReisetid.java

@@ -0,0 +1,53 @@
+package no.nav.k9.søknad.ytelse.olp.v1.kurs;
+
+import java.time.LocalDate;
+
+import javax.validation.Valid;
+import javax.validation.constraints.NotNull;
+
+import com.fasterxml.jackson.annotation.JsonAutoDetect;
+import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
+import com.fasterxml.jackson.annotation.JsonProperty;
+
+import no.nav.k9.søknad.felles.type.Periode;
+
+@JsonIgnoreProperties(ignoreUnknown = true)
+@JsonAutoDetect(fieldVisibility = JsonAutoDetect.Visibility.NONE, getterVisibility = JsonAutoDetect.Visibility.NONE, setterVisibility = JsonAutoDetect.Visibility.NONE, isGetterVisibility = JsonAutoDetect.Visibility.NONE, creatorVisibility = JsonAutoDetect.Visibility.NONE)
+public class KursPeriodeMedReisetid {
+
+    @JsonProperty(value = "periode", required = true)
+    @NotNull
+    @Valid
+    private Periode periode;
+
+    @JsonProperty(value = "avreise", required = true)
+    @NotNull
+    @Valid
+    private LocalDate avreise;
+
+    @JsonProperty(value = "hjemkomst", required = true)
+    @NotNull
+    @Valid
+    private LocalDate hjemkomst;
+
+    public KursPeriodeMedReisetid() {
+    }
+
+    public KursPeriodeMedReisetid(Periode periode, LocalDate avreise, LocalDate hjemkomst) {
+        this.periode = periode;
+        this.avreise = avreise;
+        this.hjemkomst = hjemkomst;
+    }
+
+    public Periode getPeriode() {
+        return periode;
+    }
+
+    public LocalDate getAvreise() {
+        return avreise;
+    }
+
+    public LocalDate getHjemkomst() {
+        return hjemkomst;
+    }
+}