Skip to content

Commit

Permalink
Konsekvent long/int - codelql-warning (#519)
Browse files Browse the repository at this point in the history 
* Konsekvent long/int - codelql-warning

* Litt mer sonar
  • Loading branch information
@jolarsen
jolarsen authored Jul 29, 2024
1 parent 679c829 commit 531f6f6
Show file tree
Hide file tree
Showing 8 changed files with 60 additions and 58 deletions.
7 changes: 4 additions & 3 deletions ...va/no/nav/folketrygdloven/kalkulator/adapter/vltilregelmodell/MapTilFordelingsmodell.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,10 @@ private static FordelAndelModell mapAndel(BeregningsgrunnlagPrStatusOgAndelDto b

private static boolean kanFordeleTilAndelen(BeregningsgrunnlagPrStatusOgAndelDto bgAndel, BeregningsgrunnlagInput input) {
var periode = bgAndel.getBeregningsgrunnlagPeriode().getPeriode();
if (bgAndel.getBgAndelArbeidsforhold().isPresent()) {
var ansattTidslinje = FinnArbeidsperiode.finnAnsettelseTidslinje(bgAndel.getBgAndelArbeidsforhold().get().getArbeidsgiver(),
bgAndel.getBgAndelArbeidsforhold().get().getArbeidsforholdRef(),
var arbeidsforhold = bgAndel.getBgAndelArbeidsforhold();
if (arbeidsforhold.isPresent()) {
var ansattTidslinje = FinnArbeidsperiode.finnAnsettelseTidslinje(arbeidsforhold.get().getArbeidsgiver(),
arbeidsforhold.get().getArbeidsforholdRef(),
input.getIayGrunnlag(), input.getSkjæringstidspunktForBeregning());
var aktuellPeriode = new LocalDateTimeline<>(periode.getFomDato(), periode.getTomDato(), Boolean.TRUE);
var erAnsattIPeriode = !ansattTidslinje.intersection(aktuellPeriode).isEmpty();
Expand Down
5 changes: 3 additions & 2 deletions ...lketrygdloven/kalkulator/adapter/vltilregelmodell/ytelse/OmsorgspengerGrunnlagMapper.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,9 @@ public YtelsesSpesifiktGrunnlag map(BeregningsgrunnlagDto beregningsgrunnlagDto,
if (ytelsespesifiktGrunnlag instanceof no.nav.folketrygdloven.kalkulator.input.OmsorgspengerGrunnlag omsorspengegrunnlag) {
List<UtbetalingsgradPrAktivitetDto> utbetalingsgradPrAktivitet = omsorspengegrunnlag.getUtbetalingsgradPrAktivitet();
var finnesArbeidsandelIkkeSøktOm = finnesArbeidsandelIkkeSøktOm(utbetalingsgradPrAktivitet, beregningsgrunnlagDto);
if (omsorspengegrunnlag.getBrukerSøkerPerioder().isPresent()) {
return new OmsorgspengerGrunnlag(finnesArbeidsandelIkkeSøktOm, !omsorspengegrunnlag.getBrukerSøkerPerioder().get().isEmpty());
var brukerSøkerPerioder = omsorspengegrunnlag.getBrukerSøkerPerioder();
if (brukerSøkerPerioder.isPresent()) {
return new OmsorgspengerGrunnlag(finnesArbeidsandelIkkeSøktOm, !brukerSøkerPerioder.get().isEmpty());
} else {
boolean harSøktFLEllerSN = utbetalingsgradPrAktivitet.stream()
.filter(this::erFrilansEllerNæring)
Expand Down
16 changes: 8 additions & 8 deletions ...in/java/no/nav/folketrygdloven/kalkulator/guitjenester/fakta/InntektForAndelTjeneste.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -32,16 +32,16 @@ private InntektForAndelTjeneste() {

static Optional<Beløp> finnSnittinntektForArbeidstakerIBeregningsperioden(InntektFilterDto filter, no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BeregningsgrunnlagPrStatusOgAndelDto andel) {
LocalDate tilDato = andel.getBeregningsperiodeTom();
long beregningsperiodeLengdeIMnd = finnHeleMåneder(andel.getBeregningsperiode());
var beregningsperiodeLengdeIMnd = finnHeleMåneder(andel.getBeregningsperiode());
if (beregningsperiodeLengdeIMnd == 0) {
return Optional.empty();
}
var totalBeløp = finnTotalbeløpIBeregningsperioden(filter, andel, tilDato, beregningsperiodeLengdeIMnd);
return Optional.of(totalBeløp).map(tb -> tb.divider(BigDecimal.valueOf(beregningsperiodeLengdeIMnd), 10, RoundingMode.HALF_EVEN));
}

private static int finnHeleMåneder(Intervall periode) {
int antallMåneder = 0;
private static long finnHeleMåneder(Intervall periode) {
var antallMåneder = 0L;
LocalDate date = periode.getFomDato().minusDays(1).with(TemporalAdjusters.lastDayOfMonth());
while (date.isBefore(periode.getTomDato())) {
antallMåneder++;
Expand All @@ -52,7 +52,7 @@ private InntektForAndelTjeneste() {


private static Beløp finnTotalbeløpIBeregningsperioden(InntektFilterDto filter, no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BeregningsgrunnlagPrStatusOgAndelDto andel, LocalDate tilDato,
Long beregningsperiodeLengdeIMnd) {
long beregningsperiodeLengdeIMnd) {
if (filter.isEmpty()) {
return Beløp.ZERO;
}
Expand All @@ -71,7 +71,7 @@ private InntektForAndelTjeneste() {
}
LocalDate fraDato = andel.getBeregningsperiodeFom();
LocalDate tilDato = andel.getBeregningsperiodeTom();
long beregningsperiodeLengdeIMnd = ChronoUnit.MONTHS.between(fraDato, tilDato.plusDays(1));
var beregningsperiodeLengdeIMnd = ChronoUnit.MONTHS.between(fraDato, tilDato.plusDays(1));
if (beregningsperiodeLengdeIMnd == 0) {
return Optional.empty();
}
Expand All @@ -89,9 +89,9 @@ private static InntektFilterDto finnInntekterForAndel(no.nav.folketrygdloven.kal
.filter(arbeidsgiver.get());
}

private static Beløp summerInntekterIBeregningsperioden(LocalDate tilDato, Collection<InntektspostDto> inntektsposter, Long beregningsperiodeLengdeIMnd) {
private static Beløp summerInntekterIBeregningsperioden(LocalDate tilDato, Collection<InntektspostDto> inntektsposter, long beregningsperiodeLengdeIMnd) {
Beløp totalBeløp = Beløp.ZERO;
for (int måned = 0; måned < beregningsperiodeLengdeIMnd; måned++) {
for (var måned = 0L; måned < beregningsperiodeLengdeIMnd; måned++) {
LocalDate dato = tilDato.minusMonths(måned);
var beløp = finnMånedsinntekt(inntektsposter, dato);
totalBeløp = totalBeløp.adder(beløp);
Expand All @@ -111,7 +111,7 @@ private static InntektFilterDto finnInntekterForAndel(no.nav.folketrygdloven.kal
if (!filter.isEmpty()) {
LocalDate fraDato = frilansAndel.getBeregningsperiodeFom();
LocalDate tilDato = frilansAndel.getBeregningsperiodeTom();
long beregningsperiodeLengdeIMnd = ChronoUnit.MONTHS.between(fraDato, tilDato.plusDays(1));
var beregningsperiodeLengdeIMnd = ChronoUnit.MONTHS.between(fraDato, tilDato.plusDays(1));
List<YrkesaktivitetDto> yrkesaktiviteter = finnYrkesaktiviteter(grunnlag, skjæringstidspunkt);
boolean erFrilanser = yrkesaktiviteter.stream().anyMatch(ya -> ArbeidType.FRILANSER.equals(ya.getArbeidType()));

Expand Down
8 changes: 4 additions & 4 deletions ...no/nav/folketrygdloven/kalkulator/guitjenester/BeregningsgrunnlagDtoTjenesteImplTest.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -229,17 +229,17 @@ public class BeregningsgrunnlagDtoTjenesteImplTest {
var beregningsgrunnlagPrStatusOgAndelDto = beregningsgrunnlagDto.getBeregningsgrunnlagPeriode().get(0).getBeregningsgrunnlagPrStatusOgAndel().get(0);
assertThat(beregningsgrunnlagPrStatusOgAndelDto).isNotNull();
assertThat(beregningsgrunnlagPrStatusOgAndelDto.getAktivitetStatus()).isEqualTo(AktivitetStatus.ARBEIDSTAKER);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.getSkalFastsetteGrunnlag()).isEqualTo(true);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.getSkalFastsetteGrunnlag()).isTrue();

var beregningsgrunnlagPrStatusOgAndelDto2 = beregningsgrunnlagDto.getBeregningsgrunnlagPeriode().get(0).getBeregningsgrunnlagPrStatusOgAndel().get(1);
assertThat(beregningsgrunnlagPrStatusOgAndelDto2).isNotNull();
assertThat(beregningsgrunnlagPrStatusOgAndelDto2.getAktivitetStatus()).isEqualTo(AktivitetStatus.FRILANSER);
assertThat(beregningsgrunnlagPrStatusOgAndelDto2.getSkalFastsetteGrunnlag()).isEqualTo(false);
assertThat(beregningsgrunnlagPrStatusOgAndelDto2.getSkalFastsetteGrunnlag()).isFalse();

var beregningsgrunnlagPrStatusOgAndelDto3 = beregningsgrunnlagDto.getBeregningsgrunnlagPeriode().get(0).getBeregningsgrunnlagPrStatusOgAndel().get(2);
assertThat(beregningsgrunnlagPrStatusOgAndelDto3).isNotNull();
assertThat(beregningsgrunnlagPrStatusOgAndelDto3.getAktivitetStatus()).isEqualTo(AktivitetStatus.SELVSTENDIG_NÆRINGSDRIVENDE);
assertThat(beregningsgrunnlagPrStatusOgAndelDto3.getSkalFastsetteGrunnlag()).isEqualTo(true);
assertThat(beregningsgrunnlagPrStatusOgAndelDto3.getSkalFastsetteGrunnlag()).isTrue();
}

@Test
Expand All @@ -253,7 +253,7 @@ public class BeregningsgrunnlagDtoTjenesteImplTest {
var beregningsgrunnlagPrStatusOgAndelDto = beregningsgrunnlagDto.getBeregningsgrunnlagPeriode().get(0).getBeregningsgrunnlagPrStatusOgAndel().get(0);
assertThat(beregningsgrunnlagPrStatusOgAndelDto).isNotNull();
assertThat(beregningsgrunnlagPrStatusOgAndelDto.getAktivitetStatus()).isEqualTo(AktivitetStatus.ARBEIDSTAKER);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.getSkalFastsetteGrunnlag()).isEqualTo(true);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.getSkalFastsetteGrunnlag()).isTrue();
}

private KoblingReferanse lagReferanseMedStp(KoblingReferanse koblingReferanse) {
Expand Down
32 changes: 16 additions & 16 deletions ...loven/kalkulator/guitjenester/fakta/BeregningsgrunnlagPrStatusOgAndelDtoTjenesteTest.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,9 @@ public class BeregningsgrunnlagPrStatusOgAndelDtoTjenesteTest {
var beregningsgrunnlagPrStatusOgAndelDto = tjeneste.lagBeregningsgrunnlagPrStatusOgAndelDto(input,
grunnlag.getBeregningsgrunnlagHvisFinnes().get().getBeregningsgrunnlagPerioder().get(0).getBeregningsgrunnlagPrStatusOgAndelList());
// Assert
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(0).getSkalFastsetteGrunnlag()).isEqualTo(false);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(1).getSkalFastsetteGrunnlag()).isEqualTo(false);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(2).getSkalFastsetteGrunnlag()).isEqualTo(true);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(0).getSkalFastsetteGrunnlag()).isFalse();
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(1).getSkalFastsetteGrunnlag()).isFalse();
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(2).getSkalFastsetteGrunnlag()).isTrue();
}

@Test
Expand All @@ -97,8 +97,8 @@ public class BeregningsgrunnlagPrStatusOgAndelDtoTjenesteTest {
var beregningsgrunnlagPrStatusOgAndelDto = tjeneste.lagBeregningsgrunnlagPrStatusOgAndelDto(input,
grunnlag.getBeregningsgrunnlagHvisFinnes().get().getBeregningsgrunnlagPerioder().get(0).getBeregningsgrunnlagPrStatusOgAndelList());
// Assert
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(0).getSkalFastsetteGrunnlag()).isEqualTo(true);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(1).getSkalFastsetteGrunnlag()).isEqualTo(true);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(0).getSkalFastsetteGrunnlag()).isTrue();
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(1).getSkalFastsetteGrunnlag()).isTrue();
}

@Test
Expand All @@ -122,9 +122,9 @@ public class BeregningsgrunnlagPrStatusOgAndelDtoTjenesteTest {
var beregningsgrunnlagPrStatusOgAndelDto = tjeneste.lagBeregningsgrunnlagPrStatusOgAndelDto(input,
grunnlag.getBeregningsgrunnlagHvisFinnes().get().getBeregningsgrunnlagPerioder().get(0).getBeregningsgrunnlagPrStatusOgAndelList());
// Assert
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(0).getSkalFastsetteGrunnlag()).isEqualTo(false);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(1).getSkalFastsetteGrunnlag()).isEqualTo(false);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(2).getSkalFastsetteGrunnlag()).isEqualTo(true);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(0).getSkalFastsetteGrunnlag()).isFalse();
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(1).getSkalFastsetteGrunnlag()).isFalse();
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(2).getSkalFastsetteGrunnlag()).isTrue();
}

@Test
Expand All @@ -146,7 +146,7 @@ public class BeregningsgrunnlagPrStatusOgAndelDtoTjenesteTest {
var beregningsgrunnlagPrStatusOgAndelDto = tjeneste.lagBeregningsgrunnlagPrStatusOgAndelDto(input,
grunnlag.getBeregningsgrunnlagHvisFinnes().get().getBeregningsgrunnlagPerioder().get(0).getBeregningsgrunnlagPrStatusOgAndelList());
// Assert
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(0).getSkalFastsetteGrunnlag()).isEqualTo(true);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(0).getSkalFastsetteGrunnlag()).isTrue();
}

@Test
Expand Down Expand Up @@ -185,8 +185,8 @@ public class BeregningsgrunnlagPrStatusOgAndelDtoTjenesteTest {
var beregningsgrunnlagPrStatusOgAndelDto = tjeneste.lagBeregningsgrunnlagPrStatusOgAndelDto(input,
grunnlag.getBeregningsgrunnlagHvisFinnes().get().getBeregningsgrunnlagPerioder().get(0).getBeregningsgrunnlagPrStatusOgAndelList());
// Assert
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(0).getSkalFastsetteGrunnlag()).isEqualTo(true);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(1).getSkalFastsetteGrunnlag()).isEqualTo(false);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(0).getSkalFastsetteGrunnlag()).isTrue();
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(1).getSkalFastsetteGrunnlag()).isFalse();
}

