Fixed issue current temperature issue introduced with 2.3.1

lprhodes · Apr 29, 2017 · 1d2b1f7 · 1d2b1f7
1 parent e2932d1
commit 1d2b1f7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/accessories/aircon.js b/accessories/aircon.js
@@ -323,11 +323,10 @@ class AirConAccessory extends BroadlinkRMAccessory {
     let onTemperature;
 
     onTemperature = (temperature) => {
+      if (temperature > this.config.maxTemperature) temperature = this.config.maxTemperature
+      if (temperature < this.config.minTemperature) temperature = this.config.minTemperature
       state.currentTemperature = temperature;
 
-      if (state.currentTemperature > this.config.maxTemperature) state.currentTemperature = this.config.minTemperature;
-      if (state.currentTemperature < this.config.minTemperature) state.currentTemperature = this.config.minTemperature;
-
       if (this.removeTemperatureListenerTimer) clearTimeout(this.removeTemperatureListenerTimer)
       device.removeListener('temperature', onTemperature);
       this.processQueuedCallbacks();

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "homebridge-broadlink-rm",
-  "version": "2.3.1",
+  "version": "2.3.2",
   "description": "Broadlink RM plugin (including the mini and pro) for homebridge: https://github.com/nfarina/homebridge",
   "license": "ISC",
   "keywords": [