Retter npe ved ignorert opptjening for PILS (#285)

navikt · Jun 22, 2022 · 1f3dc41 · 1f3dc41
1 parent eb2ab1c
commit 1f3dc41
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 10 deletions.
diff --git a/...c/main/java/no/nav/k9/søknad/ytelse/pls/v1/PleiepengerLivetsSluttfaseYtelseValidator.java b/...c/main/java/no/nav/k9/søknad/ytelse/pls/v1/PleiepengerLivetsSluttfaseYtelseValidator.java
@@ -17,6 +17,7 @@
 import no.nav.k9.søknad.felles.Feil;
 import no.nav.k9.søknad.felles.opptjening.AnnenAktivitet;
 import no.nav.k9.søknad.felles.opptjening.Frilanser;
+import no.nav.k9.søknad.felles.opptjening.OpptjeningAktivitet;
 import no.nav.k9.søknad.felles.opptjening.SelvstendigNæringsdrivende;
 import no.nav.k9.søknad.felles.type.Periode;
 import no.nav.k9.søknad.ytelse.Ytelse;
@@ -98,18 +99,24 @@ private void validerArbeidstid(PleipengerLivetsSluttfase søknad) throws Valider
     }
 
     private void validerOpptjening(PleipengerLivetsSluttfase søknad, List<Feil> feilene) {
-        List<SelvstendigNæringsdrivende> snAktiviteter = søknad.getOpptjeningAktivitet().getSelvstendigNæringsdrivende();
-        for (int i = 0; i < snAktiviteter.size(); i++) {
-            lagTidslinjeMedStøtteForÅpenPeriodeOgValider(snAktiviteter.get(i).getPerioder(), "opptjeningAktivitet.selvstendigNæringsdrivende[" + i + "].perioder");
+        if (!søknad.skalHaOpplysningOmOpptjeningVedNyPeriode()) {
+            feilene.add(new Feil("oppgittOpptjening", "påkrevd", "Opplysninger om opptjening må være oppgitt for ny søknadsperiode."));
         }
+        var opptjeningAktivitet = søknad.getOpptjeningAktivitet();
+        if (opptjeningAktivitet != null) {
+            List<SelvstendigNæringsdrivende> snAktiviteter = opptjeningAktivitet.getSelvstendigNæringsdrivende();
+            for (int i = 0; i < snAktiviteter.size(); i++) {
+                lagTidslinjeMedStøtteForÅpenPeriodeOgValider(snAktiviteter.get(i).getPerioder(), "opptjeningAktivitet.selvstendigNæringsdrivende[" + i + "].perioder");
+            }
 
-        List<AnnenAktivitet> andreAktiviteter = søknad.getOpptjeningAktivitet().getAndreAktiviteter();
-        for (int i = 0; i < andreAktiviteter.size(); i++) {
-            validerPerioderIkkeErInvertert(andreAktiviteter.get(i).getPeriode(), "opptjeningAktivitet.andreAktiviteter[" + i + "].perioder", feilene);
-        }
-        Frilanser fl = søknad.getOpptjeningAktivitet().getFrilanser();
-        if (fl != null) {
-            validerPerioderIkkeErInvertert(new Periode(fl.getStartdato(), fl.getSluttdato()), "opptjeningAktivitet.frilanser.startdato/sluttdato", feilene);
+            List<AnnenAktivitet> andreAktiviteter = opptjeningAktivitet.getAndreAktiviteter();
+            for (int i = 0; i < andreAktiviteter.size(); i++) {
+                validerPerioderIkkeErInvertert(andreAktiviteter.get(i).getPeriode(), "opptjeningAktivitet.andreAktiviteter[" + i + "].perioder", feilene);
+            }
+            Frilanser fl = opptjeningAktivitet.getFrilanser();
+            if (fl != null) {
+                validerPerioderIkkeErInvertert(new Periode(fl.getStartdato(), fl.getSluttdato()), "opptjeningAktivitet.frilanser.startdato/sluttdato", feilene);
+            }
         }
     }
 

diff --git a/soknad/src/main/java/no/nav/k9/søknad/ytelse/pls/v1/PleipengerLivetsSluttfase.java b/soknad/src/main/java/no/nav/k9/søknad/ytelse/pls/v1/PleipengerLivetsSluttfase.java
@@ -7,6 +7,7 @@
 import java.util.Objects;
 
 import javax.validation.Valid;
+import javax.validation.constraints.AssertTrue;
 import javax.validation.constraints.NotNull;
 
 import com.fasterxml.jackson.annotation.JsonAutoDetect;
@@ -190,4 +191,13 @@ public PleipengerLivetsSluttfase medUttak(Uttak uttak) {
         this.uttak = Objects.requireNonNull(uttak, "uttak");
         return this;
     }
+
+    @AssertTrue
+    public boolean skalHaOpplysningOmOpptjeningVedNyPeriode() {
+        if (søknadsperiode != null && !søknadsperiode.isEmpty()) {
+            return opptjeningAktivitet != null;
+        }
+        return true;
+    }
+
 }