From 8431518ee1a9c36101431f711c7cdc76ff37e573 Mon Sep 17 00:00:00 2001 From: Dragutin Vujovic Date: Thu, 6 Oct 2022 16:08:47 +0200 Subject: [PATCH 1/2] Bug fix: manglende grunn til at oppfolging ikke kan avsluttes --- .../controller/response/AvslutningStatus.java | 1 + .../domain/AvslutningStatusData.java | 1 + .../veilarboppfolging/service/OppfolgingService.java | 12 ++++++++---- .../no/nav/veilarboppfolging/utils/DtoMappers.java | 9 ++++++--- 4 files changed, 16 insertions(+), 7 deletions(-) diff --git a/src/main/java/no/nav/veilarboppfolging/controller/response/AvslutningStatus.java b/src/main/java/no/nav/veilarboppfolging/controller/response/AvslutningStatus.java index 09117f2a5..b779e7c16 100644 --- a/src/main/java/no/nav/veilarboppfolging/controller/response/AvslutningStatus.java +++ b/src/main/java/no/nav/veilarboppfolging/controller/response/AvslutningStatus.java @@ -18,5 +18,6 @@ public class AvslutningStatus { public boolean harYtelser; public boolean underKvp; public LocalDate inaktiveringsDato; + public boolean erSykmeldtMedArbeidsgiver; } diff --git a/src/main/java/no/nav/veilarboppfolging/domain/AvslutningStatusData.java b/src/main/java/no/nav/veilarboppfolging/domain/AvslutningStatusData.java index 4943c53f8..4b1b6b0d9 100644 --- a/src/main/java/no/nav/veilarboppfolging/domain/AvslutningStatusData.java +++ b/src/main/java/no/nav/veilarboppfolging/domain/AvslutningStatusData.java @@ -13,4 +13,5 @@ public class AvslutningStatusData { public final boolean harYtelser; public final boolean underKvp; public final LocalDate inaktiveringsDato; + public final boolean erSykmeldtMedArbeidsgiver; } \ No newline at end of file diff --git a/src/main/java/no/nav/veilarboppfolging/service/OppfolgingService.java b/src/main/java/no/nav/veilarboppfolging/service/OppfolgingService.java index 334e583b8..94fb33ae3 100644 --- a/src/main/java/no/nav/veilarboppfolging/service/OppfolgingService.java +++ b/src/main/java/no/nav/veilarboppfolging/service/OppfolgingService.java @@ -36,7 +36,6 @@ import static java.util.stream.Collectors.toList; import static no.nav.veilarboppfolging.config.ApplicationConfig.SYSTEM_USER_NAME; import static no.nav.veilarboppfolging.utils.ArenaUtils.*; -import static no.nav.veilarboppfolging.utils.KvpUtils.sjekkTilgangGittKvp; @Slf4j @Service @@ -392,9 +391,9 @@ private OppfolgingStatusData getOppfolgingStatusData(Fnr fnr) { long kvpId = kvpRepository.gjeldendeKvp(aktorId); boolean harSkrivetilgangTilBruker = !kvpService.erUnderKvp(kvpId) || authService.harTilgangTilEnhet( - kvpRepository.hentKvpPeriode(kvpId) - .orElseThrow() - .getEnhet() + kvpRepository.hentKvpPeriode(kvpId) + .orElseThrow() + .getEnhet() ); Boolean erInaktivIArena = maybeArenaOppfolging.map(ao -> erIserv(ao.getFormidlingsgruppe())).orElse(null); @@ -448,6 +447,10 @@ private AvslutningStatusData getAvslutningStatus(Fnr fnr) { .map(status -> ArenaUtils.erUnderOppfolging(status.getFormidlingsgruppe(), status.getServicegruppe())) .orElse(false); + boolean erSykmeldtMedArbeidsgiver = maybeArenaOppfolging + .map(status -> erSykmeldtMedArbeidsgiver(status)) + .orElse(false); + LocalDate inaktiveringsDato = maybeArenaOppfolging .map(ArenaOppfolgingTilstand::getInaktiveringsdato) .orElse(null); @@ -458,6 +461,7 @@ private AvslutningStatusData getAvslutningStatus(Fnr fnr) { .harYtelser(ytelserOgAktiviteterService.harPagaendeYtelse(fnr)) .underKvp(kvpService.erUnderKvp(aktorId)) .inaktiveringsDato(inaktiveringsDato) + .erSykmeldtMedArbeidsgiver(erSykmeldtMedArbeidsgiver) .build(); } diff --git a/src/main/java/no/nav/veilarboppfolging/utils/DtoMappers.java b/src/main/java/no/nav/veilarboppfolging/utils/DtoMappers.java index a1a6f1298..5583e38e7 100644 --- a/src/main/java/no/nav/veilarboppfolging/utils/DtoMappers.java +++ b/src/main/java/no/nav/veilarboppfolging/utils/DtoMappers.java @@ -5,10 +5,12 @@ import no.nav.veilarboppfolging.controller.response.*; import no.nav.veilarboppfolging.domain.AvslutningStatusData; import no.nav.veilarboppfolging.domain.OppfolgingStatusData; -import no.nav.veilarboppfolging.repository.entity.*; +import no.nav.veilarboppfolging.repository.entity.KvpPeriodeEntity; +import no.nav.veilarboppfolging.repository.entity.MaalEntity; +import no.nav.veilarboppfolging.repository.entity.OppfolgingsperiodeEntity; +import no.nav.veilarboppfolging.repository.entity.VeilederTilordningEntity; import java.util.Collections; -import java.util.Optional; import static java.util.Optional.ofNullable; import static java.util.stream.Collectors.toList; @@ -45,7 +47,8 @@ public static AvslutningStatus tilDto(AvslutningStatusData avslutningStatusData) avslutningStatusData.underOppfolging, avslutningStatusData.harYtelser, avslutningStatusData.underKvp, - avslutningStatusData.inaktiveringsDato + avslutningStatusData.inaktiveringsDato, + avslutningStatusData.erSykmeldtMedArbeidsgiver ); } From a2c1a81f06c7811cba24e18e3c62c3ea96118d2e Mon Sep 17 00:00:00 2001 From: Dragutin Vujovic Date: Fri, 7 Oct 2022 09:51:09 +0200 Subject: [PATCH 2/2] Code changes based on code review --- .../controller/response/AvslutningStatus.java | 2 +- .../nav/veilarboppfolging/domain/AvslutningStatusData.java | 2 +- .../no/nav/veilarboppfolging/service/OppfolgingService.java | 6 +----- .../java/no/nav/veilarboppfolging/utils/DtoMappers.java | 2 +- 4 files changed, 4 insertions(+), 8 deletions(-) diff --git a/src/main/java/no/nav/veilarboppfolging/controller/response/AvslutningStatus.java b/src/main/java/no/nav/veilarboppfolging/controller/response/AvslutningStatus.java index b779e7c16..f94eee6b7 100644 --- a/src/main/java/no/nav/veilarboppfolging/controller/response/AvslutningStatus.java +++ b/src/main/java/no/nav/veilarboppfolging/controller/response/AvslutningStatus.java @@ -18,6 +18,6 @@ public class AvslutningStatus { public boolean harYtelser; public boolean underKvp; public LocalDate inaktiveringsDato; - public boolean erSykmeldtMedArbeidsgiver; + public boolean erIserv; } diff --git a/src/main/java/no/nav/veilarboppfolging/domain/AvslutningStatusData.java b/src/main/java/no/nav/veilarboppfolging/domain/AvslutningStatusData.java index 4b1b6b0d9..70cd54071 100644 --- a/src/main/java/no/nav/veilarboppfolging/domain/AvslutningStatusData.java +++ b/src/main/java/no/nav/veilarboppfolging/domain/AvslutningStatusData.java @@ -13,5 +13,5 @@ public class AvslutningStatusData { public final boolean harYtelser; public final boolean underKvp; public final LocalDate inaktiveringsDato; - public final boolean erSykmeldtMedArbeidsgiver; + public final boolean erIserv; } \ No newline at end of file diff --git a/src/main/java/no/nav/veilarboppfolging/service/OppfolgingService.java b/src/main/java/no/nav/veilarboppfolging/service/OppfolgingService.java index 94fb33ae3..ed2903ffa 100644 --- a/src/main/java/no/nav/veilarboppfolging/service/OppfolgingService.java +++ b/src/main/java/no/nav/veilarboppfolging/service/OppfolgingService.java @@ -447,10 +447,6 @@ private AvslutningStatusData getAvslutningStatus(Fnr fnr) { .map(status -> ArenaUtils.erUnderOppfolging(status.getFormidlingsgruppe(), status.getServicegruppe())) .orElse(false); - boolean erSykmeldtMedArbeidsgiver = maybeArenaOppfolging - .map(status -> erSykmeldtMedArbeidsgiver(status)) - .orElse(false); - LocalDate inaktiveringsDato = maybeArenaOppfolging .map(ArenaOppfolgingTilstand::getInaktiveringsdato) .orElse(null); @@ -461,7 +457,7 @@ private AvslutningStatusData getAvslutningStatus(Fnr fnr) { .harYtelser(ytelserOgAktiviteterService.harPagaendeYtelse(fnr)) .underKvp(kvpService.erUnderKvp(aktorId)) .inaktiveringsDato(inaktiveringsDato) - .erSykmeldtMedArbeidsgiver(erSykmeldtMedArbeidsgiver) + .erIserv(erIserv) .build(); } diff --git a/src/main/java/no/nav/veilarboppfolging/utils/DtoMappers.java b/src/main/java/no/nav/veilarboppfolging/utils/DtoMappers.java index 5583e38e7..c5a2a0a0e 100644 --- a/src/main/java/no/nav/veilarboppfolging/utils/DtoMappers.java +++ b/src/main/java/no/nav/veilarboppfolging/utils/DtoMappers.java @@ -48,7 +48,7 @@ public static AvslutningStatus tilDto(AvslutningStatusData avslutningStatusData) avslutningStatusData.harYtelser, avslutningStatusData.underKvp, avslutningStatusData.inaktiveringsDato, - avslutningStatusData.erSykmeldtMedArbeidsgiver + avslutningStatusData.erIserv ); }