.last-branch

@@ -1 +1 @@
-v20260106-14-veeam-m365-overall-message
+v20260106-15-jobrun-popup-objects-sort

containers/backupchecks/src/templates/main/job_detail.html

@@ -274,7 +274,7 @@
         "</body></html>"
       );
     }
-function renderObjects(objects) {
+    function renderObjects(objects) {
       var container = document.getElementById("run_msg_objects_container");
       if (!container) return;
 
@@ -283,16 +283,36 @@ function renderObjects(objects) {
         return;
       }
 
+      // Sort: objects with an error_message first (alphabetically by name), then the rest (also by name).
+      var sorted = (objects || []).slice().sort(function (a, b) {
+        a = a || {};
+        b = b || {};
+        var aHasErr = !!(a.error_message && a.error_message.toString().trim());
+        var bHasErr = !!(b.error_message && b.error_message.toString().trim());
+        if (aHasErr !== bHasErr) return aHasErr ? -1 : 1;
+
+        var an = (a.name || "").toString().toLowerCase();
+        var bn = (b.name || "").toString().toLowerCase();
+        if (an < bn) return -1;
+        if (an > bn) return 1;
+        return 0;
+      });
+
       var html = "<div class=\"table-responsive\"><table class=\"table table-sm table-bordered mb-0\">";
       html += "<thead><tr><th>Object</th><th>Type</th><th>Status</th><th>Error</th></tr></thead><tbody>";
-      for (var i = 0; i < objects.length; i++) {
-        var o = objects[i] || {};
+      for (var i = 0; i < sorted.length; i++) {
+        var o = sorted[i] || {};
         html += "<tr>";
-        html += "<td>" + (o.name || "") + "</td>";
-        html += "<td>" + (o.type || "") + "</td>";
+        html += "<td>" + escapeHtml(o.name || "") + "</td>";
+        html += "<td>" + escapeHtml(o.type || "") + "</td>";
+
         var d = statusDotClass(o.status);
-                html += "<td class=\"status-text " + statusClass(o.status) + "\">" + (d ? ('<span class=\\\"status-dot ' + d + ' me-2\\\" aria-hidden=\\\"true\\\"></span>') : '') + escapeHtml(o.status || "") + "</td>";
-        html += "<td>" + (o.error_message || "") + "</td>";
+        html += "<td class=\"status-text " + statusClass(o.status) + "\">" +
+          (d ? ("<span class=\"status-dot " + d + " me-2\" aria-hidden=\"true\"></span>") : "") +
+          escapeHtml(o.status || "") +
+          "</td>";
+
+        html += "<td>" + escapeHtml(o.error_message || "") + "</td>";
         html += "</tr>";
       }
       html += "</tbody></table></div>";

docs/changelog.md

@@ -110,6 +110,16 @@ Removed an incorrectly indented redirect statement so the module loads correctly
 - Updated the Veeam Backup for Microsoft 365 mail parser to always persist the extracted overall “Details” warning/info message, even when the run is reported as Success.
 - This ensures permission/role warnings (e.g. “Missing application permissions/roles…”) are stored and can be used for overrides.
 
+---
+
+## v20260106-15-jobrun-popup-objects-sort
+
+- Fixed job run popup so objects are rendered again.
+- Restored object list handling in the job run popup.
+- Updated sorting logic:
+  - Objects with an error message are shown first, sorted alphabetically by object name.
+  - Remaining objects are shown below, also sorted alphabetically by object name.
+
 ================================================================================================================================================
 ## v0.1.16