Move BLE states to DeviceRegistry and improve logging

- Make deviceRegistry states/lists immutable
This commit is contained in:
Victor Hagelbäck 2021-02-20 12:34:30 +01:00
parent e528e2be1f
commit 0dbe0bb95f
3 changed files with 86 additions and 34 deletions

View file

@ -171,9 +171,12 @@ class PlejBLEHandler extends EventEmitter {
fixedPlejdPath = fixedPlejdPath.replace(/_/g, '');
plejd.device = this.deviceRegistry.getDeviceBySerialNumber(fixedPlejdPath);
logger.debug(`Discovered ${plejd.path} with rssi ${plejd.rssi}, name ${plejd.device.name}`);
// Todo: Connect should probably be done here
this.bleDevices.push(plejd);
if (plejd.device) {
logger.debug(`Discovered ${plejd.path} with rssi ${plejd.rssi}, name ${plejd.device.name}`);
this.bleDevices.push(plejd);
} else {
logger.warn(`Device registry does not contain device with serial ${fixedPlejdPath}`);
}
} catch (err) {
logger.error(`Failed inspecting ${path}. `, err);
}