Proper validation of prosessfullmektig in mottak.

Validate address in fullmektig update. Co-authored-by: Andreas Jonsson <[email protected]>
navikt · Jan 20, 2025 · ef6805e · ef6805e
1 parent adab4ef
commit ef6805e
Show file tree

Hide file tree

Showing 7 changed files with 60 additions and 14 deletions.
diff --git a/src/main/kotlin/no/nav/klage/dokument/api/view/DokumentUnderArbeidInput.kt b/src/main/kotlin/no/nav/klage/dokument/api/view/DokumentUnderArbeidInput.kt
@@ -1,6 +1,7 @@
 package no.nav.klage.dokument.api.view
 
 import com.fasterxml.jackson.databind.JsonNode
+import no.nav.klage.dokument.exceptions.AddressValidationException
 import java.time.LocalDate
 import java.util.*
 
@@ -70,4 +71,14 @@ data class AddressInput(
     val adresselinje3: String?,
     val landkode: String,
     val postnummer: String?,
-)
+) {
+    fun validateAddress() {
+        if (landkode == "NO") {
+            if (postnummer == null) {
+                throw AddressValidationException("Trenger postnummer for norske adresser.")
+            }
+        } else if (adresselinje1 == null) {
+            throw AddressValidationException("Trenger adresselinje1 for utenlandske adresser.")
+        }
+    }
+}
diff --git a/src/main/kotlin/no/nav/klage/dokument/exceptions/DokumentExceptions.kt b/src/main/kotlin/no/nav/klage/dokument/exceptions/DokumentExceptions.kt
@@ -4,6 +4,8 @@ import no.nav.klage.dokument.api.view.DocumentValidationResponse
 
 class DokumentValidationException(msg: String) : RuntimeException(msg)
 
+class AddressValidationException(msg: String) : RuntimeException(msg)
+
 class SmartDocumentValidationException(msg: String, val errors: List<DocumentValidationResponse>) : RuntimeException(msg)
 
 class SvarbrevPreviewException(msg: String) : RuntimeException(msg)

diff --git a/src/main/kotlin/no/nav/klage/dokument/service/DokumentUnderArbeidService.kt b/src/main/kotlin/no/nav/klage/dokument/service/DokumentUnderArbeidService.kt
@@ -5,7 +5,6 @@ import io.micrometer.core.instrument.MeterRegistry
 import jakarta.servlet.http.HttpServletRequest
 import no.nav.klage.dokument.api.mapper.DokumentMapper
 import no.nav.klage.dokument.api.view.*
-import no.nav.klage.dokument.api.view.JournalfoertDokumentReference
 import no.nav.klage.dokument.api.view.Mottaker
 import no.nav.klage.dokument.domain.PDFDocument
 import no.nav.klage.dokument.domain.dokumenterunderarbeid.*
@@ -833,15 +832,7 @@ class DokumentUnderArbeidService(
                     throw DokumentValidationException("Ugyldig landkode: ${mottaker.overriddenAddress.landkode}")
                 }
 
-                if (mottaker.overriddenAddress.landkode == "NO") {
-                    if (mottaker.overriddenAddress.postnummer == null) {
-                        throw DokumentValidationException("Trenger postnummer for norske adresser.")
-                    }
-                } else {
-                    if (mottaker.overriddenAddress.adresselinje1 == null) {
-                        throw DokumentValidationException("Trenger adresselinje1 for utenlandske adresser.")
-                    }
-                }
+                mottaker.overriddenAddress.validateAddress()
             }
         }
 

diff --git a/src/main/kotlin/no/nav/klage/oppgave/api/view/OversendtKlageAnkeV4.kt b/src/main/kotlin/no/nav/klage/oppgave/api/view/OversendtKlageAnkeV4.kt
@@ -6,6 +6,7 @@ import no.nav.klage.kodeverk.Type
 import no.nav.klage.kodeverk.hjemmel.Hjemmel
 import no.nav.klage.kodeverk.ytelse.Ytelse
 import no.nav.klage.oppgave.domain.klage.*
+import no.nav.klage.oppgave.exceptions.OversendtKlageNotValidException
 import java.time.LocalDate
 import java.time.LocalDateTime
 import java.util.*
