From 90ed775883eea7cffcc59450211c126ea6cd5444 Mon Sep 17 00:00:00 2001
From: nmemmert <67961292+nmemmert@users.noreply.github.com>
Date: Fri, 7 Nov 2025 10:07:03 -0500
Subject: [PATCH] Fix JavaScript syntax error in nested ternary operators

---
 pdf_renamer.py | 45 +++++++++++++++++++++------------------------
 1 file changed, 21 insertions(+), 24 deletions(-)

diff --git a/pdf_renamer.py b/pdf_renamer.py
index 322e7de..50cea52 100644
--- a/pdf_renamer.py
+++ b/pdf_renamer.py
@@ -1300,31 +1300,28 @@ def jobs():
                                             </div>
                                         `).join('')}
                                     ` : ''}
-                                    ${job.status === 'completed' ? `
-                                        ${job.zip_name ? `
-                                            <p><a href="/download/${job.zip_name}" class="download-btn">Download ZIP</a>
-                                            <form method="post" action="/delete/${jobId}" style="display: inline;">
-                                                <button type="submit" class="delete-btn" onclick="return confirm('Are you sure you want to delete this job?')">Delete Job</button>
-                                            </form></p>
-                                        ` : `
-                                            <p><em>No files were successfully processed to create a ZIP archive.</em>
-                                            <form method="post" action="/delete/${jobId}" style="display: inline;">
-                                                <button type="submit" class="delete-btn" onclick="return confirm('Are you sure you want to delete this job?')">Delete Job</button>
-                                            </form></p>
-                                        `}
-                                        ${job.skipped ? `
-                                            <div class="skipped-section">
-                                                <div class="skipped-toggle" onclick="toggleSkippedDetails(this)">
-                                                    ⚠️ Skipped Files (${job.skipped.length}) ▶
-                                                </div>
-                                                <div class="skipped-details">
-                                                    <ul>
-                                                        ${job.skipped.map(skipInfo => `<li><strong>${skipInfo[0]}</strong>: ${skipInfo[1]}</li>`).join('')}
-                                                    </ul>
-                                                </div>
-                                            </div>
-                                        ` : ''}
+                                    ${job.status === 'completed' ? (job.zip_name ? `
+                                        <p><a href="/download/${job.zip_name}" class="download-btn">Download ZIP</a>
+                                        <form method="post" action="/delete/${jobId}" style="display: inline;">
+                                            <button type="submit" class="delete-btn" onclick="return confirm('Are you sure you want to delete this job?')">Delete Job</button>
+                                        </form></p>
+                                    ` : `
+                                        <p><em>No files were successfully processed to create a ZIP archive.</em>
+                                        <form method="post" action="/delete/${jobId}" style="display: inline;">
+                                            <button type="submit" class="delete-btn" onclick="return confirm('Are you sure you want to delete this job?')">Delete Job</button>
+                                        </form></p>
                                     ` : ''}
+                                    ${job.skipped ? `
+                                        <div class="skipped-section">
+                                            <div class="skipped-toggle" onclick="toggleSkippedDetails(this)">
+                                                ⚠️ Skipped Files (${job.skipped.length}) ▶
+                                            </div>
+                                            <div class="skipped-details">
+                                                <ul>
+                                                    ${job.skipped.map(skipInfo => `<li><strong>${skipInfo[0]}</strong>: ${skipInfo[1]}</li>`).join('')}
+                                                </ul>
+                                            </div>
+                                        </div>
                                     ` : ''}
                                 `;
                                 jobsContainer.insertBefore(jobDiv, jobsContainer.querySelector('script') || jobsContainer.lastElementChild);