iobroker.zigbee2mqtt 3.0.21 → 3.1.1

package/README.md

@@ -22,6 +22,15 @@ This adapter allows to control the data points of the devices of a Zigbee2MQTT i
 [Adapter Documentation](https://github.com/arteck/ioBroker.zigbee2mqtt/blob/main/docs/wiki.md)
 
 ## Changelog
+### 3.1.1 (2026-04-21)
+* (arteck) fix connection handling
+
+### 3.1.0 (2026-04-20)
+* (arteck) refactoring
+
+### 3.0.22 (2026-04-12)
+* (arteck) Dependencies have been updated
+
 ### 3.0.21 (2026-01-29)
 * (arteck) add coordinator status info
 

package/admin/jsonConfig.json

@@ -560,6 +560,43 @@
           "lg": 12,
           "xl": 12,
           "newLine": true
+        },
+        "spacerDeleteStates": {
+          "type": "staticText",
+          "text": "",
+          "xs": 12,
+          "newLine": true
+        },
+        "headerDeleteStates": {
+          "type": "header",
+          "text": "coming soon ",
+          "size": 4,
+          "newLine": true
+        },
+        "buttonDeleteOldStates": {
+          "type": "sendTo",
+          "command": "deleteOldStates",
+          "label": "Delete old / unused states",
+          "variant": "contained",
+          "color": "secondary",
+          "icon": "delete",
+          "disabled": "1 === 1",
+          "xs": 12,
+          "sm": 6,
+          "md": 4,
+          "lg": 3,
+          "xl": 3,
+          "newLine": true
+        },
+        "_deleteOldStatesResult": {
+          "type": "staticText",
+          "text": "",
+          "xs": 12,
+          "newLine": false,
+          "style": {
+            "marginTop": 8,
+            "fontStyle": "italic"
+          }
         }
       }
     }

package/io-package.json