@@ -204,7 +205,17 @@ data class OversendtAdresse(
         description = "ISO 3166-1 alpha-2 kode. F.eks. NO for Norge."
     )
     val land: String,
-)
+) {
+    fun validateAddress() {
+        if (land == "NO") {
+            if (postnummer == null) {
+                throw OversendtKlageNotValidException("Trenger postnummer for norske adresser.")
+            }
+        } else if (adresselinje1 == null) {
+            throw OversendtKlageNotValidException("Trenger adresselinje1 for utenlandske adresser.")
+        }
+    }
+}
 
 data class OversendtPart(
     @Schema(

diff --git a/src/main/kotlin/no/nav/klage/oppgave/config/ProblemHandlingControllerAdvice.kt b/src/main/kotlin/no/nav/klage/oppgave/config/ProblemHandlingControllerAdvice.kt
@@ -172,6 +172,13 @@ class ProblemHandlingControllerAdvice : ResponseEntityExceptionHandler() {
     ): ProblemDetail =
         create(HttpStatus.BAD_REQUEST, ex)
 
+    @ExceptionHandler
+    fun handleAddressValidationException(
+        ex: AddressValidationException,
+        request: NativeWebRequest
+    ): ProblemDetail =
+        create(HttpStatus.BAD_REQUEST, ex)
+
     @ExceptionHandler
     fun handleDocumentDoesNotExistException(
         ex: DocumentDoesNotExistException,

diff --git a/src/main/kotlin/no/nav/klage/oppgave/service/BehandlingService.kt b/src/main/kotlin/no/nav/klage/oppgave/service/BehandlingService.kt
@@ -1297,12 +1297,15 @@ class BehandlingService(
             if ((input.address != null && input.name == null) || (input.address == null && input.name != null) ) {
                 throw IllegalOperation("Both address or name must be set")
             }
+
+            if (input.address != null) {
+                input.address.validateAddress()
+            }
         }
 
         val behandling = getBehandlingForUpdate(
             behandlingId
         )
-
         val partId: PartId? = if (input?.identifikator == null) {
             null
         } else {
@@ -1317,7 +1320,7 @@ class BehandlingService(
                     val poststed = if (it.landkode == "NO") {
                         if (it.postnummer != null) {
                             kodeverkService.getPoststed(it.postnummer)
-                        } else null
+                        } else throw IllegalOperation("Postnummer must be set for Norwegian address")
                     } else null
                     Adresse(
                         adresselinje1 = it.adresselinje1,

diff --git a/src/main/kotlin/no/nav/klage/oppgave/service/MottakService.kt b/src/main/kotlin/no/nav/klage/oppgave/service/MottakService.kt
@@ -411,6 +411,27 @@ class MottakService(
         validateOptionalDateTimeNotInFuture(sakMottattKaTidspunkt, ::sakMottattKaTidspunkt.name)
         validateKildeReferanse(kildeReferanse)
         validateEnhet(forrigeBehandlendeEnhet)
+        prosessfullmektig?.let { validateProsessfullmektig(it) }
+    }
+
+    private fun validateProsessfullmektig(prosessfullmektig: OversendtProsessfullmektig) {
+        if (prosessfullmektig.id == null && prosessfullmektig.navn == null && prosessfullmektig.adresse == null) {
+            throw OversendtKlageNotValidException("Enten id eller navn og adresse må være satt.")
+        }
+
+        if (prosessfullmektig.id != null && (prosessfullmektig.adresse != null || prosessfullmektig.navn != null)) {
+            throw OversendtKlageNotValidException("Adresse og navn kan bare settes når id ikke er satt.")
+        }
+
+        if ((prosessfullmektig.adresse != null && prosessfullmektig.navn == null) || (prosessfullmektig.adresse == null && prosessfullmektig.navn != null)) {
+            throw OversendtKlageNotValidException("Både adresse og navn må være satt.")
+        }
+
+        if (prosessfullmektig.id != null) {
+            validatePartId(prosessfullmektig.id.toPartId())
+        } else {
+            prosessfullmektig.adresse!!.validateAddress()
+        }
     }
 
     fun CreateKlageBasedOnKabinInput.validate() {