iobroker.zigbee 1.7.5 → 1.8.0

package/lib/zigbeecontroller.js

@@ -19,7 +19,7 @@ const groupConverters = [
     zigbeeHerdsmanConverters.toZigbeeConverters.light_brightness_step,
     zigbeeHerdsmanConverters.toZigbeeConverters.light_colortemp_move,
     zigbeeHerdsmanConverters.toZigbeeConverters.light_colortemp_step,
-    zigbeeHerdsmanConverters.toZigbeeConverters.ight_colortemp_startup
+    zigbeeHerdsmanConverters.toZigbeeConverters.light_colortemp_startup
 ];
 
 function isFunction(functionToCheck) {
@@ -118,13 +118,12 @@ class ZigbeeController extends EventEmitter {
             }
 
             this.debug(`Zigbee network parameters: panID=${debNetworkParam.panID} channel=${debNetworkParam.channel} extendedPanID=${extPanIDDebug}`);
-
         } catch (e) {
             this.sendError(e);
-            this.error('Starting zigbee-herdsman problem : ' + JSON.stringify(e.message));
+            this.error(`Starting zigbee-herdsman problem : ${JSON.stringify(e.message)}`);
             throw 'Error herdsman start';
         }
-        // Check if we have to turn off the led
+        // Check if we have to turn off the LED
         try {
             if (this.disableLed) {
                 this.info('Disable LED');
@@ -141,7 +140,7 @@ class ZigbeeController extends EventEmitter {
         let powerText = 'normal';
 
         if (this.transmitPower != '0') {
-            switch(this.transmitPower) {
+            switch (this.transmitPower) {
                 case '-22':
                     powerText = 'low';
                     break;
@@ -157,7 +156,7 @@ class ZigbeeController extends EventEmitter {
         }
 
 
-        this.info('  --> transmitPower : ' + powerText);
+        this.info(`  --> transmitPower : ${powerText}`);
         try {
             await this.herdsman.setTransmitPower(this.transmitPower);
         } catch (e) {
@@ -165,7 +164,6 @@ class ZigbeeController extends EventEmitter {
             this.info('Unable to set transmit power, unsupported function.');
         }
 
-
         // Call extensions
         this.callExtensionMethod('onZigbeeStarted', []);
 
@@ -176,20 +174,21 @@ class ZigbeeController extends EventEmitter {
         } else {
             this.info(`Currently no devices.`);
         }
+
         for (const device of devices) {
             const entity = await this.resolveEntity(device);
-            this.adapter.getObject(device.ieeeAddr.substr(2),(err, obj) => {
-                if (obj && obj.common && obj.common.deactivated)
-                {
+            this.adapter.getObject(device.ieeeAddr.substr(2), (err, obj) => {
+                if (obj && obj.common && obj.common.deactivated) {
                     this.callExtensionMethod('deregisterDevicePing', [device, entity]);
-                }
-                else {
+                } else {
                     this.callExtensionMethod('registerDevicePing', [device, entity]);
                 }
             });
             // ensure that objects for all found clients are present
 
-            if (entity.mapped) this.emit('new', entity);
+            if (entity.mapped) {
+                this.emit('new', entity);
+            }
             this.info(
                 (entity.device.ieeeAddr) +
                 ` (addr ${entity.device.networkAddress}): ` +
@@ -261,8 +260,7 @@ class ZigbeeController extends EventEmitter {
 
     async getGroups() {
         try {
-            const rv = await this.herdsman.getGroups();
-            return rv;
+            return await this.herdsman.getGroups();
         } catch (error) {
             this.sendError(error);
             this.error(JSON.stringify(error));
@@ -271,12 +269,12 @@ class ZigbeeController extends EventEmitter {
     }
 
     async removeGroupById(id) {
-        const group = await  this.getGroupByID(id);
+        const group = await this.getGroupByID(id);
         try {
-            if (group) group.removeFromDatabase();
+            group && group.removeFromDatabase();
         } catch (error) {
             this.sendError(error);
-            this.error('error in removeGroupById: ' + error);
+            this.error(`error in removeGroupById: ${error}`);
         }
     }
 
@@ -290,16 +288,15 @@ class ZigbeeController extends EventEmitter {
     }
 
     async verifyGroupExists(id) {
-      const nid = (typeof(id) === 'number' ? id:parseInt(id));
+        const nid = (typeof (id) === 'number' ? id : parseInt(id));
         let group = await this.herdsman.getGroupByID(nid);
         if (!group) {
             group = await this.herdsman.createGroup(nid);
             group.toZigbee = groupConverters;
             group.model = 'group';
-            this.debug('verifyGroupExists: created group ' + nid);
-        }
-        else {
-            this.debug('verifyGroupExists: group ' + nid + ' exists');
+            this.debug(`verifyGroupExists: created group ${nid}`);
+        } else {
+            this.debug(`verifyGroupExists: group ${nid} exists`);
         }
 
     }
@@ -316,15 +313,17 @@ class ZigbeeController extends EventEmitter {
                     const device = this.getDeviceByNetworkAddress(nwk);
                     if (device && device.ieeeAddr) members.push( { ieee:device.ieeeAddr, model:device.modelID, epid:epid, ep:member } );
                 }
-            }
-            else {
+            } else {
                 return undefined;
             }
 
         } catch (error) {
             this.sendError(error);
-            if (error) this.error('getGroupMembersFromController: error is  ' + JSON.stringify(error) + ' ' + JSON.stringify(new Error().stack));
-            else this.error('unidentifed error in getGroupMembersFromController');
+            if (error) {
+                this.error(`getGroupMembersFromController: error is  ${JSON.stringify(error)} ${JSON.stringify(new Error().stack)}`);
+            } else {
+                this.error('unidentified error in getGroupMembersFromController');
+            }
         }
         return members;
     }
@@ -404,7 +403,7 @@ class ZigbeeController extends EventEmitter {
         }
     }
 
-    async incMsgHandler(message){
+    async incMsgHandler(message) {
         this.debug('incoming msg', message);
         const device = await this.herdsman.getDeviceByIeeeAddr(message.srcaddr);
         if (!device) {
@@ -462,8 +461,8 @@ class ZigbeeController extends EventEmitter {
         } else {
             this.info('Zigbee: disabling joining new devices.');
         }
-        try
-        {
+
+        try {
             if (permitTime && !this.herdsman.getPermitJoin()) {
                 clearInterval(this._permitJoinInterval);
                 this._permitJoinTime = permitTime;
@@ -509,6 +508,7 @@ class ZigbeeController extends EventEmitter {
                         this.debug(`Force remove`);
                     }
                 }
+
                 try {
                     await device.removeFromDatabase();
                 } catch (error) {
@@ -518,7 +518,7 @@ class ZigbeeController extends EventEmitter {
                         this.debug(`Failed to remove from DB ${error.stack}`);
                 }
                 this.debug('Remove successful.');
-                if (callback) callback();
+                callback && callback();
                 this.callExtensionMethod(
                     'onDeviceRemove',
                     [device],
@@ -527,7 +527,7 @@ class ZigbeeController extends EventEmitter {
         } catch (error) {
             this.sendError(error);
             this.error(`Failed to remove ${error.stack}`);
-            if (callback) callback(`Failed to remove ${error.stack}`);
+            callback && callback(`Failed to remove ${error.stack}`);
         }
     }
 
@@ -536,7 +536,7 @@ class ZigbeeController extends EventEmitter {
         try {
             this.debug('handleDeviceLeave', message);
             const entity = await this.resolveEntity(message.device || message.ieeeAddr);
-            const friendlyName = (entity) ? entity.name : message.ieeeAddr;
+            const friendlyName = entity ? entity.name : message.ieeeAddr;
             this.debug(`Device '${friendlyName}' left the network`);
             this.emit('leave', message.ieeeAddr);
             // Call extensions
@@ -556,10 +556,15 @@ class ZigbeeController extends EventEmitter {
         const friendlyName = entity.name;
         this.warn(`Device '${friendlyName}' announced itself`);
 
-        if (entity && entity.mapped) {
-            this.callExtensionMethod(
-                'onZigbeeEvent',
-                [ {'device': message.device, 'type': 'deviceAnnounce'}, (entity ? entity.mapped : null)]);
+        try {
+            if (entity && entity.mapped) {
+                this.callExtensionMethod(
+                    'onZigbeeEvent',
+                    [ {'device': message.device, 'type': 'deviceAnnounce'}, (entity ? entity.mapped : null)]);
+            }
+        } catch (error) {
+            this.sendError(error);
+            this.error(`Failed to handleDeviceLeave ${error.stack}`);
         }
 
         this.emit('pairing', `Device '${friendlyName}' announced itself`);
@@ -645,12 +650,11 @@ class ZigbeeController extends EventEmitter {
 
     async getMap(callback) {
         try {
-
             const devices = this.herdsman.getDevices(true);
             const lqis = [];
             const routing = [];
 
-            for (const device of devices.filter((d) => d.type != 'EndDevice')) {
+            for (const device of devices.filter((d) => d.type !== 'EndDevice')) {
                 const resolved = await this.resolveEntity(device);
                 let result;
 
@@ -674,7 +678,7 @@ class ZigbeeController extends EventEmitter {
 
                 if (result !== undefined) {
                     for (const dev of result.neighbors) {
-                        if (dev.ieeeAddr !== '0xffffffffffffffff' && dev !== undefined) {
+                        if (dev !== undefined && dev.ieeeAddr !== '0xffffffffffffffff') {
                             lqis.push({
                                 parent: resolved.device.ieeeAddr,
                                 networkAddress: dev.networkAddress,
@@ -682,7 +686,7 @@ class ZigbeeController extends EventEmitter {
                                 lqi: dev.linkquality,
                                 relationship: dev.relationship,
                                 depth: dev.depth,
-                                status: (dev.linkquality > 0) ? 'online' : 'offline',
+                                status: dev.linkquality > 0 ? 'online' : 'offline',
                             });
                         }
                     }
@@ -713,11 +717,10 @@ class ZigbeeController extends EventEmitter {
                     }
                 }
                 this.debug(`Routing table succeeded for '${resolved.name}'`);
-
             }
             this.debug(`Get map succeeded ${safeJsonStringify(lqis)}`);
 
-            if (callback) callback({lqis: lqis, routing: routing});
+            callback && callback({lqis, routing});
         } catch (error) {
             this.sendError(error);
             this.debug(`Failed to get map: ${safeJsonStringify(error.stack)}`);
@@ -750,7 +753,7 @@ class ZigbeeController extends EventEmitter {
         if (type === 'foundation') {
             cfg.disableDefaultResponse = true;
             if (cmd === 'read' && !Array.isArray(zclData)) {
-                // needs to be iterateable (string[] | number [])
+                // needs to be iterable (string[] | number [])
                 zclData[Symbol.iterator] = function* () {
                     let k;
                     for (k in this) {
@@ -764,17 +767,15 @@ class ZigbeeController extends EventEmitter {
             } else {
                 result = await endpoint[cmd](cid, zclData, cfg);
             }
-            if (callback) callback(undefined, result);
-        }
-        else if(type === 'functionalResp'){
-              cfg.disableDefaultResponse = false;
-              const result = await endpoint.commandResponse(cid, cmd, zclData, cfg, zclSeqNum);
-              if (callback) callback(undefined, result);
-        }
-        else {
+            callback && callback(undefined, result);
+        } else if (type === 'functionalResp') {
+            cfg.disableDefaultResponse = false;
+            const result = await endpoint.commandResponse(cid, cmd, zclData, cfg, zclSeqNum);
+            callback && callback(undefined, result);
+        } else {
             cfg.disableDefaultResponse = false;
             const result = await endpoint.command(cid, cmd, zclData, cfg);
-            if (callback) callback(undefined, result);
+            callback && callback(undefined, result);
         }
     }
 
@@ -785,66 +786,61 @@ class ZigbeeController extends EventEmitter {
             this.debug(`addDevFromGroup - entity: ${safeJsonStringify(entity)}`);
             this.debug(`addDevFromGroup - group: ${safeJsonStringify(group)}`);
             if (epid != undefined) {
-              for (const ep of entity.endpoints) {
-                if (ep.ID == epid && (ep.inputClusters.includes(4) || ep.outputClusters.includes(4)))
-                {
-                  this.debug(`adding endpoint ${ep.ID} (${epid}) to group ${groupId}`)
-                  await(ep.addToGroup(group.mapped));
+                for (const ep of entity.endpoints) {
+                    if (ep.ID === epid && (ep.inputClusters.includes(4) || ep.outputClusters.includes(4))) {
+                        this.debug(`adding endpoint ${ep.ID} (${epid}) to group ${groupId}`);
+                        await (ep.addToGroup(group.mapped));
+                    }
                 }
-              }
 
-            }
-            else
-            {
-                if (entity.endpoint.inputClusters.includes(4))
-                {
-                    this.debug(`adding endpoint ${entity.endpoint.ID} to group`)
+            } else {
+                if (entity.endpoint.inputClusters.includes(4)) {
+                    this.debug(`adding endpoint ${entity.endpoint.ID} to group`);
                     await entity.endpoint.addToGroup(group.mapped);
-                }
-                else {
+                } else {
                     let added = false;
-                    for (const ep of entity.endpoints)
-                    {
-                        if (ep.inputClusters.includes(4))
-                        {
-                            this.debug(`adding endpoint ${ep.ID} to group`)
+                    for (const ep of entity.endpoints) {
+                        if (ep.inputClusters.includes(4)) {
+                            this.debug(`adding endpoint ${ep.ID} to group`);
                             await ep.addToGroup(group.mapped);
                             added = true;
                             break;
                         }
                     }
-                    if (!added) throw ('cluster genGroups not supported');
+                    if (!added) {
+                        throw ('cluster genGroups not supported');
+                    }
                 }
             }
         } catch (error) {
             this.sendError(error);
             this.error(`Exception when trying to Add ${devId}  to group ${groupId}`, error);
-            return { error:`Failed to add ${devId}  to group ${groupId}: ${JSON.stringify(error)}` };
+            return {error: `Failed to add ${devId}  to group ${groupId}: ${JSON.stringify(error)}`};
         }
         return {};
     }
 
     async removeDevFromGroup(devId, groupId, epid) {
-      try {
-          const entity = await this.resolveEntity(devId);
-          const group = await this.resolveEntity(groupId);
-          this.debug(`removeDevFromGroup - entity: ${safeJsonStringify(entity)}`);
-          this.debug(`removeDevFromGroup - group: ${safeJsonStringify(group)}`);
-          if (epid != undefined) {
-            for (const ep of entity.endpoints) {
-              if (ep.ID == epid && (ep.inputClusters.includes(4) || ep.outputClusters.includes(4)))
-              {
-                this.debug(`removing endpoint ${ep.ID} (${epid}) group ${groupId}`)
-                await(ep.removeFromGroup(group.mapped))
-              }
-            }
-          } else await entity.endpoint.removeFromGroup(group.mapped);
-      } catch (error) {
-          this.sendError(error);
-          this.error(`Exception when trying remove ${devId} (ep ${epid?epid:entity.endpoint.ID}) from group ${devId}`, error);
-          return { error: `Failed to remove dev ${devId} (ep ${epid?epid:entity.endpoint.ID}) from group ${devId}`};
-      }
-      return {};
+        let entity;
+        try {
+            entity = await this.resolveEntity(devId);
+            const group = await this.resolveEntity(groupId);
+            this.debug(`removeDevFromGroup - entity: ${safeJsonStringify(entity)}`);
+            this.debug(`removeDevFromGroup - group: ${safeJsonStringify(group)}`);
+            if (epid != undefined) {
+                for (const ep of entity.endpoints) {
+                    if (ep.ID === epid && (ep.inputClusters.includes(4) || ep.outputClusters.includes(4))) {
+                        this.debug(`removing endpoint ${ep.ID} (${epid}) group ${groupId}`);
+                        await ep.removeFromGroup(group.mapped);
+                    }
+                }
+            } else await entity.endpoint.removeFromGroup(group.mapped);
+        } catch (error) {
+            this.sendError(error);
+            this.error(`Exception when trying remove ${devId} (ep ${epid ? epid : (entity ? entity.endpoint.ID : '')}) from group ${devId}`, error);
+            return {error: `Failed to remove dev ${devId} (ep ${epid ? epid : (entity ? entity.endpoint.ID : '')}) from group ${devId}`};
+        }
+        return {};
     }
 
     async removeDevFromAllGroups(devId) {
@@ -852,14 +848,15 @@ class ZigbeeController extends EventEmitter {
             const entity = await this.resolveEntity(devId);
             this.debug(`entity: ${safeJsonStringify(entity)}`);
             for (const ep of entity.endpoints) {
-              if (ep.inputClusters.includes(4) || ep.outputClusters.includes(4))
-                await ep.removefromAllGroups();
+                if (ep.inputClusters.includes(4) || ep.outputClusters.includes(4)) {
+                    await ep.removefromAllGroups();
+                }
             }
             //await entity.endpoint.removeFromAllGroups();
         } catch (error) {
             this.sendError(error);
             this.error(`Exception when trying remove ${devId} from all groups`, error);
-            return { error: `Failed to remove dev ${devId} from all groups: ${error}`};
+            return {error: `Failed to remove dev ${devId} from all groups: ${error}`};
         }
         return {};
     }
@@ -869,7 +866,7 @@ class ZigbeeController extends EventEmitter {
         target = !target ? this.getCoordinator() : target;
 
         this.debug(`Binding ${log}`);
-        ep.bind(cluster, target, (error) => {
+        ep.bind(cluster, target, error => {
             if (error) {
                 this.sendError(error);
                 this.error(`Failed to bind ${log} - (${error})`);
@@ -900,11 +897,11 @@ class ZigbeeController extends EventEmitter {
     reset(mode, callback) {
         try {
             this.herdsman.reset(mode);
-            if (callback) callback();
+            callback && callback();
         } catch (error) {
             this.sendError(error);
             this.error(`Failed to reset ${error.stack}`);
-            if (callback) callback(error);
+            callback && callback(error);
         }
     }