homebridge-tasmota-control 0.4.4 → 0.4.45-beta.2

package/CHANGELOG.md

@@ -4,6 +4,10 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.4.5] - (08.08.2022)
+## Changes
+- fix report wrong device state [#4](https://github.com/grzegorz914/homebridge-tasmota-control/issues/4)
+
 ## [0.4.4] - (23.07.2022)
 ## Changes
 - refactor information service

package/index.js

@@ -88,8 +88,6 @@ class tasmotaDevice {
 
     //setup variables
     this.channelsCount = 0;
-    this.checkDeviceInfo = true;
-    this.checkDeviceState = false;
     this.startPrepareAccessory = true;
 
     this.prefDir = path.join(api.user.storagePath(), 'tasmota');
@@ -106,14 +104,19 @@ class tasmotaDevice {
       fs.mkdirSync(this.prefDir);
     }
 
-    //Check device state
-    setInterval(function () {
-      if (this.checkDeviceInfo) {
-        this.getDeviceInfo();
-      } else {
-        this.updateDeviceState();
-      }
-    }.bind(this), this.refreshInterval * 1000);
+    this.getDeviceInfo();
+  }
+
+  reconnect() {
+    setTimeout(() => {
+      this.getDeviceInfo();
+    }, 10000);
+  }
+
+  checkDeviceState() {
+    setTimeout(() => {
+      this.updateDeviceState();
+    }, this.refreshInterval * 1000);
   }
 
   async getDeviceInfo() {
@@ -141,9 +144,10 @@ class tasmotaDevice {
       this.firmwareRevision = firmwareRevision;
       this.channelsCount = channelsCount;
 
-      this.checkDeviceInfo = (channelsCount == 0);
+      this.updateDeviceState();
     } catch (error) {
-      this.log.error('Device: %s %s, Device Info eror: %s, state: Offline, trying to reconnect', this.host, this.name, error);
+      this.log.error('Device: %s %s, Device Info eror: %s, state: Offline, trying to reconnect in 10s.', this.host, this.name, error);
+      this.reconnect();
     }
   }
 
@@ -155,7 +159,7 @@ class tasmotaDevice {
 
       this.powerState = new Array();
       for (let i = 0; i < this.channelsCount; i++) {
-        const channel = this.channelsCount == 1 ? 'POWER' : 'POWER' + (i + 1);
+        const channel = this.channelsCount == 1 ? 'POWER' || 'POWER1' : 'POWER' + (i + 1);
         const powerState = (response.data[channel] == 'ON');
         if (this.tasmotaServices) {
           this.tasmotaServices[i]
@@ -163,16 +167,16 @@ class tasmotaDevice {
         }
         this.powerState.push(powerState);
       }
-      this.checkDeviceState = true;
+
+      this.checkDeviceState();
 
       //start prepare accessory
-      if (this.startPrepareAccessory) {
+      if (this.startPrepareAccessory && this.serialNumber) {
         this.prepareAccessory();
       }
     } catch (error) {
-      this.log.error('Device: %s %s, update Device state error: %s, state: Offline', this.host, this.name, error);
-      this.checkDeviceState = false;
-      this.checkDeviceInfo = true;
+      this.log.error('Device: %s %s, update Device state error: %s, trying again.', this.host, this.name, error);
+      this.checkDeviceState();
     }
   }
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "displayName": "Tasmota Control",
   "name": "homebridge-tasmota-control",
-  "version": "0.4.4",
+  "version": "0.4.45-beta.2",
   "description": "Homebridge plugin (https://github.com/homebridge/homebridge) to control Tasmota flashed devices.",
   "license": "MIT",
   "author": "grzegorz914",