Forenkler beregningen av utbetalingsgrad og trekkdager #359
Conversation
espenwaaga
force-pushed
the
utbetalingsgrad/trekkdager/opprydding
branch
from
0a05a6e to
6eeccf9
Compare
| RegelGrunnlag regelGrunnlag, | ||
| SaldoUtregning saldoUtregning) { | ||
| var aktuellPeriode = fastsettePeriodeGrunnlag.getAktuellPeriode(); | ||
| FastsettePeriodeGrunnlag grunnlag, |
There was a problem hiding this comment.
Hvorfor ikke beholde fastsettePeriodeGrunnlag ? Så snubler man ikke så lett i grunnlagene :-)
| // Vi skal redusere søker i forhold til annenparts uttaksprosent slik at de til sammen har 100% uttaksprosent | ||
| // TODO: Sjekke via nare? Nå er det 2 sannheter | ||
| var redusertUttaksprosentPgaSamtidigUttakMedSamletUttak100 = SamtidigUttakUtil.kanRedusereUtbetalingsgradForTapende(fastsettePeriodeGrunnlag) | ||
| ? SamtidigUttaksprosent.HUNDRED.subtract(SamtidigUttakUtil.uttaksprosentAnnenpart(fastsettePeriodeGrunnlag)) |
There was a problem hiding this comment.
Hvorfor gå fra en ZERO-konvensjon til en null-konvensjon? Blir det ikke mer if/null-sjekker som eller kunne vært håndtert aritmetisk med add/subtract
| .map(k -> oppgittPeriode.kopiMedNyPeriode(oppgittPeriode.getFom(), k.minusDays(1))) | ||
| .orElse(oppgittPeriode); | ||
|
|
||
| // Vi skal redusere søker i forhold til annenparts uttaksprosent slik at de til sammen har 100% uttaksprosent | ||
| // TODO: Sjekke via nare? Nå er det 2 sannheter |
There was a problem hiding this comment.
Nja. Du har valget mellom at regler + utregning bruker samme util eller at utregning kjenner til regelstrukturen ved å grafse i Evaluation/EvaluationSummary (etter regler med gitt ID).
Hvis det holder med å sjekke hvilken løvnode så kan man se på EvaluationSummary.allOutcomes etter 1 og (kun 1)
Ellers er delegering å foretrekke (dvs utils).
There was a problem hiding this comment.
Det blir litt mye å sette seg inn alt som tas opp i en sleng (i innspurten) - spesielt siden det er flere endringer på en gang - både restrukturering, trekkdager, utbetaling, avklaring av samtidig vs gradering.
Vurder om "grunnlag" er for generelt når det er flere å velge mellom - lesbarhet.
Vurder å ta endringen flere runder:
- Rydde i parametre som sendes ned - fx ikke sende ned to grunnlag når det man trenger finnes i ett av dem
- Utbetalingsutregningene, evt med avklaring rundt samtidig samtidg + gradering
- Evt endring i trekkdager
Vil anbefale å beholde delegering framfor å lete gjennom Evaluations på jakt etter strengkonstanter i ID. Outcomes fra løvnode er greiere.
espenwaaga
force-pushed
the
utbetalingsgrad/trekkdager/opprydding
branch
from
bfd7017 to
03b72b4
Compare
espenwaaga
force-pushed
the
utbetalingsgrad/trekkdager/opprydding
branch
from
c4fe778 to
b1d10cb
Compare
| private static Trekkdager trekkdagerMedGradering(int trekkdagerUtenGradering, BigDecimal gradertArbeidstidsprosent) { | ||
| if (gradertArbeidstidsprosent.compareTo(BigDecimal.valueOf(100)) >= 0) { | ||
| return Trekkdager.ZERO; | ||
| if (skalUtbetale) { |
There was a problem hiding this comment.
Helt sikker på denne? Finnes noen utsettelser som skal trekke dager.
| return SamtidigUttaksprosent.HUNDRED.subtract(annenpartSamtidigUttaksprosent); | ||
|
|
||
| var utbetalingsgrad = aktiviteter.stream() | ||
| .map(UttakPeriodeAktivitet::getUtbetalingsgrad) |
There was a problem hiding this comment.
Vi bør nok ta en mental refresh-runde på samvirke søkt gradering og søkt samtidig uttak
No description provided.