Fiks håndtering av null

navikt · Feb 11, 2025 · 0ad8092 · 0ad8092
1 parent 0e48d4e
commit 0ad8092
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 8 deletions.
diff --git a/src/main/java/no/nav/veilarboppfolging/repository/OppfolgingsPeriodeRepository.java b/src/main/java/no/nav/veilarboppfolging/repository/OppfolgingsPeriodeRepository.java
@@ -1,6 +1,8 @@
 package no.nav.veilarboppfolging.repository;
 
 import no.nav.common.types.identer.AktorId;
+import no.nav.common.types.identer.NavIdent;
+import no.nav.veilarboppfolging.domain.StartetAvType;
 import no.nav.veilarboppfolging.oppfolgingsbruker.OppfolgingStartBegrunnelse;
 import no.nav.veilarboppfolging.oppfolgingsbruker.Oppfolgingsbruker;
 import no.nav.veilarboppfolging.repository.entity.OppfolgingsperiodeEntity;
@@ -40,7 +42,7 @@ public OppfolgingsPeriodeRepository(JdbcTemplate db, TransactionTemplate transac
 
     public void start(Oppfolgingsbruker oppfolgingsbruker) {
         transactor.executeWithoutResult((ignored) -> {
-            insert(oppfolgingsbruker.getAktorId(), oppfolgingsbruker.getOppfolgingStartBegrunnelse(), oppfolgingsbruker.getRegistrertAv(), String.valueOf(oppfolgingsbruker.getStartetAvType()));
+            insert(oppfolgingsbruker.getAktorId(), oppfolgingsbruker.getOppfolgingStartBegrunnelse(), oppfolgingsbruker.getRegistrertAv(), oppfolgingsbruker.getStartetAvType());
             setActive(oppfolgingsbruker.getAktorId());
         });
     }
@@ -87,11 +89,15 @@ public Optional<OppfolgingsperiodeEntity> hentGjeldendeOppfolgingsperiode(AktorI
         );
     }
 
-    private void insert(AktorId aktorId, OppfolgingStartBegrunnelse getOppfolgingStartBegrunnelse, String veileder, String startetAvType) {
+    private void insert(AktorId aktorId, OppfolgingStartBegrunnelse getOppfolgingStartBegrunnelse, NavIdent veileder, StartetAvType startetAvType) {
         db.update("" +
                         "INSERT INTO OPPFOLGINGSPERIODE(uuid, aktor_id, startDato, oppdatert, start_begrunnelse, startet_av, startet_av_type) " +
                         "VALUES (?, ?, CURRENT_TIMESTAMP, CURRENT_TIMESTAMP, ?, ?, ?)",
-                UUID.randomUUID().toString(), aktorId.get(), getOppfolgingStartBegrunnelse.name(), veileder, startetAvType);
+                UUID.randomUUID().toString(),
+                aktorId.get(),
+                getOppfolgingStartBegrunnelse.name(),
+                veileder != null ? veileder.get() : null,
+                startetAvType.name());
     }
 
     private void setActive(AktorId aktorId) {

diff --git a/src/main/kotlin/no/nav/veilarboppfolging/oppfolgingsbruker/Oppfolgingsbruker.kt b/src/main/kotlin/no/nav/veilarboppfolging/oppfolgingsbruker/Oppfolgingsbruker.kt
@@ -14,13 +14,13 @@ open class Oppfolgingsbruker(
     open val aktorId: AktorId,
     open val oppfolgingStartBegrunnelse: OppfolgingStartBegrunnelse,
     open val startetAvType: StartetAvType,
-    open val registrertAv: String? = null
+    open val registrertAv: NavIdent? = null
 ) {
 
     companion object {
         @JvmStatic
         fun manueltRegistrertBruker(aktorId: AktorId, navIdent: NavIdent): Oppfolgingsbruker {
-            return BrukerManueltRegistrertAvVeileder(aktorId, navIdent.get())
+            return BrukerManueltRegistrertAvVeileder(aktorId, navIdent)
         }
 
         @JvmStatic
@@ -33,7 +33,7 @@ open class Oppfolgingsbruker(
                 aktorId,
                 OppfolgingStartBegrunnelse.ARBEIDSSOKER_REGISTRERING,
                 startetAvType,
-                navIdent?.get()
+                navIdent
             )
         }
 
@@ -51,14 +51,14 @@ open class Oppfolgingsbruker(
 
 data class BrukerManueltRegistrertAvVeileder(
     override val aktorId: AktorId,
-    override val registrertAv: String
+    override val registrertAv: NavIdent
 ): Oppfolgingsbruker(aktorId, OppfolgingStartBegrunnelse.MANUELL_REGISTRERING_VEILEDER, StartetAvType.VEILEDER, registrertAv)
 
 data class Arbeidssoker(
     override val aktorId: AktorId,
     override val oppfolgingStartBegrunnelse: OppfolgingStartBegrunnelse,
     override val startetAvType: StartetAvType,
-    override val registrertAv: String? = null
+    override val registrertAv: NavIdent? = null
 ) : Oppfolgingsbruker(
     aktorId,
     oppfolgingStartBegrunnelse,