[max] Fix inadvertently change automatic/manual mode openhab#4347 (op…

…enhab#4755) * Fix inadvertently change automatic/manual mode Addresses issue Setting temperature can inadvertently change automatic/manual mode Fixes openhab#4347 Signed-off-by: Marcel Verpaalen <[email protected]>
hreichert · Jan 30, 2019 · 8f80805 · 8f80805
1 parent e77b1ea
commit 8f80805
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/....binding.max/src/main/java/org/openhab/binding/max/internal/device/HeatingThermostat.java b/....binding.max/src/main/java/org/openhab/binding/max/internal/device/HeatingThermostat.java
@@ -76,7 +76,7 @@ public ThermostatModeType getMode() {
         return this.mode;
     }
 
-    void setMode(ThermostatModeType mode) {
+    public void setMode(ThermostatModeType mode) {
         if (this.mode != mode) {
             setUpdated(true);
         }

diff --git a/...ding.max/src/main/java/org/openhab/binding/max/internal/handler/MaxCubeBridgeHandler.java b/...ding.max/src/main/java/org/openhab/binding/max/internal/handler/MaxCubeBridgeHandler.java
@@ -436,10 +436,13 @@ private CubeCommand getCommand(SendCommand sendCommand) {
                 String commandContent = command.toString().trim().toUpperCase();
                 double setTemp = device.getTemperatureSetpoint();
                 if (commandContent.contentEquals(ThermostatModeType.AUTOMATIC.toString())) {
+                    device.setMode(ThermostatModeType.AUTOMATIC);
                     return new SCommand(device.getRFAddress(), device.getRoomId(), ThermostatModeType.AUTOMATIC, 0D);
                 } else if (commandContent.contentEquals(ThermostatModeType.BOOST.toString())) {
+                    device.setMode(ThermostatModeType.BOOST);
                     return new SCommand(device.getRFAddress(), device.getRoomId(), ThermostatModeType.BOOST, setTemp);
                 } else if (commandContent.contentEquals(ThermostatModeType.MANUAL.toString())) {
+                    device.setMode(ThermostatModeType.MANUAL);
                     logger.debug("updates to MANUAL mode with temperature '{}'", setTemp);
                     return new SCommand(device.getRFAddress(), device.getRoomId(), ThermostatModeType.MANUAL, setTemp);
                 } else {