Fjerner tiny-invariant og henter tilsagn basert på orgnr

frontend/arrangor-flate/app/components/tilsagn/TilsagnTable.tsx

@@ -1,5 +1,5 @@
 import { ArrangorflateTilsagn } from "@mr/api-client";
-import { Table } from "@navikt/ds-react";
+import { Alert, Table } from "@navikt/ds-react";
 import { Link } from "@remix-run/react";
 import { formaterDato, useOrgnrFromUrl } from "~/utils";
 import { internalNavigation } from "../../internal-navigation";
@@ -10,6 +10,15 @@ interface Props {
 
 export function TilsagnTable({ tilsagn }: Props) {
   const orgnr = useOrgnrFromUrl();
+
+  if (tilsagn.length === 0) {
+    return (
+      <Alert className="my-10" variant="info">
+        Det finnes ingen tilsagn her
+      </Alert>
+    );
+  }
+
   return (
     <>
       <div className="border-spacing-y-6 border-collapsed mt-4">

frontend/arrangor-flate/app/routes/$orgnr.refusjonskrav.$id.bekreft.tsx

@@ -9,8 +9,6 @@ import { ArrangorflateService, ArrangorflateTilsagn } from "@mr/api-client";
 import { RefusjonskravDetaljer } from "~/components/refusjonskrav/RefusjonskravDetaljer";
 import { useOrgnrFromUrl } from "../utils";
 import { internalNavigation } from "../internal-navigation";
-import invariant from "tiny-invariant";
-import React from "react";
 import { Definisjon } from "~/components/Definisjon";
 
 type BekreftRefusjonskravData = {
@@ -56,7 +54,9 @@ export const action: ActionFunction = async ({ request }) => {
     return json({ error: "Mangler kontonummer" }, { status: 400 });
   }
 
-  invariant(orgnr, "Mangler orgnr");
+  if (!orgnr) {
+    throw new Error("Mangler orgnr");
+  }
 
   await ArrangorflateService.godkjennRefusjonskrav({
     id: refusjonskravId as string,

frontend/arrangor-flate/app/routes/$orgnr.refusjonskrav.$id.for-du-begynner.tsx

@@ -5,7 +5,6 @@ import { checkValidToken } from "../auth/auth.server";
 import { PageHeader } from "../components/PageHeader";
 import { internalNavigation } from "../internal-navigation";
 import { useOrgnrFromUrl } from "../utils";
-import invariant from "tiny-invariant";
 
 export const meta: MetaFunction = () => {
   return [
@@ -22,7 +21,9 @@ export const loader: LoaderFunction = async ({ request, params }): Promise<objec
 
 export default function ForDuBegynner() {
   const { id } = useParams<{ id: string }>();
-  invariant(id, "Mangler id");
+  if (!id) {
+    throw new Error("Mangler id");
+  }
   const orgnr = useOrgnrFromUrl();
   return (
     <>

frontend/arrangor-flate/app/routes/$orgnr_.refusjonskrav.tsx

@@ -7,7 +7,6 @@ import { RefusjonskravTable } from "~/components/refusjonskrav/RefusjonskravTabl
 import { TilsagnTable } from "~/components/tilsagn/TilsagnTable";
 import { checkValidToken, setupOpenApi } from "../auth/auth.server";
 import { PageHeader } from "../components/PageHeader";
-import invariant from "tiny-invariant";
 
 export const meta: MetaFunction = () => {
   return [
@@ -20,9 +19,11 @@ export async function loader({ request, params }: LoaderFunctionArgs) {
   await checkValidToken(request);
   await setupOpenApi(request);
   const { orgnr } = params;
-  invariant(orgnr, "Mangler orgnr");
+  if (!orgnr) {
+    throw new Error("Mangler orgnr");
+  }
   const krav = await ArrangorflateService.getAllRefusjonKrav({ orgnr });
-  const tilsagn = await ArrangorflateService.getAllArrangorflateTilsagn();
+  const tilsagn = await ArrangorflateService.getAllArrangorflateTilsagn({ orgnr });
 
   return json({ krav, tilsagn });
 }

frontend/arrangor-flate/package.json

@@ -43,7 +43,6 @@
     "prom-client": "^15.1.3",
     "react": "^18.3.1",
     "react-dom": "^18.3.1",
-    "tiny-invariant": "^1.3.3",
     "uuid": "11.0.1"
   },
   "devDependencies": {

mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/okonomi/refusjon/ArrangorflateRoutes.kt

@@ -89,10 +89,11 @@ fun Route.arrangorflateRoutes() {
     route("/arrangorflate") {
         route("/refusjonskrav") {
             get("alle/{orgnr}") {
-                val arrangorIds = arrangorIderMedTilgang()
                 val orgnr = call.parameters.getOrFail("orgnr")
+                val organisasjonsnummer = Organisasjonsnummer(orgnr)
+                requireTilgangHosArrangor(organisasjonsnummer)
 
-                val krav = refusjonskrav.getByArrangorIds(arrangorIds, Organisasjonsnummer(orgnr))
+                val krav = refusjonskrav.getByArrangorIds(organisasjonsnummer)
                     .map { toRefusjonskravKompakt(it) }
 
                 call.respond(krav)
@@ -161,8 +162,13 @@ fun Route.arrangorflateRoutes() {
         }
 
         route("/tilsagn") {
-            get {
-                call.respond(tilsagnService.getAllArrangorflateTilsagn(arrangorIderMedTilgang()))
+            route("/alle/{orgnr}") {
+                get {
+                    val orgnr = call.parameters.getOrFail("orgnr")
+                    val organisasjonsnummer = Organisasjonsnummer(orgnr)
+                    requireTilgangHosArrangor(organisasjonsnummer)
+                    call.respond(tilsagnService.getAllArrangorflateTilsagn(organisasjonsnummer))
+                }
             }
 
             get("/{id}") {

mulighetsrommet-api/src/main/kotlin/no/nav/mulighetsrommet/api/okonomi/refusjon/RefusjonskravRepository.kt

@@ -174,22 +174,21 @@ class RefusjonskravRepository(private val db: Database) {
             .runWithSession(tx)
     }
 
-    fun getByArrangorIds(ids: List<UUID>, organisasjonsnummer: Organisasjonsnummer): List<RefusjonskravDto> =
-        db.transaction { getByArrangorIds(ids, organisasjonsnummer, it) }
+    fun getByArrangorIds(organisasjonsnummer: Organisasjonsnummer): List<RefusjonskravDto> =
+        db.transaction { getByArrangorIds(organisasjonsnummer, it) }
 
     fun getByArrangorIds(
-        ids: List<UUID>,
         organisasjonsnummer: Organisasjonsnummer,
         tx: Session,
     ): List<RefusjonskravDto> {
         @Language("PostgreSQL")
         val query = """
             select * from refusjonskrav_aft_view
-            where arrangor_id = any(:ids) and arrangor_organisasjonsnummer = :organisasjonsnummer
+            where arrangor_organisasjonsnummer = :organisasjonsnummer
         """.trimIndent()
 
         return tx.run(
-            queryOf(query, mapOf("ids" to db.createUuidArray(ids), "organisasjonsnummer" to organisasjonsnummer.value))
+            queryOf(query, mapOf("organisasjonsnummer" to organisasjonsnummer.value))
                 .map { it.toRefusjonsKravAft() }
                 .asList,
         )

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

@@ -96,14 +96,14 @@ class TilsagnRepository(private val db: Database) {
             .let { db.run(it) }
     }
 
-    fun getAllArrangorflateTilsagn(arrangorIds: List<UUID>): List<ArrangorflateTilsagn> {
+    fun getAllArrangorflateTilsagn(organisasjonsnummer: Organisasjonsnummer): List<ArrangorflateTilsagn> {
         @Language("PostgreSQL")
         val query = """
             select * from tilsagn_arrangorflate_view
-            where arrangor_id = any (?)
+            where arrangor_organisasjonsnummer = :organisasjonsnummer
         """.trimIndent()
 
-        return queryOf(query, db.createUuidArray(arrangorIds))
+        return queryOf(query, mapOf("organisasjonsnummer" to organisasjonsnummer.value))
             .map { it.toArrangorflateTilsagn() }
             .asList
             .let { db.run(it) }

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

@@ -11,6 +11,7 @@ import no.nav.mulighetsrommet.api.repositories.TiltaksgjennomforingRepository
 import no.nav.mulighetsrommet.api.responses.*
 import no.nav.mulighetsrommet.database.Database
 import no.nav.mulighetsrommet.domain.dto.NavIdent
+import no.nav.mulighetsrommet.domain.dto.Organisasjonsnummer
 import java.time.LocalDate
 import java.time.LocalDateTime
 import java.util.*
@@ -100,8 +101,8 @@ class TilsagnService(
         }.right()
     }
 
-    fun getAllArrangorflateTilsagn(arrangorIds: List<UUID>): List<ArrangorflateTilsagn> =
-        tilsagnRepository.getAllArrangorflateTilsagn(arrangorIds)
+    fun getAllArrangorflateTilsagn(organisasjonsnummer: Organisasjonsnummer): List<ArrangorflateTilsagn> =
+        tilsagnRepository.getAllArrangorflateTilsagn(organisasjonsnummer)
 
     fun getArrangorflateTilsagnTilRefusjon(
         gjennomforingId: UUID,

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

@@ -1803,11 +1803,13 @@ paths:
                 items:
                   $ref: "#/components/schemas/AFTSats"
 
-  /api/v1/intern/arrangorflate/tilsagn:
+  /api/v1/intern/arrangorflate/tilsagn/alle/{orgnr}:
     get:
       tags:
         - Arrangorflate
       operationId: getAllArrangorflateTilsagn
+      parameters:
+        - $ref: "#/components/parameters/OrgnrParam"
       responses:
         200:
           description: Hent tilsagn til arrangør

mulighetsrommet-api/src/test/kotlin/no/nav/mulighetsrommet/api/okonomi/refusjon/RefusjonServiceTest.kt

@@ -8,7 +8,6 @@ import io.kotest.matchers.shouldBe
 import io.kotest.matchers.types.shouldBeTypeOf
 import no.nav.mulighetsrommet.api.createDatabaseTestConfig
 import no.nav.mulighetsrommet.api.domain.dbo.TiltaksgjennomforingDbo
-import no.nav.mulighetsrommet.api.fixtures.ArrangorFixtures
 import no.nav.mulighetsrommet.api.fixtures.DeltakerFixtures
 import no.nav.mulighetsrommet.api.fixtures.MulighetsrommetTestDomain
 import no.nav.mulighetsrommet.api.fixtures.TiltaksgjennomforingFixtures.AFT1
@@ -61,7 +60,6 @@ class RefusjonServiceTest : FunSpec({
             service.genererRefusjonskravForMonth(LocalDate.of(2024, 1, 1))
 
             val allKrav = refusjonskravRepository.getByArrangorIds(
-                listOf(ArrangorFixtures.underenhet1.id),
                 getOrgnrForArrangor(
                     AFT1,
                     domain,
@@ -126,7 +124,6 @@ class RefusjonServiceTest : FunSpec({
             service.genererRefusjonskravForMonth(LocalDate.of(2024, 1, 1))
 
             val krav = refusjonskravRepository.getByArrangorIds(
-                listOf(ArrangorFixtures.underenhet1.id),
                 getOrgnrForArrangor(
                     AFT1,
                     domain,
@@ -187,7 +184,6 @@ class RefusjonServiceTest : FunSpec({
             service.genererRefusjonskravForMonth(LocalDate.of(2024, 1, 1))
 
             val krav = refusjonskravRepository.getByArrangorIds(
-                listOf(ArrangorFixtures.underenhet1.id),
                 getOrgnrForArrangor(
                     AFT1,
                     domain,
@@ -229,7 +225,6 @@ class RefusjonServiceTest : FunSpec({
             service.genererRefusjonskravForMonth(LocalDate.of(2024, 1, 1))
 
             val krav = refusjonskravRepository.getByArrangorIds(
-                listOf(ArrangorFixtures.underenhet1.id),
                 getOrgnrForArrangor(
                     AFT1,
                     domain,