Refine deployment

.gitea/workflows/build.yml

@@ -1,5 +1,8 @@
 name: Build
-on: [push]
+on:
+  push:
+    branches:
+      - '*'
 jobs:
   build:
     runs-on: standard-latest

.gitea/workflows/deploy.yml

@@ -10,13 +10,23 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v4
 
+      - name: Setup Java
+        uses: actions/setup-java@v4
+        with:
+          distribution: 'temurin' # See 'Supported distributions' for available options
+          java-version: '21'
+          cache: 'gradle'
+
       - name: Login
         with: # Set the secret as an input
           package_rw: ${{ secrets.PACKAGE_RW }}
         run: docker login gitea.seeseepuff.be -u seeseemelk -p ${{ secrets.PACKAGE_RW }}
 
-      - name: Build
+      - name: Build Jar
-        run: ./gradlew bootBuildImage --no-daemon --imageName=gitea.seeseepuff.be/seeseemelk/pcinv:${{github.ref_name}}
+        run: ./gradlew bootJar
 
-      - name: Push
+      - name: Build Container
+        run: docker build --tag gitea.seeseepuff.be/seeseemelk/pcinv:${{github.ref_name}} .
+
+      - name: Push Container
         run: docker push gitea.seeseepuff.be/seeseemelk/pcinv:${{github.ref_name}}

Dockerfile

@@ -1,9 +1,5 @@
-FROM eclipse-temurin:21-jdk-alpine AS builder
-WORKDIR /app
-COPY . /app
-RUN ./gradlew jar --no-daemon
-
 FROM eclipse-temurin:21-alpine
-COPY --from=builder /app/build/libs/pcinv-0.0.1-SNAPSHOT.jar ./
+WORKDIR /app
-ENTRYPOINT ["java", "-jar", "pcinv-0.0.1-SNAPSHOT.jar"]
+ADD ./build/libs/pcinv-0.0.1-SNAPSHOT.jar /app/pcinv.jar
+ENTRYPOINT ["java", "-jar", "pcinv.jar"]
 EXPOSE 8088/tcp

src/main/java/be/seeseepuff/pcinv/controllers/WebController.java

@@ -1,5 +1,6 @@
 package be.seeseepuff.pcinv.controllers;
 
+import be.seeseepuff.pcinv.models.WorkLogEntry;
 import be.seeseepuff.pcinv.services.AssetService;
 import lombok.RequiredArgsConstructor;
 import org.springframework.data.repository.query.Param;
@@ -12,6 +13,8 @@ import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.PostMapping;
 import org.springframework.web.bind.annotation.RequestBody;
 
