Merge pull request #5324 from navikt/fix/public-360

Rename websaknummer -> sakarkivNummer

Author: fredrikpe

common/domain/src/main/kotlin/no/nav/mulighetsrommet/model/SakarkivNummer.kt

@@ -0,0 +1,15 @@
+package no.nav.mulighetsrommet.model
+
+import kotlinx.serialization.Serializable
+
+private val SAKARKIV_NUMMER_REGEX = "^(\\d{2,4})/(\\d+)$".toRegex()
+
+@Serializable
+@JvmInline
+value class SakarkivNummer(val value: String) {
+    init {
+        require(SAKARKIV_NUMMER_REGEX.matches(value)) {
+            "The format of 'sakarkiv_nummer' is invalid. Expected '{year}/{id}'."
+        }
+    }
+}

common/domain/src/main/kotlin/no/nav/mulighetsrommet/model/Websaknummer.kt

@@ -54,7 +54,7 @@ export function defaultAvtaleData(
       startDato: avtale?.startDato ? avtale.startDato : undefined,
       sluttDato: avtale?.sluttDato ? avtale.sluttDato : undefined,
     },
-    websaknummer: avtale?.websaknummer,
+    sakarkivNummer: avtale?.sakarkivNummer,
     prisbetingelser: avtale?.prisbetingelser ?? undefined,
     beskrivelse: avtale?.beskrivelse ?? null,
     faneinnhold: avtale?.faneinnhold ?? null,

frontend/mr-admin-flate/src/components/avtaler/AvtaleFormConst.tsx

