From: Mark Hilbush <mark@hilbush.com>
Date: Thu, 22 Oct 2020 23:38:19 +0000 (-0400)
Subject: [ecobee] Fix setHold when using holdHours (#8834)
X-Git-Url: https://git.basschouten.com/?a=commitdiff_plain;h=e8280b2ef95552463c15f311732e164082de04c7;p=openhab-addons.git

[ecobee] Fix setHold when using holdHours (#8834)

Signed-off-by: Mark Hilbush <mark@hilbush.com>
---

diff --git a/bundles/org.openhab.binding.ecobee/src/main/java/org/openhab/binding/ecobee/internal/action/EcobeeActions.java b/bundles/org.openhab.binding.ecobee/src/main/java/org/openhab/binding/ecobee/internal/action/EcobeeActions.java
index a9ae8d4ebe..911d53d9e0 100644
--- a/bundles/org.openhab.binding.ecobee/src/main/java/org/openhab/binding/ecobee/internal/action/EcobeeActions.java
+++ b/bundles/org.openhab.binding.ecobee/src/main/java/org/openhab/binding/ecobee/internal/action/EcobeeActions.java
@@ -306,9 +306,7 @@ public class EcobeeActions implements ThingActions {
         Map<String, Object> params = new HashMap<String, Object>();
         params.put("coolHoldTemp", coolHoldTemp);
         params.put("heatHoldTemp", heatHoldTemp);
-        params.put("holdType", HoldType.HOLD_HOURS);
-        params.put("holdHours", Integer.valueOf(holdHours.intValue()));
-        return setHold(params, null, null, null, null);
+        return setHold(params, HoldType.HOLD_HOURS.toString(), holdHours, null, null);
     }
 
     public static boolean setHold(ThingActions actions, @Nullable QuantityType<Temperature> coolHoldTemp,
@@ -359,9 +357,7 @@ public class EcobeeActions implements ThingActions {
         }
         Map<String, Object> params = new HashMap<String, Object>();
         params.put("holdClimateRef", holdClimateRef);
-        params.put("holdType", HoldType.HOLD_HOURS);
-        params.put("holdHours", Integer.valueOf(holdHours.intValue()));
-        return setHold(params, null, null, null, null);
+        return setHold(params, HoldType.HOLD_HOURS.toString(), holdHours, null, null);
     }
 
     public static boolean setHold(ThingActions actions, @Nullable String holdClimateRef, @Nullable Number holdHours) {