+import java.time.format.DateTimeFormatter;
+import java.util.Comparator;
 import java.util.HashMap;
 
 /**
@@ -37,6 +40,13 @@ public class WebController {
     private static final String TIME = "time";
     /// The name of the model attribute that holds the input lists for creating or editing assets.
     private static final String INPUT_LIST = "inputLists";
+    /// The name of the model attribute that holds the current work log entries.
+    private static final String WORKLOG = "worklog";
+
+    /// The name of the input field for the current size of the work log.
+    private static final String WORKLOG_SIZE = "worklog_size";
+
+    private static final DateTimeFormatter DATE_TIME_FORMATTER = DateTimeFormatter.ofPattern("dd MMM yyyy 'at' HH:mm");
 
     private final AssetService assetService;
 
@@ -84,7 +94,7 @@ public class WebController {
      * Handles the view of an asset by its QR code.
      * If the asset does not exist, it redirects to the index page.
      *
-     * @param qr    The QR code of the asset to view.
+     * @param qr The QR code of the asset to view.
      */
     @GetMapping("/view/{qr}")
     public String view(Model model, @PathVariable long qr) {
@@ -93,6 +103,35 @@ public class WebController {
         return renderView(model, qr);
     }
 
+    /**
+     * Handles the form submission for adding a work log entry to an asset.
+     *
+     * @param qr The QR code of the asset to which the work log entry is added.
+     */
+    @PostMapping("/view/{qr}")
+    public String viewPost(Model model, @PathVariable long qr, @RequestBody MultiValueMap<String, String> formData) {
+        model.addAttribute(TIME, System.currentTimeMillis());
+        model.addAttribute(ACTION, "view");
+
+        var asset = assetService.getAssetByQr(qr);
+        if (asset == null) {
+            return "redirect:/";
+        }
+
+        var workLogSizeStr = formData.getFirst(WORKLOG_SIZE);
+        if (workLogSizeStr != null) {
+            var workLogSize = Integer.parseInt(workLogSizeStr);
+            if (asset.getAsset().getWorkLog().size() == workLogSize) {
+                var comment = formData.getFirst("comment");
+                if (comment != null && !comment.isBlank()) {
+                    assetService.addWorkLogEntry(asset, comment);
+                }
+            }
+        }
+
+        return renderView(model, qr);
+    }
+
     /**
      * Show a page asking if the user wants to delete an asset.
      *
@@ -130,6 +169,9 @@ public class WebController {
         model.addAttribute(ASSET, asset);
         model.addAttribute(DESCRIPTORS, assetService.getAssetDescriptorTree(asset.getAsset().getType()));
         model.addAttribute(DESCRIPTOR, assetService.getAssetDescriptor(asset.getAsset().getType()));
+        model.addAttribute(WORKLOG, asset.getAsset().getWorkLog().stream()
+                .sorted(Comparator.comparing(WorkLogEntry::getDate).reversed())
+                .toList());
         return "view";
     }
 

src/main/java/be/seeseepuff/pcinv/meta/AssetProperty.java

@@ -203,7 +203,7 @@ public class AssetProperty {
         }
         var value = getValue(asset);
         if (value == null) {
-            return "Unknown";
+            return "?";
         } else if (type == Type.BOOLEAN) {
             return (boolean) value ? "Yes" : "No";
         } else if (type == Type.INTEGER || type == Type.STRING) {

src/main/java/be/seeseepuff/pcinv/models/AssetCondition.java

@@ -14,7 +14,7 @@ public enum AssetCondition implements AssetEnum
 	/// The asset is in perfect working order.
 	HEALTHY("healthy", "Healthy"),
 	///  The condition of the asset is unknown. E.g.: it is untested.
-	UNKNOWN("unknown", "Not known"),
+	UNKNOWN("unknown", "?"),
 	/// The asset generally works, but has some known issues.
 	PARTIAL("partial", "Partially working"),
 	/// The asset is in need of repair, but is not completely broken.

src/main/java/be/seeseepuff/pcinv/models/GenericAsset.java

@@ -5,6 +5,8 @@ import jakarta.persistence.*;
 import lombok.Getter;
 import lombok.Setter;
 
+import java.util.List;
+
 /**
  * Represents a generic asset in the inventory system.
  */
@@ -65,4 +67,7 @@ public class GenericAsset
 	@Property("Warranty Expiration Date")
 	@HideInOverview
 	private String warrantyExpirationDate;
+
+	@OneToMany(mappedBy = "asset", cascade = CascadeType.ALL, orphanRemoval = true)
+	private List<WorkLogEntry> workLog;
 }

src/main/java/be/seeseepuff/pcinv/models/OpticalDriveAsset.java

@@ -9,12 +9,12 @@ import lombok.Setter;
 @Setter
 @Entity
 @AssetInfo(
-		displayName = "CD Drive",
+		displayName = "Optical Drive",
-		pluralName = "CD Drives",
+		pluralName = "Optical Drives",
-		type = "cd_drive"
+		type = "optical_drive"
 )
