From b39dd1251c0cdc9c4ed7d01ad03dfba3e5aac823 Mon Sep 17 00:00:00 2001
From: Samir Boulahtit <samir.boulahtit@wizard.lu>
Date: Fri, 5 Dec 2025 21:59:32 +0100
Subject: [PATCH] fix: use consistent optional chaining for selectedJob in
 templates
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Fix JavaScript TypeError "can't access property 'error_details',
selectedJob is null" by using optional chaining consistently.

Changed in all three templates:
- x-show="selectedJob?.error_details?.length > 0" (was missing ?. before length)
- x-text="JSON.stringify(selectedJob?.error_details, ...)" (was missing ?.)

Affected templates:
- app/templates/admin/marketplace.html
- app/templates/admin/imports.html
- app/templates/vendor/marketplace.html

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 app/templates/admin/imports.html      | 4 ++--
 app/templates/admin/marketplace.html  | 4 ++--
 app/templates/vendor/marketplace.html | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/app/templates/admin/imports.html b/app/templates/admin/imports.html
index 652ccd7d..faf9cdd6 100644
--- a/app/templates/admin/imports.html
+++ b/app/templates/admin/imports.html
@@ -410,10 +410,10 @@
             </div>
 
             <!-- Error Details -->
-            <div x-show="selectedJob?.error_details && selectedJob.error_details.length > 0" class="mt-4">
+            <div x-show="selectedJob?.error_details?.length > 0" class="mt-4">
                 <p class="text-sm font-medium text-gray-600 dark:text-gray-400 mb-2">Error Details</p>
                 <div class="p-3 bg-red-50 dark:bg-red-900/20 rounded-lg max-h-48 overflow-y-auto">
-                    <pre class="text-xs text-red-700 dark:text-red-300 whitespace-pre-wrap" x-text="JSON.stringify(selectedJob.error_details, null, 2)"></pre>
+                    <pre class="text-xs text-red-700 dark:text-red-300 whitespace-pre-wrap" x-text="JSON.stringify(selectedJob?.error_details, null, 2)"></pre>
                 </div>
             </div>
         </div>
diff --git a/app/templates/admin/marketplace.html b/app/templates/admin/marketplace.html
index 3c70ba2d..0e0f36ff 100644
--- a/app/templates/admin/marketplace.html
+++ b/app/templates/admin/marketplace.html
@@ -487,10 +487,10 @@
             </div>
 
             <!-- Error Details -->
-            <div x-show="selectedJob?.error_details && selectedJob.error_details.length > 0" class="mt-4">
+            <div x-show="selectedJob?.error_details?.length > 0" class="mt-4">
                 <p class="text-sm font-medium text-gray-600 dark:text-gray-400 mb-2">Error Details</p>
                 <div class="p-3 bg-red-50 dark:bg-red-900/20 rounded-lg max-h-48 overflow-y-auto">
-                    <pre class="text-xs text-red-700 dark:text-red-300 whitespace-pre-wrap" x-text="JSON.stringify(selectedJob.error_details, null, 2)"></pre>
+                    <pre class="text-xs text-red-700 dark:text-red-300 whitespace-pre-wrap" x-text="JSON.stringify(selectedJob?.error_details, null, 2)"></pre>
                 </div>
             </div>
         </div>
diff --git a/app/templates/vendor/marketplace.html b/app/templates/vendor/marketplace.html
index 33ba6ba5..405d1576 100644
--- a/app/templates/vendor/marketplace.html
+++ b/app/templates/vendor/marketplace.html
@@ -394,10 +394,10 @@
             </div>
 
             <!-- Error Details -->
-            <div x-show="selectedJob?.error_details && selectedJob.error_details.length > 0" class="mt-4">
+            <div x-show="selectedJob?.error_details?.length > 0" class="mt-4">
                 <p class="text-sm font-medium text-gray-600 dark:text-gray-400 mb-2">Error Details</p>
                 <div class="p-3 bg-red-50 dark:bg-red-900/20 rounded-lg max-h-48 overflow-y-auto">
-                    <pre class="text-xs text-red-700 dark:text-red-300 whitespace-pre-wrap" x-text="JSON.stringify(selectedJob.error_details, null, 2)"></pre>
+                    <pre class="text-xs text-red-700 dark:text-red-300 whitespace-pre-wrap" x-text="JSON.stringify(selectedJob?.error_details, null, 2)"></pre>
                 </div>
             </div>
         </div>