From 571452d15fc66a617e19f06189cb558f7d1fed7f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Halvor=20Grizzly=20Bj=C3=B8rn?=
 <halvor.grizzly.bjorn@nav.no>
Date: Tue, 12 Mar 2024 12:40:12 +0100
Subject: [PATCH] =?UTF-8?q?:fire:=20Skal=20ikke=20hente=20mellomlagret=20s?=
 =?UTF-8?q?=C3=B8knad=20fra=20s=C3=B8knad-api?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 components/SoknadFormWrapper/SlettModal.tsx   | 11 ++----
 .../standard/Vedlegg/Vedlegg.test.tsx         |  1 -
 .../standard/Vedlegg/Vedlegg.tsx              | 25 ++++++-------
 context/soknadcontext/actions.tsx             | 35 ++++---------------
 context/soknadcontext/reducer.tsx             |  6 ----
 context/soknadcontext/soknadContext.tsx       |  2 --
 hooks/useDebounceLagreSoknad.tsx              |  4 +--
 pages/[step].tsx                              | 31 +++-------------
 pages/api/mellomlagring/les.ts                |  1 -
 pages/index.tsx                               | 20 +----------
 10 files changed, 26 insertions(+), 110 deletions(-)

diff --git a/components/SoknadFormWrapper/SlettModal.tsx b/components/SoknadFormWrapper/SlettModal.tsx
index 0a68264a..4d96360a 100644
--- a/components/SoknadFormWrapper/SlettModal.tsx
+++ b/components/SoknadFormWrapper/SlettModal.tsx
@@ -20,15 +20,8 @@ const SlettModal = ({ isOpen, onClose }: Props) => {
   const router = useRouter();
 
   async function onDelete() {
-    await deleteOpplastedeVedlegg(
-      søknadState.brukerMellomLagretSøknadFraAApInnsending,
-      søknadState.søknad,
-    );
-    await slettLagretSoknadState(
-      søknadDispatch,
-      søknadState,
-      søknadState.brukerMellomLagretSøknadFraAApInnsending,
-    );
+    await deleteOpplastedeVedlegg(søknadState.søknad);
+    await slettLagretSoknadState();
   }
 
   const slettSøknadOgAvbryt = async () => {
diff --git a/components/pageComponents/standard/Vedlegg/Vedlegg.test.tsx b/components/pageComponents/standard/Vedlegg/Vedlegg.test.tsx
index 2ff914af..33321b49 100644
--- a/components/pageComponents/standard/Vedlegg/Vedlegg.test.tsx
+++ b/components/pageComponents/standard/Vedlegg/Vedlegg.test.tsx
@@ -49,7 +49,6 @@ const requiredVedlegg = [
 const soknadContextInitial: SoknadContextState = {
   version: 0,
   type: undefined,
-  brukerMellomLagretSøknadFraAApInnsending: true,
   søknad: {
     manuelleBarn: [
       {
diff --git a/components/pageComponents/standard/Vedlegg/Vedlegg.tsx b/components/pageComponents/standard/Vedlegg/Vedlegg.tsx
index 6d5aecc7..5ca9b94f 100644
--- a/components/pageComponents/standard/Vedlegg/Vedlegg.tsx
+++ b/components/pageComponents/standard/Vedlegg/Vedlegg.tsx
@@ -42,12 +42,9 @@ const Vedlegg = ({ onBackClick }: Props) => {
 
   const harPåkrevdeVedlegg = søknadState.requiredVedlegg.length > 0;
 
-  const deleteUrl = søknadState.brukerMellomLagretSøknadFraAApInnsending
-    ? '/aap/soknad/api/vedlegginnsending/slett/?uuid='
-    : '/aap/soknad/api/vedlegg/slett/?uuid=';
-  const uploadUrl = søknadState.brukerMellomLagretSøknadFraAApInnsending
-    ? '/aap/soknad/api/vedlegginnsending/lagre/'
-    : '/aap/soknad/api/vedlegg/lagre/';
+  const deleteUrl = '/aap/soknad/api/vedlegginnsending/slett/?uuid=';
+  const uploadUrl = '/aap/soknad/api/vedlegginnsending/lagre/';
+
   const readAttachmentUrl = '/aap/soknad/vedlegg/';
 
   return (
@@ -110,7 +107,7 @@ const Vedlegg = ({ onBackClick }: Props) => {
           uploadUrl={uploadUrl}
           readAttachmentUrl={readAttachmentUrl}
           files={søknadState.søknad?.vedlegg?.AVBRUTT_STUDIE || []}
-          brukFileInputInnsending={søknadState.brukerMellomLagretSøknadFraAApInnsending}
+          brukFileInputInnsending={true}
         />
       )}
       {søknadState?.requiredVedlegg?.find((e) => e.type === 'OMSORGSSTØNAD') && (
@@ -129,7 +126,7 @@ const Vedlegg = ({ onBackClick }: Props) => {
           uploadUrl={uploadUrl}
           readAttachmentUrl={readAttachmentUrl}
           files={søknadState.søknad?.vedlegg?.OMSORGSSTØNAD || []}
-          brukFileInputInnsending={søknadState.brukerMellomLagretSøknadFraAApInnsending}
+          brukFileInputInnsending={true}
         />
       )}
       {søknadState?.requiredVedlegg?.find((e) => e.type === 'LØNN_OG_ANDRE_GODER') && (
@@ -148,7 +145,7 @@ const Vedlegg = ({ onBackClick }: Props) => {
           uploadUrl={uploadUrl}
           readAttachmentUrl={readAttachmentUrl}
           files={søknadState.søknad?.vedlegg?.LØNN_OG_ANDRE_GODER || []}
-          brukFileInputInnsending={søknadState.brukerMellomLagretSøknadFraAApInnsending}
+          brukFileInputInnsending={true}
         />
       )}
 
@@ -168,7 +165,7 @@ const Vedlegg = ({ onBackClick }: Props) => {
           uploadUrl={uploadUrl}
           readAttachmentUrl={readAttachmentUrl}
           files={søknadState.søknad?.vedlegg?.UTLANDSSTØNAD || []}
-          brukFileInputInnsending={søknadState.brukerMellomLagretSøknadFraAApInnsending}
+          brukFileInputInnsending={true}
         />
       )}
 
@@ -188,7 +185,7 @@ const Vedlegg = ({ onBackClick }: Props) => {
           uploadUrl={uploadUrl}
           readAttachmentUrl={readAttachmentUrl}
           files={søknadState.søknad?.vedlegg?.LÅN || []}
-          brukFileInputInnsending={søknadState.brukerMellomLagretSøknadFraAApInnsending}
+          brukFileInputInnsending={true}
         />
       )}
 
@@ -208,7 +205,7 @@ const Vedlegg = ({ onBackClick }: Props) => {
           uploadUrl={uploadUrl}
           readAttachmentUrl={readAttachmentUrl}
           files={søknadState.søknad?.vedlegg?.SYKESTIPEND || []}
-          brukFileInputInnsending={søknadState.brukerMellomLagretSøknadFraAApInnsending}
+          brukFileInputInnsending={true}
         />
       )}
 
@@ -236,7 +233,7 @@ const Vedlegg = ({ onBackClick }: Props) => {
             uploadUrl={uploadUrl}
             readAttachmentUrl={readAttachmentUrl}
             files={søknadState.søknad?.vedlegg?.[barn.internId] || []}
-            brukFileInputInnsending={søknadState.brukerMellomLagretSøknadFraAApInnsending}
+            brukFileInputInnsending={true}
           />
         );
       })}
@@ -256,7 +253,7 @@ const Vedlegg = ({ onBackClick }: Props) => {
         uploadUrl={uploadUrl}
         readAttachmentUrl={readAttachmentUrl}
         files={søknadState.søknad?.vedlegg?.ANNET || []}
-        brukFileInputInnsending={søknadState.brukerMellomLagretSøknadFraAApInnsending}
+        brukFileInputInnsending={true}
       />
 
       <Textarea
diff --git a/context/soknadcontext/actions.tsx b/context/soknadcontext/actions.tsx
index e11336a4..88a112bd 100644
--- a/context/soknadcontext/actions.tsx
+++ b/context/soknadcontext/actions.tsx
@@ -14,7 +14,6 @@ export enum SoknadActionKeys {
   ADD_BEHANDLER_IF_MISSING = 'ADD_BEHANDLER_IF_MISSING',
   ADD_REQUIRED_VEDLEGG = 'ADD_REQUIRED_VEDLEGG',
   REMOVE_REQUIRED_VEDLEGG = 'REMOVE_REQUIRED_VEDLEGG',
-  ADD_SØKNAD_URL = 'ADD_SØKNAD_URL',
   ADD_VEDLEGG = 'ADD_VEDLEGG',
   DELETE_VEDLEGG = 'DELETE_VEDLEGG',
 }
@@ -47,10 +46,6 @@ type RemoveRequiredVedlegg = {
   type: SoknadActionKeys.REMOVE_REQUIRED_VEDLEGG;
   payload?: AttachmentType;
 };
-type AddSøknadUrl = {
-  type: SoknadActionKeys.ADD_SØKNAD_URL;
-  payload: string;
-};
 
 type AddVedlegg = {
   type: SoknadActionKeys.ADD_VEDLEGG;
@@ -72,7 +67,6 @@ export type SoknadAction =
   | AddBehandlerIfMissing
   | AddRequiredVedlegg
   | RemoveRequiredVedlegg
-  | AddSøknadUrl
   | AddVedlegg
   | DeleteVedlegg;
 
@@ -87,16 +81,9 @@ export function setSoknadStateFraProps(
   return props;
 }
 
-export async function slettLagretSoknadState(
-  dispatch: Dispatch<SoknadAction>,
-  state: SoknadContextState,
-  brukerMellomLagretSøknadFraAApInnsending: boolean,
-) {
-  const deleteResponse = brukerMellomLagretSøknadFraAApInnsending
-    ? await fetch('/aap/soknad/api/mellomlagring/slett')
-    : await fetch(`/aap/soknad/api/buckets/slett/?type=${state.type}`, {
-        method: 'DELETE',
-      });
+export async function slettLagretSoknadState() {
+  const deleteResponse = await fetch('/aap/soknad/api/mellomlagring/slett');
+
   return !!deleteResponse?.ok;
 }
 
@@ -151,21 +138,11 @@ export const getVedleggUuidsFromSoknad = (søknad?: Soknad) => {
     .map((vedlegg) => vedlegg?.vedleggId);
 };
 
-export const deleteOpplastedeVedlegg = async (
-  brukMellomlagretSøknadFraInnsending: boolean,
-  søknad?: Soknad,
-) => {
+export const deleteOpplastedeVedlegg = async (søknad?: Soknad) => {
   const vedleggUuids = getVedleggUuidsFromSoknad(søknad);
   if (vedleggUuids.length > 0) {
-    if (brukMellomlagretSøknadFraInnsending) {
-      for (const vedlegg of vedleggUuids) {
-        await fetch(`/aap/soknad/api/vedlegginnsending/slett/?uuid=${vedlegg}`);
-      }
-    } else {
-      const commaSeparatedUuids = vedleggUuids.join(',');
-      await fetch(`/aap/soknad/api/vedlegg/slett/?uuids=${commaSeparatedUuids}`, {
-        method: 'DELETE',
-      });
+    for (const vedlegg of vedleggUuids) {
+      await fetch(`/aap/soknad/api/vedlegginnsending/slett/?uuid=${vedlegg}`);
     }
   }
 };
diff --git a/context/soknadcontext/reducer.tsx b/context/soknadcontext/reducer.tsx
index f354e451..62bdbec9 100644
--- a/context/soknadcontext/reducer.tsx
+++ b/context/soknadcontext/reducer.tsx
@@ -76,12 +76,6 @@ export function soknadReducer(state: SoknadContextState, action: SoknadAction):
         requiredVedlegg: [...newVedleggList],
       };
     }
-    case SoknadActionKeys.ADD_SØKNAD_URL: {
-      return {
-        ...state,
-        søknadUrl: action?.payload,
-      };
-    }
 
     case SoknadActionKeys.ADD_VEDLEGG: {
       const filesToAdd = [...(state.søknad?.vedlegg?.[action.key] || []), ...action.payload];
diff --git a/context/soknadcontext/soknadContext.tsx b/context/soknadcontext/soknadContext.tsx
index bf801064..07b09786 100644
--- a/context/soknadcontext/soknadContext.tsx
+++ b/context/soknadcontext/soknadContext.tsx
@@ -9,7 +9,6 @@ export interface SoknadContextState {
   version: number;
   type?: SøknadType;
   søknad?: Soknad;
-  brukerMellomLagretSøknadFraAApInnsending: boolean;
   lagretStepList?: Array<StepType>;
   requiredVedlegg: RequiredVedlegg[];
   søknadUrl?: string;
@@ -20,7 +19,6 @@ export const SOKNAD_VERSION = 1;
 export const soknadContextInititalState: SoknadContextState = {
   version: SOKNAD_VERSION,
   søknad: undefined,
-  brukerMellomLagretSøknadFraAApInnsending: true,
   requiredVedlegg: [],
   søknadUrl: undefined,
   type: SøknadType.STANDARD,
diff --git a/hooks/useDebounceLagreSoknad.tsx b/hooks/useDebounceLagreSoknad.tsx
index 9bba73cf..387bf93a 100644
--- a/hooks/useDebounceLagreSoknad.tsx
+++ b/hooks/useDebounceLagreSoknad.tsx
@@ -18,9 +18,7 @@ export function useDebounceLagreSoknad<SoknadStateType>() {
       lagretStepList: stepList,
       sistLagret: formatDateTime(new Date()),
     };
-    const res = state.brukerMellomLagretSøknadFraAApInnsending
-      ? await fetchPOST(`/aap/soknad/api/mellomlagring/lagre`, payload)
-      : await fetchPOST(`/aap/soknad/api/buckets/lagre/?type=${payload.type}`, payload);
+    const res = await fetchPOST(`/aap/soknad/api/mellomlagring/lagre`, payload);
     if (res.ok) setSistLagret(formatDateTime(new Date()), appStateDispatch);
   }
   const debouncedLagre = () => {
diff --git a/pages/[step].tsx b/pages/[step].tsx
index 05506d6d..63f94b00 100644
--- a/pages/[step].tsx
+++ b/pages/[step].tsx
@@ -120,9 +120,10 @@ const Steps = ({ søker, mellomlagretSøknad, kontaktinformasjon, barn }: PagePr
         søknadState?.requiredVedlegg,
       );
 
-      const postResponse = søknadState.brukerMellomLagretSøknadFraAApInnsending
-        ? await postSøknadMedAAPInnsending(søknadState.søknad, søknadState.requiredVedlegg)
-        : await postSøknadMedSoknadApi({ søknad, kvittering: søknadPdf });
+      const postResponse = await postSøknadMedAAPInnsending(
+        søknadState.søknad,
+        søknadState.requiredVedlegg,
+      );
 
       if (postResponse?.ok) {
         const harVedlegg = søknadState.requiredVedlegg && søknadState?.requiredVedlegg?.length > 0;
@@ -135,10 +136,6 @@ const Steps = ({ søker, mellomlagretSøknad, kontaktinformasjon, barn }: PagePr
           søknadState?.søknad?.tilleggsopplysninger.length > 0;
         logSkjemaFullførtEvent({ harVedlegg, erIkkeKomplett, brukerFritekstfelt, yrkesskade });
 
-        if (!søknadState.brukerMellomLagretSøknadFraAApInnsending) {
-          const url = postResponse?.data?.uri;
-          søknadDispatch({ type: SoknadActionKeys.ADD_SØKNAD_URL, payload: url });
-        }
         router.push('kvittering');
         return true;
       } else {
@@ -242,25 +239,7 @@ export const getServerSideProps = beskyttetSide(
     let mellomlagretSøknad: SoknadContextState | undefined;
 
     try {
-      const [mellomlagretSøknadFraSoknadApi, mellomlagretSøknadFraAapInnsending] =
-        await Promise.all([lesBucket('STANDARD', bearerToken), hentMellomlagring(bearerToken)]);
-
-      if (mellomlagretSøknadFraAapInnsending && mellomlagretSøknadFraSoknadApi) {
-        logError('pages/step: finner mellomlagring fra begge kilder');
-      }
-      if (mellomlagretSøknadFraSoknadApi) {
-        logInfo('pages/step: velger mellomlagring fra søknad-api');
-        mellomlagretSøknad = {
-          ...mellomlagretSøknadFraSoknadApi,
-          brukerMellomLagretSøknadFraAApInnsending: false,
-        };
-      } else if (mellomlagretSøknadFraAapInnsending) {
-        logInfo('pages/step: velger mellomlagring fra innsending');
-        mellomlagretSøknad = {
-          ...mellomlagretSøknadFraAapInnsending,
-          brukerMellomLagretSøknadFraAApInnsending: true,
-        };
-      }
+      mellomlagretSøknad = await hentMellomlagring(bearerToken);
     } catch (e) {
       logError('Noe gikk galt i innhenting av mellomlagret søknad', e);
     }
diff --git a/pages/api/mellomlagring/les.ts b/pages/api/mellomlagring/les.ts
index 00abe479..f4cf5671 100644
--- a/pages/api/mellomlagring/les.ts
+++ b/pages/api/mellomlagring/les.ts
@@ -20,7 +20,6 @@ export const hentMellomlagring = async (
   if (isFunctionalTest()) {
     return {
       version: SOKNAD_VERSION,
-      brukerMellomLagretSøknadFraAApInnsending: true,
       søknad: {},
       lagretStepList: defaultStepList,
       requiredVedlegg: [],
diff --git a/pages/index.tsx b/pages/index.tsx
index 1aaa71e0..614d2beb 100644
--- a/pages/index.tsx
+++ b/pages/index.tsx
@@ -114,25 +114,7 @@ export const getServerSideProps = beskyttetSide(
     let mellomlagretSøknad: SoknadContextState | undefined;
 
     try {
-      const [mellomlagretSøknadFraSoknadApi, mellomlagretSøknadFraAapInnsending] =
-        await Promise.all([lesBucket('STANDARD', bearerToken), hentMellomlagring(bearerToken)]);
-
-      if (mellomlagretSøknadFraAapInnsending && mellomlagretSøknadFraSoknadApi) {
-        logError('pages/index: finner mellomlagring fra begge kilder');
-      }
-      if (mellomlagretSøknadFraSoknadApi) {
-        logInfo('pages/index: velger mellomlagring fra søknad-api');
-        mellomlagretSøknad = {
-          ...mellomlagretSøknadFraSoknadApi,
-          brukerMellomLagretSøknadFraAApInnsending: false,
-        };
-      } else if (mellomlagretSøknadFraAapInnsending) {
-        logInfo('pages/index: velger mellomlagring fra innsending');
-        mellomlagretSøknad = {
-          ...mellomlagretSøknadFraAapInnsending,
-          brukerMellomLagretSøknadFraAApInnsending: true,
-        };
-      }
+      mellomlagretSøknad = await hentMellomlagring(bearerToken);
     } catch (e) {
       logError('Noe gikk galt i innhenting av mellomlagret søknad', e);
     }