From 80fc5db373755b8722daa511966962e04859fcee Mon Sep 17 00:00:00 2001
From: lolodomo <lg.hc@free.fr>
Date: Tue, 1 Oct 2024 22:07:55 +0200
Subject: [PATCH] [warmup] Set Dimension to QuantityType (#17492)

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
---
 .../warmup/internal/action/WarmupActions.java      | 11 +++++++----
 .../warmup/internal/handler/RoomHandler.java       | 14 +++++++++-----
 .../internal/handler/WarmupThingHandler.java       |  4 +++-
 3 files changed, 19 insertions(+), 10 deletions(-)

diff --git a/bundles/org.openhab.binding.warmup/src/main/java/org/openhab/binding/warmup/internal/action/WarmupActions.java b/bundles/org.openhab.binding.warmup/src/main/java/org/openhab/binding/warmup/internal/action/WarmupActions.java
index 3960205068ca6..964127926e6de 100644
--- a/bundles/org.openhab.binding.warmup/src/main/java/org/openhab/binding/warmup/internal/action/WarmupActions.java
+++ b/bundles/org.openhab.binding.warmup/src/main/java/org/openhab/binding/warmup/internal/action/WarmupActions.java
@@ -12,6 +12,9 @@
  */
 package org.openhab.binding.warmup.internal.action;
 
+import javax.measure.quantity.Temperature;
+import javax.measure.quantity.Time;
+
 import org.eclipse.jdt.annotation.NonNullByDefault;
 import org.eclipse.jdt.annotation.Nullable;
 import org.openhab.binding.warmup.internal.handler.RoomHandler;
@@ -53,8 +56,8 @@ public void setThingHandler(@Nullable ThingHandler handler) {
 
     @RuleAction(label = "override", description = "Overrides the thermostat state for a specified time")
     public void setOverride(
-            @ActionInput(name = "temperature", label = "Temperature") @Nullable QuantityType<?> temperature,
-            @ActionInput(name = "duration", label = "Duration") @Nullable QuantityType<?> duration) {
+            @ActionInput(name = "temperature", label = "Temperature") @Nullable QuantityType<Temperature> temperature,
+            @ActionInput(name = "duration", label = "Duration") @Nullable QuantityType<Time> duration) {
         logger.debug("setOverride action called");
         RoomHandler handler = this.handler;
         if (handler != null && temperature != null && duration != null) {
@@ -64,8 +67,8 @@ public void setOverride(
         }
     }
 
-    public static void setOverride(@Nullable ThingActions actions, @Nullable QuantityType<?> temperature,
-            @Nullable QuantityType<?> duration) {
+    public static void setOverride(@Nullable ThingActions actions, @Nullable QuantityType<Temperature> temperature,
+            @Nullable QuantityType<Time> duration) {
         if (actions instanceof WarmupActions warmupActions) {
             warmupActions.setOverride(temperature, duration);
         } else {
diff --git a/bundles/org.openhab.binding.warmup/src/main/java/org/openhab/binding/warmup/internal/handler/RoomHandler.java b/bundles/org.openhab.binding.warmup/src/main/java/org/openhab/binding/warmup/internal/handler/RoomHandler.java
index 9840d8a72b2b6..9f8fbf2a90580 100644
--- a/bundles/org.openhab.binding.warmup/src/main/java/org/openhab/binding/warmup/internal/handler/RoomHandler.java
+++ b/bundles/org.openhab.binding.warmup/src/main/java/org/openhab/binding/warmup/internal/handler/RoomHandler.java
@@ -18,8 +18,12 @@
 import java.util.Map;
 import java.util.Set;
 
+import javax.measure.quantity.Temperature;
+import javax.measure.quantity.Time;
+
 import org.eclipse.jdt.annotation.NonNullByDefault;
 import org.eclipse.jdt.annotation.Nullable;
+import org.openhab.binding.warmup.internal.WarmupBindingConstants.RoomMode;
 import org.openhab.binding.warmup.internal.action.WarmupActions;
 import org.openhab.binding.warmup.internal.api.MyWarmupApi;
 import org.openhab.binding.warmup.internal.api.MyWarmupApiException;
@@ -68,11 +72,11 @@ public void handleCommand(ChannelUID channelUID, Command command) {
         super.handleCommand(channelUID, command);
         if (CHANNEL_TARGET_TEMPERATURE.equals(channelUID.getId())
                 && command instanceof QuantityType<?> quantityCommand) {
-            setOverride(quantityCommand);
+            setOverride((QuantityType<Temperature>) quantityCommand);
         }
         if (CHANNEL_FIXED_TEMPERATURE.equals(channelUID.getId())
                 && command instanceof QuantityType<?> quantityCommand) {
-            setFixed(quantityCommand);
+            setFixed((QuantityType<Temperature>) quantityCommand);
         }
         if (CHANNEL_FROST_PROTECTION_MODE.equals(channelUID.getId()) && command instanceof OnOffType onOffCommand) {
             toggleFrostProtectionMode(onOffCommand);
@@ -138,11 +142,11 @@ public Collection<Class<? extends ThingHandlerService>> getServices() {
         return Set.of(WarmupActions.class);
     }
 
-    private void setOverride(final QuantityType<?> command) {
+    private void setOverride(final QuantityType<Temperature> command) {
         setOverride(command, new QuantityType<>(config.getOverrideDuration(), Units.MINUTE));
     }
 
-    public void setOverride(final QuantityType<?> temperature, final QuantityType<?> duration) {
+    public void setOverride(final QuantityType<Temperature> temperature, final QuantityType<Time> duration) {
         setOverride(formatTemperature(temperature), duration.toUnit(Units.MINUTE).intValue());
     }
 
@@ -163,7 +167,7 @@ private void setOverride(final int temperature, final int duration) {
         }
     }
 
-    private void setFixed(final QuantityType<?> command) {
+    private void setFixed(final QuantityType<Temperature> command) {
         try {
             RoomCallout rc = getCallout();
             rc.api.setFixed(rc.locationId, rc.roomId, formatTemperature(command));
diff --git a/bundles/org.openhab.binding.warmup/src/main/java/org/openhab/binding/warmup/internal/handler/WarmupThingHandler.java b/bundles/org.openhab.binding.warmup/src/main/java/org/openhab/binding/warmup/internal/handler/WarmupThingHandler.java
index faafbb5bc966d..b05c2cfe30359 100644
--- a/bundles/org.openhab.binding.warmup/src/main/java/org/openhab/binding/warmup/internal/handler/WarmupThingHandler.java
+++ b/bundles/org.openhab.binding.warmup/src/main/java/org/openhab/binding/warmup/internal/handler/WarmupThingHandler.java
@@ -12,6 +12,8 @@
  */
 package org.openhab.binding.warmup.internal.handler;
 
+import javax.measure.quantity.Temperature;
+
 import org.eclipse.jdt.annotation.NonNullByDefault;
 import org.eclipse.jdt.annotation.Nullable;
 import org.openhab.core.library.types.QuantityType;
@@ -104,7 +106,7 @@ protected State parseTemperature(@Nullable Integer temperature) {
      * @param temperature {@link QuantityType} a temperature
      * @return the temperature as an int in degrees C * 10. i.e. 21.5 degrees C = 215
      */
-    protected int formatTemperature(QuantityType<?> temperature) {
+    protected int formatTemperature(QuantityType<Temperature> temperature) {
         return (int) (temperature.toUnit(SIUnits.CELSIUS).doubleValue() * 10);
     }