From 585d3acf8899155afca02fc06ad4e80c3e3d5219 Mon Sep 17 00:00:00 2001 From: Abhinav Date: Sat, 8 Jan 2022 13:09:43 +0530 Subject: [PATCH] fix multiple folder detection for sibling folder with no common ancestor --- src/components/pages/gallery/Upload.tsx | 42 +++++++++++++------------ 1 file changed, 22 insertions(+), 20 deletions(-) diff --git a/src/components/pages/gallery/Upload.tsx b/src/components/pages/gallery/Upload.tsx index a8a6a2962..ac0379797 100644 --- a/src/components/pages/gallery/Upload.tsx +++ b/src/components/pages/gallery/Upload.tsx @@ -273,10 +273,14 @@ export default function Upload(props: Props) { }; const uploadToSingleNewCollection = (collectionName: string) => { - uploadFilesToNewCollections( - UPLOAD_STRATEGY.SINGLE_COLLECTION, - collectionName - ); + if (collectionName) { + uploadFilesToNewCollections( + UPLOAD_STRATEGY.SINGLE_COLLECTION, + collectionName + ); + } else { + showCollectionCreateModal(analysisResult); + } }; const showCollectionCreateModal = (analysisResult: AnalysisResult) => { props.setCollectionNamerAttributes({ @@ -291,25 +295,23 @@ export default function Upload(props: Props) { analysisResult: AnalysisResult, isFirstUpload: boolean ) => { - if (!analysisResult.suggestedCollectionName) { - if (isFirstUpload) { - uploadToSingleNewCollection(FIRST_ALBUM_NAME); - } else { - props.setCollectionSelectorAttributes({ - callback: uploadFilesToExistingCollection, - showNextModal: () => - showCollectionCreateModal(analysisResult), - title: constants.UPLOAD_TO_COLLECTION, - }); - } + if (isFirstUpload) { + uploadToSingleNewCollection(FIRST_ALBUM_NAME); } else { + let showNextModal = () => {}; if (analysisResult.multipleFolders) { - setChoiceModalView(true); - } else if (analysisResult.suggestedCollectionName) { - uploadToSingleNewCollection( - analysisResult.suggestedCollectionName - ); + showNextModal = () => setChoiceModalView(true); + } else { + showNextModal = () => + uploadToSingleNewCollection( + analysisResult.suggestedCollectionName + ); } + props.setCollectionSelectorAttributes({ + callback: uploadFilesToExistingCollection, + showNextModal, + title: constants.UPLOAD_TO_COLLECTION, + }); } };