-@Table(name = "cd_drive_assets")
+@Table(name = "optical_drive_assets")
-public class CdDriveAsset implements Asset
+public class OpticalDriveAsset implements Asset
 {
 	@Id
 	@GeneratedValue

src/main/java/be/seeseepuff/pcinv/models/PowerSupplyAsset.java

@@ -0,0 +1,64 @@
+package be.seeseepuff.pcinv.models;
+
+import be.seeseepuff.pcinv.meta.*;
+import jakarta.persistence.*;
+import lombok.Getter;
+import lombok.Setter;
+
+@Getter
+@Setter
+@Entity
+@AssetInfo(
+		displayName = "Power Supply",
+		pluralName = "Power Supplies",
+		type = "psu"
+)
+@Table(name = "psu_assets")
+public class PowerSupplyAsset implements Asset
+{
+	@Id
+	@GeneratedValue
+	private long id;
+
+	/// The generic asset associated with this RAM.
+	@OneToOne(orphanRemoval = true)
+	private GenericAsset asset;
+
+	@Description("The wattage rating of the power supply in watts.")
+	@Property("Wattage")
+	private int wattage;
+
+	@Description("The efficiency rating of the power supply, e.g., 80 Plus Bronze, Silver, Gold, Platinum, Titanium.")
+	@Property("Efficiency Rating")
+	@InputList
+	private String efficiencyRating;
+
+	@Description("The form factor of the power supply, e.g., ATX, SFX, etc.")
+	@Property("Form Factor")
+	@InputList
+	private String formFactor;
+
+	@Description("The number number of Molex connectors.")
+	@Property("4-pin Molex Connectors")
+	private int molexConnectors;
+
+	@Description("The number of 4-pin floppy connectors.")
+	@Property("4-pin Floppy Connectors")
+	private int floppyConnectors;
+
+	@Description("The number of SATA power connectors.")
+	@Property("15-pin SATA Connectors")
+	private int sataConnectors;
+
+	@Description("The number of 6-pin PCIe connectors.")
+	@Property("6-pin PCIe Connectors")
+	private int pcie6Connectors;
+
+	@Description("The number of 8-pin PCIe connectors.")
+	@Property("8-pin PCIe Connectors")
+	private int pcie8Connectors;
+
+	@Description("The number of 4-pin ATX 12V connectors.")
+	@Property("4-pin ATX 12V Connectors")
+	private int atx12vConnectors;
+}

src/main/java/be/seeseepuff/pcinv/models/ReadWrite.java

@@ -12,7 +12,7 @@ import lombok.RequiredArgsConstructor;
 @RequiredArgsConstructor
 public enum ReadWrite implements AssetEnum {
     /// The capacbilities are unknown.
-    UNKNOWN("unknown", "Unknown"),
+    UNKNOWN("unknown", "?"),
     /// The device can only read data.
     READ("read", "Read Only"),
     /// The device can only write data.

src/main/java/be/seeseepuff/pcinv/models/WorkLogEntry.java

@@ -0,0 +1,31 @@
+package be.seeseepuff.pcinv.models;
+
+import jakarta.persistence.Entity;
+import jakarta.persistence.GeneratedValue;
+import jakarta.persistence.Id;
+import jakarta.persistence.ManyToOne;
+import lombok.Getter;
+import lombok.Setter;
+
+import java.time.ZonedDateTime;
+
+/**
+ * Represents a work log entry in the system.
+ */
+@Setter
+@Getter
+@Entity
+public class WorkLogEntry {
+    @Id
+    @GeneratedValue
+    private long id;
+
+    @ManyToOne(optional = false)
+    private GenericAsset asset;
+
+    /// The description of the work log entry.
+    private String comment;
+
+    /// The date and time when the work log entry was created.
+    private ZonedDateTime date;
+}

src/main/java/be/seeseepuff/pcinv/repositories/CdDriveRepository.java

@@ -1,12 +0,0 @@
-package be.seeseepuff.pcinv.repositories;
-
-import be.seeseepuff.pcinv.models.CdDriveAsset;
-import org.springframework.data.jpa.repository.JpaRepository;
-
-@SuppressWarnings("unused")
-public interface CdDriveRepository extends JpaRepository<CdDriveAsset, Long>, AssetRepository<CdDriveAsset> {
-    @Override
-    default Class<CdDriveAsset> getAssetType() {
-        return CdDriveAsset.class;
-    }
-}

src/main/java/be/seeseepuff/pcinv/repositories/OpticalDriveRepository.java

@@ -0,0 +1,12 @@
+package be.seeseepuff.pcinv.repositories;
+
+import be.seeseepuff.pcinv.models.OpticalDriveAsset;
+import org.springframework.data.jpa.repository.JpaRepository;
+
+@SuppressWarnings("unused")
+public interface OpticalDriveRepository extends JpaRepository<OpticalDriveAsset, Long>, AssetRepository<OpticalDriveAsset> {
+    @Override
+    default Class<OpticalDriveAsset> getAssetType() {
+        return OpticalDriveAsset.class;
+    }
+}

src/main/java/be/seeseepuff/pcinv/repositories/PowerSupplyRepository.java

@@ -0,0 +1,12 @@
+package be.seeseepuff.pcinv.repositories;
+
+import be.seeseepuff.pcinv.models.PowerSupplyAsset;
+import org.springframework.data.jpa.repository.JpaRepository;
+
+@SuppressWarnings("unused")
+public interface PowerSupplyRepository extends JpaRepository<PowerSupplyAsset, Long>, AssetRepository<PowerSupplyAsset> {
+    @Override
+    default Class<PowerSupplyAsset> getAssetType() {
+        return PowerSupplyAsset.class;
+    }
+}

src/main/java/be/seeseepuff/pcinv/repositories/WorkLogRepository.java

@@ -0,0 +1,7 @@
+package be.seeseepuff.pcinv.repositories;
+
+import be.seeseepuff.pcinv.models.WorkLogEntry;
+import org.springframework.data.jpa.repository.JpaRepository;
+
+public interface WorkLogRepository extends JpaRepository<WorkLogEntry, Long> {
+}

src/main/java/be/seeseepuff/pcinv/services/AssetService.java

@@ -6,12 +6,16 @@ import be.seeseepuff.pcinv.meta.AssetInfo;
 import be.seeseepuff.pcinv.meta.AssetProperty;
 import be.seeseepuff.pcinv.models.Asset;
 import be.seeseepuff.pcinv.models.GenericAsset;
+import be.seeseepuff.pcinv.models.WorkLogEntry;
 import be.seeseepuff.pcinv.repositories.AssetRepository;
 import be.seeseepuff.pcinv.repositories.GenericAssetRepository;
+import be.seeseepuff.pcinv.repositories.WorkLogRepository;
+import jakarta.persistence.EntityManager;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
+import java.time.ZonedDateTime;
 import java.util.*;
 
 /**
@@ -22,7 +26,9 @@ import java.util.*;
 @RequiredArgsConstructor
 public class AssetService {
     private final GenericAssetRepository genericRepository;
+    private final WorkLogRepository workLogRepository;
     private final Collection<AssetRepository<?>> repositories;
+    private final EntityManager entityManager;
 
     /**
      * Returns the count of all assets in the repository.
@@ -260,7 +266,7 @@ public class AssetService {
         for (var descriptor : tree) {
             for (var property : descriptor.getProperties()) {
                 if (property.isInputList()) {
-                    var inputList = getInputList(descriptor, property);
+                    var inputList = getInputList(descriptor, property, type);
                     map.put(descriptor.asString(property), inputList);
                 }
             }
@@ -273,9 +279,10 @@ public class AssetService {
      *
      * @param descriptor the asset descriptor containing the property
      * @param property the asset property to retrieve the input list for
+     * @param type Limit the search to a specific type
      * @return a set of input values for the specified property
      */
-    private Set<String> getInputList(AssetDescriptor descriptor, AssetProperty property) {
+    private Set<String> getInputList(AssetDescriptor descriptor, AssetProperty property, String type) {
         List<?> entries;
         if (descriptor.getType().equals(GenericAsset.TYPE)) {
             entries = genericRepository.findAll();
@@ -286,6 +293,19 @@ public class AssetService {
 
         Set<String> inputList = new TreeSet<>();
         for (var entry : entries) {
+            String entryType;
+            if (entry instanceof Asset asset) {
+                entryType = asset.getAsset().getType();
+            } else if (entry instanceof GenericAsset asset) {
+                entryType = asset.getType();
+            } else {
+                throw new IllegalArgumentException("Unsupported entry type: " + entry.getClass().getName());
+            }
+
+            if (!entryType.equals(type)) {
+                continue; // Skip entries that do not match the specified type
+            }
+
             var value = property.getValue(entry);
             if (value != null) {
                 inputList.add(value.toString());
@@ -293,4 +313,23 @@ public class AssetService {
         }
         return inputList;
     }
+
+    /**
+     * Adds a work log entry to the asset with the specified QR code.
+     *
+     * @param asset the asset to which the work log entry will be added
+     * @param comment the comment for the work log entry
+     */
+    @Transactional
+    public void addWorkLogEntry(Asset asset, String comment) {
+        var genericAsset = asset.getAsset();
+
+        var workLogEntry = new WorkLogEntry();
+        workLogEntry.setAsset(genericAsset);
+        workLogEntry.setComment(comment);
+        workLogEntry.setDate(ZonedDateTime.now());
+
+        workLogRepository.saveAndFlush(workLogEntry);
+        entityManager.refresh(genericAsset);
+    }
 }

src/main/resources/templates/view.html

@@ -6,7 +6,7 @@
     <h2 th:text="${d.displayName}"></h2>
     <table border="1" cellpadding="4">
       <tr th:each="p : ${d.properties}">
-        <td bgcolor="lightgray"><b th:text="${p.displayName}"></b></td>
+        <th bgcolor="lightgray"><b th:text="${p.displayName}"></b></th>
         <td th:text="${p.renderValue(asset)}"></td>
       </tr>
     </table>
@@ -22,5 +22,25 @@
       <li><a th:href="'/delete/'+${asset.getQr()}+'?confirm=true'">Yes, delete it</a></li>
     </ul>
   </p>
+  <p th:if="${action == 'view'}">
+    <h2>Work Log</h2>
+    <form th:action="'/view/'+${asset.getQr()}" method="post">
+      <input type="hidden" th:value="${worklog.size()}" name="worklog_size">
+      <table border="1" cellpadding="4">
+        <tr>
+          <th bgcolor="lightgray"><b>Date</b></th>
+          <th bgcolor="lightgray"><b>Comment</b></th>
+        </tr>
+        <tr>
+          <td align="center"><input type="submit" value="Post Entry"></td>
+          <td><input type="text" name="comment" placeholder="Comment"></td>
+        </tr>
+        <tr th:each="w : ${worklog}">
+          <td th:text="${#temporals.format(w.date, 'dd/MM/yyyy')} + ' at ' + ${#temporals.format(w.date, 'HH:mm')}"></td>
+          <td th:text="${w.comment}"></td>
+        </tr>
+      </table>
+    </form>
+  </p>
 </div>
 </body>