1.0.5

nicoduj · Aug 22, 2019 · c3cf88e · c3cf88e
1 parent 58257fa
commit c3cf88e
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 20 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,10 @@
 
 All notable changes to this project will be documented in this file.
 
+## 1.0.5
+
+- [FIX] trying to better handle some problems with clean cache / name caching
+
 ## 1.0.4
 
 - [FIX] sequencesToPublishAsAccessoriesSwitch switches don't do anything #165

diff --git a/harmonyPlatform.js b/harmonyPlatform.js
@@ -57,29 +57,31 @@ function HarmonyPlatform(log, config, api) {
           let platform = this.platforms[i];
           platform.harmonyBase.harmony.removeAllListeners();
           platform.harmonyBase.harmony.close();
+
+          if (this.cleanCache)
+          {
+            this.log(
+              'WARNING - shutdown - cleaning cache.'   
+            );
+
+            this.api.unregisterPlatformAccessories(
+              'homebridge-harmonyHub',
+              'HarmonyHubWebSocket',
+              platform._foundAccessories
+            );
+          }
+
         }
+
+
+
       }.bind(this)
     )
     .on(
       'didFinishLaunching',
       function() {
         this.log('DidFinishLaunching');
 
-        if (this.cleanCache) {
-          this.log('WARNING - Removing Accessories');
-          this.api.unregisterPlatformAccessories(
-            'homebridge-harmonyHub',
-            'HarmonyHubWebSocket',
-            this._foundAccessories
-          );
-          this._foundAccessories = [];
-
-          for (let i = 0, len = this.platforms.length; i < len; i++) {
-            let platform = this.platforms[i];
-            platform._foundAccessories = [];
-          }
-        }
-
         for (let i = 0, len = this.platforms.length; i < len; i++) {
           let platform = this.platforms[i];
           platform.harmonyBase.configureAccessories(platform);
@@ -98,10 +100,10 @@ HarmonyPlatform.prototype = {
     if (this.platforms && this.platforms.length > 0)
       platform = this.platforms.find(x => x.name == platformName);
 
-    if (platform == undefined)
+    if (platform == undefined || this.cleanCache || this.config == undefined)
     {
       this.log(
-        'WARNING - configureAccessory - The platform ' + platformName + ' is not there anymore in your config (name property). It won\'t be loaded and will be removed from cache.'   
+        'WARNING - configureAccessory - The platform ' + platformName + ' is not there anymore in your config (name property) or cleanCached set to true. It won\'t be loaded and will be removed from cache.'   
       );
 
       this.api.unregisterPlatformAccessories(

diff --git a/harmonySubPlatform.js b/harmonySubPlatform.js
@@ -294,7 +294,7 @@ HarmonySubPlatform.prototype = {
       accessory.addService(this.mainService);
     }
 
-    if (this.savedNames && this.savedNames[0]) {
+    if (this.savedNames && this.savedNames[0] && this.savedNames[0] !== "" ) {
       mainServiceName = this.savedNames[0];
     } else {
       mainServiceName = this.name;
@@ -400,7 +400,7 @@ HarmonySubPlatform.prototype = {
       inputSourceService
     );
 
-    if (this.savedNames && this.savedNames[inputId]) {
+    if (this.savedNames && this.savedNames[inputId] && this.savedNames[inputId]!== "") {
       inputServiceName = this.savedNames[inputId];
     } else {
       inputServiceName = inputName;

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "homebridge-harmony",
-  "version": "1.0.4",
+  "version": "1.0.5",
   "author": "Nicolas Dujardin",
   "description": "Publish your harmony activities as homekit accessories",
   "main": "index.js",