Bump spring boot and refactor deprecations (#850)

* Bump spring boot and refactor deprecations * Legg til kotlin-spring plugin for å åpne alle Configuration klasser * Fix unintended merge of deps * Rydde vekk utkommentert kode * Null-safe håndtering i veilarbarenamapper etter sonar qualitygate * Null-safe håndtering i veilarbarenamapper etter sonar qualitygate --------- Co-authored-by: Sigurd Grøneng <[email protected]>
navikt · Nov 13, 2024 · 53ce286 · 53ce286
1 parent 4d96513
commit 53ce286
Show file tree

Hide file tree

Showing 37 changed files with 207 additions and 215 deletions.
diff --git a/build.gradle.kts b/build.gradle.kts
@@ -8,9 +8,10 @@ val _version: String by project
 
 plugins {
     id("java")
+    id("org.jetbrains.kotlin.plugin.spring") version "2.0.21"
     id("application")
     id("maven-publish")
-    kotlin("jvm") version "2.0.20"
+    kotlin("jvm") version "2.0.21"
     id("org.openapi.generator") version "6.4.0"
     id("com.github.davidmc24.gradle.plugin.avro") version "1.3.0"
     id("project-report")
@@ -33,15 +34,6 @@ kotlin {
 
 configurations.all {
     resolutionStrategy.failOnNonReproducibleResolution()
-    // Kan mest sannsynlig fjernes når vi oppgrader poao-tilgang og springboot
-//    resolutionStrategy {
-//        force("com.fasterxml.jackson.core:jackson-databind:2.16.0")
-//        force("com.fasterxml.jackson.core:jackson-core:2.16.0")
-//        force("com.fasterxml.jackson.core:jackson-annotations:2.16.0")
-//        force("com.fasterxml.jackson.core:jackson-datatype-jdk8:2.16.0")
-//        force("com.fasterxml.jackson.module:jackson-module-scala:2.16.0")
-//        force("com.fasterxml.jackson.module:jackson-module-scala_2.13:2.16.0")
-//    }
 }
 
 tasks.test {
@@ -182,10 +174,6 @@ dependencies {
     implementation("org.flywaydb:flyway-database-postgresql:10.15.2")
     implementation("org.postgresql:postgresql:42.7.3")
     implementation("com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:2.17.1")
-//    implementation("com.fasterxml.jackson.module:jackson-module-kotlin:2.17.0")
-
-    implementation("org.springframework.cloud:spring-cloud-starter-gateway-mvc:4.1.4")
-
     // Hvis det ønskes swagger doc, foreslås å bruke springdoc (springdoc-openapi-starter-webmvc-ui - se no.nav.fo.veilarbdialog.rest.SwaggerConfig for eksempelconfig)
     implementation("io.swagger.core.v3:swagger-annotations:2.2.8")
 

diff --git a/src/main/java/no/nav/veilarbaktivitet/aktivitet/AktivitetAppService.java b/src/main/java/no/nav/veilarbaktivitet/aktivitet/AktivitetAppService.java
@@ -1,7 +1,6 @@
 package no.nav.veilarbaktivitet.aktivitet;
 
 import lombok.RequiredArgsConstructor;
-import lombok.val;
 import no.nav.poao.dab.spring_auth.IAuthService;
 import no.nav.veilarbaktivitet.aktivitet.domain.AktivitetData;
 import no.nav.veilarbaktivitet.aktivitet.domain.AktivitetTransaksjonsType;
@@ -203,7 +202,7 @@ private void kanEndreAktivitetEtikettGuard(AktivitetData orginalAktivitet, Aktiv
 
     @Transactional
     public AktivitetData oppdaterStatus(AktivitetData aktivitet) {
-        val originalAktivitet = hentAktivitet(aktivitet.getId());
+        final var originalAktivitet = hentAktivitet(aktivitet.getId());
         kanEndreAktivitetGuard(originalAktivitet, aktivitet.getVersjon(), aktivitet.getAktorId());
 
         if (authService.erEksternBruker() && !TYPER_SOM_KAN_ENDRES_EKSTERNT.contains(originalAktivitet.getAktivitetType())) {
@@ -219,15 +218,15 @@ public AktivitetData oppdaterStatus(AktivitetData aktivitet) {
 
     @Transactional
     public AktivitetData oppdaterEtikett(AktivitetData aktivitet) {
-        val originalAktivitet = hentAktivitet(aktivitet.getId());
+        final var originalAktivitet = hentAktivitet(aktivitet.getId());
         kanEndreAktivitetEtikettGuard(originalAktivitet, aktivitet);
         aktivitetService.oppdaterEtikett(originalAktivitet, aktivitet);
         return aktivitetService.hentAktivitetMedForhaandsorientering(aktivitet.getId());
     }
 
     @Transactional
     public AktivitetData oppdaterReferat(AktivitetData aktivitet) {
-        val originalAktivitet = hentAktivitet(aktivitet.getId());
+        final var originalAktivitet = hentAktivitet(aktivitet.getId());
         kanEndreAktivitetGuard(originalAktivitet, aktivitet.getVersjon(), aktivitet.getAktorId());
 
         var oppdatertAktivtiet = aktivitetService.oppdaterReferat(

diff --git a/src/main/java/no/nav/veilarbaktivitet/aktivitet/AktivitetDataRowMapper.java b/src/main/java/no/nav/veilarbaktivitet/aktivitet/AktivitetDataRowMapper.java
@@ -1,6 +1,5 @@
 package no.nav.veilarbaktivitet.aktivitet;
 
-import lombok.val;
 import no.nav.veilarbaktivitet.aktivitet.domain.*;
 import no.nav.veilarbaktivitet.aktivitet.dto.KanalDTO;
 import no.nav.veilarbaktivitet.aktivitetskort.dto.AktivitetskortType;
@@ -29,10 +28,10 @@ public AktivitetData mapRow(ResultSet rs, int rowNum) throws SQLException {
     }
 
     public static AktivitetData mapAktivitet(ResultSet resultSet) throws SQLException {
-        val rs = new VeilarbAktivitetResultSet(resultSet);
-        val type = AktivitetTypeData.valueOf(rs.getString("aktivitet_type_kode"));
+        final var rs = new VeilarbAktivitetResultSet(resultSet);
+        final var type = AktivitetTypeData.valueOf(rs.getString("aktivitet_type_kode"));
 
-        val aktivitet = AktivitetData
+        final var aktivitet = AktivitetData
                 .builder()
                 .id(rs.getLong("aktivitet_id"))
                 .funksjonellId(Database.hentMaybeUUID(rs, "funksjonell_id"))
@@ -79,7 +78,7 @@ public static AktivitetData mapAktivitet(ResultSet resultSet) throws SQLExceptio
     }
 
     private static MoteData mapMoteData(ResultSet resultSet) throws SQLException {
-        val rs = new VeilarbAktivitetResultSet(resultSet);
+        final var rs = new VeilarbAktivitetResultSet(resultSet);
 
         return MoteData.builder()
                 .adresse(rs.getString("adresse"))
@@ -169,7 +168,7 @@ private static StillingFraNavData mapStillingFraNav(ResultSet resultSet) throws
     }
 
     private static EksternAktivitetData mapEksternAktivitetData(ResultSet resultSet) throws SQLException {
-        val rs = new VeilarbAktivitetResultSet(resultSet);
+        final var rs = new VeilarbAktivitetResultSet(resultSet);
         var arenaId = rs.getString("ARENA_ID");
         return new EksternAktivitetData(
             rs.getString("SOURCE"),

diff --git a/src/main/java/no/nav/veilarbaktivitet/aktivitet/AktivitetService.java b/src/main/java/no/nav/veilarbaktivitet/aktivitet/AktivitetService.java
@@ -1,7 +1,6 @@
 package no.nav.veilarbaktivitet.aktivitet;
 
 import lombok.AllArgsConstructor;
-import lombok.val;
 import no.nav.veilarbaktivitet.aktivitet.domain.*;
 import no.nav.veilarbaktivitet.avtalt_med_nav.AvtaltMedNavService;
 import no.nav.veilarbaktivitet.avtalt_med_nav.Forhaandsorientering;
@@ -104,10 +103,10 @@ public AktivitetData oppdaterStatus(AktivitetData originalAktivitet, AktivitetDa
     }
 
     public AktivitetData avsluttStillingFraNav(AktivitetData originalAktivitet, Ident endretAv) {
-        val originalStillingFraNav = originalAktivitet.getStillingFraNavData();
-        val nyStillingFraNav = originalStillingFraNav.withLivslopsStatus(LivslopsStatus.AVBRUTT_AV_SYSTEM);
+        final var originalStillingFraNav = originalAktivitet.getStillingFraNavData();
+        final var nyStillingFraNav = originalStillingFraNav.withLivslopsStatus(LivslopsStatus.AVBRUTT_AV_SYSTEM);
 
-        val nyAktivitet = originalAktivitet
+        final var nyAktivitet = originalAktivitet
                 .toBuilder()
                 .endretAv(endretAv.ident())
                 .endretAvType(endretAv.identType().toInnsender())
@@ -122,10 +121,10 @@ public AktivitetData avsluttStillingFraNav(AktivitetData originalAktivitet, Iden
     }
 
     public void oppdaterEtikett(AktivitetData originalAktivitet, AktivitetData aktivitet) {
-        val nyEtikett = aktivitet.getStillingsSoekAktivitetData().getStillingsoekEtikett();
-        val originalStillingsAktivitet = originalAktivitet.getStillingsSoekAktivitetData();
-        val nyStillingsAktivitet = originalStillingsAktivitet.withStillingsoekEtikett(nyEtikett);
-        val nyAktivitet = originalAktivitet
+        final var nyEtikett = aktivitet.getStillingsSoekAktivitetData().getStillingsoekEtikett();
+        final var originalStillingsAktivitet = originalAktivitet.getStillingsSoekAktivitetData();
+        final var nyStillingsAktivitet = originalStillingsAktivitet.withStillingsoekEtikett(nyEtikett);
+        final var nyAktivitet = originalAktivitet
                 .toBuilder()
                 .endretAvType(aktivitet.getEndretAvType())
                 .endretAv(aktivitet.getEndretAv())
@@ -137,7 +136,7 @@ public void oppdaterEtikett(AktivitetData originalAktivitet, AktivitetData aktiv
     }
 
     public void oppdaterAktivitetFrist(AktivitetData originalAktivitet, AktivitetData aktivitetData) {
-        val oppdatertAktivitetMedNyFrist = originalAktivitet
+        final var oppdatertAktivitetMedNyFrist = originalAktivitet
                 .toBuilder()
                 .endretAvType(aktivitetData.getEndretAvType())
                 .endretAv(aktivitetData.getEndretAv())
@@ -149,7 +148,7 @@ public void oppdaterAktivitetFrist(AktivitetData originalAktivitet, AktivitetDat
     }
 
     public void oppdaterMoteTidStedOgKanal(AktivitetData originalAktivitet, AktivitetData aktivitetData) {
-        val oppdatertAktivitetMedNyFrist = originalAktivitet
+        final var oppdatertAktivitetMedNyFrist = originalAktivitet
                 .toBuilder()
                 .endretAvType(aktivitetData.getEndretAvType())
                 .endretAv(aktivitetData.getEndretAv())
@@ -169,9 +168,9 @@ public AktivitetData oppdaterReferat(
             AktivitetData originalAktivitet,
             AktivitetData aktivitetData
     ) {
-        val transaksjon = getReferatTransakjsonType(originalAktivitet, aktivitetData);
+        final var transaksjon = getReferatTransakjsonType(originalAktivitet, aktivitetData);
 
-        val merger = MappingUtils.merge(originalAktivitet, aktivitetData);
+        final var merger = MappingUtils.merge(originalAktivitet, aktivitetData);
         return aktivitetDAO.oppdaterAktivitet(originalAktivitet
                 .withEndretDato(aktivitetData.getEndretDato())
                 .withEndretAv(aktivitetData.getEndretAv())
@@ -183,7 +182,7 @@ public AktivitetData oppdaterReferat(
 
     private AktivitetTransaksjonsType getReferatTransakjsonType(AktivitetData originalAktivitet,
                                                                 AktivitetData aktivitetData) {
-        val transaksjon = nullOrEmpty(originalAktivitet.getMoteData().getReferat())
+        final var transaksjon = nullOrEmpty(originalAktivitet.getMoteData().getReferat())
                 ? AktivitetTransaksjonsType.REFERAT_OPPRETTET : AktivitetTransaksjonsType.REFERAT_ENDRET;
 
         if (!originalAktivitet.getMoteData().isReferatPublisert() && aktivitetData.getMoteData().isReferatPublisert()) {
@@ -208,13 +207,13 @@ public void svarPaaKanCvDeles(AktivitetData originalAktivitet, AktivitetData akt
     }
 
     public AktivitetData oppdaterAktivitet(AktivitetData originalAktivitet, AktivitetData aktivitet) {
-        val blittAvtalt = originalAktivitet.isAvtalt() != aktivitet.isAvtalt();
+        final var blittAvtalt = originalAktivitet.isAvtalt() != aktivitet.isAvtalt();
         if (blittAvtalt) {
             throw new IllegalArgumentException(String.format("Kan ikke sette avtalt for aktivitetsid: %s gjennom oppdaterAktivitet", originalAktivitet.getId()));
         }
-        val transType = AktivitetTransaksjonsType.DETALJER_ENDRET;
-        val merger = MappingUtils.merge(originalAktivitet, aktivitet);
-        val result = aktivitetDAO.oppdaterAktivitet(originalAktivitet
+        final var transType = AktivitetTransaksjonsType.DETALJER_ENDRET;
+        final var merger = MappingUtils.merge(originalAktivitet, aktivitet);
+        final var result = aktivitetDAO.oppdaterAktivitet(originalAktivitet
                 .toBuilder()
                 .avsluttetKommentar(aktivitet.getAvsluttetKommentar())
                 .behandlingAktivitetData(merger.map(AktivitetData::getBehandlingAktivitetData).merge(this::mergeBehandlingAktivitetData))

diff --git a/src/main/java/no/nav/veilarbaktivitet/aktivitet/mappers/AktivitetDataMapperService.java b/src/main/java/no/nav/veilarbaktivitet/aktivitet/mappers/AktivitetDataMapperService.java
@@ -1,7 +1,6 @@
 package no.nav.veilarbaktivitet.aktivitet.mappers;
 
 import lombok.RequiredArgsConstructor;
-import lombok.val;
 import no.nav.common.client.aktoroppslag.AktorOppslagClient;
 import no.nav.common.types.identer.*;
 import no.nav.poao.dab.spring_auth.IAuthService;
@@ -35,19 +34,19 @@ private String getEndretAv(Id bruker) {
     }
 
     public AktivitetData mapTilAktivitetData(AktivitetDTO aktivitetDTO) {
-        val id = Optional.ofNullable(aktivitetDTO.getId())
+        final var id = Optional.ofNullable(aktivitetDTO.getId())
                 .filter(s -> !s.isEmpty())
                 .map(Long::parseLong)
                 .orElse(null);
-        val versjon = Optional.ofNullable(aktivitetDTO.getVersjon()).map(Long::parseLong).orElse(0L);
-        val aktivitetType = Helpers.Type.getData(aktivitetDTO.getType());
-        val innloggetBruker = authService.getLoggedInnUser();
-        val endretAvType = innloggetBruker instanceof EksternBrukerId ? Innsender.BRUKER : Innsender.NAV;
-        val endretAv = getEndretAv(innloggetBruker);
-        val aktorId = userInContext.getAktorId();
+        final var versjon = Optional.ofNullable(aktivitetDTO.getVersjon()).map(Long::parseLong).orElse(0L);
+        final var aktivitetType = Helpers.Type.getData(aktivitetDTO.getType());
+        final var innloggetBruker = authService.getLoggedInnUser();
+        final var endretAvType = innloggetBruker instanceof EksternBrukerId ? Innsender.BRUKER : Innsender.NAV;
+        final var endretAv = getEndretAv(innloggetBruker);
+        final var aktorId = userInContext.getAktorId();
         var kontorSperreEnhet = kvpService.getKontorSperreEnhet(aktorId);
 
-        val aktivitetData = AktivitetData
+        final var aktivitetData = AktivitetData
                 .builder()
                 .id(id)
                 .aktorId(aktorId)

diff --git a/src/main/java/no/nav/veilarbaktivitet/arena/VeilarbarenaClient.java b/src/main/java/no/nav/veilarbaktivitet/arena/VeilarbarenaClient.java
@@ -28,7 +28,7 @@ public class VeilarbarenaClient {
     private String veilarbarenaServiceUrl;
 
     public HealthStatus  ping() {
-        String uri = String.format("%s/veilarbarena/internal/selftest", veilarbarenaServiceUrl);
+        String uri = "%s/veilarbarena/internal/selftest".formatted(veilarbarenaServiceUrl);
         // This endpoint this not need auth and therfore uses baseClient
         var basicHttpClient = RestClient.baseClientBuilder().build();
         Request request = new Request.Builder()
@@ -46,7 +46,7 @@ public HealthStatus  ping() {
 
     @Timed
     public Optional<AktiviteterDTO> hentAktiviteter(Person.Fnr fnr) {
-        String uri = String.format("%s/veilarbarena/api/arena/aktiviteter?fnr=%s", veilarbarenaServiceUrl, fnr.get());
+        String uri = "%s/veilarbarena/api/arena/aktiviteter?fnr=%s".formatted(veilarbarenaServiceUrl, fnr.get());
         Request request = new Request.Builder()
                 .url(uri)
                 .build();