NPE i RoadObjectRequest::toMutable

[okkero]

RoadObjectRequest::toMutable kaster en NPE hvis allVersions ikke har blitt satt (f.eks ved å kalle withAllVersions på builderen).

Problemet bryter ned til bruk av Boolean wrapper-typen i feltene, mens parameteret til withAllVersions bruker boolean-primitiven. Så hvis et RoadObjectRequest-objekt har blitt bygget uten at allVersions har blitt satt, ender man opp med allVersions lik null. toMutable prøver så å kalle RoadObjectRequest.Builder::withAllVersions med RoadObjectRequest-objektets allVersions-verdi som umiddelbart konverteres til boolean, som ender opp med å kaste en NPE fordi null ikke er en gyldig boolean-verdi.

https://github.com/nvdb-vegdata/nvdb-api-client/blob/master/src/main/java/no/vegvesen/nvdbapi/client/clients/RoadObjectRequest.java#L532

[johannsl]

@okkero Tror du vi slipper unna med å bare bytte parameter typen til referansetypen Boolean?

Det kan jo potensielt brekke systemene til brukere, men det er jo ikke all verden å endre feks. true til Boolean.TRUE

[okkero]

Den løsningen skal være 100% bakoverkompatibel. Autoboxing tar seg av konverteringen fra true/false.

[johannsl]

ab914bf
https://bintray.com/nvdb-vegdata/vegdata/nvdb-read-api-v3-client/1.16.2