Alder på barn regel forbedring/forenkling #2453
Conversation
…KAL_IKKE_VURDERES
…ikke trenger noen if-sjekker
…an i stedet for SvarId|null
| barnId: UUID?, | ||
| ): Boolean { | ||
| val fødselsdato = metadata.barn.firstOrNull { it.id == barnId } | ||
| ?.personIdent |
There was a problem hiding this comment.
Burde man egentlige bruke fødselsdato fra PDL her, og ikke personIdent? Det er ikke sikkert personIdent -> fødselsdato er lik faktiskt fødselsdato
There was a problem hiding this comment.
Det høres jo rimelig ut - jeg vet ikke helt hvilke potensielle fallgruver som dukker opp hvis vi bytter til fødselsdato?
There was a problem hiding this comment.
Problemet er vel først og fremst at det ikke finnes noe fødselsdato i det objektet der, eller?
There was a problem hiding this comment.
Det er mulig, men det burde kunne legges til då :-)
src/main/kotlin/no/nav/familie/ef/sak/vilkår/regler/vilkår/AlderPåBarnRegel.kt
Outdated
Show resolved
Hide resolved
| * 2023 fyller 10 år - fullfører 4 trinn i juni | ||
| */ | ||
| fun harFullførtFjerdetrinn(fødselsdato: LocalDate, datoForBeregning: LocalDate = LocalDate.now()): Boolean { | ||
| return if (datoForBeregning.month >= Month.JUNE) { |
There was a problem hiding this comment.
Vet ikke helt hvorfor vi gjorde det på denne måten tidligere, men det blir vel litt forskjell her nå?
if (datoForBeregning.month.plus(1).value < 6) { // Legger til en sikkerhetsmargin på 1 mnd tilfelle de fyller år mens saken behandles
```
There was a problem hiding this comment.
Ja, forskjellen blir at man i den tidligere løsningen fikk at man "fullført fjerne trinn" hvis man beregner i mai, det året barnet skal fullføre, som på en måte høres rart ut? 👀
Hva mener man egentlige med "sikkerhetsmargin"?
There was a problem hiding this comment.
Synes dette er en positiv endring.
I norsk målestokk er det svært få som starter i første klasse før "normalen" og følgelig heller ikke noe vi klarer å fange opp via søknad eller andre steder. Men du vil heller ikke få SFO-tilbud som 5. klassing - så da har du isåfall et spesielt tilbud fordi du trenger ekstra oppfølging... Og da starter du sjelden på skolen ett år FØR tiden, mest sannsynlig ett år ETTER normaltid.
|
Kudos, SonarCloud Quality Gate passed!
|
|
… ikke i mai. Det gjør valideringen og automatisk vurdering litt strengere da den ekstra måneden "slakk" var ment som at en behandling kan være opprettet i mai, men fullført i juni, noe som vi nå går bort ifra. I norsk målestokk er det svært få som starter i første klasse før "normalen" og følgelig heller ikke noe som blir fanget opp via søknad eller andre steder. Men barn vil heller ikke få SFO-tilbud som 5. klassing - så da har barnet isåfall et spesielt tilbud fordi det trengs ekstra oppfølging... Og da starter du sjelden på skolen ett år FØR tiden, mest sannsynlig ett år ETTER normaltid. - Alder på barn kan ikke automatisk vurderes dersom barnId er null, noe som skjer ved migreringer da BehandlingBarn ikke er opprettet enda ved initiering av vilkår - Fjerner enhetstester på harfullførtfjerdetrinn da det allerede finnes cucumber-tester på dette - Laget enhetstester for AlderPåBarnRegel som sjekker at vilkårsresultat blir riktig
|
Hvorfor er denne endringen nødvendig? ✨
Tilleggsstønader skulle gjenbruke denne og følte at den hadde litt forbedringspotensiale.
Sjekk commit etter commit.
SKAL_IKKE_VURDERESharFullførtFjedetrinnvar vanskelig å skjønneNoe som dere burde vurdere:
AlderPåBarnRegelhar ingen tester, det burde den kanskje få?