Bruk string heller enn enum for å slippe at jsonValue automatisk gir …

[johannbm]

…kode

[jolarsen]

Dere kan jo vurdere å endre Enum.name til å bli lik Enum.kode
Da blir det enklere å kvitte seg med kode - som er planen på sikt.

[palfi]

Ev lag egen enum og legg den lokalt i FpSakInntektsmeldingDto med en mapping

[johannbm]

Jeg kjenner ikke historikken her godt nok til å forstå hvorfor man har flere permutasjoner av samme Enum heller, så her gjør jeg det dere tenker er best.

Først vil jeg fikse feilen, og ikke gjøre videre refaktoreringer som kan gi bugs.

Vet ikke om jeg har lyst til å innføre enda en enum i FpSakInntektsmeldingDto (som er lik, men med annen verdi som mappes til JsonValue).

Enum.name til å bli lik Enum.kode (basicly kvitte seg med kode) høres jo fristen ut, men kjenner ikke scope på hvordan det kan være breaking

[palfi]

Tror det er best å ikke legge en default, men bruke UKJENT -> throw new IllegalStateException("Kunne ikke mappe NaturalytelseType til FpSakInntektsmeldingDto.NaturalytelseType: " + naturalytelseType);

Da vil man få kompileringsfeil istf runtimeexception hvis det senere kommer nye typer naturytelser

[palfi]

ellers ok!

[johannbm]

Ja, godt poeng!