@@ -1,8 +1,21 @@
 {
   "common": {
     "name": "zigbee2mqtt",
-    "version": "3.0.21",
+    "version": "3.1.1",
     "news": {
+      "3.1.1": {
+        "en": "fix connection handling",
+        "de": "befestigungstechnik",
+        "ru": "фиксировать связь",
+        "pt": "corrigir o tratamento da ligação",
+        "nl": "fix verbinding handling",
+        "fr": "fixer la manipulation de la connexion",
+        "it": "gestione del collegamento",
+        "es": "corrección de conexión",
+        "pl": "naprawić obsługę połączenia",
+        "uk": "кріплення з'єднання",
+        "zh-cn": "固定连接处理"
+      },
       "3.0.21": {
         "en": "add coordinator status info",
         "de": "koordinator statusinfo hinzufügen",
@@ -54,45 +67,6 @@
         "pl": "typo",
         "uk": "типи",
         "zh-cn": "类型"
-      },
-      "3.0.17": {
-        "en": "typo",
-        "de": "typo",
-        "ru": "опечатка",
-        "pt": "erro de digitação",
-        "nl": "type",
-        "fr": "typo",
-        "it": "tipo",
-        "es": "typo",
-        "pl": "typo",
-        "uk": "типи",
-        "zh-cn": "类型"
-      },
-      "3.0.16": {
-        "en": "fix action dp",
-        "de": "funktion dp",
-        "ru": "исправить действие dp",
-        "pt": "corrigir ação dp",
-        "nl": "fix actie dp",
-        "fr": "corriger l'action dp",
-        "it": "fix azione dp",
-        "es": "arreglar acción dp",
-        "pl": "fix action dp",
-        "uk": "фіксувати дію dp",
-        "zh-cn": "固定动作 dp"
-      },
-      "3.0.15": {
-        "en": "update",
-        "de": "aktualisierung",
-        "ru": "обновление",
-        "pt": "atualizar",
-        "nl": "bijwerken",
-        "fr": "mise à jour",
-        "it": "aggiornamento",
-        "es": "actualización",
-        "pl": "aktualizacja",
-        "uk": "оновлення",
-        "zh-cn": "更新"
       }
     },
     "messages": [
@@ -220,12 +194,12 @@
     },
     "dependencies": [
       {
-        "js-controller": ">= 5.0.19"
+        "js-controller": ">= 6.0.11"
       }
     ],
     "globalDependencies": [
       {
-        "admin": ">=7.4.10"
+        "admin": ">=7.6.20"
       }
     ]
   },

package/lib/check.js

@@ -5,28 +5,36 @@
  * @param version
  */
 function checkConfig(config, log, version) {
+    if (!config || !log) {
+        return;
+    }
+
+    const advanced     = config.advanced     || {};
+    const deviceOptions = config.device_options || {};
+
     const checkAPIOptions = {
-        legacy_api_enabled: config.advanced.legacy_api != false,
-        legacy_availability_payload_enabled: config.advanced.legacy_availability_payload != false,
-        device_legacy_enabled: config.device_options.legacy != false
+        legacy_api_enabled:                   advanced.legacy_api !== false,
+        legacy_availability_payload_enabled:  advanced.legacy_availability_payload !== false,
+        device_legacy_enabled:                deviceOptions.legacy !== false,
     };
 
     const checkAPIOptionsOutput = {
-        payload_contains_not_json: config.advanced.output != 'attribute_and_json' && config.advanced.output != 'json'
+        payload_contains_not_json: advanced.output !== 'attribute_and_json' && advanced.output !== 'json',
     };
 
-    if (version.startsWith('1.')) {      // wird in version 2.x immer auf false gesetzt sein
-        if (Object.values(checkAPIOptions).filter((x) => x == true).length > 0) {
+    // Fix: version kann undefined/null sein (ältere Z2M-Versionen)
+    if (version && version.startsWith('1.')) {
+        if (Object.values(checkAPIOptions).filter((x) => x === true).length > 0) {
             log.error('===================================================');
             log.error('===================================================');
-            if (checkAPIOptions.legacy_api_enabled == true) {
+            if (checkAPIOptions.legacy_api_enabled === true) {
                 log.error('Legacy api is activated, so the adapter can not work correctly!!!');
                 log.error('Please add the following lines to your Zigbee2MQTT configuration.yaml:');
                 log.error('advanced:');
                 log.error(' legacy_api: false');
                 log.error('');
             }
-            if (checkAPIOptions.legacy_availability_payload_enabled == true) {
+            if (checkAPIOptions.legacy_availability_payload_enabled === true) {
                 log.error(
                     'Legacy Availability Payload is activated, thus the adapter cannot represent the availability of the devices!!!'
                 );
@@ -35,7 +43,7 @@ function checkConfig(config, log, version) {
                 log.error('legacy_availability_payload: false');
                 log.error('');
             }
-            if (checkAPIOptions.device_legacy_enabled == true) {
+            if (checkAPIOptions.device_legacy_enabled === true) {
                 log.error(
                     'Device Legacy Payload is activated, therefore the adapter may process the states of the devices correctly!!!'
                 );
@@ -46,8 +54,8 @@ function checkConfig(config, log, version) {
             log.error('===================================================');
         }
     }
-    if (Object.values(checkAPIOptionsOutput).filter((x) => x == true).length > 0) {
-        if (checkAPIOptions.payload_contains_not_json == true ) {
+    if (Object.values(checkAPIOptionsOutput).filter((x) => x === true).length > 0) {
+        if (checkAPIOptions.payload_contains_not_json === true ) {
             log.error('===================================================');
             log.error(
                 'MQTT output type must "attribute_and_json" or "json" , therefore the adapter may process the states of the devices correctly!!!'

package/lib/deviceController.js

@@ -33,8 +33,17 @@ class DeviceController {
      * @param devicesMessage
      */
     async createDeviceDefinitions(devicesMessage) {
+        // Fix 1: devicesMessage kann null/kein Array sein (z.B. leerer bridge/devices payload)
+        if (!Array.isArray(devicesMessage)) {
+            this.adapter.log.warn('createDeviceDefinitions: payload ist kein Array');
+            return;
+        }
         utils.clearArray(this.deviceCache);
         for (const devicesMessag of devicesMessage) {
+            if (!devicesMessag || !devicesMessag.ieee_address) {
+                continue;
+            }
+
             if (this.logCustomizations.debugDevices.includes(devicesMessag.ieee_address)) {
                 this.adapter.log.warn(
                     `--->>> fromZ2M -> ${devicesMessag.ieee_address} exposes: ${JSON.stringify(devicesMessag)}`
@@ -42,16 +51,17 @@ class DeviceController {
             }
 
             if (devicesMessag.definition != null) {
-                // if the device is already present in the cache, remove it
                 this.removeDeviceByIeee(this.deviceCache, devicesMessag.ieee_address);
 
-                if (devicesMessag.definition.exposes) {
+                // Fix 3: exposes muss auch ein Array sein
+                if (Array.isArray(devicesMessag.definition.exposes)) {
                     try {
                         const newDevice = await createDeviceFromExposes(devicesMessag, this.adapter);
                         newDevice.icon = await this.imageController.getDeviceIcon(devicesMessag);
                         this.deviceCache.push(newDevice);
                     } catch (err) {
-                        this.adapter.log.warn(`Cannot ${devicesMessag.friendly_name} create Device from Exposes!`);
+                        // Fix 2: friendly_name im Fehlerlog verwenden
+                        this.adapter.log.warn(`Cannot create Device from Exposes for ${devicesMessag.friendly_name || devicesMessag.ieee_address}!`);
                         this.adapter.log.debug(JSON.stringify(devicesMessag));
                         this.adapter.log.debug(err);
                     }
@@ -67,23 +77,22 @@ class DeviceController {
      * @param scenes
      */
     async defineGroupDevice(groupID, ieee_address, scenes) {
-        const brmPropName =
-			this.adapter.config.brightnessMoveOnOff == true ? 'brightness_move_onoff' : 'brightness_move';
-        const brsPropName =
-			this.adapter.config.brightnessStepOnOff == true ? 'brightness_step_onoff' : 'brightness_step';
+        const brmPropName = this.adapter.config.brightnessMoveOnOff === true ? 'brightness_move_onoff' : 'brightness_move';
+        const brsPropName = this.adapter.config.brightnessStepOnOff === true ? 'brightness_step_onoff' : 'brightness_step';
         const newDevice = {
             id: groupID,
             ieee_address: ieee_address,
             icon: undefined, //  await imageController.getDeviceIcon(devicesMessag), device.definition.model
             optionsValues: {},
             states: [
-                statesDefs.available,
-                statesDefs.brightness,
-                statesDefs.colortemp_move,
-                statesDefs.transition,
+                // Fix 1: Klone aller statesDefs-Objekte – sonst werden geteilte Referenzen mutiert
+                Object.assign({}, statesDefs.available),
+                Object.assign({}, statesDefs.brightness),
+                Object.assign({}, statesDefs.colortemp_move),
+                Object.assign({}, statesDefs.transition),
                 {
                     id: 'state',
-                    prob: 'state',
+                    prop: 'state',
                     name: 'Switch state',
                     options: ['transition'],
                     icon: undefined,
@@ -133,29 +142,28 @@ class DeviceController {
                     type: 'string',
                     def: '#ff00ff',
                     setter: (value) => {
-                        let xy = [0, 0];
+                        // Fix 6: redundante [0,0]-Initialisierung entfernt
                         const rgbcolor = colors.ParseColor(value);
-                        xy = rgb.rgb_to_cie(rgbcolor.r, rgbcolor.g, rgbcolor.b);
+                        const xy = rgb.rgb_to_cie(rgbcolor.r, rgbcolor.g, rgbcolor.b);
                         return {
                             x: xy[0],
                             y: xy[1],
                         };
                     },
                     getter: (payload) => {
-                        if (payload.color_mode != 'xy' && this.config.colorTempSyncColor == false) {
+                        if (payload.color_mode !== 'xy' && !this.config.colorTempSyncColor) {
                             return undefined;
                         }
-                        if (payload.color && payload.color.x && payload.color.y) {
+                        // Fix 2: x=0 / y=0 sind gültige CIE-Koordinaten → != null statt truthy
+                        if (payload.color && payload.color.x != null && payload.color.y != null) {
                             const colorval = rgb.cie_to_rgb(payload.color.x, payload.color.y);
                             return (
-                                `#${ 
-								utils.decimalToHex(colorval[0]) 
-								}${utils.decimalToHex(colorval[1]) 
-								}${utils.decimalToHex(colorval[2])}`
+                                `#${utils.decimalToHex(colorval[0])
+                                }${utils.decimalToHex(colorval[1])
+                                }${utils.decimalToHex(colorval[2])}`
                             );
-                        } 
-                            return undefined;
-                        
+                        }
+                        return undefined;
                     },
                 },
                 {
@@ -168,22 +176,21 @@ class DeviceController {
                     write: true,
                     read: true,
                     type: 'number',
-                    min: this.config.useKelvin == true ? utils.miredKelvinConversion(550) : 150,
-                    max: this.config.useKelvin == true ? utils.miredKelvinConversion(153) : 500,
-                    def: this.config.useKelvin == true ? utils.miredKelvinConversion(153) : 500,
-                    unit: this.config.useKelvin == true ? 'K' : 'mired',
+                    min: this.config.useKelvin === true ? utils.miredKelvinConversion(550) : 150,
+                    max: this.config.useKelvin === true ? utils.miredKelvinConversion(153) : 500,
+                    def: this.config.useKelvin === true ? utils.miredKelvinConversion(153) : 500,
+                    unit: this.config.useKelvin === true ? 'K' : 'mired',
                     setter: (value) => {
                         return utils.toMired(value);
                     },
                     getter: (payload) => {
-                        if (payload.color_mode != 'color_temp') {
+                        if (payload.color_mode !== 'color_temp') {
                             return undefined;
                         }
-                        if (this.config.useKelvin == true) {
+                        if (this.config.useKelvin === true) {
                             return utils.miredKelvinConversion(payload.color_temp);
-                        } 
-                            return payload.color_temp;
-                        
+                        }
+                        return payload.color_temp;
                     },
                 },
                 {
@@ -248,12 +255,21 @@ class DeviceController {
      * @param groupsMessage
      */
     async createGroupDefinitions(groupsMessage) {
+        // Fix 4: groupsMessage kann null/kein Array sein
+        if (!Array.isArray(groupsMessage)) {
+            this.adapter.log.warn('createGroupDefinitions: payload ist kein Array');
+            return;
+        }
         utils.clearArray(this.groupCache);
         for (const groupMessage of groupsMessage) {
-            if (this.logCustomizations.debugDevices.includes(groupMessage.id)) {
+            // Fix 5: fehlende Pflichtfelder abfangen
+            if (!groupMessage || groupMessage.id == null || !groupMessage.friendly_name) {
+                continue;
+            }
+            if (this.logCustomizations.debugDevices.includes(String(groupMessage.id))) {
                 this.adapter.log.warn(`--->>> fromZ2M -> ${groupMessage.id} exposes: ${JSON.stringify(groupMessage)}`);
             }
-            await this.defineGroupDevice(groupMessage.friendly_name, `group_${groupMessage.id}`, groupMessage.scenes);
+            await this.defineGroupDevice(groupMessage.friendly_name, `group_${groupMessage.id}`, groupMessage.scenes || []);
         }
     }
 
@@ -261,144 +277,158 @@ class DeviceController {
      *
      */
     async createOrUpdateDevices() {
-        for (const device of this.groupCache.concat(this.deviceCache)) {
-            let deviceName = await this.getDeviceName(device);
-            let description = await this.getDeviceDescription(device);
+        // Gruppen und Geräte getrennt verarbeiten
+        for (const device of this.groupCache) {
+            await this._createOrUpdateSingleDevice(device, true);
+        }
+        for (const device of this.deviceCache) {
+            await this._createOrUpdateSingleDevice(device, false);
+        }
+    }
 
-            if (deviceName == '' && device.description) {
-                deviceName = device.description;
-                description = '';
-            }
+    /**
+     * @param device
+     * @param isGroup
+     */
+    async _createOrUpdateSingleDevice(device, isGroup) {
+        let deviceName = await this.getDeviceName(device);
+        let description = await this.getDeviceDescription(device);
 
-            // Manipulate deviceName if the device is disabled, so the update of the device is triggered as well
-            if (device.disabled && device.disabled == true) {
-                if (this.config.useEventInDesc == true) {
-                    description = 'Device is disabled!';
-                } else {
-                    deviceName = `[Disabled] ${deviceName}`;
-                }
+        if (deviceName === '' && device.description) {
+            deviceName = device.description;
+            description = '';
+        }
+
+        // Disabled-Flag nur bei echten Geräten – Gruppen können nicht disabled sein
+        if (!isGroup && device.disabled === true) {
+            if (this.config.useEventInDesc === true) {
+                description = 'Device is disabled!';
+            } else {
+                deviceName = `[Disabled] ${deviceName}`;
             }
+        }
 
-            if (!this.createCache[device.ieee_address] || this.createCache[device.ieee_address].name != deviceName || this.createCache[device.ieee_address].description != description) {
-                const deviceObj = {
-                    type: 'device',
-                    common: {
-                        icon: device.icon,
-                        name: deviceName,
-                        desc: description,
-                        statusStates: { onlineId: '' },
-                    },
-                    native: {
-                        deviceRemoved: false,
-                        groupDevice: false,
-                    },
-                };
+        if (!this.createCache[device.ieee_address] ||
+            this.createCache[device.ieee_address].name !== deviceName ||
+            this.createCache[device.ieee_address].description !== description) {
 
-                // Group Device
-                if (device.ieee_address.includes('group_')) {
-                    deviceObj.native.groupDevice = true;
-                    deviceObj.common.statusStates.onlineId = `${this.adapter.name}.${this.adapter.instance}.${device.ieee_address}.available`;
-                }
-                // Disabled Device
-                else if (device.disabled || device.disabled == true) {
-                    // Placeholder for possible later logic
-                }
-                // Only the onlineId is set if the device is not disabled
-                else {
-                    deviceObj.common.statusStates.onlineId = `${this.adapter.name}.${this.adapter.instance}.${device.ieee_address}.available`;
-                }               
-                await this.adapter.extendObjectAsync(device.ieee_address, deviceObj);
-                this.createCache[device.ieee_address] = { name: deviceName, description: description };
-            }
+            const deviceObj = {
+                type: 'device',
+                common: {
+                    icon: device.icon,
+                    name: deviceName,
+                    desc: description,
+                    statusStates: { onlineId: '' },
+                },
+                native: {
+                    deviceRemoved: false,
+                    groupDevice: isGroup,
+                },
+            };
 
-            // Here it is checked whether the scenes match the current data from z2m.
-            // If necessary, scenes are automatically deleted from ioBroker.
-            const sceneStates = await this.adapter.getStatesAsync(`${device.ieee_address}.scene_*`);
-            const sceneIDs = Object.keys(sceneStates);
-            for (const sceneID of sceneIDs) {
-                const stateID = sceneID.split('.')[3];
-                if (device.states.find((x) => x.id == stateID) == null) {
-                    this.adapter.delObject(sceneID);
-                }
+            if (isGroup || device.disabled !== true) {
+                deviceObj.common.statusStates.onlineId = `${this.adapter.name}.${this.adapter.instance}.${device.ieee_address}.available`;
             }
 
-            if (device != undefined && device.states != undefined) {
-                for (const state of device.states) {
-                    if (state && (!this.createCache[device.ieee_address][state.id] || this.createCache[device.ieee_address][state.id].name != state.name)) {
-                        const iobState = {
-                            type: 'state',
-                            common: await this.copyAndCleanStateObj(state),
-                            native: {},
-                        };
+            await this.adapter.extendObjectAsync(device.ieee_address, deviceObj);
+            this.createCache[device.ieee_address] = { name: deviceName, description: description };
+        }
 
-                        await this.adapter.extendObjectAsync(`${device.ieee_address}.${state.id}`, iobState);
-                        this.createCache[device.ieee_address][state.id] = {name: state.name, created: true};
-                    }
+        if (!Array.isArray(device.states)) {
+            return;
+        }
+
+        // Veraltete scene_*-States bereinigen – getForeignObjectsAsync mit vollem Namespace-Pfad
+        const sceneObjects = await this.adapter.getForeignObjectsAsync(
+            `${this.adapter.namespace}.${device.ieee_address}.scene_*`
+        );
+        if (sceneObjects) {
+            for (const sceneObjId of Object.keys(sceneObjects)) {
+                const parts = sceneObjId.split('.');
+                const stateID = parts[parts.length - 1];
+                if (!device.states.find((x) => x.id === stateID)) {
+                    await this.adapter.delForeignObjectAsync(sceneObjId);
                 }
             }
         }
-    }
 
-    /**
-     *
-     * @param messageObj
-     */
-    async renameDeviceInCache(messageObj) {
-        const renamedDevice = this.groupCache
-            .concat(this.deviceCache)
-            .find((x) => x.id == messageObj.payload.data.from);
-        if (renamedDevice) {
-            renamedDevice.id = messageObj.payload.data.to;
+        for (const state of device.states) {
+            if (state && (!this.createCache[device.ieee_address][state.id] ||
+                this.createCache[device.ieee_address][state.id].name !== state.name)) {
+
+                const iobState = {
+                    type: 'state',
+                    common: await this.copyAndCleanStateObj(state),
+                    native: {},
+                };
+
+                await this.adapter.extendObjectAsync(`${device.ieee_address}.${state.id}`, iobState);
+                this.createCache[device.ieee_address][state.id] = { name: state.name, created: true };
+            }
         }
     }
 
     /**
-     *
+     * Nur echte Geräte prüfen – Gruppen werden hier nie verarbeitet
      */
     async checkAndProgressDeviceRemove() {
-        let description = '';
-        let deviceName = '';
         let iobDevices = await this.adapter.getDevicesAsync();
-        // Do not consider devices already marked as "deviceRemoved"
-        iobDevices = iobDevices.filter((x) => x.native.deviceRemoved == false);
-        // Do not consider groups
-        iobDevices = iobDevices.filter((x) => x.native.groupDevice == false);
+        // Nur nicht-entfernte, echte Geräte (keine Gruppen)
+        iobDevices = iobDevices.filter((x) => x.native && x.native.deviceRemoved === false);
+        iobDevices = iobDevices.filter((x) => x.native && x.native.groupDevice === false);
 
         for (const iobDevice of iobDevices) {
-            const ieee_address = iobDevice._id.split('.')[2];
-            //Check whether the devices found from the object tree are also available in the DeviceCache
-            if (!this.deviceCache.find((x) => x.ieee_address == ieee_address)) {
-                deviceName = iobDevice.common.name;
+            const idParts = iobDevice._id.split('.');
+            if (idParts.length < 3) {continue;}
+            const ieee_address = idParts[2];
 
-                if (this.config.useEventInDesc == true) {
+            // Gruppen-Einträge im ioBroker-Baum explizit überspringen
+            if (ieee_address.startsWith('group_')) {continue;}
+
+            if (!this.deviceCache.find((x) => x.ieee_address === ieee_address)) {
+                let deviceName = iobDevice.common && iobDevice.common.name ? iobDevice.common.name : ieee_address;
+                let description = '';
+
+                if (this.config.useEventInDesc === true) {
                     description = 'Device was removed!';
                 } else {
                     deviceName = `[Removed] ${deviceName}`;
                 }
 
-                this.adapter.extendObject(`${ieee_address}`, {
-                    common: {
-                        name: deviceName,
-                        desc: description,
-                    },
-                    native: {
-                        deviceRemoved: true,
-                    },
+                await this.adapter.extendObjectAsync(ieee_address, {
+                    common: { name: deviceName, desc: description },
+                    native: { deviceRemoved: true },
                 });
-                this.adapter.setStateChangedAsync(`${ieee_address}.available`, false, true);
+                await this.adapter.setStateChangedAsync(`${ieee_address}.available`, false, true);
 
                 delete this.createCache[ieee_address];
             }
         }
     }
 
+    /**
+     *
+     * @param messageObj
+     */
+    async renameDeviceInCache(messageObj) {
+        if (!messageObj.payload || !messageObj.payload.data) {
+            return;
+        }
+        const renamedDevice = this.groupCache
+            .concat(this.deviceCache)
+            .find((x) => x.id === messageObj.payload.data.from);
+        if (renamedDevice) {
+            renamedDevice.id = messageObj.payload.data.to;
+        }
+    }
+
     /**
      *
      * @param devices
      * @param ieee_address
      */
     removeDeviceByIeee(devices, ieee_address) {
-        const idx = devices.findIndex((x) => x.ieee_address == ieee_address);
+        const idx = devices.findIndex((x) => x.ieee_address === ieee_address);
         if (idx > -1) {
             devices.splice(idx, 1);
         }
@@ -434,7 +464,7 @@ class DeviceController {
      * @param device
      */
     getDeviceName(device) {
-        return device.id == device.ieee_address ? '' : device.id;
+        return device.id === device.ieee_address ? '' : device.id;
     }
 
     /**
@@ -456,7 +486,11 @@ class DeviceController {
             this.adapter.setState('info.missing_routers_count', missingRoutersCount, true);
 
             if (missingRoutersCount > 0) {
-                this.adapter.log[this.config.coordinatorCheckLogLvl](
+                const logLvl = this.config.coordinatorCheckLogLvl;
+                const logFn = (logLvl && typeof this.adapter.log[logLvl] === 'function')
+                    ? this.adapter.log[logLvl].bind(this.adapter.log)
+                    : this.adapter.log.warn.bind(this.adapter.log);
+                logFn(
                     `Coordinator check: ${missingRoutersCount} missing routers were found, please check the data point 'zigbee2mqtt.x.info.missing_routers'!`
                 );
             } else {