Merge branch 'release/1.0.2'

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## v1.0.2 -  2021-03-19
+- Fixed a bug that can crash homebridge when no devices are defined in the config
+
 ## v1.0.1 -  2021-03-12
 - Complete rewrite of the Smartglass plugin with xbox api functionalities. Allows you to launch apps and have realtime status of your console.
 

README.md

@@ -2,7 +2,7 @@
 
 # Homebridge-Smartglass
 
-[![Build and Lint](https://github.com/unknownskl/homebridge-smartglass/actions/workflows/build.yml/badge.svg?branch=release%2F1.0.1)](https://github.com/unknownskl/homebridge-smartglass/actions/workflows/build.yml)
+[![Build and Lint](https://github.com/unknownskl/homebridge-smartglass/actions/workflows/build.yml/badge.svg?branch=release%2F1.0.2)](https://github.com/unknownskl/homebridge-smartglass/actions/workflows/build.yml)
 [![npm](https://img.shields.io/npm/v/homebridge-smartglass.svg?style=flat-square)](https://www.npmjs.com/package/homebridge-smartglass)
 [![npm](https://img.shields.io/npm/dt/homebridge-smartglass.svg?style=flat-square)](https://www.npmjs.com/package/homebridge-smartglass)
 

package.json

@@ -1,7 +1,7 @@
 {
   "displayName": "Xbox Smartglass",
   "name": "homebridge-smartglass",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "Smartglass plugin for homebridge. Allows you to control your Xbox using Homekit",
   "main": "dist/index.js",
   "license": "Apache-2.0",

src/platform.ts

@@ -31,7 +31,7 @@ export class SmartglassPlatform implements DynamicPlatformPlugin {
     public readonly config: PlatformConfig,
     public readonly api: API,
   ) {
-    this.log.debug('Finished initializing platform:', this.config.name);
+    this.log.debug('Finished initializing platform:', this.config.platform);
 
     // When this event is fired it means Homebridge has restored all cached accessories from disk.
     // Dynamic Platform plugins should only register new accessories after this event was fired,
@@ -62,25 +62,11 @@ export class SmartglassPlatform implements DynamicPlatformPlugin {
    */
   discoverDevices() {
 
-    // EXAMPLE ONLY
-    // A real plugin you would discover accessories from the local network, cloud services
-    // or a user-defined array in the platform config.
-    // const exampleDevices = [
-    //   {
-    //     exampleUniqueId: 'ABCD',
-    //     exampleDisplayName: 'Bedroom',
-    //   },
-    //   {
-    //     exampleUniqueId: 'EFGH',
-    //     exampleDisplayName: 'Kitchen',
-    //   },
-    // ];
-
-    const exampleDevices = this.config.devices as DeviceConfig[];
+    const configDevices = this.config.devices as DeviceConfig[] || [];
 
 
     // loop over the discovered devices and register each one if it has not already been registered
-    for (const device of exampleDevices) {
+    for (const device of configDevices) {
 
       // generate a unique id for the accessory this should be generated from
       // something globally unique, but constant, for example, the device serial