lagt på Optional for felter som kan retunere null (#147)

Author: christofferlohne

soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/Omsorg.java

@@ -1,6 +1,7 @@
 package no.nav.k9.søknad.ytelse.psb.v1;
 
 import java.util.Objects;
+import java.util.Optional;
 
 import com.fasterxml.jackson.annotation.JsonAutoDetect;
 import com.fasterxml.jackson.annotation.JsonCreator;
@@ -26,9 +27,8 @@ public class Omsorg {
     public Omsorg() {
     }
 
-    //TODO optinal returverdi
-    public BarnRelasjon getRelasjonTilBarnet() {
-        return relasjonTilBarnet;
+    public Optional<BarnRelasjon> getRelasjonTilBarnet() {
+        return Optional.ofNullable(relasjonTilBarnet);
     }
 
     public Omsorg medRelasjonTilBarnet(BarnRelasjon relasjonTilBarnet) {
@@ -37,8 +37,8 @@ public Omsorg medRelasjonTilBarnet(BarnRelasjon relasjonTilBarnet) {
     }
 
     //TODO optional returverdi
-    public String getBeskrivelseAvOmsorgsrollen() {
-        return beskrivelseAvOmsorgsrollen;
+    public Optional<String> getBeskrivelseAvOmsorgsrollen() {
+        return Optional.ofNullable(beskrivelseAvOmsorgsrollen);
     }
 
     public Omsorg medBeskrivelseAvOmsorgsrollen(String beskrivelseAvOmsorgsrollen) {

soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/PleiepengerSyktBarn.java

@@ -5,6 +5,7 @@
 import java.util.Collections;
 import java.util.List;
 import java.util.Objects;
+import java.util.Optional;
 
 import javax.validation.Valid;
 import javax.validation.constraints.NotNull;
@@ -51,7 +52,6 @@ public class PleiepengerSyktBarn implements Ytelse {
     private List<Periode> endringsperiode = new ArrayList<>();
 
     @Valid
-    @NotNull
     @JsonProperty(value = "opptjeningAktivitet")
     private OpptjeningAktivitet opptjeningAktivitet = new OpptjeningAktivitet();
 
@@ -72,37 +72,30 @@ public class PleiepengerSyktBarn implements Ytelse {
     private Utenlandsopphold utenlandsopphold = new Utenlandsopphold();
 
     @Valid
-    @NotNull
     @JsonProperty(value = "beredskap", required = true)
     private Beredskap beredskap = new Beredskap();
 
     @Valid
-    @NotNull
     @JsonProperty(value = "nattevåk", required = true)
     private Nattevåk nattevåk = new Nattevåk();
 
     @Valid
-    @NotNull
     @JsonProperty(value = "tilsynsordning", required = true)
     private Tilsynsordning tilsynsordning = new Tilsynsordning();
 
     @Valid
-    @NotNull
     @JsonProperty(value = "lovbestemtFerie", required = true)
     private LovbestemtFerie lovbestemtFerie = new LovbestemtFerie();
 
     @Valid
-    @NotNull
     @JsonProperty(value = "arbeidstid", required = true)
     private Arbeidstid arbeidstid = new Arbeidstid();
 
     @Valid
-    @NotNull
     @JsonProperty(value = "uttak", required = true)
     private Uttak uttak = new Uttak();
 
     @Valid
-    @NotNull
     @JsonProperty(value = "omsorg", required = true)
     private Omsorg omsorg = new Omsorg();
 
@@ -188,17 +181,17 @@ public PleiepengerSyktBarn medOpptjeningAktivitet(OpptjeningAktivitet arbeidAkti
         return this;
     }
 
-    public DataBruktTilUtledning getSøknadInfo() {
-        return dataBruktTilUtledning;
+    public Optional<DataBruktTilUtledning> getSøknadInfo() {
+        return Optional.ofNullable(dataBruktTilUtledning);
     }
 
     public PleiepengerSyktBarn medSøknadInfo(DataBruktTilUtledning dataBruktTilUtledning) {
         this.dataBruktTilUtledning = Objects.requireNonNull(dataBruktTilUtledning, "dataBruktTilUtledning");
         return this;
     }
 
-    public InfoFraPunsj getInfoFraPunsj() {
-        return infoFraPunsj;
+    public Optional<InfoFraPunsj> getInfoFraPunsj() {
+        return Optional.ofNullable(infoFraPunsj);
     }
 
     public PleiepengerSyktBarn medInfoFraPunsj(InfoFraPunsj infoFraPunsj) {

soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/PleiepengerSyktBarnValidator.java

@@ -110,8 +110,12 @@ private void validerLovbestemtFerie(LovbestemtFerie lovbestemtFerie, TidsserieVa
 
     private void validerArbeidstid(Arbeidstid arbeidstid, TidsserieValidator.Perioder søknadsperiode, List<Feil> feil) {
         validerArbeidstaker(arbeidstid.getArbeidstakerList(), søknadsperiode, feil);
-        validerFrilanser(arbeidstid.getFrilanserArbeidstidInfo(), søknadsperiode, feil);
-        validerSelvstendigNæringsdrivende(arbeidstid.getSelvstendigNæringsdrivendeArbeidstidInfo(), søknadsperiode, feil);
+        if (arbeidstid.getFrilanserArbeidstidInfo().isPresent()) {
+            validerFrilanser(arbeidstid.getFrilanserArbeidstidInfo().get(), søknadsperiode, feil);
+        }
+        if (arbeidstid.getSelvstendigNæringsdrivendeArbeidstidInfo().isPresent()) {
+            validerSelvstendigNæringsdrivende(arbeidstid.getSelvstendigNæringsdrivendeArbeidstidInfo().get(), søknadsperiode, feil);
+        }
     }
 
     private void validerArbeidstaker(List<Arbeidstaker> arbeidstakerList, TidsserieValidator.Perioder søknadsperiode, List<Feil> feil) {
@@ -127,19 +131,13 @@ private void validerArbeidstaker(List<Arbeidstaker> arbeidstakerList, TidsserieV
     }
 
     private void validerFrilanser(ArbeidstidInfo frilanser, TidsserieValidator.Perioder søknadsperiode, List<Feil> feil) {
-        if (frilanser == null) {
-            return;
-        }
         finnIkkeKomplettePerioderOgPerioderUtenfor(
                 toLocalDateTimeline(frilanser.getPerioder()),
                 søknadsperiode)
                 .valider("arbeidstid.frilanser", feil);
     }
 
     private void validerSelvstendigNæringsdrivende(ArbeidstidInfo selvstendigNæringsdrivende, TidsserieValidator.Perioder søknadsperiode, List<Feil> feil) {
-        if (selvstendigNæringsdrivende == null) {
-            return;
-        }
         finnIkkeKomplettePerioderOgPerioderUtenfor(
                 toLocalDateTimeline(selvstendigNæringsdrivende.getPerioder()),
                 søknadsperiode)

soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/arbeidstid/Arbeidstid.java

@@ -53,17 +53,17 @@ public Arbeidstid leggeTilArbeidstaker(Arbeidstaker arbeidstaker) {
         return this;
     }
 
-    public ArbeidstidInfo getFrilanserArbeidstidInfo() {
-        return frilanserArbeidstidInfo;
+    public Optional<ArbeidstidInfo> getFrilanserArbeidstidInfo() {
+        return Optional.ofNullable(frilanserArbeidstidInfo);
     }
 
     public Arbeidstid medFrilanserArbeidstid(ArbeidstidInfo frilanserArbeidstidInfo) {
         this.frilanserArbeidstidInfo = Objects.requireNonNull(frilanserArbeidstidInfo, "Arbeidstid.frilanserArbeidstidInfo");
         return this;
     }
 
-    public ArbeidstidInfo getSelvstendigNæringsdrivendeArbeidstidInfo() {
-        return selvstendigNæringsdrivendeArbeidstidInfo;
+    public Optional<ArbeidstidInfo> getSelvstendigNæringsdrivendeArbeidstidInfo() {
+        return Optional.ofNullable(selvstendigNæringsdrivendeArbeidstidInfo);
     }
 
     public Arbeidstid medSelvstendigNæringsdrivendeArbeidstidInfo(ArbeidstidInfo selvstendigNæringsdrivendeArbeidstidInfo) {