Another attempt at fixing sendTemperatureOnlyWhenOff

lprhodes · Mar 20, 2018 · ac8aefa · ac8aefa
1 parent d789c71
commit ac8aefa
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/accessories/aircon.js b/accessories/aircon.js
@@ -246,20 +246,18 @@ class AirConAccessory extends BroadlinkRMAccessory {
       }
     }
 
-    if (state.currentHeatingCoolingState !== Characteristic.TargetHeatingCoolingState.OFF && sendTemperatureOnlyWhenOff) {
-      return;
-    }
-
     state.firstTemperatureUpdate = false;
 
     // Send the temperature hex
     this.log(`${name} sendTemperature (${state.targetTemperature}`);
     await this.performSend(hexData.data);
 
 
-    // Update the heating/cooling mode based on the temperature.
-
+    if (!state.currentHeatingCoolingState && sendTemperatureOnlyWhenOff) {
+      return;
+    }
 
+    // Update the heating/cooling mode based on the temperature.
     let mode = hexData['pseudo-mode'];
 
     if (mode) assert.oneOf(mode, [ 'heat', 'cool', 'auto' ], '\x1b[31m[CONFIG ERROR] \x1b[33mpseudo-mode\x1b[0m should be one of "heat", "cool" or "auto"');