fix: fjerner public fields som går i beina på annotation mapping. (#118)

* fix: fjerner public fields som går i beina på annotation mapping.
lagt til begrunnelse på midlertidig alene

* fix: ctor

Author: frode-carlsen

ettersendelse/src/main/java/no/nav/k9/ettersendelse/Ettersendelse.java

@@ -26,29 +26,29 @@ public class Ettersendelse implements Innsending {
     @JsonProperty(value="søknadId", required = true)
     @NotNull
     @Valid
-    public final SøknadId søknadId;
+    private final SøknadId søknadId;
 
     @JsonProperty(value="versjon")
     @Valid
-    public final Versjon versjon;
+    private final Versjon versjon;
 
 
     @JsonProperty(value="mottattDato")
     @Valid
     @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd'T'HH:mm:ss.SSSX", timezone = "UTC")
-    public final ZonedDateTime mottattDato;
+    private final ZonedDateTime mottattDato;
 
 
     @JsonProperty(value="søker", required = true)
     @Valid
     @NotNull
-    public final Søker søker;
+    private final Søker søker;
 
 
     @JsonProperty(value="ytelse", required = true)
     @Valid
     @NotNull
-    public final Ytelse ytelse;
+    private final Ytelse ytelse;
 
     @JsonCreator
     private Ettersendelse(
@@ -84,6 +84,10 @@ public Versjon getVersjon() {
         return versjon;
     }
 
+    public Ytelse getYtelse() {
+        return ytelse;
+    }
+    
     public static Builder builder() {
         return new Builder();
     }

ettersendelse/src/main/java/no/nav/k9/ettersendelse/EttersendelseValidator.java

@@ -16,10 +16,10 @@ public List<Feil> valider(Ettersendelse ettersendelse) {
 
         // I en ettersendelse må ikke søknadId settes.
         // Men om den er satt kan vi ruete deen samme veg som selve søknaden.
-        validerVersjon(ettersendelse.versjon, feil);
-        validerMottattDato(ettersendelse.mottattDato, feil);
-        validerSøker(ettersendelse.søker, feil);
-        validerYtelse(ettersendelse.ytelse, feil);
+        validerVersjon(ettersendelse.getVersjon(), feil);
+        validerMottattDato(ettersendelse.getMottattDato(), feil);
+        validerSøker(ettersendelse.getSøker(), feil);
+        validerYtelse(ettersendelse.getYtelse(), feil);
 
         return feil;
     }

soknad-omsorgspenger-utbetaling/src/main/java/no/nav/k9/søknad/omsorgspenger/utbetaling/arbeidstaker/OmsorgspengerUtbetalingSøknad.java

@@ -92,10 +92,10 @@ private List<Feil> getValiderAngittFosterbarn() {
         var index = 0;
         List<Feil> feil = new ArrayList<>();
         for (Barn b : barn) {
-            if (b.norskIdentitetsnummer == null && b.fødselsdato == null) {
+            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.norskIdentitetsnummer != null && b.fødselsdato != null) {
+            } 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."));
             }

soknad-omsorgspenger-utbetaling/src/main/java/no/nav/k9/søknad/omsorgspenger/utbetaling/snf/OmsorgspengerUtbetalingSøknad.java

@@ -111,10 +111,10 @@ private List<Feil> getValiderAngittFosterbarn() {
         var index = 0;
         List<Feil> feil = new ArrayList<>();
         for (Barn b : barn) {
-            if (b.norskIdentitetsnummer == null && b.fødselsdato == null) {
+            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.norskIdentitetsnummer != null && b.fødselsdato != null) {
+            } 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."));
             }

soknad-omsorgspenger/src/main/java/no/nav/k9/søknad/omsorgspenger/OmsorgspengerSøknadValidator.java

@@ -50,9 +50,9 @@ private static void validerMottattDato(ZonedDateTime mottatDato, List<Feil> feil
     private static void validerBarn(Barn barn, List<Feil> feil) {
         if (barn == null) {
             feil.add(new Feil("barn", PÅKREVD, "Barn må settes i søknaden."));
-        } else if (barn.norskIdentitetsnummer == null && barn.fødselsdato == null) {
+        } else if (barn.getPersonIdent() == null && barn.getFødselsdato() == null) {
             feil.add(new Feil("barn", "norskIdentitetsnummerEllerFødselsdatoPåkrevd", "Må sette enten Personnummer/D-nummer på barn, eller fødselsdato."));
-        } else if (barn.norskIdentitetsnummer != null && barn.fødselsdato != null) {
+        } else if (barn.getPersonIdent() != null && barn.getFødselsdato() != null) {
             feil.add(new Feil("barn", "ikkeEntydigIdPåBarnet", "Må sette enten Personnummer/D-nummer på barn, eller fødselsdato."));
         }
     }

soknad/src/main/java/no/nav/k9/søknad/felles/Feil.java

@@ -2,18 +2,40 @@
 
 import java.util.Objects;
 
+import com.fasterxml.jackson.annotation.JsonAutoDetect;
+import com.fasterxml.jackson.annotation.JsonCreator;
+import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
+import com.fasterxml.jackson.annotation.JsonProperty;
+
+@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 Feil {
 
-    public final String felt;
-    public final String feilkode;
-    public final String feilmelding;
+    private final String felt;
+    private final String feilkode;
+    private final String feilmelding;
 
-    public Feil(String felt, String feilkode, String feilmelding) {
+    @JsonCreator
+    public Feil(@JsonProperty(value = "felt") String felt,
+                @JsonProperty(value = "feilkode") String feilkode,
+                @JsonProperty(value = "feilmelding") String feilmelding) {
         this.felt = felt;
         this.feilkode = feilkode;
         this.feilmelding = feilmelding;
     }
 
+    public String getFeilkode() {
+        return feilkode;
+    }
+
+    public String getFeilmelding() {
+        return feilmelding;
+    }
+
+    public String getFelt() {
+        return felt;
+    }
+
     @Override
     public boolean equals(Object o) {
         if (this == o)

soknad/src/main/java/no/nav/k9/søknad/felles/Versjon.java

@@ -17,15 +17,20 @@ public class Versjon {
     @JsonValue
     @Size(max = 10)
     @Pattern(regexp = SEMVER_REGEX, message = "'${validatedValue}' matcher ikke tillatt pattern '{regexp}'")
-    public final String verdi;
+    private final String verdi;
 
+    @JsonCreator
     public Versjon(String verdi) {
         if (verdi == null || verdi.isBlank()) {
             throw new IllegalArgumentException("Kan ikke ha null eller blank versjon");
         }
         this.verdi = verdi;
     }
 
+    public String getVerdi() {
+        return verdi;
+    }
+    
     @JsonCreator
     public static Versjon of(String verdi) {
         if (verdi == null || verdi.isBlank()) {

soknad/src/main/java/no/nav/k9/søknad/felles/aktivitet/Arbeidstaker.java

@@ -1,17 +1,18 @@
 package no.nav.k9.søknad.felles.aktivitet;
 
+import java.util.List;
+
+import javax.validation.Valid;
+
 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 no.nav.k9.søknad.felles.Feil;
 import no.nav.k9.søknad.felles.type.NorskIdentitetsnummer;
 import no.nav.k9.søknad.ytelse.psb.v1.arbeidstid.ArbeidstidInfo;
 
-import javax.validation.Valid;
-import javax.validation.constraints.AssertTrue;
-import java.util.List;
-
 @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 Arbeidstaker {

soknad/src/main/java/no/nav/k9/søknad/felles/aktivitet/Frilanser.java

@@ -14,10 +14,10 @@ public class Frilanser {
 
     @JsonProperty(value = "startdato")
     @NotNull
-    public final LocalDate startdato;
+    private final LocalDate startdato;
 
     @JsonProperty(value = "jobberFortsattSomFrilans")
-    public final Boolean jobberFortsattSomFrilans;
+    private final Boolean jobberFortsattSomFrilans;
 
     public static Frilanser.Builder builder() {
         return new Frilanser.Builder();

soknad/src/main/java/no/nav/k9/søknad/felles/aktivitet/Organisasjonsnummer.java

@@ -14,12 +14,16 @@ public class Organisasjonsnummer {
     @NotNull
     @Size(max = 20)
     @Pattern(regexp = "^\\d+$", message = "'${validatedValue}' matcher ikke tillatt pattern '{regexp}'")
-    public final String verdi;
+    private final String verdi;
 
     private Organisasjonsnummer(String verdi) {
         this.verdi = Objects.requireNonNull(verdi, "organisasjonsnummer");
     }
 
+    public String getVerdi() {
+        return verdi;
+    }
+    
     @JsonCreator
     public static Organisasjonsnummer of(String verdi) {
         if (verdi == null || verdi.isBlank()) {