FIX: Fjerner all brukt av arbeidsperiode utenom visning i GUI (#499)

* FIX: Fjerner all brukt av arbeidsperiode utenom visning i GUI
navikt · Apr 29, 2024 · aeef7c4 · aeef7c4
1 parent 2b19347
commit aeef7c4
Show file tree

Hide file tree

Showing 6 changed files with 3 additions and 61 deletions.
diff --git a/.../nav/folketrygdloven/kalkulator/guitjenester/GraderingUtenBeregningsgrunnlagTjeneste.java b/.../nav/folketrygdloven/kalkulator/guitjenester/GraderingUtenBeregningsgrunnlagTjeneste.java
@@ -5,7 +5,6 @@
 import java.util.Objects;
 import java.util.Optional;
 
-import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BGAndelArbeidsforholdDto;
 import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BeregningsgrunnlagDto;
 import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BeregningsgrunnlagPeriodeDto;
 import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BeregningsgrunnlagPrStatusOgAndelDto;
@@ -14,7 +13,6 @@
 import no.nav.folketrygdloven.kalkulator.modell.gradering.AndelGradering.Gradering;
 import no.nav.folketrygdloven.kalkulator.modell.typer.Beløp;
 import no.nav.folketrygdloven.kalkulator.modell.typer.InternArbeidsforholdRefDto;
-import no.nav.folketrygdloven.kalkulator.tid.Intervall;
 
 public class GraderingUtenBeregningsgrunnlagTjeneste {
 
@@ -36,7 +34,7 @@ private static List<BeregningsgrunnlagPrStatusOgAndelDto> finnTilsvarendeAndelIT
         andelGradering.getGraderinger().forEach(gradering ->{
             Optional<BeregningsgrunnlagPeriodeDto> korrektBGPeriode = finnTilsvarendeBGPeriode(gradering, beregningsgrunnlag.getBeregningsgrunnlagPerioder());
             Optional<BeregningsgrunnlagPrStatusOgAndelDto> korrektBGAndel = korrektBGPeriode.flatMap(p -> finnTilsvarendeAndelIPeriode(andelGradering, p));
-            if (korrektBGAndel.isPresent() && harIkkeTilkjentBGEtterRedusering(korrektBGAndel.get()) && arbeidsforholdErAktivtIGraderingsperiode(gradering, korrektBGAndel.get())) {
+            if (korrektBGAndel.isPresent() && harIkkeTilkjentBGEtterRedusering(korrektBGAndel.get())) {
                 andeler.add(korrektBGAndel.get());
             }
         });
@@ -64,10 +62,4 @@ private static boolean bgAndelMatcherGraderingAndel(BeregningsgrunnlagPrStatusOg
     private static Optional<BeregningsgrunnlagPeriodeDto> finnTilsvarendeBGPeriode(Gradering gradering, List<BeregningsgrunnlagPeriodeDto> beregningsgrunnlagPerioder) {
         return beregningsgrunnlagPerioder.stream().filter(p -> gradering.getPeriode().overlapper(p.getPeriode())).findFirst();
     }
-
-    private static boolean arbeidsforholdErAktivtIGraderingsperiode(Gradering gradering, BeregningsgrunnlagPrStatusOgAndelDto andel) {
-        Optional<Intervall> arbeidsperiode = andel.getBgAndelArbeidsforhold().map(BGAndelArbeidsforholdDto::getArbeidsperiode);
-        return arbeidsperiode.map(ap -> ap.overlapper(gradering.getPeriode())).orElse(true);
-    }
-
 }
diff --git a/...no/nav/folketrygdloven/kalkulator/modell/beregningsgrunnlag/BGAndelArbeidsforholdDto.java b/...no/nav/folketrygdloven/kalkulator/modell/beregningsgrunnlag/BGAndelArbeidsforholdDto.java
@@ -10,7 +10,6 @@
 import no.nav.folketrygdloven.kalkulator.modell.typer.Beløp;
 import no.nav.folketrygdloven.kalkulator.modell.typer.InternArbeidsforholdRefDto;
 import no.nav.folketrygdloven.kalkulator.modell.typer.Refusjon;
-import no.nav.folketrygdloven.kalkulator.tid.Intervall;
 import no.nav.folketrygdloven.kalkulus.kodeverk.Hjemmel;
 import no.nav.folketrygdloven.kalkulus.kodeverk.Utfall;
 
@@ -73,13 +72,6 @@ public Optional<LocalDate> getArbeidsperiodeTom() {
         return Optional.ofNullable(arbeidsperiodeTom);
     }
 
-    public Intervall getArbeidsperiode() {
-        if (arbeidsperiodeTom == null) {
-            return Intervall.fraOgMed(arbeidsperiodeFom);
-        }
-        return Intervall.fraOgMedTilOgMed(arbeidsperiodeFom, arbeidsperiodeTom);
-    }
-
     public String getArbeidsforholdOrgnr() {
         return getArbeidsgiver().getOrgnr();
     }

diff --git a/...r/src/main/java/no/nav/folketrygdloven/kalkulator/steg/BeregningsgrunnlagVerifiserer.java b/...r/src/main/java/no/nav/folketrygdloven/kalkulator/steg/BeregningsgrunnlagVerifiserer.java
@@ -13,7 +13,6 @@
 import no.nav.folketrygdloven.kalkulator.avklaringsbehov.PerioderTilVurderingTjeneste;
 import no.nav.folketrygdloven.kalkulator.input.ForeldrepengerGrunnlag;
 import no.nav.folketrygdloven.kalkulator.input.YtelsespesifiktGrunnlag;
-import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BGAndelArbeidsforholdDto;
 import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BeregningsgrunnlagDto;
 import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BeregningsgrunnlagPeriodeDto;
 import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BeregningsgrunnlagPrStatusOgAndelDto;
@@ -71,15 +70,10 @@ private static void verifiserOpprettetAndel(BeregningsgrunnlagPrStatusOgAndelDto
             if (andel.getArbeidsforholdType().equals(OpptjeningAktivitetType.ARBEID)) {
                 verifiserOptionalPresent(andel.getBgAndelArbeidsforhold(), "BgAndelArbeidsforhold " + andel);
             }
-            if (andel.getKilde() == null || AndelKilde.PROSESS_START.equals(andel.getKilde()) || AndelKilde.SAKSBEHANDLER_KOFAKBER.equals(andel.getKilde())) {
+            if (AndelKilde.PROSESS_START.equals(andel.getKilde()) || AndelKilde.SAKSBEHANDLER_KOFAKBER.equals(andel.getKilde())) {
                 Objects.requireNonNull(andel.getBeregningsperiodeFom(), "BeregningsperiodeFom " + andel);
                 Objects.requireNonNull(andel.getBeregningsperiodeTom(), "BeregningsperiodeTom " + andel);
             }
-            if (andel.getBgAndelArbeidsforhold().isPresent()) {
-                BGAndelArbeidsforholdDto arbFor = andel.getBgAndelArbeidsforhold().get();
-                Objects.requireNonNull(arbFor.getArbeidsperiodeFom(), "arbeidsperiodeFom " + andel);
-                Objects.requireNonNull(arbFor.getArbeidsperiodeTom(), "arbeidsperiodeTom " + andel);
-            }
         }
     }
 

diff --git a/.../no/nav/folketrygdloven/kalkulator/ytelse/fp/GraderingUtenBeregningsgrunnlagTjeneste.java b/.../no/nav/folketrygdloven/kalkulator/ytelse/fp/GraderingUtenBeregningsgrunnlagTjeneste.java
@@ -5,7 +5,6 @@
 import java.util.Objects;
 import java.util.Optional;
 
-import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BGAndelArbeidsforholdDto;
 import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BeregningsgrunnlagDto;
 import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BeregningsgrunnlagPeriodeDto;
 import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BeregningsgrunnlagPrStatusOgAndelDto;
@@ -14,7 +13,6 @@
 import no.nav.folketrygdloven.kalkulator.modell.gradering.AndelGradering.Gradering;
 import no.nav.folketrygdloven.kalkulator.modell.typer.Beløp;
 import no.nav.folketrygdloven.kalkulator.modell.typer.InternArbeidsforholdRefDto;
-import no.nav.folketrygdloven.kalkulator.tid.Intervall;
 
 public class GraderingUtenBeregningsgrunnlagTjeneste {
 
@@ -36,7 +34,7 @@ private static List<BeregningsgrunnlagPrStatusOgAndelDto> finnTilsvarendeAndelIT
         andelGradering.getGraderinger().forEach(gradering ->{
             Optional<BeregningsgrunnlagPeriodeDto> korrektBGPeriode = finnTilsvarendeBGPeriode(gradering, beregningsgrunnlag.getBeregningsgrunnlagPerioder());
             Optional<BeregningsgrunnlagPrStatusOgAndelDto> korrektBGAndel = korrektBGPeriode.flatMap(p -> finnTilsvarendeAndelIPeriode(andelGradering, p));
-            if (korrektBGAndel.isPresent() && harIkkeTilkjentBGEtterRedusering(korrektBGAndel.get()) && arbeidsforholdErAktivtIGraderingsperiode(gradering, korrektBGAndel.get())) {
+            if (korrektBGAndel.isPresent() && harIkkeTilkjentBGEtterRedusering(korrektBGAndel.get())) {
                 andeler.add(korrektBGAndel.get());
             }
         });
@@ -64,9 +62,4 @@ private static boolean bgAndelMatcherGraderingAndel(BeregningsgrunnlagPrStatusOg
     private static Optional<BeregningsgrunnlagPeriodeDto> finnTilsvarendeBGPeriode(Gradering gradering, List<BeregningsgrunnlagPeriodeDto> beregningsgrunnlagPerioder) {
         return beregningsgrunnlagPerioder.stream().filter(p -> gradering.getPeriode().overlapper(p.getPeriode())).findFirst();
     }
-
-    private static boolean arbeidsforholdErAktivtIGraderingsperiode(Gradering gradering, BeregningsgrunnlagPrStatusOgAndelDto andel) {
-        Optional<Intervall> arbeidsperiode = andel.getBgAndelArbeidsforhold().map(BGAndelArbeidsforholdDto::getArbeidsperiode);
-        return arbeidsperiode.map(ap -> ap.overlapper(gradering.getPeriode())).orElse(true);
-    }
 }
diff --git a/...o/nav/folketrygdloven/kalkulator/ytelse/frisinn/BeregningsgrunnlagVerifisererFRISINN.java b/...o/nav/folketrygdloven/kalkulator/ytelse/frisinn/BeregningsgrunnlagVerifisererFRISINN.java
@@ -8,7 +8,6 @@
 import java.util.function.Consumer;
 
 import no.nav.folketrygdloven.kalkulator.KalkulatorException;
-import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BGAndelArbeidsforholdDto;
 import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BeregningsgrunnlagDto;
 import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BeregningsgrunnlagPeriodeDto;
 import no.nav.folketrygdloven.kalkulator.modell.beregningsgrunnlag.BeregningsgrunnlagPrStatusOgAndelDto;
@@ -54,11 +53,6 @@ private static void verifiserOpprettetAndel(BeregningsgrunnlagPrStatusOgAndelDto
             }
             Objects.requireNonNull(andel.getBeregningsperiodeFom(), "BeregningsperiodeFom " + andel.toString());
             Objects.requireNonNull(andel.getBeregningsperiodeTom(), "BeregningsperiodeTom " + andel.toString());
-            if (andel.getBgAndelArbeidsforhold().isPresent()) {
-                BGAndelArbeidsforholdDto arbFor = andel.getBgAndelArbeidsforhold().get();
-                Objects.requireNonNull(arbFor.getArbeidsperiodeFom(), "arbeidsperiodeFom " + andel.toString());
-                Objects.requireNonNull(arbFor.getArbeidsperiodeTom(), "arbeidsperiodeTom " + andel.toString());
-            }
         }
     }
 

diff --git a/...t/java/no/nav/folketrygdloven/kalkulator/GraderingUtenBeregningsgrunnlagTjenesteTest.java b/...t/java/no/nav/folketrygdloven/kalkulator/GraderingUtenBeregningsgrunnlagTjenesteTest.java
@@ -390,29 +390,6 @@ public class GraderingUtenBeregningsgrunnlagTjenesteTest {
         assertThat(harAndelerMedGraderingUtenGrunnlag).isFalse();
     }
 
-    @Test
-    void skalIkkeSlåUtNårDetManglerBGMenGraderingErUtenforArbeidsperiode() {
-        // Arrange
-        LocalDate skjæringstidspunkt = SKJÆRINGSTIDSPUNKT;
-        LocalDate graderingFom = skjæringstidspunkt;
-        LocalDate graderingTom = skjæringstidspunkt.plusMonths(4);
-
-        AktivitetGradering aktivitetGradering = new AktivitetGradering(AndelGradering.builder()
-                .medArbeidsgiver(arbeidsgiver)
-                .medStatus(AktivitetStatus.ARBEIDSTAKER)
-                .medGradering(graderingFom, graderingTom, 50)
-                .build());
-
-        var beregningsgrunnlagPeriode = lagBeregningsgrunnlagPeriode(skjæringstidspunkt, null);
-        lagBeregningsgrunnlagAndel(beregningsgrunnlagPeriode, AktivitetStatus.ARBEIDSTAKER, BigDecimal.ZERO, graderingFom.minusYears(3), graderingFom.minusDays(1));
-
-        // Act
-        boolean harAndelerMedGraderingUtenGrunnlag = harAndelerMedGraderingUtenGrunnlag(aktivitetGradering);
-
-        // Assert
-        assertThat(harAndelerMedGraderingUtenGrunnlag).isFalse();
-    }
-
     private List<BeregningsgrunnlagPrStatusOgAndelDto> finnAndelerMedGraderingUtenBG(BeregningsgrunnlagDto beregningsgrunnlag, AktivitetGradering aktivitetGradering) {
         return GraderingUtenBeregningsgrunnlagTjeneste.finnAndelerMedGraderingUtenBG(beregningsgrunnlag, aktivitetGradering);
     }