Update app.py

app.py

@@ -228,10 +228,10 @@ class PaperDownloader:
                 with open(filepath, 'wb') as f:
                     f.write(pdf_content)
                 logger.info(f"Successfully downloaded: {filename}")
-                return filepath, "Successfully downloaded", ""
+                return filepath, f'<a href="https://doi.org/{doi}">{doi}</a>', ""
             else:
                 logger.warning(f"Could not download: {doi}")
-                return None, f"Could not download {doi}", f"Could not download {doi}"
+                return None, f"Could not download {doi}", f'<a href="https://doi.org/{doi}">{doi}</a>'
 
         except Exception as e:
             logger.error(f"Error processing {doi}: {e}")
@@ -248,6 +248,7 @@ class PaperDownloader:
 
         downloaded_files = []
         failed_dois = []
+        downloaded_links = []
         for i, doi in enumerate(tqdm(dois, desc="Downloading papers")):
             filepath, success_message, fail_message = self.download_single_doi(doi)
             if filepath:
@@ -256,9 +257,10 @@ class PaperDownloader:
               filepath_unique = os.path.join(self.output_dir, filename)
               os.rename(filepath,filepath_unique)
               downloaded_files.append(filepath_unique)
-            else:
-               failed_dois.append(doi)
+              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'
@@ -267,8 +269,7 @@ class PaperDownloader:
                     zipf.write(file_path, arcname=os.path.basename(file_path))
             logger.info(f"ZIP file created: {zip_filename}")
 
-        return zip_filename if downloaded_files else None, "\n".join(downloaded_files), "\n".join(failed_dois)
-
+        return zip_filename if downloaded_files else None, "\n".join(downloaded_links), "\n".join(failed_dois)
 
 
     def process_bibtex(self, bib_file):
@@ -279,14 +280,14 @@ class PaperDownloader:
                  bib_content = f.read()
         except Exception as e:
              logger.error(f"Error reading uploaded file {bib_file.name}: {e}")
-             return None, f"Error reading uploaded file {bib_file.name}: {e}", f"Error reading uploaded file {bib_file.name}: {e}"
+             return None, f"Error reading uploaded file {bib_file.name}: {e}", f"Error reading uploaded file {bib_file.name}: {e}", None
 
         # Parse BibTeX data
         try:
             bib_database = bibtexparser.loads(bib_content)
         except Exception as e:
              logger.error(f"Error parsing BibTeX data: {e}")
-             return None, f"Error parsing BibTeX data: {e}", f"Error parsing BibTeX data: {e}"
+             return None, f"Error parsing BibTeX data: {e}", f"Error parsing BibTeX data: {e}", None
 
         # Extract DOIs
         dois = [entry.get('doi') for entry in bib_database.entries if entry.get('doi')]
@@ -295,6 +296,7 @@ class PaperDownloader:
         # Result lists
         downloaded_files = []
         failed_dois = []
+        downloaded_links = []
 
         # Download PDFs
         for doi in tqdm(dois, desc="Downloading papers"):
@@ -313,13 +315,13 @@ class PaperDownloader:
                         f.write(pdf_content)
                     
                     downloaded_files.append(filepath)
+                    downloaded_links.append(f'<a href="https://doi.org/{doi}">{doi}</a>')
                     logger.info(f"Successfully downloaded: {filename}")
                 else:
-                    failed_dois.append(doi)
-                    logger.warning(f"Could not download: {doi}")
+                    failed_dois.append(f'<a href="https://doi.org/{doi}">{doi}</a>')
 
             except Exception as e:
-                failed_dois.append(doi)
+                failed_dois.append(f'<a href="https://doi.org/{doi}">{doi}</a>')
                 logger.error(f"Error processing {doi}: {e}")
 
         # Create ZIP of downloaded papers
@@ -330,8 +332,7 @@ class PaperDownloader:
                     zipf.write(file_path, arcname=os.path.basename(file_path))
             logger.info(f"ZIP file created: {zip_filename}")
 
-        return zip_filename, "\n".join(downloaded_files), "\n".join(failed_dois), None
-
+        return zip_filename, "\n".join(downloaded_links), "\n".join(failed_dois), None
 
 def create_gradio_interface():
     """Create Gradio interface for Paper Downloader"""
@@ -365,8 +366,25 @@ def create_gradio_interface():
         ],
         outputs=[
             gr.File(label="Download Papers (ZIP) or Single PDF"),
-            gr.Textbox(label="Downloaded DOIs/Message"),
-            gr.Textbox(label="Failed DOIs"),
+             gr.HTML(label="""
+               <div style='padding-bottom: 5px; font-weight: bold;'>
+                        Enter Single DOI
+               </div>
+               <div style='border: 1px solid #ddd; padding: 5px; border-radius: 5px;'>
+                    <div style='padding-bottom: 5px; font-weight: bold;'>
+                        Downloaded DOIs
+                    </div>
+                    <div id="downloaded-dois"></div>
+                 </div>
+            """),
+           gr.HTML(label="""
+                <div style='border: 1px solid #ddd; padding: 5px; border-radius: 5px;'>
+                    <div style='padding-bottom: 5px; font-weight: bold;'>
+                        Failed DOIs
+                    </div>
+                    <div id="failed-dois"></div>
+                 </div>
+            """),
             gr.File(label="Downloaded Single PDF")
         ],
         title="🔬 Academic Paper Batch Downloader",
@@ -390,10 +408,33 @@ def create_gradio_interface():
             border-radius: 10px;
             box-shadow: 0 4px 6px rgba(0, 0, 0, 0.1);
         }
+           .output-text a {
+               color: #007bff; /* Blue color for hyperlinks */
+            }
         """,
-        cache_examples = False
+        cache_examples = False,
     )
+    
+    # Add Javascript to update HTML
+    interface.load = """
+        function(downloaded_dois, failed_dois){
+             let downloaded_html =  '<ul>';
+             downloaded_dois.split('\\n').filter(Boolean).forEach(doi => {
+                downloaded_html +=  '<li>' + doi + '</li>';
+            });
+            downloaded_html +=  '</ul>';
+            document.querySelector("#downloaded-dois").innerHTML = downloaded_html;
+
+             let failed_html = '<ul>';
+             failed_dois.split('\\n').filter(Boolean).forEach(doi => {
+                failed_html += '<li>' + doi + '</li>';
+            });
+             failed_html += '</ul>';
+            document.querySelector("#failed-dois").innerHTML = failed_html;
+            return [downloaded_html, failed_html];
 
+        }
+    """
     return interface
 
 def main():