From 0dbab45651639135d3c146c06721e6219380a988 Mon Sep 17 00:00:00 2001
From: Sebastiaan de Schaetzen <sebastiaan.de.schaetzen@gmail.com>
Date: Sun, 15 Jun 2025 09:26:08 +0200
Subject: [PATCH] Enhance asset display by handling null builds and sorting
 assets by QR code

---
 .../java/be/seeseepuff/pcinv/controllers/WebController.java  | 5 ++++-
 src/main/resources/templates/view.html                       | 5 ++++-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/main/java/be/seeseepuff/pcinv/controllers/WebController.java b/src/main/java/be/seeseepuff/pcinv/controllers/WebController.java
index a0573d2..c2eb028 100644
--- a/src/main/java/be/seeseepuff/pcinv/controllers/WebController.java
+++ b/src/main/java/be/seeseepuff/pcinv/controllers/WebController.java
@@ -1,5 +1,6 @@
 package be.seeseepuff.pcinv.controllers;
 
+import be.seeseepuff.pcinv.models.Asset;
 import be.seeseepuff.pcinv.models.WorkLogEntry;
 import be.seeseepuff.pcinv.services.AssetService;
 import be.seeseepuff.pcinv.services.BuildService;
@@ -94,7 +95,9 @@ public class WebController {
         model.addAttribute(DESCRIPTOR, assetService.getAssetDescriptor(type));
         model.addAttribute(DESCRIPTORS, tree);
         model.addAttribute(PROPERTIES, tree.stream().flatMap(d -> d.getProperties().stream()).toList());
-        model.addAttribute(ASSETS, assetService.getAssetsByType(type));
+        var assets = assetService.getAssetsByType(type);
+        assets.sort(Comparator.comparing(Asset::getQr));
+        model.addAttribute(ASSETS, assets);
         return "browse_type";
     }
 
diff --git a/src/main/resources/templates/view.html b/src/main/resources/templates/view.html
index 241fa4d..5199060 100644
--- a/src/main/resources/templates/view.html
+++ b/src/main/resources/templates/view.html
@@ -7,7 +7,10 @@
     <table border="1" cellpadding="4">
       <tr th:each="p : ${d.properties}">
         <th bgcolor="lightgray"><b th:text="${p.displayName}"></b></th>
-        <td th:if="${p.name == 'build'}"><a th:href="'/build/' + ${asset.getAsset().getBuild().id}" th:text="${p.renderValue(asset)}"></a></td>
+        <td th:if="${p.name == 'build'}">
+          <a th:if="${asset.getAsset().getBuild() != null}" th:href="'/build/' + ${asset.getAsset().getBuild().id}" th:text="${p.renderValue(asset)}"></a>
+          <span th:if="${asset.getAsset().getBuild() == null}" th:text="${p.renderValue(asset)}"></span>
+        </td>
         <td th:if="${p.name != 'build'}" th:text="${p.renderValue(asset)}"></td>
       </tr>
     </table>