Skip to content

Commit d6e50fb

Browse files
christofferlohnechristofferlohne
authored
lag på at mer null tester ved opprettelse (#139)
* lag på at mer null tester ved opprettelse * fikset arbeidstid arbeidstaker med
1 parent c03f5b4 commit d6e50fb

File tree

14 files changed

+101
-102
lines changed

14 files changed

+101
-102
lines changed

‎soknad/src/main/java/no/nav/k9/søknad/felles/personopplysninger/Bosteder.java‎

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,11 @@
77

88
import java.util.HashMap;
99
import java.util.Map;
10+
import java.util.Objects;
1011
import java.util.TreeMap;
1112

1213
import javax.validation.Valid;
14+
import javax.validation.constraints.NotNull;
1315

1416
import com.fasterxml.jackson.annotation.JsonAutoDetect;
1517
import com.fasterxml.jackson.annotation.JsonCreator;
@@ -35,6 +37,7 @@ public class Bosteder {
3537
@JsonInclude(value = Include.ALWAYS)
3638
private Map<Periode, BostedPeriodeInfo> perioderSomSkalSlettes = new TreeMap<>();
3739

40+
/**@deprecated brukt tom ctor.*/
3841
@JsonCreator
3942
public Bosteder(
4043
@JsonProperty("perioder") Map<Periode, BostedPeriodeInfo> perioder) {
@@ -98,12 +101,15 @@ public Bosteder build() {
98101
@JsonAutoDetect(fieldVisibility = JsonAutoDetect.Visibility.NONE, getterVisibility = JsonAutoDetect.Visibility.NONE, setterVisibility = JsonAutoDetect.Visibility.NONE, isGetterVisibility = JsonAutoDetect.Visibility.NONE, creatorVisibility = JsonAutoDetect.Visibility.NONE)
99102
public static class BostedPeriodeInfo {
100103

101-
@JsonProperty(value = "land")
104+
@JsonProperty(value = "land", required = true)
105+
@NotNull
106+
@Valid
102107
private Landkode land;
103108

109+
/**@deprecated brukt tom ctor.*/
104110
@JsonCreator
105-
public BostedPeriodeInfo(@JsonProperty(value = "land") Landkode land) {
106-
this.land = land;
111+
public BostedPeriodeInfo(@JsonProperty(value = "land", required = true) @NotNull @Valid Landkode land) {
112+
this.land = Objects.requireNonNull(land, "land");
107113
}
108114

109115
public BostedPeriodeInfo() {
@@ -115,7 +121,7 @@ public Landkode getLand() {
115121
}
116122

117123
public BostedPeriodeInfo medLand(Landkode land) {
118-
this.land = land;
124+
this.land = Objects.requireNonNull(land, "land");
119125
return this;
120126
}
121127

‎soknad/src/main/java/no/nav/k9/søknad/felles/personopplysninger/Utenlandsopphold.java‎

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77

88
import java.util.HashMap;
99
import java.util.Map;
10+
import java.util.Objects;
1011
import java.util.TreeMap;
1112

1213
import static java.util.Collections.emptyMap;
@@ -15,6 +16,7 @@
1516
import static no.nav.k9.søknad.felles.type.Periode.Utils.leggTilPerioder;
1617

1718
import javax.validation.Valid;
19+
import javax.validation.constraints.NotNull;
1820

1921
@JsonIgnoreProperties(ignoreUnknown = true)
2022
@JsonAutoDetect(fieldVisibility = JsonAutoDetect.Visibility.NONE, getterVisibility = JsonAutoDetect.Visibility.NONE, setterVisibility = JsonAutoDetect.Visibility.NONE, isGetterVisibility = JsonAutoDetect.Visibility.NONE, creatorVisibility = JsonAutoDetect.Visibility.NONE)
@@ -28,6 +30,7 @@ public class Utenlandsopphold {
2830
@JsonProperty(value = "perioderSomSkalSlettes")
2931
private Map<Periode, UtenlandsoppholdPeriodeInfo> perioderSomSkalSlettes = new TreeMap<>();
3032

33+
/**@deprecated brukt tom ctor.*/
3134
@JsonCreator
3235
public Utenlandsopphold(
3336
@JsonProperty("perioder") Map<Periode, UtenlandsoppholdPeriodeInfo> perioder) {
@@ -92,16 +95,18 @@ public Utenlandsopphold build() {
9295
public static class UtenlandsoppholdPeriodeInfo {
9396

9497
@JsonProperty(value = "land", required = true)
98+
@NotNull
9599
private Landkode land;
96100

97101
@JsonProperty(value = "årsak")
98102
private UtenlandsoppholdÅrsak årsak;
99103

104+
/**@Deprecated bruk tom ctor*/
100105
@JsonCreator
101106
private UtenlandsoppholdPeriodeInfo(
102107
@JsonProperty("land") Landkode land,
103108
@JsonProperty("årsak") UtenlandsoppholdÅrsak årsak) {
104-
this.land = land;
109+
this.land = Objects.requireNonNull(land, "UtenlandsoppholdPeriodeInfo.land");
105110
this.årsak = årsak;
106111
}
107112

@@ -114,7 +119,7 @@ public Landkode getLand() {
114119
}
115120

116121
public UtenlandsoppholdPeriodeInfo medLand(Landkode land) {
117-
this.land = land;
122+
this.land = Objects.requireNonNull(land, "UtenlandsoppholdPeriodeInfo.land");
118123
return this;
119124
}
120125

@@ -123,10 +128,11 @@ public UtenlandsoppholdPeriodeInfo medLand(Landkode land) {
123128
}
124129

125130
public UtenlandsoppholdPeriodeInfo medÅrsak(UtenlandsoppholdÅrsak årsak) {
126-
this.årsak = årsak;
131+
this.årsak = Objects.requireNonNull(årsak, "UtenlandsoppholdPeriodeInfo.årsak");
127132
return this;
128133
}
129-
134+
135+
@Deprecated
130136
public static Builder builder() {
131137
return new Builder();
132138
}

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
import javax.validation.constraints.NotEmpty;
1111
import javax.validation.constraints.NotNull;
1212
import java.util.Map;
13+
import java.util.Objects;
1314
import java.util.TreeMap;
1415

1516
import static java.util.Collections.unmodifiableMap;
@@ -69,7 +70,7 @@ public String getTilleggsinformasjon() {
6970
}
7071

7172
public BeredskapPeriodeInfo medTilleggsinformasjon(String tilleggsinformasjon) {
72-
this.tilleggsinformasjon = tilleggsinformasjon;
73+
this.tilleggsinformasjon = Objects.requireNonNull(tilleggsinformasjon, "BeredskapPeriodeInfo.tilleggsinformasjon");
7374
return this;
7475
}
7576
}

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212

1313
import java.util.Collections;
1414
import java.util.Map;
15+
import java.util.Objects;
1516
import java.util.TreeMap;
1617

1718
@JsonIgnoreProperties(ignoreUnknown = true)

‎soknad/src/main/java/no/nav/k9/søknad/ytelse/psb/v1/Nattevåk.java‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
import javax.validation.constraints.NotNull;
1212
import java.util.Collections;
1313
import java.util.Map;
14+
import java.util.Objects;
1415
import java.util.TreeMap;
1516

1617
@JsonIgnoreProperties(ignoreUnknown = true)
@@ -68,7 +69,7 @@ public String getTilleggsinformasjon() {
6869
}
6970

7071
public NattevåkPeriodeInfo medTilleggsinformasjon(String tilleggsinformasjon) {
71-
this.tilleggsinformasjon = tilleggsinformasjon;
72+
this.tilleggsinformasjon = Objects.requireNonNull(tilleggsinformasjon, "NattevåkPeriodeInfo.tilleggsinformasjon");
7273
return this;
7374
}
7475
}

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

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package no.nav.k9.søknad.ytelse.psb.v1;
22

3+
import java.util.Objects;
4+
35
import com.fasterxml.jackson.annotation.JsonAutoDetect;
46
import com.fasterxml.jackson.annotation.JsonCreator;
57
import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
@@ -29,7 +31,7 @@ public BarnRelasjon getRelasjonTilBarnet() {
2931
}
3032

3133
public Omsorg medRelasjonTilBarnet(BarnRelasjon relasjonTilBarnet) {
32-
this.relasjonTilBarnet = relasjonTilBarnet;
34+
this.relasjonTilBarnet = Objects.requireNonNull(relasjonTilBarnet, "relasjonTilBarnet");
3335
return this;
3436
}
3537

@@ -38,7 +40,7 @@ public String getBeskrivelseAvOmsorgsrollen() {
3840
}
3941

4042
public Omsorg medBeskrivelseAvOmsorgsrollen(String beskrivelseAvOmsorgsrollen) {
41-
this.beskrivelseAvOmsorgsrollen = beskrivelseAvOmsorgsrollen;
43+
this.beskrivelseAvOmsorgsrollen = Objects.requireNonNull(beskrivelseAvOmsorgsrollen, "beskrivelseAvOmsorgsrollen");
4244
return this;
4345
}
4446

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

Lines changed: 24 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@
44
import java.util.ArrayList;
55
import java.util.Collections;
66
import java.util.List;
7+
import java.util.Objects;
8+
import java.util.TreeMap;
79

810
import javax.validation.Valid;
911
import javax.validation.constraints.NotNull;
@@ -110,7 +112,12 @@ public PleiepengerSyktBarn() {
110112
public Barn getBarn() {
111113
return barn;
112114
}
113-
115+
116+
public PleiepengerSyktBarn medBarn(Barn barn) {
117+
this.barn = Objects.requireNonNull(barn, "barn");
118+
return this;
119+
}
120+
114121
@Override
115122
public Person getPleietrengende() {
116123
return barn;
@@ -121,11 +128,6 @@ public Person getAnnenPart() {
121128
// ikke relevant for pleiepenger sykt barn
122129
return null;
123130
}
124-
125-
public PleiepengerSyktBarn medBarn(Barn barn) {
126-
this.barn = barn;
127-
return this;
128-
}
129131

130132
@Override
131133
public Periode getSøknadsperiode() {
@@ -149,7 +151,7 @@ public PleiepengerSyktBarn medBarn(Barn barn) {
149151
return søknadsperiode == null? null: Collections.unmodifiableList(søknadsperiode);
150152
}
151153

152-
public PleiepengerSyktBarn medSøknadsperiodeList(List<Periode> søknadsperiodeList) {
154+
public PleiepengerSyktBarn medSøknadsperiode(List<Periode> søknadsperiodeList) {
153155
if (this.søknadsperiode == null)
154156
this.søknadsperiode = new ArrayList<>();
155157
this.søknadsperiode.addAll(søknadsperiodeList);
@@ -163,11 +165,11 @@ public PleiepengerSyktBarn medBarn(Barn barn) {
163165
return this;
164166
}
165167

166-
public List<Periode> getEndringsperiodeList() {
167-
return endringsperiode == null? null: Collections.unmodifiableList(endringsperiode);
168+
public List<Periode> getEndringsperiode() {
169+
return (endringsperiode == null) ? null : Collections.unmodifiableList(endringsperiode);
168170
}
169171

170-
public PleiepengerSyktBarn medEndringsperiodeList(List<Periode> endringsperiodeList) {
172+
public PleiepengerSyktBarn medEndringsperiode(List<Periode> endringsperiodeList) {
171173
if (this.endringsperiode == null)
172174
this.endringsperiode = new ArrayList<>();
173175
this.endringsperiode.addAll(endringsperiodeList);
@@ -186,7 +188,7 @@ public OpptjeningAktivitet getOpptjeningAktivitet() {
186188
}
187189

188190
public PleiepengerSyktBarn medOpptjeningAktivitet(OpptjeningAktivitet arbeidAktivitet) {
189-
this.opptjeningAktivitet = arbeidAktivitet;
191+
this.opptjeningAktivitet = Objects.requireNonNull(arbeidAktivitet, "arbeidAktivitet");
190192
return this;
191193
}
192194

@@ -195,7 +197,7 @@ public PleiepengerSyktBarn medOpptjeningAktivitet(OpptjeningAktivitet arbeidAkti
195197
}
196198

197199
public PleiepengerSyktBarn medSøknadInfo(DataBruktTilUtledning dataBruktTilUtledning) {
198-
this.dataBruktTilUtledning = dataBruktTilUtledning;
200+
this.dataBruktTilUtledning = Objects.requireNonNull(dataBruktTilUtledning, "dataBruktTilUtledning");
199201
return this;
200202
}
201203

@@ -204,7 +206,7 @@ public InfoFraPunsj getInfoFraPunsj() {
204206
}
205207

206208
public PleiepengerSyktBarn medInfoFraPunsj(InfoFraPunsj infoFraPunsj) {
207-
this.infoFraPunsj = infoFraPunsj;
209+
this.infoFraPunsj = Objects.requireNonNull(infoFraPunsj, "infoFraPunsj");
208210
return this;
209211
}
210212

@@ -213,7 +215,7 @@ public Bosteder getBosteder() {
213215
}
214216

215217
public PleiepengerSyktBarn medBosteder(Bosteder bosteder) {
216-
this.bosteder = bosteder;
218+
this.bosteder = Objects.requireNonNull(bosteder, "bosteder");
217219
return this;
218220
}
219221

@@ -222,7 +224,7 @@ public Utenlandsopphold getUtenlandsopphold() {
222224
}
223225

224226
public PleiepengerSyktBarn medUtenlandsopphold(Utenlandsopphold utenlandsopphold) {
225-
this.utenlandsopphold = utenlandsopphold;
227+
this.utenlandsopphold = Objects.requireNonNull(utenlandsopphold, "utenlandsopphold");
226228
return this;
227229
}
228230

@@ -231,7 +233,7 @@ public Beredskap getBeredskap() {
231233
}
232234

233235
public PleiepengerSyktBarn medBeredskap(Beredskap beredskap) {
234-
this.beredskap = beredskap;
236+
this.beredskap = Objects.requireNonNull(beredskap, "beredskap");
235237
return this;
236238
}
237239

@@ -240,7 +242,7 @@ public PleiepengerSyktBarn medBeredskap(Beredskap beredskap) {
240242
}
241243

242244
public PleiepengerSyktBarn medNattevåk(Nattevåk nattevåk) {
243-
this.nattevåk = nattevåk;
245+
this.nattevåk = Objects.requireNonNull(nattevåk, "nattevåk");
244246
return this;
245247
}
246248

@@ -249,7 +251,7 @@ public Tilsynsordning getTilsynsordning() {
249251
}
250252

251253
public PleiepengerSyktBarn medTilsynsordning(Tilsynsordning tilsynsordning) {
252-
this.tilsynsordning = tilsynsordning;
254+
this.tilsynsordning = Objects.requireNonNull(tilsynsordning, "tilsynsordning");
253255
return this;
254256
}
255257

@@ -258,7 +260,7 @@ public LovbestemtFerie getLovbestemtFerie() {
258260
}
259261

260262
public PleiepengerSyktBarn medLovbestemtFerie(LovbestemtFerie lovbestemtFerie) {
261-
this.lovbestemtFerie = lovbestemtFerie;
263+
this.lovbestemtFerie = Objects.requireNonNull(lovbestemtFerie, "lovbestemtFerie");
262264
return this;
263265
}
264266

@@ -267,7 +269,7 @@ public Arbeidstid getArbeidstid() {
267269
}
268270

269271
public PleiepengerSyktBarn medArbeidstid(Arbeidstid arbeidstid) {
270-
this.arbeidstid = arbeidstid;
272+
this.arbeidstid = Objects.requireNonNull(arbeidstid, "arbeidstid");
271273
return this;
272274
}
273275

@@ -276,7 +278,7 @@ public Uttak getUttak() {
276278
}
277279

278280
public PleiepengerSyktBarn medUttak(Uttak uttak) {
279-
this.uttak = uttak;
281+
this.uttak = Objects.requireNonNull(uttak, "uttak");
280282
return this;
281283
}
282284

@@ -285,7 +287,7 @@ public Omsorg getOmsorg() {
285287
}
286288

287289
public PleiepengerSyktBarn medOmsorg(Omsorg omsorg) {
288-
this.omsorg = omsorg;
290+
this.omsorg = Objects.requireNonNull(omsorg, "omsorg");
289291
return this;
290292
}
291293

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

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
import javax.validation.Validation;
1414
import javax.validation.ValidatorFactory;
1515
import java.util.List;
16-
import java.util.Optional;
1716
import java.util.stream.Collectors;
1817

1918
import static no.nav.k9.søknad.TidsserieValidator.TidsserieUtils.toLocalDateTimeline;
@@ -73,11 +72,11 @@ private Feil toFeil(ConstraintViolation<PleiepengerSyktBarn> constraintViolation
7372

7473
private TidsserieValidator.Perioder validerSøknadsOgEndringsPerioder(PleiepengerSyktBarn psb, List<Feil> feil) {
7574
manglerIkkeSøknadEllerEndringsPerioder(psb, feil);
76-
return new TidsserieValidator.Perioder(psb.getSøknadsperiodeList(), psb.getEndringsperiodeList());
75+
return new TidsserieValidator.Perioder(psb.getSøknadsperiodeList(), psb.getEndringsperiode());
7776
}
7877

7978
private void manglerIkkeSøknadEllerEndringsPerioder(PleiepengerSyktBarn psb, List<Feil> feil) {
80-
if ( (psb.getSøknadsperiodeList().isEmpty() && psb.getEndringsperiodeList().isEmpty())) {
79+
if ( (psb.getSøknadsperiodeList().isEmpty() && psb.getEndringsperiode().isEmpty())) {
8180
feil.add(new Feil("søknadsperiode/endringsperiode", "missingArgument","Mangler søknadsperiode eller endringsperiode."));
8281
}
8382
}

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

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212

1313
import java.util.Collections;
1414
import java.util.Map;
15+
import java.util.Objects;
1516
import java.util.TreeMap;
1617

1718
import static java.util.Collections.unmodifiableMap;
@@ -29,11 +30,6 @@ public class Uttak {
2930
@Valid
3031
private Map<Periode, UttakPeriodeInfo> perioderSomSkalSlettes = new TreeMap<>();
3132

32-
@JsonCreator
33-
public Uttak( @JsonProperty(value = "perioder") @Valid Map<Periode, UttakPeriodeInfo> perioder ) {
34-
this.perioder = (perioder == null ) ? new TreeMap<>() : new TreeMap<>(perioder);
35-
}
36-
3733
public Uttak() {
3834
}
3935

0 commit comments

Comments
 (0)