Update app.py

app.py

@@ -324,37 +324,40 @@ class PaperDownloader:
                 return None, f"Error processing {doi}: {e}", f"Error processing {doi}: {e}"
     
     async def download_multiple_dois_async(self, dois_text, progress_callback):
-            """Downloads multiple papers from a list of DOIs and uses a callback for UI"""
-            if not dois_text:
-                return None, "Error: No DOIs provided", "Error: No DOIs provided"
+        if not dois_text:
+            return None, "Error: No DOIs provided", "Error: No DOIs provided"
     
-            dois = [doi.strip() for doi in dois_text.split('\n') if doi.strip()]
-            if not dois:
-                return None, "Error: No valid DOIs provided", "Error: No valid DOIs provided"
+        dois = [doi.strip() for doi in dois_text.split('\n') if doi.strip()]
+        if not dois:
+            return None, "Error: No valid DOIs provided", "Error: No valid DOIs provided"
     
-            downloaded_files = []
-            failed_dois = []
-            downloaded_links = []
-           
-            for i, doi in enumerate(dois):
-                 filepath, success_message, fail_message = await self.download_single_doi_async(doi, progress_callback)
-                 if filepath:
+        downloaded_files = []
+        failed_dois = []
+        downloaded_links = []
+       
+        for i, doi in enumerate(dois):
+            try:
+                filepath, success_message, fail_message = await self.download_single_doi_async(doi, progress_callback)
+                if filepath:
                     # Unique filename for zip
                     filename = f"{str(doi).replace('/', '_').replace('.', '_')}_{i}.pdf"
                     filepath_unique = os.path.join(self.output_dir, filename)
                     os.rename(filepath, filepath_unique)
                     downloaded_files.append(filepath_unique)
                     downloaded_links.append(f'<a href="https://doi.org/{doi}">{doi}</a>')
-                 else:
-                     failed_dois.append(f'<a href="https://doi.org/{doi}">{doi}</a>')
-            
-            if downloaded_files:
-                zip_filename = 'papers.zip'
-                loop = asyncio.get_running_loop()
-                await loop.run_in_executor(self.executor, lambda:  self.create_zip(zip_filename,downloaded_files))
-                logger.info(f"ZIP file created: {zip_filename}")
-            
-            return zip_filename if downloaded_files else None, "\n".join(downloaded_links), "\n".join(failed_dois)
+                else:
+                    failed_dois.append(f'<a href="https://doi.org/{doi}">{doi}</a> - {fail_message}')
+            except Exception as e:
+                failed_dois.append(f'<a href="https://doi.org/{doi}">{doi}</a> - Unexpected error: {str(e)}')
+                continue  # Continue to next DOI even if this one fails
+        
+        if downloaded_files:
+            zip_filename = 'papers.zip'
+            loop = asyncio.get_running_loop()
+            await loop.run_in_executor(self.executor, lambda:  self.create_zip(zip_filename,downloaded_files))
+            logger.info(f"ZIP file created: {zip_filename}")
+        
+        return zip_filename if downloaded_files else None, "\n".join(downloaded_links), "\n".join(failed_dois)
     
     def create_zip(self, zip_filename, downloaded_files):
         with zipfile.ZipFile(zip_filename, 'w') as zipf: