feat: Improve resume flow and decouple it

Author: acezard

src/drive/web/modules/views/Drive/index.jsx

@@ -9,12 +9,6 @@ import { useQuery, useClient } from 'cozy-client'
 import { useVaultClient } from 'cozy-keys-lib'
 import { useI18n } from 'cozy-ui/transpiled/react/I18n'
 import useBreakpoints from 'cozy-ui/transpiled/react/hooks/useBreakpoints'
-import { useWebviewIntent } from 'cozy-intent'
-import {
-  removeFileToUploadQueue,
-  uploadFilesFromNative,
-  purgeUploadQueue
-} from 'drive/web/modules/upload'
 import Dropzone from 'drive/web/modules/upload/Dropzone'
 import { ModalContext } from 'drive/lib/ModalContext'
 import useActions from 'drive/web/modules/actions/useActions'
@@ -56,6 +50,7 @@ import FabWithMenuContext from 'drive/web/modules/drive/FabWithMenuContext'
 import AddMenuProvider from 'drive/web/modules/drive/AddMenu/AddMenuProvider'
 import useHead from 'components/useHead'
 import { useSelectionContext } from 'drive/web/modules/selection/SelectionProvider'
+import { useResumeUploadFromFlagship } from 'drive/web/modules/views/Upload/useUploadFromFlagship'
 
 const desktopExtraColumnsNames = ['carbonCopy', 'electronicSafe']
 const mobileExtraColumnsNames = []
@@ -67,7 +62,6 @@ const DriveView = () => {
   const currentFolderId = useCurrentFolderId() || ROOT_DIR_ID
   useHead()
   const { isSelectionBarVisible } = useSelectionContext()
-  const webviewIntent = useWebviewIntent()
 
   const { isMobile } = useBreakpoints()
   const { isFabDisplayed, setIsFabDisplayed } = useContext(FabContext)
@@ -179,37 +173,7 @@ const DriveView = () => {
     [t]
   )
 
-  useEffect(() => {
-    if (!webviewIntent) return
-
-    webviewIntent
-      .call('hasFilesToHandle')
-      .then(({ filesToHandle, uploadedFiles, uploading }) => {
-        if (!uploading) return false
-
-        actions.dispatch(
-          uploadFilesFromNative(
-            filesToHandle,
-            displayedFolder?.id,
-            undefined,
-            { client },
-            () => Promise.resolve()
-          )
-        )
-
-        return uploadedFiles
-      })
-      .then(uploadedFiles => {
-        if (!uploadedFiles) return false
-
-        return uploadedFiles.forEach(file => {
-          return actions.dispatch(removeFileToUploadQueue(file))
-        })
-      })
-      .catch(() => {
-        return actions.dispatch(purgeUploadQueue())
-      })
-  }, [actions, client, displayedFolder?.id, webviewIntent])
+  useResumeUploadFromFlagship()
 
   useEffect(() => {
     if (canWriteToCurrentFolder) {

src/drive/web/modules/views/Upload/useUploadFromFlagship.ts

@@ -5,7 +5,13 @@ import { useWebviewIntent } from 'cozy-intent'
 
 import logger from 'lib/logger'
 import { FileFromNative } from './UploadTypes'
-import CozyClient from 'cozy-client'
+import CozyClient, { useClient } from 'cozy-client'
+import { useDispatch } from 'react-redux'
+import {
+  uploadFilesFromNative,
+  removeFileToUploadQueue,
+  purgeUploadQueue
+} from '../../upload'
 
 const typedLogger = logger as unknown & {
   info: (message: string, ...rest: unknown[]) => void
@@ -104,3 +110,45 @@ export const useUploadFromFlagship = (): UploadFromFlagship => {
     }
   }
 }
+
+export const useResumeUploadFromFlagship = (): void => {
+  const client = useClient()
+  const dispatch = useDispatch()
+  const webviewIntent = useWebviewIntent()
+  useEffect(() => {
+    const doResumeCheck = async (): Promise<void> => {
+      try {
+        const { filesToHandle, uploadedFiles, uploading } =
+          (await webviewIntent?.call('hasFilesToHandle')) as {
+            filesToHandle: FileFromNative[]
+            uploadedFiles: FileFromNative[]
+            uploading: boolean
+          }
+
+        if (!uploading) return
+
+        dispatch(
+          uploadFilesFromNative(
+            filesToHandle.map(file => ({
+              file: { ...file, name: file.fileName },
+              isDirectory: false
+            })),
+            filesToHandle[0].dirId,
+            undefined,
+            { client },
+            () => Promise.resolve()
+          )
+        )
+
+        uploadedFiles.forEach(file => {
+          dispatch(removeFileToUploadQueue({ ...file, name: file.fileName }))
+        })
+      } catch (error) {
+        typedLogger.info('hasFilesToHandle error', { error })
+        dispatch(purgeUploadQueue())
+      }
+    }
+
+    void doResumeCheck()
+  }, [client, dispatch, webviewIntent])
+}