Add CHS mapping to hard drive

.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: Build Jar
+        run: ./gradlew bootJar
+
+      - name: Build Container
+        run: docker build --tag gitea.seeseepuff.be/seeseemelk/pcinv:${{github.ref_name}} .
+
       - 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
-        run: ./gradlew bootBuildImage --no-daemon --imageName=gitea.seeseepuff.be/seeseemelk/pcinv:${{github.ref_name}}
-
-      - name: Push
+      - 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 ./
-ENTRYPOINT ["java", "-jar", "pcinv-0.0.1-SNAPSHOT.jar"]
+WORKDIR /app
+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;
 
@@ -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,13 +203,13 @@ 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) {
             return value.toString();
         } else if (type == Type.CAPACITY) {
-            return String.format("%s bytes", value);
+            return convertCapacity((Long) value).toString();
         } else if (type.isEnum) {
             if (value instanceof AssetEnum assetEnum) {
                 return assetEnum.getDisplayName();
@@ -220,6 +220,24 @@ public class AssetProperty {
         }
     }
 
+    public CapacityInfo convertCapacity(Long value) {
+        if (value == null) {
+            return null;
+        }
+        if (type != Type.CAPACITY) {
+            throw new IllegalStateException("Property '" + name + "' is not a capacity type.");
+        }
+        return CapacityInfo.of(value, capacityAsIEC, capacityAsSI);
+    }
+
+    public CapacityInfo asCapacity(@Nullable Object object) {
+        var value = getValue(object);
+        if (value == null) {
+            return null;
+        }
+        return convertCapacity((Long) value);
+    }
+
     @Override
     public String toString() {
         var enumOptions = "";

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

@@ -0,0 +1,62 @@
+package be.seeseepuff.pcinv.meta;
+
+import lombok.Getter;
+import lombok.RequiredArgsConstructor;
+
+import java.util.Arrays;
+import java.util.Comparator;
+
+/**
+ * Represents a capacity in bytes, with various units for display.
+ * This class is used to encapsulate the capacity information and provide
+ * a way to represent it in different units.
+ */
+@Getter
+@RequiredArgsConstructor
+public class CapacityInfo {
+    private final long capacity;
+    private final CapacityUnit idealUnit;
+
+    public static CapacityInfo of(long capacity, CapacityUnit idealUnit) {
+        return new CapacityInfo(capacity, idealUnit);
+    }
+
+    public static CapacityInfo of(long capacity) {
+        return of(capacity, idealUnitForCapacity(capacity, CapacityUnit.values()));
+    }
+
+    public static CapacityInfo ofSI(long capacity) {
+        return of(capacity, idealUnitForCapacity(capacity, CapacityUnit.SI_UNITS));
+    }
+
+    public static CapacityInfo ofIEC(long capacity) {
+        return of(capacity, idealUnitForCapacity(capacity, CapacityUnit.IEC_UNITS));
+    }
+
+    public static CapacityInfo of(long capacity, boolean iec, boolean si) {
+        if (iec && !si) {
+            return ofIEC(capacity);
+        } else if (si && !iec) {
+            return ofSI(capacity);
+        } else {
+            return of(capacity);
+        }
+    }
+
+    public static CapacityUnit idealUnitForCapacity(long capacity, CapacityUnit[] units) {
+        return Arrays.stream(units)
+                .sorted(Comparator.comparing(CapacityUnit::getBytes).reversed())
+                .filter(unit -> capacity % unit.getBytes() == 0)
+                .findFirst()
+                .orElse(CapacityUnit.BYTES);
+    }
+
+    public long getCapacityInUnit() {
+        return capacity / idealUnit.getBytes();
+    }
+
+    @Override
+    public String toString() {
+        return String.format("%d %s", capacity / idealUnit.getBytes(), idealUnit.getDisplayName());
+    }
+}

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

@@ -0,0 +1,28 @@
+package be.seeseepuff.pcinv.meta;
+
+import lombok.Getter;
+import lombok.RequiredArgsConstructor;
+
+/**
+ * Represents a unit of capacity, either in binary (IEC) or decimal (SI) format.
+ */
+@Getter
+@RequiredArgsConstructor
+public enum CapacityUnit {
+    BYTES("Bytes", 1),
+    KIBIBYTES("KiB", 1024),
+    MEBIBYTES("MiB", 1024 * 1024),
+    GIBIBYTES("GiB", 1024 * 1024 * 1024),
+    TEBIBYTES("TiB", 1024L * 1024 * 1024 * 1024),
+    KILOBYTES("kB", 1000),
+    MEGABYTES("MB", 1000 * 1000),
+    GIGABYTES("GB", 1000 * 1000 * 1000),
+    TERABYTES("TB", 1000L * 1000 * 1000 * 1000),
+    ;
+
+    public static final CapacityUnit[] SI_UNITS = {BYTES, KILOBYTES, MEGABYTES, GIGABYTES, TERABYTES};
+    public static final CapacityUnit[] IEC_UNITS = {BYTES, KIBIBYTES, MEBIBYTES, GIBIBYTES, TEBIBYTES};
+
+    private final String displayName;
+    private final long bytes;
+}

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/CustomAsset.java

@@ -0,0 +1,28 @@
+package be.seeseepuff.pcinv.models;
+
+import be.seeseepuff.pcinv.meta.*;
+import jakarta.persistence.*;
+import lombok.Getter;
+import lombok.Setter;
+
+/**
+ * Represents a CPU or similar device.
+ */
+@Getter
+@Setter
+@Entity
+@AssetInfo(
+		displayName = "Custom Device",
+		pluralName = "Custom Devices",
+		type = "custom"
+)
+@Table(name = "custom_assets")
+public class CustomAsset implements Asset
+{
+	@Id
+	@GeneratedValue
+	private long id;
+
+	@OneToOne(orphanRemoval = true)
+	private GenericAsset asset;
+}

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/HddAsset.java

@@ -40,12 +40,12 @@ public class HddAsset implements Asset
 	@InputList
 	private String formFactor;
 
-	@Description("The drive's RPM (Revolutions Per Minute) speed, if applicable.")
+	@Description("The drive's RPM (Revolutions Per Minute) speed.")
 	@Property("RPM Speed")
 	@HideInOverview
 	private Long rpmSpeed;
 
-	@Description("The drive's cache size, if applicable.")
+	@Description("The drive's cache size.")
 	@Property("Cache Size")
 	@HideInOverview
 	private Long cacheSize;
@@ -54,4 +54,19 @@ public class HddAsset implements Asset
 	@Property("Drive Type")
 	@InputList
 	private String driveType;
+
+	@Description("Number of heads in the drive.")
+	@Property("Heads")
+	@HideInOverview
+	private Integer heads;
+
+	@Description("Number of cylinders in the drive.")
+	@Property("Cylinders")
+	@HideInOverview
+	private Integer cylinders;
+
+	@Description("Number of sectors in the drive.")
+	@Property("Sectors")
+	@HideInOverview
+	private Integer sectors;
 }

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

@@ -9,12 +9,12 @@ import lombok.Setter;
 @Setter
 @Entity
 @AssetInfo(
-		displayName = "CD Drive",
-		pluralName = "CD Drives",
-		type = "cd_drive"
+		displayName = "Optical Drive",
+		pluralName = "Optical Drives",
+		type = "optical_drive"
 )
-@Table(name = "cd_drive_assets")
-public class CdDriveAsset implements Asset
+@Table(name = "optical_drive_assets")
+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/RamAsset.java

@@ -37,5 +37,5 @@ public class RamAsset implements Asset
 
 	@Description("The speed of the memory in MHz.")
 	@Property("Speed")
-	private Long speed;
+	private Integer speed;
 }

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/CustomRepository.java

