fix: korjaa lomakkeiden lataus tallennuksen jälkeen, jotta optimistin…

…en lukitus ei antaisi virhettä (#574)
finnishtransportagency · Jan 31, 2023 · 4f11676 · 4f11676
1 parent e5af4f2
commit 4f11676
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 19 deletions.
diff --git a/src/components/projekti/nahtavillaolo/kuulutuksentiedot/Painikkeet.tsx b/src/components/projekti/nahtavillaolo/kuulutuksentiedot/Painikkeet.tsx
@@ -39,7 +39,7 @@ export default function Painikkeet({ projekti }: Props) {
     }; // ... and to false on unmount
   }, []);
 
-  const { handleSubmit, reset } = useFormContext<KuulutuksenTiedotFormValues>();
+  const { handleSubmit } = useFormContext<KuulutuksenTiedotFormValues>();
 
   const api = useApi();
 
@@ -49,9 +49,8 @@ export default function Painikkeet({ projekti }: Props) {
       if (reloadProjekti) {
         await reloadProjekti();
       }
-      reset(formData);
     },
-    [api, reloadProjekti, reset]
+    [api, reloadProjekti]
   );
 
   const saveDraft = async (formData: KuulutuksenTiedotFormValues) => {

diff --git a/...ponents/projekti/nahtavillaolo/nahtavilleAsetettavatAineistot/NahtavillaoloPainikkeet.tsx b/...ponents/projekti/nahtavillaolo/nahtavilleAsetettavatAineistot/NahtavillaoloPainikkeet.tsx
@@ -23,20 +23,19 @@ const mapFormValuesToTallennaProjektiInput = ({
   const aineistoNahtavillaFlat = Object.values(aineistoNahtavilla).flat();
   deleteFieldArrayIds(aineistoNahtavillaFlat);
   deleteFieldArrayIds(lisaAineisto);
-  const result: TallennaProjektiInput = {
+  return {
     oid,
     versio,
     nahtavillaoloVaihe: { aineistoNahtavilla: aineistoNahtavillaFlat, lisaAineisto },
   };
-  return result;
 };
 
 export default function NahtavillaoloPainikkeet() {
   const { mutate: reloadProjekti, data: projekti } = useProjekti();
   const [isFormSubmitting, setIsFormSubmitting] = useState(false);
   const { showSuccessMessage } = useSnackbars();
 
-  const { handleSubmit, reset, watch } = useFormContext<NahtavilleAsetettavatAineistotFormValues>();
+  const { handleSubmit, watch } = useFormContext<NahtavilleAsetettavatAineistotFormValues>();
   const api = useApi();
 
   const aineistoNahtavilla = watch("aineistoNahtavilla");
@@ -59,7 +58,6 @@ export default function NahtavillaoloPainikkeet() {
       if (reloadProjekti) {
         await reloadProjekti();
       }
-      reset(formData);
       showSuccessMessage("Tallennus onnistui!");
       await afterSaveCallback?.();
     } catch (e) {
@@ -69,11 +67,6 @@ export default function NahtavillaoloPainikkeet() {
     }
   };
 
-  // useEffect(() => {
-  //   console.log("Reset", defaultValues);
-  //   reset(defaultValues);
-  // }, [defaultValues, reset]);
-
   const saveDraft = async (formData: NahtavilleAsetettavatAineistotFormValues) => {
     await saveNahtavillaoloAineisto(formData);
   };

diff --git a/src/components/projekti/paatos/aineistot/HyvaksymisPaatosVaihePainikkeet.tsx b/src/components/projekti/paatos/aineistot/HyvaksymisPaatosVaihePainikkeet.tsx
@@ -33,7 +33,7 @@ export default function PaatosPainikkeet({ paatosTyyppi }: { paatosTyyppi: Paato
   const [isFormSubmitting, setIsFormSubmitting] = useState(false);
   const { showSuccessMessage } = useSnackbars();
 
-  const { handleSubmit, reset, watch } = useFormContext<HyvaksymisPaatosVaiheAineistotFormValues>();
+  const { handleSubmit, watch } = useFormContext<HyvaksymisPaatosVaiheAineistotFormValues>();
   const api = useApi();
 
   const aineistoNahtavilla = watch("aineistoNahtavilla");
@@ -57,7 +57,6 @@ export default function PaatosPainikkeet({ paatosTyyppi }: { paatosTyyppi: Paato
       if (reloadProjekti) {
         await reloadProjekti();
       }
-      reset(formData);
       showSuccessMessage("Tallennus onnistui!");
       await afterSaveCallback?.();
     } catch (e) {

diff --git a/src/components/projekti/paatos/aineistot/Muokkausnakyma.tsx b/src/components/projekti/paatos/aineistot/Muokkausnakyma.tsx
@@ -1,6 +1,6 @@
 import { yupResolver } from "@hookform/resolvers/yup";
 import { Aineisto, AineistoInput, TallennaProjektiInput } from "@services/api";
-import React, { ReactElement, useMemo } from "react";
+import React, { ReactElement, useEffect, useMemo } from "react";
 import { UseFormProps, useForm, FormProvider } from "react-hook-form";
 import { useProjekti } from "src/hooks/useProjekti";
 import { nahtavillaoloAineistotSchema } from "src/schemas/nahtavillaoloAineistot";
@@ -121,6 +121,12 @@ function MuokkausnakymaForm({
 
   useLeaveConfirm(isDirty);
 
+  const { reset } = useFormReturn;
+  useEffect(() => {
+    console.log("Reset", defaultValues);
+    reset(defaultValues);
+  }, [defaultValues, reset]);
+
   return (
     <FormProvider {...useFormReturn}>
       <form>

diff --git a/src/components/projekti/paatos/kuulutuksenTiedot/Painikkeet.tsx b/src/components/projekti/paatos/kuulutuksenTiedot/Painikkeet.tsx
@@ -46,17 +46,16 @@ export default function Painikkeet({ projekti, julkaisu, paatosTyyppi, julkaisem
     }; // ... and to false on unmount
   }, []);
 
-  const { handleSubmit, reset } = useFormContext<KuulutuksenTiedotFormValues>();
+  const { handleSubmit } = useFormContext<KuulutuksenTiedotFormValues>();
 
   const api = useApi();
 
   const saveHyvaksymisPaatosVaihe = useCallback(
     async (formData: KuulutuksenTiedotFormValues) => {
       await api.tallennaProjekti(convertFormDataToTallennaProjektiInput(formData, paatosTyyppi));
       if (reloadProjekti) await reloadProjekti();
-      reset(formData);
     },
-    [api, paatosTyyppi, reloadProjekti, reset]
+    [api, paatosTyyppi, reloadProjekti]
   );
 
   const saveDraft = async (formData: KuulutuksenTiedotFormValues) => {

diff --git a/src/components/projekti/paatos/kuulutuksenTiedot/index.tsx b/src/components/projekti/paatos/kuulutuksenTiedot/index.tsx
@@ -1,7 +1,7 @@
 import { yupResolver } from "@hookform/resolvers/yup";
 import { HyvaksymisPaatosVaiheInput, KirjaamoOsoite, MuokkausTila, TallennaProjektiInput, YhteystietoInput } from "@services/api";
 import Notification, { NotificationType } from "@components/notification/Notification";
-import React, { ReactElement, useMemo } from "react";
+import React, { ReactElement, useEffect, useMemo } from "react";
 import { FormProvider, useForm, UseFormProps } from "react-hook-form";
 import { ProjektiLisatiedolla } from "src/hooks/useProjekti";
 import { hyvaksymispaatosKuulutusSchema } from "src/schemas/hyvaksymispaatosKuulutus";
@@ -102,6 +102,12 @@ function KuulutuksenTiedotForm({ kirjaamoOsoitteet, paatosTyyppi, projekti }: Ku
 
   useLeaveConfirm(isDirty);
 
+  const { reset } = useFormReturn;
+  useEffect(() => {
+    console.log("Reset", defaultValues);
+    reset(defaultValues);
+  }, [defaultValues, reset]);
+
   const voiMuokata = !julkaisematonPaatos?.muokkausTila || julkaisematonPaatos?.muokkausTila === MuokkausTila.MUOKKAUS;
 
   return (