From d951fa5e8a39bfb52c206ff71fd69150c0d05977 Mon Sep 17 00:00:00 2001
From: Stefano Ricci <1219739+SteRiccio@users.noreply.github.com>
Date: Wed, 20 Sep 2023 13:57:58 +0200
Subject: [PATCH] survey import: prevent starting multiple import jobs (#3024)

Co-authored-by: Stefano Ricci <SteRiccio@users.noreply.github.com>
---
 webapp/components/survey/SurveyCreate/SurveyCreate.js    | 7 +++++--
 .../survey/SurveyCreate/store/actions/useOnImport.js     | 9 +++++----
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/webapp/components/survey/SurveyCreate/SurveyCreate.js b/webapp/components/survey/SurveyCreate/SurveyCreate.js
index 78ddbdeac7..9ac5f28bad 100644
--- a/webapp/components/survey/SurveyCreate/SurveyCreate.js
+++ b/webapp/components/survey/SurveyCreate/SurveyCreate.js
@@ -50,6 +50,7 @@ const SurveyCreate = (props) => {
     cloneFromCycle,
     options,
     file,
+    uploading,
     uploadProgressPercent,
   } = newSurvey
 
@@ -62,6 +63,7 @@ const SurveyCreate = (props) => {
     <div className="home-survey-create">
       <div className="row">
         <ButtonGroup
+          disabled={uploading}
           groupName={template ? 'templateCreateType' : 'surveyCreateType'}
           selectedItemKey={createType}
           onChange={onCreateTypeUpdate}
@@ -89,6 +91,7 @@ const SurveyCreate = (props) => {
       </div>
       <FormItem label={i18n.t('common.name')}>
         <Input
+          disabled={uploading}
           id={TestId.surveyCreate.surveyName}
           value={name}
           validation={Validation.getFieldValidation('name')(validation)}
@@ -159,7 +162,7 @@ const SurveyCreate = (props) => {
 
       {createType === createTypes.import && showImport && (
         <>
-          {uploadProgressPercent >= 0 ? (
+          {uploading && uploadProgressPercent >= 0 ? (
             <div className="row">
               <ProgressBar indeterminate={false} progress={uploadProgressPercent} />
             </div>
@@ -204,7 +207,7 @@ const SurveyCreate = (props) => {
               <div className="row">
                 <Button
                   className="btn-primary"
-                  disabled={!file}
+                  disabled={!file || uploading}
                   label={'homeView.surveyCreate.startImport'}
                   onClick={onImport}
                   testId={TestId.surveyCreate.startImportBtn}
diff --git a/webapp/components/survey/SurveyCreate/store/actions/useOnImport.js b/webapp/components/survey/SurveyCreate/store/actions/useOnImport.js
index 6a0f034613..5c21b73e6e 100644
--- a/webapp/components/survey/SurveyCreate/store/actions/useOnImport.js
+++ b/webapp/components/survey/SurveyCreate/store/actions/useOnImport.js
@@ -24,17 +24,17 @@ export const useOnImport = ({ newSurvey, setNewSurvey }) => {
   return useCallback(async () => {
     const { file, source, ...surveyObj } = newSurvey
 
+    // reset upload progress (hide progress bar)
+    setNewSurvey({ ...newSurvey, uploadProgressPercent: -1, uploading: true })
+
     const formData = objectToFormData({ file, survey: JSON.stringify(surveyObj) })
 
     const onUploadProgress = (progressEvent) => {
       const uploadProgressPercent = Math.round((progressEvent.loaded / progressEvent.total) * 100)
-      setNewSurvey({ ...newSurvey, uploadProgressPercent })
+      setNewSurvey({ ...newSurvey, uploadProgressPercent, uploading: uploadProgressPercent < 100 })
     }
     const { data } = await axios.post(urlBasedOnSource[source], formData, { onUploadProgress })
 
-    // reset upload progress (hide progress bar)
-    setNewSurvey({ ...newSurvey, uploadProgressPercent: -1 })
-
     const { job, validation } = data
 
     if (job && (!validation || Validation.isValid(validation))) {
@@ -53,6 +53,7 @@ export const useOnImport = ({ newSurvey, setNewSurvey }) => {
       setNewSurvey({
         ...newSurvey,
         validation,
+        uploading: false,
       })
     }
   }, [dispatch, newSurvey, setNewSurvey])