Corrections @c727 suggestions

This commit is contained in:
Marius Ciotlos
2018-06-11 23:15:45 +03:00
committed by Koenkk
parent 06142115af
commit dc5a090d44
+6 -4
View File
@@ -14,6 +14,8 @@ const mqttDevicePrefixRegex = new RegExp(`${settings.get().mqtt.base_topic}/[\\w
const pollInterval = 60 * 1000; // seconds * 1000.
const softResetTimeout = 3600 * 1000; // seconds * 1000.
const allowedLogLevels = ['error', 'warn', 'info', 'debug'];
/**
* Home Assistant requires ALL attributes to be present in ALL MQTT messages send by the device.
* https://community.home-assistant.io/t/missing-value-with-mqtt-only-last-data-set-is-shown/47070/9
@@ -340,11 +342,11 @@ class Controller {
if (option === 'permit_join') {
this.zigbee.permitJoin(message.toString().toLowerCase() === 'true');
} else if (option === 'log_level') {
let level = message.toString().toLowerCase();
if (['error', 'warn', 'info', 'debug'].indexOf(level) > -1) {
const level = message.toString().toLowerCase();
if (allowedLogLevels.includes(level)) {
logger.warn(`Switching log level to '${level}'`);
logger.transports.console.level = message.toString().toLowerCase();
logger.transports.file.level = message.toString().toLowerCase();
logger.transports.console.level = level;
logger.transports.file.level = level;
} else {
logger.error(`Could not set log level to '${level}'`);
}