@Test
Expand All @@ -206,8 +206,8 @@ public class BeregningsgrunnlagPrStatusOgAndelDtoTjenesteTest {
var beregningsgrunnlagPrStatusOgAndelDto = tjeneste.lagBeregningsgrunnlagPrStatusOgAndelDto(input,
grunnlag.getBeregningsgrunnlagHvisFinnes().get().getBeregningsgrunnlagPerioder().get(0).getBeregningsgrunnlagPrStatusOgAndelList());
// Assert
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(0).getSkalFastsetteGrunnlag()).isEqualTo(true);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(1).getSkalFastsetteGrunnlag()).isEqualTo(true);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(0).getSkalFastsetteGrunnlag()).isTrue();
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(1).getSkalFastsetteGrunnlag()).isTrue();
}

@Test
Expand All @@ -231,9 +231,9 @@ public class BeregningsgrunnlagPrStatusOgAndelDtoTjenesteTest {
var beregningsgrunnlagPrStatusOgAndelDto = tjeneste.lagBeregningsgrunnlagPrStatusOgAndelDto(input,
grunnlag.getBeregningsgrunnlagHvisFinnes().get().getBeregningsgrunnlagPerioder().get(0).getBeregningsgrunnlagPrStatusOgAndelList());
// Assert
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(0).getSkalFastsetteGrunnlag()).isEqualTo(false);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(1).getSkalFastsetteGrunnlag()).isEqualTo(false);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(2).getSkalFastsetteGrunnlag()).isEqualTo(true);
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(0).getSkalFastsetteGrunnlag()).isFalse();
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(1).getSkalFastsetteGrunnlag()).isFalse();
assertThat(beregningsgrunnlagPrStatusOgAndelDto.get(2).getSkalFastsetteGrunnlag()).isTrue();
}

@Test
Expand Down
Loading

0 comments on commit 531f6f6

Please sign in to comment.