@@ -0,0 +1,12 @@
+package be.seeseepuff.pcinv.repositories;
+
+import be.seeseepuff.pcinv.models.CustomAsset;
+import org.springframework.data.jpa.repository.JpaRepository;
+
+@SuppressWarnings("unused")
+public interface CustomRepository extends JpaRepository<CustomAsset, Long>, AssetRepository<CustomAsset> {
+    @Override
+    default Class<CustomAsset> getAssetType() {
+        return CustomAsset.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();
@@ -284,8 +291,21 @@ public class AssetService {
             entries = repository.findAll();
         }
 
-        Set<String> inputList = new TreeSet<>();
+        var inputList = new TreeSet<String>(Comparator.comparing(String::toLowerCase));
         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/create_asset.html

@@ -31,17 +31,17 @@
 							<option th:each="o : ${p.options}" th:value="${o.value}" th:text="${o.displayName}" th:selected="${asset != null ? (p.getValue(asset) == o.enumConstant) : o.defaultValue}">Good</option>
 						</select>
 						<span th:case="CAPACITY">
-							<input type="number" th:id="${d.asString(p)+'-value'}" th:name="${d.asString(p)+'-value'}" th:required="${p.required}"/>
+							<input type="number" th:id="${d.asString(p)+'-value'}" th:name="${d.asString(p)+'-value'}" th:required="${p.required}" th:value="${p.asCapacity(asset)?.getCapacityInUnit() ?: ''}"/>
 							<select th:id="${d.asString(p)}+'-unit'" th:name="${d.asString(p)}+'-unit'">
-								<option value="1">Bytes</option>
-								<option th:if="${p.capacityAsSI}">kB</option>
-								<option th:if="${p.capacityAsIEC}">KiB</option>
-								<option th:if="${p.capacityAsSI}">MB</option>
-								<option th:if="${p.capacityAsIEC}">MiB</option>
-								<option th:if="${p.capacityAsSI}">GB</option>
-								<option th:if="${p.capacityAsIEC}">GiB</option>
-								<option th:if="${p.capacityAsSI}">TB</option>
-								<option th:if="${p.capacityAsIEC}">TiB</option>
+								<option value="1" th:selected="${p.asCapacity(asset)?.getIdealUnit()?.name() == 'BYTES'}">Bytes</option>
+								<option th:value="${1000}" th:if="${p.capacityAsSI}" th:selected="${p.asCapacity(asset)?.getIdealUnit()?.name() == 'KILOBYTES'}">kB</option>
+								<option th:value="${1024}" th:if="${p.capacityAsIEC}" th:selected="${p.asCapacity(asset)?.getIdealUnit()?.name() == 'KIBIBYTES'}">KiB</option>
+								<option th:value="${1000*1000}" th:if="${p.capacityAsSI}" th:selected="${p.asCapacity(asset)?.getIdealUnit()?.name() == 'MEGABYTES'}">MB</option>
+								<option th:value="${1024*1024}" th:if="${p.capacityAsIEC}" th:selected="${p.asCapacity(asset)?.getIdealUnit()?.name() == 'MEBIBYTES'}">MiB</option>
+								<option th:value="${1000*1000*1000}" th:if="${p.capacityAsSI}" th:selected="${p.asCapacity(asset)?.getIdealUnit()?.name() == 'GIGABYTES'}">GB</option>
+								<option th:value="${1024*1024*1024}" th:if="${p.capacityAsIEC}" th:selected="${p.asCapacity(asset)?.getIdealUnit()?.name() == 'GIBIBYTES'}">GiB</option>
+								<option value="1000000000000" th:if="${p.capacityAsSI}" th:selected="${p.asCapacity(asset)?.getIdealUnit()?.name() == 'TERABYTES'}">TB</option>
+								<option value="1099511627776" th:if="${p.capacityAsIEC}" th:selected="${p.asCapacity(asset)?.getIdealUnit()?.name() == 'TEBIBYTES'}">TiB</option>
 							</select>
 						</span>
 						<b th:case="*">Bad input type for <span th:text="${d.type}+'-'+${p.type}"></span></b>

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>
@@ -15,6 +15,7 @@
     <ul th:if="${action == 'view'}">
       <li><a th:href="'/edit/'+${asset.getQr()}">Edit</a></li>
       <li><a th:href="'/delete/'+${asset.getQr()}">Delete</a></li>
+      <li><a th:href="'/create/'+${asset.getAsset().type}">Create another <span th:text="${descriptor.displayName}">Hard Drive</span> </a></li>
       <li><a th:href="'/browse/'+${descriptor.type}">Browse all <span th:text="${descriptor.pluralName}">Hard Drives</span></a></li>
     </ul>
     <ul th:if="${action == 'delete'}">
@@ -22,5 +23,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>