From 6ec67b2ba0593c3e13404a88e37969ceb915d63c Mon Sep 17 00:00:00 2001 From: malazalkoj Date: Fri, 6 Sep 2024 11:38:39 +0200 Subject: [PATCH] Remove commented code --- src/main/kotlin/no/nav/syfo/aareg/AaregClient.kt | 2 -- .../aareg/service/ArbeidsforholdController.kt | 3 --- .../syfo/narmesteleder/NarmesteLederController.kt | 15 +-------------- src/main/kotlin/no/nav/syfo/pdl/PdlClient.kt | 6 ------ .../nav/syfo/aareg/utils/AaregClientTestUtils.kt | 2 -- 5 files changed, 1 insertion(+), 27 deletions(-) diff --git a/src/main/kotlin/no/nav/syfo/aareg/AaregClient.kt b/src/main/kotlin/no/nav/syfo/aareg/AaregClient.kt index c799772..e6314c9 100644 --- a/src/main/kotlin/no/nav/syfo/aareg/AaregClient.kt +++ b/src/main/kotlin/no/nav/syfo/aareg/AaregClient.kt @@ -37,7 +37,6 @@ class AaregClient( LOG.info("AaregClient with scope: $scope") LOG.info("AaregClient with cleanedScope: $cleanedScope") metrikk.tellHendelse("call_aareg") - // check which function to use for getting token, or we need to use getOnBehalfOfToken val token = azureAdTokenClient.getSystemToken(cleanedScope) return try { @@ -61,7 +60,6 @@ class AaregClient( private fun entity(fnr: String, token: String): HttpEntity<*> { val headers = HttpHeaders() headers.add(HttpHeaders.AUTHORIZATION, bearerHeader(token)) - // headers.add(NAV_CONSUMER_TOKEN_HEADER, bearerHeader(token)) headers.add(NAV_PERSONIDENT_HEADER, fnr) return HttpEntity(headers) } diff --git a/src/main/kotlin/no/nav/syfo/aareg/service/ArbeidsforholdController.kt b/src/main/kotlin/no/nav/syfo/aareg/service/ArbeidsforholdController.kt index 1bdcbf3..3d9cc65 100644 --- a/src/main/kotlin/no/nav/syfo/aareg/service/ArbeidsforholdController.kt +++ b/src/main/kotlin/no/nav/syfo/aareg/service/ArbeidsforholdController.kt @@ -30,9 +30,6 @@ class ArbeidsforholdController( @RequestParam date: LocalDate, @RequestParam orgnummer: String ): ResponseEntity> { - /*val innloggetFnr = - TokenXUtil.validateTokenXClaims(contextHolder, oppfolgingsplanClientId).fnrFromIdportenTokenX().value -*/ log.info("Henter stillinger for aktorId $aktorId, orgnummer $orgnummer") val stillinger = arbeidsforholdService.arbeidstakersStillingerForOrgnummer(aktorId, date, orgnummer) log.info("Hentet ${stillinger.size} stillinger for aktorId $aktorId, orgnummer $orgnummer") diff --git a/src/main/kotlin/no/nav/syfo/narmesteleder/NarmesteLederController.kt b/src/main/kotlin/no/nav/syfo/narmesteleder/NarmesteLederController.kt index 2f65488..084ffe8 100644 --- a/src/main/kotlin/no/nav/syfo/narmesteleder/NarmesteLederController.kt +++ b/src/main/kotlin/no/nav/syfo/narmesteleder/NarmesteLederController.kt @@ -86,23 +86,10 @@ class NarmesteLederController @Autowired constructor( val innloggetIdent = TokenXUtil.validateTokenXClaims(contextHolder, oppfolgingsplanClientId) .fnrFromIdportenTokenX() .value + // Henting og logging av stillinger TO be removed when we use it from where it should be used LOG.info("Henter stillinger from ArbeidsforholdService for $innloggetIdent") val stillinger = arbeidsforholdService.arbeidstakersStillinger(innloggetIdent) LOG.info("Hentet alle stillinger from ArbeidsforholdService for $innloggetIdent: $stillinger") - /*LOG.info("Henter nærmeste leder for $innloggetIdent i virksomhet $virksomhetsnummer") - val stillingerIFlereVirksomhet = - arbeidsforholdService.arbeidstakersStillingerForOrgnummer(fnr, listOf(virksomhetsnummer)) - LOG.info("Hentet stillinger for $fnr i virksomhet $virksomhetsnummer: $stillingerIFlereVirksomhet") - LOG.info("Henter stillinger for $fnr i virksomhet $virksomhetsnummer") - val stillingerForVirksomhet = arbeidsforholdService.arbeidstakersStillingerForOrgnummer( - fnr, - LocalDate.now(), - virksomhetsnummer - ) - LOG.info( - "Hentet stillinger for $fnr i virksomhet $virksomhetsnummer: $stillingerForVirksomhet from date:" + - "${LocalDate.now()}" - )*/ val narmesteLedere = narmesteLederClient.alleLedereForSykmeldt(ansattFnr = innloggetIdent) LOG.info("Hentet nærmeste ledere for $innloggetIdent: $narmesteLedere") LOG.info("Hentet nærmeste ledere for $innloggetIdent: ${narmesteLedere.size}") diff --git a/src/main/kotlin/no/nav/syfo/pdl/PdlClient.kt b/src/main/kotlin/no/nav/syfo/pdl/PdlClient.kt index 75e5611..e3b644d 100644 --- a/src/main/kotlin/no/nav/syfo/pdl/PdlClient.kt +++ b/src/main/kotlin/no/nav/syfo/pdl/PdlClient.kt @@ -82,7 +82,6 @@ class PdlClient( val entity = createRequestEntity( PdlRequest(query ?: "", Variables(ident = ident, grupper = gruppe)) ) - // try { val pdlIdenter = RestTemplate().exchange( pdlUrl, HttpMethod.POST, @@ -106,11 +105,6 @@ class PdlClient( throw RestClientException("Error while requesting $gruppe from PDL") } } - /*} catch (exception: RestClientResponseException) { - metric.tellHendelse("call_pdl_fail") - LOG.error("Error from PDL with request-url: $pdlUrl", exception) - throw exception - }*/ } companion object { diff --git a/src/test/kotlin/no/nav/syfo/aareg/utils/AaregClientTestUtils.kt b/src/test/kotlin/no/nav/syfo/aareg/utils/AaregClientTestUtils.kt index 35210e7..4d1cfff 100644 --- a/src/test/kotlin/no/nav/syfo/aareg/utils/AaregClientTestUtils.kt +++ b/src/test/kotlin/no/nav/syfo/aareg/utils/AaregClientTestUtils.kt @@ -18,7 +18,6 @@ object AaregClientTestUtils { val PASSED_DATE: LocalDate = LocalDate.of(1970, 1, 2) const val YRKESKODE = "1234567" - // Unused code may will be used in ArbeidsforholdSErviceTest const val YRKESNAVN = "yrkesnavn" const val YRKESNAVN_CAPITALIZED = "Yrkesnavn" const val STILLINGSPROSENT = 50.0 @@ -39,7 +38,6 @@ object AaregClientTestUtils { ) } - // To be used later in ArbeidsforholdServiceTest fun validArbeidsforhold(): Arbeidsforhold { return mockArbeidsforhold(ORGNUMMER, Type.Organisasjon, VALID_DATE) }