From 341bb9a209c27da4dbe185302aacafe14103fc0e Mon Sep 17 00:00:00 2001
From: soupette <cyril.lpz@gmail.com>
Date: Tue, 7 Apr 2020 10:57:59 +0200
Subject: [PATCH 1/6] Disable buttons when in uploading state

Signed-off-by: soupette <cyril.lpz@gmail.com>
---
 .../admin/src/components/CheckControl/index.js            | 2 +-
 .../admin/src/components/EditForm/index.js                | 3 ++-
 .../src/containers/InputModalStepper/InputModalStepper.js | 8 +++++---
 .../admin/src/containers/ModalStepper/index.js            | 8 +++++---
 4 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/packages/strapi-plugin-upload/admin/src/components/CheckControl/index.js b/packages/strapi-plugin-upload/admin/src/components/CheckControl/index.js
index 0012138580..5b9cc8ef12 100644
--- a/packages/strapi-plugin-upload/admin/src/components/CheckControl/index.js
+++ b/packages/strapi-plugin-upload/admin/src/components/CheckControl/index.js
@@ -40,7 +40,7 @@ const CheckControl = ({ title, onSubmitEdit }) => {
         color="#6DBB1A"
         onClick={handleToggle}
         type="check"
-        title={formatMessage({ id: getTrad(`control-card.${title}`) })}
+        title={title}
         iconStyle={{ height: '1.4rem', width: '1.4rem' }}
       />
       <CustomDropdownSection isOpen={isOpen}>
diff --git a/packages/strapi-plugin-upload/admin/src/components/EditForm/index.js b/packages/strapi-plugin-upload/admin/src/components/EditForm/index.js
index cb373068e5..be929cdc9f 100644
--- a/packages/strapi-plugin-upload/admin/src/components/EditForm/index.js
+++ b/packages/strapi-plugin-upload/admin/src/components/EditForm/index.js
@@ -55,6 +55,7 @@ const EditForm = forwardRef(
     const [isCropping, setIsCropping] = useState(false);
     const [infos, setInfos] = useState({ width: null, height: null });
     const [src, setSrc] = useState(null);
+    const cacheRef = useRef(performance.now());
 
     const fileURL = get(fileToEdit, ['file', 'url'], null);
     const prefixedFileURL = fileURL ? prefixFileUrlWithBackendUrl(fileURL) : null;
@@ -274,7 +275,7 @@ const EditForm = forwardRef(
                       {isImg ? (
                         <CropWrapper>
                           <img
-                            src={src}
+                            src={`${src}?${cacheRef.current}`}
                             alt={get(fileToEdit, ['file', 'name'], '')}
                             ref={isCropping ? imgRef : null}
                           />
diff --git a/packages/strapi-plugin-upload/admin/src/containers/InputModalStepper/InputModalStepper.js b/packages/strapi-plugin-upload/admin/src/containers/InputModalStepper/InputModalStepper.js
index b2480a2e49..dde21c51de 100644
--- a/packages/strapi-plugin-upload/admin/src/containers/InputModalStepper/InputModalStepper.js
+++ b/packages/strapi-plugin-upload/admin/src/containers/InputModalStepper/InputModalStepper.js
@@ -237,7 +237,7 @@ const InputModalStepper = ({ isOpen, onToggle, onInputMediaChange }) => {
       const errorMessage = get(
         err,
         ['response', 'payload', 'message', '0', 'messages', '0', 'message'],
-        get(err, ['response', 'payload', 'message'], null)
+        get(err, ['response', 'payload', 'message'], 'An error occurred')
       );
 
       if (errorMessage) {
@@ -263,6 +263,8 @@ const InputModalStepper = ({ isOpen, onToggle, onInputMediaChange }) => {
 
   const shouldDisplayNextButton = currentStep === 'browse' && displayNextButton;
   const isFinishButtonDisabled = filesToUpload.some(file => file.isDownloading || file.isUploading);
+  const areButtonsDisabledOnEditExistingFile =
+    currentStep === 'edit' && fileToEdit.isUploading === true;
 
   return (
     <>
@@ -348,7 +350,7 @@ const InputModalStepper = ({ isOpen, onToggle, onInputMediaChange }) => {
             {currentStep === 'edit' && (
               <div style={{ margin: 'auto 0' }}>
                 <Button
-                  disabled={isFormDisabled}
+                  disabled={isFormDisabled || areButtonsDisabledOnEditExistingFile}
                   color="primary"
                   onClick={handleReplaceMedia}
                   style={{ marginRight: 10 }}
@@ -357,7 +359,7 @@ const InputModalStepper = ({ isOpen, onToggle, onInputMediaChange }) => {
                 </Button>
 
                 <Button
-                  disabled={isFormDisabled}
+                  disabled={isFormDisabled || areButtonsDisabledOnEditExistingFile}
                   color="success"
                   type="button"
                   onClick={handleSubmitEditExistingFile}
diff --git a/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js b/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js
index f2b2f6ee1e..843789d64c 100644
--- a/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js
+++ b/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js
@@ -315,7 +315,7 @@ const ModalStepper = ({
       const errorMessage = get(
         err,
         ['response', 'payload', 'message', '0', 'messages', '0', 'message'],
-        get(err, ['response', 'payload', 'message'], null)
+        get(err, ['response', 'payload', 'message'], 'An error occurred')
       );
 
       if (errorMessage) {
@@ -441,6 +441,8 @@ const ModalStepper = ({
 
   const shouldDisplayNextButton = currentStep === 'browse' && displayNextButton;
   const isFinishButtonDisabled = filesToUpload.some(file => file.isDownloading || file.isUploading);
+  const areButtonsDisabledOnEditExistingFile =
+    currentStep === 'edit' && fileToEdit.isUploading === true;
 
   return (
     <>
@@ -525,7 +527,7 @@ const ModalStepper = ({
             {currentStep === 'edit' && (
               <div style={{ margin: 'auto 0' }}>
                 <Button
-                  disabled={isFormDisabled}
+                  disabled={isFormDisabled || areButtonsDisabledOnEditExistingFile}
                   color="primary"
                   onClick={handleReplaceMedia}
                   style={{ marginRight: 10 }}
@@ -534,7 +536,7 @@ const ModalStepper = ({
                 </Button>
 
                 <Button
-                  disabled={isFormDisabled}
+                  disabled={isFormDisabled || areButtonsDisabledOnEditExistingFile}
                   color="success"
                   type="button"
                   onClick={handleSubmitEditExistingFile}

From ead77e08c444a053f6e2818356a59ebfe6937607 Mon Sep 17 00:00:00 2001
From: soupette <cyril.lpz@gmail.com>
Date: Tue, 7 Apr 2020 11:18:26 +0200
Subject: [PATCH 2/6] Fix errors

Signed-off-by: soupette <cyril.lpz@gmail.com>
---
 .../InputModalStepper/InputModalStepper.js       |  4 +++-
 .../InputModalStepperProvider/index.js           | 16 ++++++++++------
 .../admin/src/containers/ModalStepper/index.js   | 10 +++++++---
 3 files changed, 20 insertions(+), 10 deletions(-)

diff --git a/packages/strapi-plugin-upload/admin/src/containers/InputModalStepper/InputModalStepper.js b/packages/strapi-plugin-upload/admin/src/containers/InputModalStepper/InputModalStepper.js
index dde21c51de..b23f1a0e3f 100644
--- a/packages/strapi-plugin-upload/admin/src/containers/InputModalStepper/InputModalStepper.js
+++ b/packages/strapi-plugin-upload/admin/src/containers/InputModalStepper/InputModalStepper.js
@@ -234,13 +234,15 @@ const InputModalStepper = ({ isOpen, onToggle, onInputMediaChange }) => {
       handleEditExistingFile(editedFile);
       goToList();
     } catch (err) {
+      console.error(err);
+      const status = get(err, 'response.status', null);
       const errorMessage = get(
         err,
         ['response', 'payload', 'message', '0', 'messages', '0', 'message'],
         get(err, ['response', 'payload', 'message'], 'An error occurred')
       );
 
-      if (errorMessage) {
+      if (status) {
         handleSetFileToEditError(errorMessage);
       }
     }
diff --git a/packages/strapi-plugin-upload/admin/src/containers/InputModalStepperProvider/index.js b/packages/strapi-plugin-upload/admin/src/containers/InputModalStepperProvider/index.js
index e2555dd1be..9a0d401586 100644
--- a/packages/strapi-plugin-upload/admin/src/containers/InputModalStepperProvider/index.js
+++ b/packages/strapi-plugin-upload/admin/src/containers/InputModalStepperProvider/index.js
@@ -402,17 +402,21 @@ const InputModalStepperProvider = ({
             multiple,
           });
         } catch (err) {
+          console.error(err);
+          const status = get(err, 'response.status', null);
           const errorMessage = get(
             err,
             ['response', 'payload', 'message', '0', 'messages', '0', 'message'],
-            null
+            'An error occured'
           );
 
-          dispatch({
-            type: 'SET_FILE_ERROR',
-            fileIndex: originalIndex,
-            errorMessage,
-          });
+          if (status) {
+            dispatch({
+              type: 'SET_FILE_ERROR',
+              fileIndex: originalIndex,
+              errorMessage,
+            });
+          }
         }
       }
     );
diff --git a/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js b/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js
index 843789d64c..e06d671a7f 100644
--- a/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js
+++ b/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js
@@ -312,13 +312,15 @@ const ModalStepper = ({
       // Close the modal and refetch data
       toggleRef.current(true);
     } catch (err) {
+      console.error(err);
+      const status = get(err, 'response.status', null);
       const errorMessage = get(
         err,
         ['response', 'payload', 'message', '0', 'messages', '0', 'message'],
         get(err, ['response', 'payload', 'message'], 'An error occurred')
       );
 
-      if (errorMessage) {
+      if (status) {
         dispatch({
           type: 'SET_FILE_TO_EDIT_ERROR',
           errorMessage,
@@ -394,13 +396,15 @@ const ModalStepper = ({
             fileIndex: originalIndex,
           });
         } catch (err) {
+          console.error(err);
+          const status = get(err, 'response.status', null);
           const errorMessage = get(
             err,
             ['response', 'payload', 'message', '0', 'messages', '0', 'message'],
-            get(err, ['response', 'payload', 'message'], null)
+            get(err, ['response', 'payload', 'message'], 'An error occured')
           );
 
-          if (errorMessage) {
+          if (status) {
             dispatch({
               type: 'SET_FILE_ERROR',
               fileIndex: originalIndex,

From 19af1482af2bb26b8c26ae056df7da06ae55dbff Mon Sep 17 00:00:00 2001
From: soupette <cyril.lpz@gmail.com>
Date: Tue, 7 Apr 2020 13:23:08 +0200
Subject: [PATCH 3/6] Fix errors

Signed-off-by: soupette <cyril.lpz@gmail.com>
---
 .../lib/src/utils/request.js                  | 29 ++++++++-----------
 .../InputModalStepper/InputModalStepper.js    |  6 ++--
 .../InputModalStepperProvider/index.js        |  6 ++--
 .../src/containers/ModalStepper/index.js      | 10 ++++---
 4 files changed, 24 insertions(+), 27 deletions(-)

diff --git a/packages/strapi-helper-plugin/lib/src/utils/request.js b/packages/strapi-helper-plugin/lib/src/utils/request.js
index 9e7fea5369..1d28fd9e7f 100644
--- a/packages/strapi-helper-plugin/lib/src/utils/request.js
+++ b/packages/strapi-helper-plugin/lib/src/utils/request.js
@@ -21,10 +21,7 @@ function parseJSON(response) {
  * @return {object|undefined} Returns either the response, or throws an error
  */
 function checkStatus(response, checkToken = true) {
-  if (
-    (response.status >= 200 && response.status < 300) ||
-    response.status === 0
-  ) {
+  if ((response.status >= 200 && response.status < 300) || response.status === 0) {
     return response;
   }
 
@@ -32,12 +29,16 @@ function checkStatus(response, checkToken = true) {
     return checkTokenValidity(response);
   }
 
-  return parseJSON(response).then(responseFormatted => {
-    const error = new Error(response.statusText);
-    error.response = response;
-    error.response.payload = responseFormatted;
-    throw error;
-  });
+  return parseJSON(response)
+    .then(responseFormatted => {
+      const error = new Error(response.statusText);
+      error.response = response;
+      error.response.payload = responseFormatted;
+      throw error;
+    })
+    .catch(() => {
+      throw response;
+    });
 }
 
 function checkTokenValidity(response) {
@@ -114,13 +115,7 @@ function serverRestartWatcher(response) {
  * @return {object}           The response data
  */
 export default function request(...args) {
-  let [
-    url,
-    options = {},
-    shouldWatchServerRestart,
-    stringify = true,
-    ...rest
-  ] = args;
+  let [url, options = {}, shouldWatchServerRestart, stringify = true, ...rest] = args;
   let noAuth;
 
   try {
diff --git a/packages/strapi-plugin-upload/admin/src/containers/InputModalStepper/InputModalStepper.js b/packages/strapi-plugin-upload/admin/src/containers/InputModalStepper/InputModalStepper.js
index b23f1a0e3f..d4f8ec2dbb 100644
--- a/packages/strapi-plugin-upload/admin/src/containers/InputModalStepper/InputModalStepper.js
+++ b/packages/strapi-plugin-upload/admin/src/containers/InputModalStepper/InputModalStepper.js
@@ -234,12 +234,12 @@ const InputModalStepper = ({ isOpen, onToggle, onInputMediaChange }) => {
       handleEditExistingFile(editedFile);
       goToList();
     } catch (err) {
-      console.error(err);
-      const status = get(err, 'response.status', null);
+      const status = get(err, 'response.status', get(err, 'status', null));
+      const statusText = get(err, 'response.statusText', get(err, 'statusText', null));
       const errorMessage = get(
         err,
         ['response', 'payload', 'message', '0', 'messages', '0', 'message'],
-        get(err, ['response', 'payload', 'message'], 'An error occurred')
+        get(err, ['response', 'payload', 'message'], statusText)
       );
 
       if (status) {
diff --git a/packages/strapi-plugin-upload/admin/src/containers/InputModalStepperProvider/index.js b/packages/strapi-plugin-upload/admin/src/containers/InputModalStepperProvider/index.js
index 9a0d401586..c8c97d4726 100644
--- a/packages/strapi-plugin-upload/admin/src/containers/InputModalStepperProvider/index.js
+++ b/packages/strapi-plugin-upload/admin/src/containers/InputModalStepperProvider/index.js
@@ -402,12 +402,12 @@ const InputModalStepperProvider = ({
             multiple,
           });
         } catch (err) {
-          console.error(err);
-          const status = get(err, 'response.status', null);
+          const status = get(err, 'response.status', get(err, 'status', null));
+          const statusText = get(err, 'response.statusText', get(err, 'statusText', null));
           const errorMessage = get(
             err,
             ['response', 'payload', 'message', '0', 'messages', '0', 'message'],
-            'An error occured'
+            get(err, ['response', 'payload', 'message'], statusText)
           );
 
           if (status) {
diff --git a/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js b/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js
index e06d671a7f..181fae915d 100644
--- a/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js
+++ b/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js
@@ -313,11 +313,12 @@ const ModalStepper = ({
       toggleRef.current(true);
     } catch (err) {
       console.error(err);
-      const status = get(err, 'response.status', null);
+      const status = get(err, 'response.status', get(err, 'status', null));
+      const statusText = get(err, 'response.statusText', get(err, 'statusText', null));
       const errorMessage = get(
         err,
         ['response', 'payload', 'message', '0', 'messages', '0', 'message'],
-        get(err, ['response', 'payload', 'message'], 'An error occurred')
+        get(err, ['response', 'payload', 'message'], statusText)
       );
 
       if (status) {
@@ -397,11 +398,12 @@ const ModalStepper = ({
           });
         } catch (err) {
           console.error(err);
-          const status = get(err, 'response.status', null);
+          const status = get(err, 'response.status', get(err, 'status', null));
+          const statusText = get(err, 'response.statusText', get(err, 'statusText', null));
           const errorMessage = get(
             err,
             ['response', 'payload', 'message', '0', 'messages', '0', 'message'],
-            get(err, ['response', 'payload', 'message'], 'An error occured')
+            get(err, ['response', 'payload', 'message'], statusText)
           );
 
           if (status) {

From e7f4f8bfa158fbe5d3638201fd018f7400a2cfe7 Mon Sep 17 00:00:00 2001
From: soupette <cyril.lpz@gmail.com>
Date: Tue, 7 Apr 2020 14:24:12 +0200
Subject: [PATCH 4/6] Fix edit file when lib not initialized

Signed-off-by: soupette <cyril.lpz@gmail.com>
---
 .../admin/src/components/EditForm/index.js    | 44 ++++++++++++-------
 1 file changed, 27 insertions(+), 17 deletions(-)

diff --git a/packages/strapi-plugin-upload/admin/src/components/EditForm/index.js b/packages/strapi-plugin-upload/admin/src/components/EditForm/index.js
index be929cdc9f..c8ebc9ccbe 100644
--- a/packages/strapi-plugin-upload/admin/src/components/EditForm/index.js
+++ b/packages/strapi-plugin-upload/admin/src/components/EditForm/index.js
@@ -135,6 +135,7 @@ const EditForm = forwardRef(
     };
 
     const handleClick = async () => {
+      console.log('click');
       const cropResult = await getCroppedResult();
 
       setCropResult(cropResult);
@@ -143,30 +144,39 @@ const EditForm = forwardRef(
     };
 
     const getCroppedResult = () => {
-      return new Promise(resolve => {
-        const canvas = cropper.current.getCroppedCanvas();
+      return new Promise((resolve, reject) => {
+        try {
+          const canvas = cropper.current.getCroppedCanvas();
 
-        canvas.toBlob(async blob => {
-          const {
-            file: { lastModifiedDate, lastModified, name },
-          } = fileToEdit;
+          canvas.toBlob(async blob => {
+            const {
+              file: { lastModifiedDate, lastModified, name },
+            } = fileToEdit;
 
-          resolve(
-            new File([blob], name, {
-              type: mimeType,
-              lastModified,
-              lastModifiedDate,
-            })
-          );
-        });
+            resolve(
+              new File([blob], name, {
+                type: mimeType,
+                lastModified,
+                lastModifiedDate,
+              })
+            );
+          });
+        } catch (err) {
+          reject();
+        }
       });
     };
 
     const handleClickEditCroppedFile = async (e, shouldDuplicate = false) => {
-      const file = await getCroppedResult();
+      try {
+        const file = await getCroppedResult();
 
-      setIsCropping(false);
-      onSubmitEdit(e, shouldDuplicate, file);
+        onSubmitEdit(e, shouldDuplicate, file);
+      } catch (err) {
+        // Silent
+      } finally {
+        setIsCropping(false);
+      }
     };
 
     const handleClickDelete = () => {

From 9532212e864273efe4d7b99a85a3f31100fe2b3d Mon Sep 17 00:00:00 2001
From: soupette <cyril.lpz@gmail.com>
Date: Tue, 7 Apr 2020 14:28:54 +0200
Subject: [PATCH 5/6] Update timeout

Signed-off-by: soupette <cyril.lpz@gmail.com>
---
 .../admin/src/containers/InputModalStepperProvider/index.js     | 2 +-
 .../admin/src/containers/ModalStepper/index.js                  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/packages/strapi-plugin-upload/admin/src/containers/InputModalStepperProvider/index.js b/packages/strapi-plugin-upload/admin/src/containers/InputModalStepperProvider/index.js
index c8c97d4726..5f57f06ae3 100644
--- a/packages/strapi-plugin-upload/admin/src/containers/InputModalStepperProvider/index.js
+++ b/packages/strapi-plugin-upload/admin/src/containers/InputModalStepperProvider/index.js
@@ -75,7 +75,7 @@ const InputModalStepperProvider = ({
               headers: { Authorization: `Bearer ${auth.getToken()}` },
               responseType: 'blob',
               cancelToken: source.token,
-              timeout: 30000,
+              timeout: 60000,
             })
             .then(({ data }) => {
               const createdFile = new File([data], file.fileURL, {
diff --git a/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js b/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js
index 181fae915d..e1bbb8ae06 100644
--- a/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js
+++ b/packages/strapi-plugin-upload/admin/src/containers/ModalStepper/index.js
@@ -95,7 +95,7 @@ const ModalStepper = ({
               headers: { Authorization: `Bearer ${auth.getToken()}` },
               responseType: 'blob',
               cancelToken: source.token,
-              timeout: 30000,
+              timeout: 60000,
             })
             .then(({ data }) => {
               const fileName = file.fileInfo.name;

From c5441f1d6142a1dcd6c1944c44562e2b0a6a8e29 Mon Sep 17 00:00:00 2001
From: soupette <cyril.lpz@gmail.com>
Date: Tue, 7 Apr 2020 14:55:26 +0200
Subject: [PATCH 6/6] Fix init request

Signed-off-by: soupette <cyril.lpz@gmail.com>
---
 .../strapi-admin/admin/src/containers/App/index.js  | 13 +++++--------
 .../admin/src/components/EditForm/index.js          |  1 -
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/packages/strapi-admin/admin/src/containers/App/index.js b/packages/strapi-admin/admin/src/containers/App/index.js
index f7d718c78e..b9f7fd6829 100644
--- a/packages/strapi-admin/admin/src/containers/App/index.js
+++ b/packages/strapi-admin/admin/src/containers/App/index.js
@@ -42,7 +42,9 @@ function App(props) {
       try {
         const requestURL = '/users-permissions/init';
 
-        const { hasAdmin } = await request(requestURL, { method: 'GET' });
+        const { hasAdmin } = await request(requestURL, { method: 'GET' }, false, false, {
+          noAuth: true,
+        });
         const { data } = await request('/admin/init', { method: 'GET' });
         const { uuid } = data;
 
@@ -86,9 +88,7 @@ function App(props) {
           <Switch>
             <Route
               path="/auth/:authType"
-              render={routerProps => (
-                <AuthPage {...routerProps} hasAdminUser={state.hasAdmin} />
-              )}
+              render={routerProps => <AuthPage {...routerProps} hasAdminUser={state.hasAdmin} />}
               exact
             />
             <PrivateRoute path="/" component={Admin} />
@@ -108,10 +108,7 @@ export function mapDispatchToProps(dispatch) {
   return bindActionCreators({ getDataSucceeded }, dispatch);
 }
 
-const withConnect = connect(
-  null,
-  mapDispatchToProps
-);
+const withConnect = connect(null, mapDispatchToProps);
 
 export default compose(withConnect)(App);
 export { App };
diff --git a/packages/strapi-plugin-upload/admin/src/components/EditForm/index.js b/packages/strapi-plugin-upload/admin/src/components/EditForm/index.js
index c8ebc9ccbe..67682d2d4a 100644
--- a/packages/strapi-plugin-upload/admin/src/components/EditForm/index.js
+++ b/packages/strapi-plugin-upload/admin/src/components/EditForm/index.js
@@ -135,7 +135,6 @@ const EditForm = forwardRef(
     };
 
     const handleClick = async () => {
-      console.log('click');
       const cropResult = await getCroppedResult();
 
       setCropResult(cropResult);