iobroker.zigbee 1.8.0 → 1.8.1

package/lib/commands.js

@@ -5,7 +5,7 @@ const statesMapping = require('./devices');
 const disallowedDashStates = [
     'link_quality', 'available', 'battery', 'groups', 'device_query',
     'hue_move', 'color_temp_move', 'satuation_move', 'brightness_move', 'brightness_step', 'hue_calibration',
-    'msg_from_zigbee','send_payload'
+    'msg_from_zigbee', 'send_payload',
 ];
 
 class Commands {
@@ -183,16 +183,16 @@ class Commands {
                     result.forEach(async devInfo => {
                         if (devInfo._id) {
                             // groups
-//                            const grState = alls[`${devInfo._id}.groups`];
-//                            if (grState && grState.val) {
-//                                groups[devInfo._id] = JSON.parse(grState.val);
-//                            }
+                            // const grState = alls[`${devInfo._id}.groups`];
+                            // if (grState && grState.val) {
+                            //     groups[devInfo._id] = JSON.parse(grState.val);
+                            // }
                             // battery and link_quality
                             const lqState = alls[`${devInfo._id}.link_quality`];
-                            devInfo.link_quality = lqState ? lqState.val: undefined;
-                            devInfo.link_quality_lc = lqState ? lqState.lc: undefined;
+                            devInfo.link_quality = lqState ? lqState.val : undefined;
+                            devInfo.link_quality_lc = lqState ? lqState.lc : undefined;
                             const batState = alls[`${devInfo._id}.battery`];
-                            devInfo.battery = batState ? batState.val: undefined;
+                            devInfo.battery = batState ? batState.val : undefined;
                             // devInfo.states = states || {};
 
                             const states = allst.filter(item => item._id.startsWith(devInfo._id));
@@ -241,11 +241,12 @@ class Commands {
                                         if (groups) {
                                             const grouparray = groups[member.ieee];
                                             if (grouparray) {
-                                                if (!grouparray.includes(match[1]))  {
+                                                if (!grouparray.includes(match[1])) {
                                                     groups[member.ieee].push(match[1]);
                                                 }
+                                            } else {
+                                                groups[member.ieee] = [match[1]];
                                             }
-                                            else groups[member.ieee] = [match[1]];
                                         }
                                         const device = await this.adapter.getObjectAsync(`${this.adapter.namespace}.${member.ieee.substr(2)}`);
                                         if (device) {
@@ -291,7 +292,7 @@ class Commands {
                     // fill group info
                     for (const groupdev in groups) {
                         this.debug(`GetDevices scanning group ${groupdev} ${JSON.stringify(groups[groupdev])}`);
-                        const device = devices.find((dev) =>( groupdev === getZbId(dev._id) ));
+                        const device = devices.find(dev => (groupdev === getZbId(dev._id)));
                         if (device) {
                             device.groups = groups[groupdev];
                             this.debug(`adding group info to device ${groupdev}`);
@@ -394,10 +395,11 @@ class Commands {
                             coordinatorinfo.version = vt;
                         }
                     }
+                } catch {
+                    this.warn('exception raised in getCoordinatorInfo');
                 }
-                catch { this.warn('exception raised in getCoordinatorInfo');}
 
-                this.debug(`getCoorinatorInfo result: ${JSON.stringify(coordinatorinfo)}`);
+                this.debug(`getCoordinatorInfo result: ${JSON.stringify(coordinatorinfo)}`);
                 this.adapter.sendTo(from, command, coordinatorinfo, callback);
             });
         } else {

package/lib/developer.js

@@ -91,11 +91,9 @@ class Developer {
         const zclData = obj.message.zclData;
         if (cmdType === 'functional') {
             cmd = zcl.Utils.getCluster(cid).getCommand(obj.message.cmd);
-        }
-        else if (cmdType === 'functionalResp') {
+        } else if (cmdType === 'functionalResp') {
             cmd = zcl.Utils.getCluster(cid).getCommandResponse(obj.message.cmd);
-        }
-        else if (cmdType === 'foundation') {
+        } else if (cmdType === 'foundation') {
             cmd = zcl.Utils.getGlobalCommand((obj.message.cmd));
         } else {
             this.adapter.sendTo(obj.from, obj.command, {localErr: 'Invalid cmdType'}, obj.callback);
@@ -124,7 +122,7 @@ class Developer {
                 }
                 result.statusCode = 0;
                 this.adapter.sendTo(obj.from, obj.command, result, obj.callback);
-            },obj.message.zclSeqNum);
+            }, obj.message.zclSeqNum);
         } catch (exception) {
             // report exceptions
             // happens for example if user tries to send write command but did not provide value/type

package/lib/devices.js

@@ -52,8 +52,7 @@ const comb = {
                     effectjson.colors = rgb.colorsArrayFromString(options.effect_colors);
                 }
                 //               effectjson = JSON.parse(value);
-            }
-            catch {
+            } catch {
                 const effectjson = {
                     colors: [{r: 255, g: 0, b: 0}, {r: 0, g: 255, b: 0}, {r: 0, g: 0, b: 255}],
                     speed: 10,
@@ -120,11 +119,11 @@ const sync = {
     },
 };
 
-const lightStatesWithColortemp = [states.state, states.brightness, states.colortemp,states.brightness_move, states.colortemp_move, states.transition_time];
-const lightStatesWithColor = [states.state, states.brightness, states.colortemp, states.color,states.brightness_move, states.colortemp_move, states.transition_time];
-const lightStatesWithColor_hue = [states.state, states.brightness, states.colortemp, states.color,states.brightness_move, states.colortemp_move, states.hue_move, states.transition_time, states.effect_type_hue];
+const lightStatesWithColortemp = [states.state, states.brightness, states.colortemp, states.brightness_move, states.colortemp_move, states.transition_time];
+const lightStatesWithColor = [states.state, states.brightness, states.colortemp, states.color, states.brightness_move, states.colortemp_move, states.transition_time];
+const lightStatesWithColor_hue = [states.state, states.brightness, states.colortemp, states.color, states.brightness_move, states.colortemp_move, states.hue_move, states.transition_time, states.effect_type_hue];
 const lightStatesWithColorNoTemp = [states.state, states.brightness, states.color, states.brightness_move, states.transition_time];
-const lightStates = [states.state, states.brightness, states.brightness_move,  states.transition_time];
+const lightStates = [states.state, states.brightness, states.brightness_move, states.transition_time];
 
 const gl_lightStatesWithColor = [states.gl_state, states.gl_brightness, states.gl_colortemp, states.gl_color, states.transition_time];
 const gl_white_channel = [states.white_brightness, states.white_state, states.white_colortemp];
@@ -233,8 +232,8 @@ const generator = {
         }
         return freepad_states.concat(freepad_st);
     },
-    ptvo_switch_channels: entity =>{
-        const devstates = [states.ptvo_trigger,states.ptvo_interval];
+    ptvo_switch_channels: entity => {
+        const devstates = [states.ptvo_trigger, states.ptvo_interval];
         for (const endpoint of entity.device.endpoints) {
             const epID = endpoint.ID;
             const epName = getKey(entity.mapped.endpoint(entity.device), epID);
@@ -260,7 +259,7 @@ const generator = {
                     //          l7:l7=143voltage_l7=143
 
                     if (payload.hasOwnProperty(`device_${epName}`)) {
-                        id = (typeof(payload[`device_${epName}`]) == 'string' ? payload[`device_${epName}`].replace(/[-. ]/g, ''):'unset');
+                        id = (typeof payload[`device_${epName}`] === 'string' ? payload[`device_${epName}`].replace(/[-. ]/g, '') : 'unset');
                     } else if (payload.hasOwnProperty(`voltage_${epName}`)) {
                         id = 'voltage';
                     }
@@ -277,12 +276,13 @@ const generator = {
                             }
                         }
                     }
-                    if (measurement && id !== 'voltage')
+                    if (measurement && id !== 'voltage') {
                         rv.stateid = `${id}_${measurement}`;
-                    else
+                    } else {
                         rv.stateid = id;
+                    }
                     rv.value = payload[epName];
-                    rv.name = measurement? `${measurement} 0x${id}`: `0x${id}`;
+                    rv.name = measurement ? `${measurement} 0x${id}` : `0x${id}`;
                     rv.unit = unit;
                     rv.role = measurement;
 
@@ -569,13 +569,13 @@ const generator = {
     },
 };
 
-function states_with_epname (entity, states) {
+function states_with_epname(entity, states) {
     const devstates = [];
     for (const endpoint of entity.device.endpoints) {
         const epID = endpoint.ID;
         const epName = getKey(entity.mapped.endpoint(entity.device), epID);
         for (const state of states) {
-            const newState = { ...state };
+            const newState = {...state};
             newState.id = `${newState.id}_${epName}`;
             if (newState.name) newState.name = `${newState.name} (${epName})`;
             if (newState.prop) newState.prop = `${newState.prop}_${epName}`;
@@ -622,7 +622,7 @@ const devices = [
         ],
     },
     {
-        models: ['WXKG03LM','WXKG03LM_rev2'],
+        models: ['WXKG03LM', 'WXKG03LM_rev2'],
         icon: 'img/86sw1.png',
         states: [
             states.click, states.double_click, states.voltage, states.battery,
@@ -638,7 +638,7 @@ const devices = [
         ],
     },
     {
-        models: ['WXKG02LM','WXKG02LM_rev2'],
+        models: ['WXKG02LM', 'WXKG02LM_rev2'],
         icon: 'img/86sw2.png',
         states: [
             states.left_click, states.right_click, states.both_click,
@@ -995,7 +995,7 @@ const devices = [
         states: [states.occupancy, states.temperature, states.temp_calibration],
     },
     {
-        models: ['595UGR22','4058075181472'],
+        models: ['595UGR22', '4058075181472'],
         icon: 'img/FLOALT.panel.WS.png',
         states: lightStatesWithColortemp,
         syncStates: [sync.brightness],
@@ -1168,7 +1168,7 @@ const devices = [
         syncStates: [sync.brightness],
     },
     {
-        models: ['5055131P7','5055148P7'],
+        models: ['5055131P7', '5055148P7'],
         icon: 'img/philips_hue_centura_square.png',
         states: lightStatesWithColor_hue,
         syncStates: [sync.brightness],
@@ -1490,7 +1490,7 @@ const devices = [
     {
         models: ['E1757', 'E1926'],
         states: [states.battery, states.blind_position, states.blind_open, states.blind_close, states.blind_stop],
-        icon:  'img/Ikea_fyrtur.png',
+        icon: 'img/Ikea_fyrtur.png',
     },
     {
         models: ['ICPSHC24-10EU-IL-1', 'ICPSHC24-30EU-IL-1'],
@@ -1512,7 +1512,7 @@ const devices = [
     {
         models: ['E1744'],
         icon: 'img/ikea_SYMFONISK_Sound_Controller.png',
-        states: [states.button_action_skip_back, states.button_action_skip_forward,  states.action_play_pause,
+        states: [states.button_action_skip_back, states.button_action_skip_forward, states.action_play_pause,
             states.rotate_left, states.rotate_right, states.rotate_stop, states.battery],
     },
     // Hive
@@ -1611,7 +1611,7 @@ const devices = [
     {
         models: ['SP 120', 'SP 220', 'SP 222', 'SP 224'],
         icon: 'img/innr_plug.png',
-        states: [states.state,states.load_power],
+        states: [states.state, states.load_power],
     },
     {
         models: ['FL 130 C'],
@@ -1918,7 +1918,7 @@ const devices = [
         syncStates: [sync.brightness],
     },
     {
-        models: ['798.09','371000002'],
+        models: ['798.09', '371000002'],
         icon: 'img/Paulmann_79809.png',
         states: lightStatesWithColor,
         syncStates: [sync.brightness],
@@ -1938,7 +1938,7 @@ const devices = [
             states.brightness_move_down, states.brightness_move_down_size,
             states.enhanced_move_to_hue_and_saturation, states.enhanced_move_enhanced_hue,
             states.enhanced_move_hue, states.enhanced_move_saturation,
-            states.scene,states.action_group
+            states.scene, states.action_group,
         ],
     },
 
@@ -2091,7 +2091,7 @@ const devices = [
         linkedStates: [comb.brightnessAndState],
     },
     {
-        models: ['MLI-404011','MLI-404049','MLI-404011/MLI-404049'],
+        models: ['MLI-404011', 'MLI-404049', 'MLI-404011/MLI-404049'],
         icon: 'img/MLI-404011-MLI-404049.png',
         states: [
             states.tint404011_scene_sunset, states.tint404011_scene_party,
@@ -2228,7 +2228,7 @@ const devices = [
     {
         models: ['07048L'],
         icon: 'img/innr_plug.png',
-        states: [states.state,states.load_power],
+        states: [states.state, states.load_power],
     },
     {
         models: ['AV2010/32'],
@@ -2404,12 +2404,12 @@ const devices = [
         states: [states.state],
     },
     {
-        models: [ 'X712A'],
+        models: ['X712A'],
         icon: 'img/tuya_switch_2.png',
         states: [states.l1, states.l2],
     },
     {
-        models: [ 'X713A'],
+        models: ['X713A'],
         icon: 'img/tuya_switch_3.png',
         states: [states.l1, states.l2, states.l3],
     },
@@ -2553,7 +2553,7 @@ const devices = [
         ],
     },
     {
-        models: ['SEA802-Zigbee','SEA801-Zigbee/SEA802-Zigbee'],
+        models: ['SEA802-Zigbee', 'SEA801-Zigbee/SEA802-Zigbee'],
         icon: 'img/tuya_TS0601.png',
         states: [
             states.child_lock,
@@ -3108,7 +3108,6 @@ function fillStatesWithExposes(allExcludesObj) {
     applyExposes(devices, byModel, allExcludesObj);
 
     removeEmptyStates(devices);
-
 }
 
 // remove empty states
@@ -3122,7 +3121,7 @@ function removeEmptyStates(devices) {
 
 function findModel(model) {
     const byModel = getByModel();
-    const stripModel = (model) ? model.replace(/0.*$/g, '').trim() : '';
+    const stripModel = model ? model.replace(/0.*$/g, '').trim() : '';
     return byModel.get(stripModel);
 }
 

package/lib/exposes.js

@@ -29,9 +29,8 @@ function genState(expose, role, name, desc) {
                 type: 'boolean',
             };
             if (readable) {
-                state.getter = payload => (payload[propName] === (expose.value_on || 'ON'));
-            }
-            else {
+                state.getter = payload => payload[propName] === (expose.value_on || 'ON');
+            } else {
                 state.getter = payload => undefined;
             }
             if (writable) {
@@ -124,7 +123,7 @@ function createFromExposes(model, def) {
         }
         state.readable = (access & ea.STATE) > 0;
         state.writable = (access & ea.SET) > 0;
-        const stateExists = states.findIndex( (element, index, array) => element.id === state.id);
+        const stateExists = states.findIndex((element, index, array) => element.id === state.id);
         if (stateExists < 0) {
             state.write = state.writable;
             if (!state.writable) {
@@ -137,13 +136,12 @@ function createFromExposes(model, def) {
             }
             if (!state.readable) {
                 if (state.hasOwnProperty('getter')) {
-                    //to awid some worning on unprocessed data
-                    state.getter = payload => ( undefined );
+                    // to awoid some warnings on unprocessed data
+                    state.getter = payload => undefined;
                 }
             }
             return states.push(state);
-        }
-        else {
+        } else {
             if ((state.readable) && (!states[stateExists].readable)) {
                 states[stateExists].read = state.read;
                 // as state is readable, it can't be button or event
@@ -154,14 +152,14 @@ function createFromExposes(model, def) {
                     delete states[stateExists].isEvent;
                 }
                 // we have to use the getter from "new" state
-                if ( state.hasOwnProperty('getter') ) {
+                if (state.hasOwnProperty('getter')) {
                     states[stateExists].getter = state.getter;
                 }
                 // trying to remove the `prop` property, as main key for get and set,
                 // as it can be different in new and old states, and leave only:
                 // setattr for old and id for new
                 if ((state.hasOwnProperty('prop')) && (state.prop === state.id)) {
-                    if ( states[stateExists].hasOwnProperty('prop') ) {
+                    if (states[stateExists].hasOwnProperty('prop')) {
                         if (states[stateExists].prop !== states[stateExists].id) {
                             if (!states[stateExists].hasOwnProperty('setattr')) {
                                 states[stateExists].setattr = states[stateExists].prop;
@@ -169,7 +167,7 @@ function createFromExposes(model, def) {
                         }
                         delete states[stateExists].prop;
                     }
-                } else if ( state.hasOwnProperty('prop') ) {
+                } else if (state.hasOwnProperty('prop')) {
                     states[stateExists].prop = state.prop;
                 }
                 states[stateExists].readable = true;
@@ -191,7 +189,7 @@ function createFromExposes(model, def) {
                 // as we have new state, responsible for set, we have to use new `isOption`
                 // or remove it
                 if (((!state.hasOwnProperty('isOption')) || (state.isOptions === false))
-                     && (states[stateExists].hasOwnProperty('isOption'))) {
+                    && (states[stateExists].hasOwnProperty('isOption'))) {
                     delete states[stateExists].isOption;
                 } else {
                     states[stateExists].isOption = state.isOption;
@@ -217,6 +215,7 @@ function createFromExposes(model, def) {
             return states.length;
         }
     }
+
     const icon = utils.getDeviceIcon(def);
     for (const expose of def.exposes) {
         let state;
@@ -318,7 +317,7 @@ function createFromExposes(model, def) {
                                 read: true,
                                 type: 'string',
                                 setter: value => {
-                                // convert RGB to XY for set
+                                    // convert RGB to XY for set
                                     /*
                                     const result = /^#?([a-f\d]{2})([a-f\d]{2})([a-f\d]{2})$/i.exec(value);
                                     let xy = [0, 0];
@@ -375,7 +374,7 @@ function createFromExposes(model, def) {
                                     const _rgb = colors.ParseColor(value);
                                     const hsv = rgb.rgbToHSV(_rgb.r, _rgb.g, _rgb.b, true);
                                     return {
-                                        hue: Math.min(Math.max(hsv.h,1),359),
+                                        hue: Math.min(Math.max(hsv.h, 1), 359),
                                         saturation: hsv.s,
                                         //                                        brightness: Math.floor(hsv.v * 2.55),
                                     };
@@ -412,17 +411,27 @@ function createFromExposes(model, def) {
                                     const hasHueCalibrationTable = options && options.hasOwnProperty('hue_calibration');
                                     if (hasHueCalibrationTable)
                                         try {
-                                            return {...options, transition: transitionTime, hue_correction: JSON.parse(options.hue_calibration)};
-                                        }
-                                        catch {
+                                            return {
+                                                ...options,
+                                                transition: transitionTime,
+                                                hue_correction: JSON.parse(options.hue_calibration)
+                                            };
+                                        } catch {
                                             const hue_correction_table = [];
                                             options.hue_calibration.split(',').forEach(element => {
                                                 const match = /([0-9]+):([0-9]+)/.exec(element);
                                                 if (match && match.length === 3)
-                                                    hue_correction_table.push({in: Number(match[1]), out: Number(match[2])});
+                                                    hue_correction_table.push({
+                                                        in: Number(match[1]),
+                                                        out: Number(match[2])
+                                                    });
                                             });
                                             if (hue_correction_table.length > 0) {
-                                                return {...options, transition: transitionTime, hue_correction: hue_correction_table};
+                                                return {
+                                                    ...options,
+                                                    transition: transitionTime,
+                                                    hue_correction: hue_correction_table
+                                                };
                                             }
                                         }
                                     return {...options, transition: transitionTime};
@@ -451,17 +460,27 @@ function createFromExposes(model, def) {
                                     const hasHueCalibrationTable = options && options.hasOwnProperty('hue_calibration');
                                     if (hasHueCalibrationTable)
                                         try {
-                                            return {...options, transition: transitionTime, hue_correction: JSON.parse(options.hue_calibration)};
-                                        }
-                                        catch {
+                                            return {
+                                                ...options,
+                                                transition: transitionTime,
+                                                hue_correction: JSON.parse(options.hue_calibration)
+                                            };
+                                        } catch {
                                             const hue_correction_table = [];
                                             options.hue_calibration.split(',').forEach(element => {
                                                 const match = /([0-9]+):([0-9]+)/.exec(element);
                                                 if (match && match.length === 3)
-                                                    hue_correction_table.push({ in: Number(match[1]), out: Number(match[2])});
+                                                    hue_correction_table.push({
+                                                        in: Number(match[1]),
+                                                        out: Number(match[2])
+                                                    });
                                             });
                                             if (hue_correction_table.length > 0) {
-                                                return {...options, transition: transitionTime, hue_correction: hue_correction_table};
+                                                return {
+                                                    ...options,
+                                                    transition: transitionTime,
+                                                    hue_correction: hue_correction_table
+                                                };
                                             }
                                         }
                                     return {...options, transition: transitionTime};
@@ -490,17 +509,28 @@ function createFromExposes(model, def) {
                                     const hasHueCalibrationTable = options && options.hasOwnProperty('hue_calibration');
                                     if (hasHueCalibrationTable)
                                         try {
-                                            return {...options, transition: transitionTime, hue_correction: JSON.parse(options.hue_calibration)};
+                                            return {
+                                                ...options,
+                                                transition: transitionTime,
+                                                hue_correction: JSON.parse(options.hue_calibration)
+                                            };
                                         } catch {
                                             const hue_correction_table = [];
                                             options.hue_calibration.split(',').forEach(element => {
                                                 const match = /([0-9]+):([0-9]+)/.exec(element);
                                                 if (match && match.length === 3) {
-                                                    hue_correction_table.push({ in: Number(match[1]), out: Number(match[2])});
+                                                    hue_correction_table.push({
+                                                        in: Number(match[1]),
+                                                        out: Number(match[2])
+                                                    });
                                                 }
                                             });
                                             if (hue_correction_table.length > 0) {
-                                                return {...options, transition: transitionTime, hue_correction: hue_correction_table};
+                                                return {
+                                                    ...options,
+                                                    transition: transitionTime,
+                                                    hue_correction: hue_correction_table
+                                                };
                                             }
                                         }
                                     return {...options, transition: transitionTime};
@@ -739,10 +769,9 @@ function createFromExposes(model, def) {
                     // if we have a composite expose, the payload will be an object {expose.property : {prop.property: value}}
                     if (prop.access & ea.STATE) {
                         st.getter = payload => {
-                            if ( (payload.hasOwnProperty(expose.property)) && (payload[expose.property]  !== null) && payload[expose.property].hasOwnProperty(prop.property)) {
-                                return !isNaN(payload[expose.property][prop.property]) ? payload[expose.property][prop.property] : undefined
-                            }
-                            else {
+                            if ((payload.hasOwnProperty(expose.property)) && (payload[expose.property] !== null) && payload[expose.property].hasOwnProperty(prop.property)) {
+                                return !isNaN(payload[expose.property][prop.property]) ? payload[expose.property][prop.property] : undefined;
+                            } else {
                                 return undefined;
                             }
                         };

package/lib/groups.js

@@ -92,7 +92,7 @@ class Groups {
     }
 
     async getGroups(obj) {
-        const response = { groups: {} };
+        const response = {groups: {}};
         try {
             // const groupsState = await this.adapter.getStateAsync('info.groups');
             const herdsmanGroups = await this.zbController.getGroups();
@@ -104,7 +104,7 @@ class Groups {
                 for (const group of herdsmanGroups) {
                     const gid = group.groupID;
                     if (gid) {
-                        groups[gid]=this.stController.verifyDeviceName(`group_${gid}`,`Group ${gid}`);
+                        groups[gid] = this.stController.verifyDeviceName(`group_${gid}`, `Group ${gid}`);
                     }
                 }
             }
@@ -137,13 +137,13 @@ class Groups {
                 for (const gpid of groups[epid]) {
                     const gpidn = parseInt(gpid);
                     if (gpidn < 0) {
-                        let response = await this.zbController.removeDevFromGroup(sysid,(-gpidn), epid);
+                        const response = await this.zbController.removeDevFromGroup(sysid, (-gpidn), epid);
                         if (response && response.error) {
                             errors.push(response.error);
                             this.error(`remove dev from group Error: ${JSON.stringify(response.error)}`);
                         }
-                    } else if (gpidn > 0){
-                        let response = await this.zbController.addDevToGroup(sysid,(gpidn), epid);
+                    } else if (gpidn > 0) {
+                        const response = await this.zbController.addDevToGroup(sysid, (gpidn), epid);
                         if (response && response.error) {
                             errors.push(response.error);
                             this.error(`add dev to group Error: ${JSON.stringify(response.error)}`);
@@ -157,7 +157,7 @@ class Groups {
             this.adapter.sendTo(from, command, {error: e}, callback);
             return;
         }
-        this.adapter.sendTo(from, command, { }, callback);
+        this.adapter.sendTo(from, command, {}, callback);
     }
 
 
@@ -188,43 +188,43 @@ class Groups {
                 }
             }
         }
-        if (result.unsupported.length>0) {
+        if (result.unsupported.length > 0) {
             const error = {
                 code: 134,
                 message: `unsupported ${stateDesc.id} change for group members ${result.unsupported.join()}`
             };
             throw error;
         }
-        if (result.unread.length>0) {
+        if (result.unread.length > 0) {
             this.warn(`unread ${stateDesc.id} change for group members ${JSON.stringify(result.unread)}`);
         }
     }
 
     async deleteGroup(from, command, message) {
-    /*
-        const members = await this.getGroupMembersFromController(parseInt(message));
-        if (members && members.length) {
-            for (const member of members) {
-                const devName = member.device.substring(2);
-                const groupEntry = this.adapter.getStateAsync(`${devName}.groups`);
-                const memberarray = (groupEntry && groupEntry.val) ? JSON.parse(groupEntry.val) : [];
-                const index = memberarray.indexOf(message.toString());
-                if (index > -1) {
-                    memberarray.splice(index, 1);
-                }
-                if (memberarray.length > 0) {
-                    await this.adapter.setStateAsync(`${devName}.groups`, JSON.stringify(memberarray), true);
-                }
-                else {
-                    await this.adapter.setStateAsync(`${devName}.groups`, '', true);
+        /*
+            const members = await this.getGroupMembersFromController(parseInt(message));
+            if (members && members.length) {
+                for (const member of members) {
+                    const devName = member.device.substring(2);
+                    const groupEntry = this.adapter.getStateAsync(`${devName}.groups`);
+                    const memberarray = (groupEntry && groupEntry.val) ? JSON.parse(groupEntry.val) : [];
+                    const index = memberarray.indexOf(message.toString());
+                    if (index > -1) {
+                        memberarray.splice(index, 1);
+                    }
+                    if (memberarray.length > 0) {
+                        await this.adapter.setStateAsync(`${devName}.groups`, JSON.stringify(memberarray), true);
+                    }
+                    else {
+                        await this.adapter.setStateAsync(`${devName}.groups`, '', true);
+                    }
                 }
             }
-        }
-        const groupsEntry = await this.adapter.getStateAsync('info.groups');
-        const objGroups = (groupsEntry && groupsEntry.val ? JSON.parse(groupsEntry.val) : {});
-        delete objGroups[message.toString()];
-        await this.adapter.setStateAsync('info.groups', JSON.stringify(objGroups), true);
-    */
+            const groupsEntry = await this.adapter.getStateAsync('info.groups');
+            const objGroups = (groupsEntry && groupsEntry.val ? JSON.parse(groupsEntry.val) : {});
+            delete objGroups[message.toString()];
+            await this.adapter.setStateAsync('info.groups', JSON.stringify(objGroups), true);
+        */
         await this.zbController.removeGroupById(message);
         await this.stController.deleteDeviceStatesAsync(`group_${parseInt(message)}`);
     }
@@ -317,7 +317,7 @@ class Groups {
                 usedGroupsIds.push(parseInt(j));
             }
         }
-        chain.push(new Promise(resolve => {
+        chain.push(new Promise(resolve =>
             // remove unused adapter groups
             this.adapter.getDevices((err, devices) => {
                 if (!err) {
@@ -331,8 +331,7 @@ class Groups {
                     });
                 }
                 resolve();
-            });
-        }));
+            })));
 
         await Promise.all(chain);
     }