@@ -81,7 +81,7 @@ export function AvtaleFormContainer({
       id: avtale?.id ?? uuidv4(),
       navEnheter: data.navEnheter.concat(data.navRegioner),
       avtalenummer: avtale?.avtalenummer || null,
-      websaknummer: data.websaknummer || null,
+      sakarkivNummer: data.sakarkivNummer || null,
       arrangor:
         data.arrangorHovedenhet && data.arrangorUnderenheter
           ? {

frontend/mr-admin-flate/src/components/avtaler/AvtaleFormContainer.tsx

@@ -128,13 +128,13 @@ export function AvtaleFormDetaljer({ tiltakstyper, ansatt, enheter, avtale }: Pr
             <TextField
               size="small"
               placeholder="åå/12345"
-              error={errors.websaknummer?.message}
+              error={errors.sakarkivNummer?.message}
               label={
                 <LabelWithHelpText
-                  label={avtaletekster.websaknummerLabel}
-                  helpTextTitle={avtaletekster.websaknummerHelpTextTitle}
+                  label={avtaletekster.sakarkivNummerLabel}
+                  helpTextTitle={avtaletekster.sakarkivNummerHelpTextTitle}
                 >
-                  I Websak skal det opprettes tre typer arkivsaker med egne saksnummer:
+                  I Public360 skal det opprettes tre typer arkivsaker med egne saksnummer:
                   <ol>
                     <li>En sak for hver anskaffelse.</li>
                     <li>En sak for kontrakt/avtale med hver leverandør (Avtalesaken).</li>
@@ -145,7 +145,7 @@ export function AvtaleFormDetaljer({ tiltakstyper, ansatt, enheter, avtale }: Pr
                   Det er <b>2) Saksnummeret til Avtalesaken</b> som skal refereres til herfra.
                 </LabelWithHelpText>
               }
-              {...register("websaknummer")}
+              {...register("sakarkivNummer")}
             />
           </HGrid>
         </FormGroup>

frontend/mr-admin-flate/src/components/avtaler/AvtaleFormDetaljer.tsx

@@ -14,8 +14,8 @@ export const avtaletekster = {
   prisOgBetalingLabel: "Pris- og betalingsbetingelser",
   administratorerForAvtalenLabel: "Administratorer for avtalen",
   ingenAdministratorerSattLabel: "Ingen administratorer satt for avtalen",
-  websaknummerLabel: "Saksnummer til Avtalesaken i Websak",
-  websaknummerHelpTextTitle: "Hvilket saksnummer er dette?",
+  sakarkivNummerLabel: "Saksnummer til Avtalesaken i Public360",
+  sakarkivNummerHelpTextTitle: "Hvilket saksnummer er dette?",
   fylkessamarbeidLabel: "Fylkessamarbeid",
   navRegionerLabel: "Nav-regioner",
   navEnheterLabel: "Nav-enheter (kontorer)",

frontend/mr-admin-flate/src/components/ledetekster/avtaleLedetekster.ts

@@ -50,7 +50,7 @@ export const AvtaleSchema = z
       customOpsjonsmodellNavn: z.string().optional().nullable(),
     }),
     administratorer: z.string().array().min(1, "Du må velge minst én administrator"),
-    websaknummer: z
+    sakarkivNummer: z
       .string()
       .nullable()
       .refine(
@@ -59,7 +59,7 @@ export const AvtaleSchema = z
           return /^\d{2}\/\d+$/.test(value);
         },
         {
-          message: "Websaknummer må være på formatet 'år/løpenummer'",
+          message: "Saksnummer må være på formatet 'år/løpenummer'",
         },
       ),
     prisbetingelser: z.string().optional(),
@@ -76,7 +76,7 @@ export const AvtaleSchema = z
   .superRefine((data, ctx) => {
     if (
       [Avtaletype.AVTALE, Avtaletype.RAMMEAVTALE].includes(data.avtaletype) &&
-      !data.websaknummer
+      !data.sakarkivNummer
     ) {
       ctx.addIssue({
         code: z.ZodIssueCode.custom,

frontend/mr-admin-flate/src/components/redaksjoneltInnhold/AvtaleSchema.ts

@@ -196,7 +196,7 @@ export const mockAvtaler: AvtaleDto[] = [
     },
     navn: "Avtale hos Kulinarisk akademi",
     avtalenummer: "2020#4929",
-    websaknummer: "24/12345",
+    sakarkivNummer: "24/12345",
     arrangor: {
       ...mockArrangorer.data[0],
       slettet: false,

frontend/mr-admin-flate/src/mocks/fixtures/mock_avtaler.ts

@@ -34,7 +34,7 @@ export function AvtaleDetaljer({ avtale }: Props) {
     startDato,
     sluttDato,
     administratorer,
-    websaknummer,
+    sakarkivNummer,
     kontorstruktur,
     arenaAnsvarligEnhet,
     arrangor,
@@ -53,8 +53,8 @@ export function AvtaleDetaljer({ avtale }: Props) {
 
         <Bolk aria-label="Eksterne referanser">
           <Metadata header={avtaletekster.avtalenummerLabel} verdi={avtalenummer} />
-          {websaknummer ? (
-            <Metadata header={avtaletekster.websaknummerLabel} verdi={websaknummer} />
+          {sakarkivNummer ? (
+            <Metadata header={avtaletekster.sakarkivNummerLabel} verdi={sakarkivNummer} />
           ) : null}
         </Bolk>
 

frontend/mr-admin-flate/src/pages/avtaler/AvtaleDetaljer.tsx

@@ -11,7 +11,7 @@ const fyllInnAvtale = async (page: Page) => {
   const heading = page.locator("h2");
   await test.expect(heading).toHaveText("Opprett ny avtale");
   await locateAndFillInput(page, "navn", "Testavtale fra Playwright");
-  await locateAndFillInput(page, "websaknummer", "24/123");
+  await locateAndFillInput(page, "sakarkivNummer", "24/123");
   await locateAndFillComboboxFirst(page, "input#tiltakstype", "AFT");
   await page.fill('.navds-form-field:has(label:text("Startdato")) input', "01.02.2025");
   await locateAndFillComboboxFirst(page, "input#tiltakstype", "AFT");

frontend/mr-admin-flate/tests/adminflate.avtale.spec.ts

@@ -33,7 +33,7 @@ data class AvtaleRequest(
     val tiltakstypeId: UUID,
     val arrangor: Arrangor?,
     val avtalenummer: String?,
-    val websaknummer: Websaknummer?,
+    val sakarkivNummer: SakarkivNummer?,
     @Serializable(with = LocalDateSerializer::class)
     val startDato: LocalDate,
     @Serializable(with = LocalDateSerializer::class)

mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/avtale/AvtaleRoutes.kt

@@ -239,7 +239,7 @@ private fun toAvtaleDbo(
         id = id,
         navn = navn,
         avtalenummer = avtalenummer,
-        websaknummer = websaknummer,
+        sakarkivNummer = sakarkivNummer,
         tiltakstypeId = tiltakstypeId,
         arrangor = arrangor,
         startDato = startDato,

mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/avtale/AvtaleService.kt

@@ -104,8 +104,8 @@ class AvtaleValidator(
                 )
             }
 
-            if (avtale.avtaletype.kreverWebsaknummer() && avtale.websaknummer == null) {
-                add(FieldError.of(AvtaleDbo::websaknummer, "Du må skrive inn Websaknummer til avtalesaken"))
+            if (avtale.avtaletype.kreverWebsaknummer() && avtale.sakarkivNummer == null) {
+                add(FieldError.of(AvtaleDbo::sakarkivNummer, "Du må skrive inn Websaknummer til avtalesaken"))
             }
 
             if (avtale.arrangor?.underenheter?.isEmpty() == true) {

mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/avtale/AvtaleValidator.kt

@@ -11,7 +11,7 @@ data class AvtaleDbo(
     val navn: String,
     val tiltakstypeId: UUID,
     val avtalenummer: String?,
-    val websaknummer: Websaknummer?,
+    val sakarkivNummer: SakarkivNummer?,
     // @todo: Make nullable
     val arrangor: Arrangor?,
     val startDato: LocalDate,

mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/avtale/db/AvtaleDbo.kt

@@ -42,7 +42,7 @@ class AvtaleQueries(private val session: Session) {
                 navn,
                 tiltakstype_id,
                 avtalenummer,
-                websaknummer,
+                sakarkiv_nummer,
                 arrangor_hovedenhet_id,
                 start_dato,
                 slutt_dato,
@@ -62,7 +62,7 @@ class AvtaleQueries(private val session: Session) {
                 :navn,
                 :tiltakstype_id::uuid,
                 :avtalenummer,
-                :websaknummer,
+                :sakarkiv_nummer,
                 :arrangor_hovedenhet_id,
                 :start_dato,
                 :slutt_dato,
@@ -81,7 +81,7 @@ class AvtaleQueries(private val session: Session) {
                 navn                        = excluded.navn,
                 tiltakstype_id              = excluded.tiltakstype_id,
                 avtalenummer                = excluded.avtalenummer,
-                websaknummer                = excluded.websaknummer,
+                sakarkiv_nummer             = excluded.sakarkiv_nummer,
                 arrangor_hovedenhet_id      = excluded.arrangor_hovedenhet_id,
                 start_dato                  = excluded.start_dato,
                 slutt_dato                  = excluded.slutt_dato,
@@ -439,7 +439,7 @@ class AvtaleQueries(private val session: Session) {
         "navn" to navn,
         "tiltakstype_id" to tiltakstypeId,
         "avtalenummer" to avtalenummer,
-        "websaknummer" to websaknummer?.value,
+        "sakarkiv_nummer" to sakarkivNummer?.value,
         "arrangor_hovedenhet_id" to arrangor?.hovedenhet,
         "start_dato" to startDato,
         "slutt_dato" to sluttDato,
@@ -540,7 +540,7 @@ class AvtaleQueries(private val session: Session) {
             id = uuid("id"),
             navn = string("navn"),
             avtalenummer = stringOrNull("avtalenummer"),
-            websaknummer = stringOrNull("websaknummer")?.let { Websaknummer(it) },
+            sakarkivNummer = stringOrNull("sakarkiv_nummer")?.let { SakarkivNummer(it) },
             startDato = startDato,
             sluttDato = sluttDato,
             opphav = ArenaMigrering.Opphav.valueOf(string("opphav")),

mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/avtale/db/AvtaleQueries.kt

@@ -27,7 +27,7 @@ data class AvtaleDto(
     val tiltakstype: Tiltakstype,
     val navn: String,
     val avtalenummer: String?,
-    val websaknummer: Websaknummer?,
+    val sakarkivNummer: SakarkivNummer?,
     val arrangor: ArrangorHovedenhet?,
     @Serializable(with = LocalDateSerializer::class)
     val startDato: LocalDate,
@@ -110,7 +110,7 @@ fun AvtaleDto.toDbo(): AvtaleDbo {
         navn = navn,
         tiltakstypeId = tiltakstype.id,
         avtalenummer = avtalenummer,
-        websaknummer = websaknummer,
+        sakarkivNummer = sakarkivNummer,
         arrangor = arrangor?.id?.let {
             AvtaleDbo.Arrangor(
                 hovedenhet = it,

mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/avtale/model/AvtaleDto.kt

@@ -132,7 +132,7 @@ class OkonomiBestillingService(
             arrangor = arrangor,
             kostnadssted = NavEnhetNummer(tilsagn.kostnadssted.enhetsnummer),
             // TODO: håndter avtalenummer fra p360, eller erstatter til Mercell
-            avtalenummer = avtale.websaknummer?.value,
+            avtalenummer = avtale.sakarkivNummer?.value,
             belop = tilsagn.beregning.output.belop,
             periode = tilsagn.periode,
             behandletAv = opprettelse.behandletAv.toOkonomiPart(),

mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/tilsagn/OkonomiBestillingService.kt

@@ -7,7 +7,7 @@ select avtale.id,
        avtale.fts,
        avtale.navn,
        coalesce(avtale.avtalenummer, avtale.lopenummer) as avtalenummer,
-       avtale.websaknummer,
+       avtale.sakarkiv_nummer,
        avtale.start_dato,
        avtale.slutt_dato,
        avtale.opsjon_maks_varighet,

mulighetsrommet-api/src/main/resources/db/migration/R__view_avtale_admin.sql

@@ -0,0 +1 @@
+alter table avtale rename column websaknummer to sakarkiv_nummer;

mulighetsrommet-api/src/main/resources/db/migration/V266__sakarkiv_lenke_rename.sql

@@ -2795,7 +2795,7 @@ components:
           type: string
         avtalenummer:
           type: string
-        websaknummer:
+        sakarkivNummer:
           type: [string, "null"]
         arrangor:
           $ref: "#/components/schemas/ArrangorHovedenhet"
@@ -2885,7 +2885,7 @@ components:
           format: UUID
         avtalenummer:
           type: [string, "null"]
-        websaknummer:
+        sakarkivNummer:
           type: [string, "null"]
         arrangor:
           anyOf:

mulighetsrommet-api/src/main/resources/web/openapi.yaml

@@ -54,7 +54,7 @@ class AvtaleValidatorTest : FunSpec({
             kontaktpersoner = emptyList(),
         ),
         avtalenummer = "123456",
-        websaknummer = Websaknummer("24/1234"),
+        sakarkivNummer = SakarkivNummer("24/1234"),
         startDato = LocalDate.now().minusDays(1),
         sluttDato = LocalDate.now().plusMonths(1),
         administratorer = listOf(NavAnsattFixture.ansatt1.navIdent),
@@ -303,22 +303,22 @@ class AvtaleValidatorTest : FunSpec({
         validator.validate(gruppeAmoOffentlig, null).shouldBeRight()
     }
 
-    test("Websak-referanse må være med når avtalen er avtale eller rammeavtale") {
+    test("SakarkivNummer må være med når avtalen er avtale eller rammeavtale") {
         val validator = createValidator()
 
-        val rammeavtale = AvtaleFixtures.oppfolging.copy(avtaletype = Avtaletype.Rammeavtale, websaknummer = null)
+        val rammeavtale = AvtaleFixtures.oppfolging.copy(avtaletype = Avtaletype.Rammeavtale, sakarkivNummer = null)
         validator.validate(rammeavtale, null).shouldBeLeft(
-            listOf(FieldError("/websaknummer", "Du må skrive inn Websaknummer til avtalesaken")),
+            listOf(FieldError("/sakarkivNummer", "Du må skrive inn Websaknummer til avtalesaken")),
         )
 
-        val avtale = AvtaleFixtures.oppfolging.copy(avtaletype = Avtaletype.Avtale, websaknummer = null)
+        val avtale = AvtaleFixtures.oppfolging.copy(avtaletype = Avtaletype.Avtale, sakarkivNummer = null)
         validator.validate(avtale, null).shouldBeLeft(
-            listOf(FieldError("/websaknummer", "Du må skrive inn Websaknummer til avtalesaken")),
+            listOf(FieldError("/sakarkivNummer", "Du må skrive inn Websaknummer til avtalesaken")),
         )
 
         val offentligOffentligSamarbeid = AvtaleFixtures.gruppeAmo.copy(
             avtaletype = Avtaletype.OffentligOffentlig,
-            websaknummer = null,
+            sakarkivNummer = null,
             amoKategorisering = AmoKategorisering.Studiespesialisering,
         )
         validator.validate(offentligOffentligSamarbeid, null).shouldBeRight()

mulighetsrommet-api/src/test/kotlin/no/nav/mulighetsrommet/api/avtale/AvtaleValidatorTest.kt

@@ -25,7 +25,7 @@ class OpsjonLoggValidatorTest : FunSpec({
         ),
         navn = "Avtale for opsjoner",
         avtalenummer = "24/123",
-        websaknummer = Websaknummer("24/1234"),
+        sakarkivNummer = SakarkivNummer("24/1234"),
         arrangor = AvtaleDto.ArrangorHovedenhet(
             id = UUID.randomUUID(),
             organisasjonsnummer = Organisasjonsnummer("123456789"),