Protect maximum temperature setting

Signed-off-by: clinique <gael@lhopital.org>
openhab · Feb 25, 2022 · df04e05 · df04e05
1 parent 6526f69
commit df04e05
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/...hab.binding.netatmo/src/main/java/org/openhab/binding/netatmo/internal/api/EnergyApi.java b/...hab.binding.netatmo/src/main/java/org/openhab/binding/netatmo/internal/api/EnergyApi.java
@@ -88,7 +88,7 @@ public void setRoomThermpoint(String homeId, String roomId, SetpointMode mode, l
         if (mode == SetpointMode.MANUAL || mode == SetpointMode.MAX) {
             uriBuilder.queryParam("endtime", endtime);
             if (mode == SetpointMode.MANUAL) {
-                uriBuilder.queryParam("temp", temp);
+                uriBuilder.queryParam("temp", temp > 30 ? 30 : temp);
             }
         }
         post(uriBuilder, ApiResponse.Ok.class, null);

diff --git a/...c/main/java/org/openhab/binding/netatmo/internal/handler/capability/EnergyCapability.java b/...c/main/java/org/openhab/binding/netatmo/internal/handler/capability/EnergyCapability.java
@@ -103,7 +103,7 @@ public int getSetpointDefaultDuration() {
     public void callSetRoomThermTemp(String roomId, double temperature) {
         try {
             api.setRoomThermpoint(homeId, roomId, SetpointMode.MANUAL, setpointEndTimeFromNow(setPointDefaultDuration),
-                    temperature > 30 ? 30 : temperature);
+                    temperature);
             expireData();
         } catch (NetatmoException e) {
             logger.warn("Error setting room target temperature '{}' : {}", roomId, e.getMessage());

diff --git a/bundles/org.openhab.binding.netatmo/src/main/resources/OH-INF/thing/channels.xml b/bundles/org.openhab.binding.netatmo/src/main/resources/OH-INF/thing/channels.xml
@@ -155,9 +155,9 @@
 		<label>Setpoint</label>
 		<description>Thermostat temperature setpoint.</description>
 		<category>Temperature</category>
-        <tags>
-            <tag>Setpoint</tag>
-            <tag>Temperature</tag>
+		<tags>
+			<tag>Setpoint</tag>
+			<tag>Temperature</tag>
 		</tags>
 		<state pattern="%.1f %unit%" readOnly="false" min="7" max="30" step="0.5"/>
 	</channel-type>