homebridge-denon-tv 5.7.2 → 5.7.4

package/config.schema.json

@@ -4371,6 +4371,7 @@
                   "namePrefix": {
                     "title": "Prefix",
                     "type": "boolean",
+                    "default": false,
                     "description": "Here enable/disable the accessory name as a prefix for button name.",
                     "required": false
                   }
@@ -5083,6 +5084,7 @@
                   "namePrefix": {
                     "title": "Prefix",
                     "type": "boolean",
+                    "default": false,
                     "description": "Here enable/disable the accessory name as a prefix for button name.",
                     "required": false
                   }
@@ -5783,6 +5785,7 @@
                   "namePrefix": {
                     "title": "Prefix",
                     "type": "boolean",
+                    "default": false,
                     "description": "Here enable/disable the accessory name as a prefix for button name.",
                     "required": false
                   }
@@ -6365,6 +6368,7 @@
                   "namePrefix": {
                     "title": "Prefix",
                     "type": "boolean",
+                    "default": false,
                     "description": "Here enable/disable the accessory name as a prefix for sensor name.",
                     "required": false
                   }
@@ -6562,6 +6566,7 @@
                   "namePrefix": {
                     "title": "Prefix",
                     "type": "boolean",
+                    "default": false,
                     "description": "Here enable/disable the accessory name as a prefix for sensor name.",
                     "required": false
                   }

package/index.js

@@ -32,9 +32,9 @@ class DenonPlatform {
 				//check accessory is enabled
 				const disableAccessory = device.disableAccessory || false;
 				if (disableAccessory) {
-				  continue;
+					continue;
 				}
-				
+
 				const deviceName = device.name;
 				const host = device.host;
 				const port = device.port;
@@ -137,7 +137,7 @@ class DenonPlatform {
 							//start impulse generator
 							await impulseGenerator.start([{ name: 'start', sampling: 45000 }]);
 						} catch (error) {
-							throw new Error(`Device: ${host} ${deviceName}, Did finish launching error: ${error}.`);
+							const emitLog = disableLogError ? false : log.error(`Device: ${host} ${deviceName}, Did finish launching error: ${error}.`);
 						}
 						break;
 					case 1: //zone 2
@@ -185,7 +185,7 @@ class DenonPlatform {
 							//start impulse generator
 							await impulseGenerator.start([{ name: 'start', sampling: 45000 }]);
 						} catch (error) {
-							throw new Error(`Device: ${host} ${deviceName}, Did finish launching error: ${error}.`);
+							const emitLog = disableLogError ? false : log.error(`Device: ${host} ${deviceName}, Did finish launching error: ${error}.`);
 						}
 						break;
 					case 2: //zone 3
@@ -233,7 +233,7 @@ class DenonPlatform {
 							//start impulse generator
 							await impulseGenerator.start([{ name: 'start', sampling: 45000 }]);
 						} catch (error) {
-							throw new Error(`Device: ${host} ${deviceName}, Did finish launching error: ${error}.`);
+							const emitLog = disableLogError ? false : log.error(`Device: ${host} ${deviceName}, Did finish launching error: ${error}.`);
 						}
 						break;
 					case 3: //surrounds
@@ -281,7 +281,7 @@ class DenonPlatform {
 							//start impulse generator
 							await impulseGenerator.start([{ name: 'start', sampling: 45000 }]);
 						} catch (error) {
-							throw new Error(`Device: ${host} ${deviceName}, Did finish launching error: ${error}.`);
+							const emitLog = disableLogError ? false : log.error(`Device: ${host} ${deviceName}, Did finish launching error: ${error}.`);
 						}
 						break;
 					case 4: //pass through inputs
@@ -329,7 +329,7 @@ class DenonPlatform {
 							//start impulse generator
 							await impulseGenerator.start([{ name: 'start', sampling: 45000 }]);
 						} catch (error) {
-							throw new Error(`Device: ${host} ${deviceName}, Did finish launching error: ${error}.`);
+							const emitLog = disableLogError ? false : log.error(`Device: ${host} ${deviceName}, Did finish launching error: ${error}.`);
 						}
 						break;
 					default:

package/package.json

@@ -1,7 +1,7 @@
 {
   "displayName": "Denon TV",
   "name": "homebridge-denon-tv",
-  "version": "5.7.2",
+  "version": "5.7.4",
   "description": "Homebridge plugin to control Denon/Marantz AV Receivers.",
   "license": "MIT",
   "author": "grzegorz914",
@@ -30,14 +30,14 @@
     "LICENSE"
   ],
   "engines": {
-    "homebridge": "^1.9.0 || ^2.0.0 || ^2.0.0-beta.26 || ^2.0.0-alpha.37",
+    "homebridge": "^1.9.0 || ^2.0.0 || ^2.0.0-beta.27 || ^2.0.0-alpha.37",
     "node": "^18.20.4 || ^20.15.1 || ^22.7.0 || ^23.2.0"
   },
   "dependencies": {
     "async-mqtt": "^2.6.3",
     "axios": "^1.8.4",
-    "fast-xml-parser": "^5.0.9",
-    "express": "^4.21.2"
+    "fast-xml-parser": "^5.2.0",
+    "express": "^5.1.0"
   },
   "keywords": [
     "homebridge",

package/src/impulsegenerator.js

@@ -12,6 +12,10 @@ class ImpulseGenerator extends EventEmitter {
             return true;
         }
 
+        //update state
+        const updateState = timers.length > 0 ? this.state(true) : false;
+
+        //add timers
         this.timers = [];
         for (const timer of timers) {
             this.emit(timer.name);
@@ -22,8 +26,6 @@ class ImpulseGenerator extends EventEmitter {
             this.timers.push(newTimer);
         };
 
-        //update state
-        this.state(true);
         return true;
     }
 
@@ -36,6 +38,7 @@ class ImpulseGenerator extends EventEmitter {
         //update state
         this.timers.forEach(timer => clearInterval(timer));
         this.timers = [];
+
         this.state(false);
         return true
     }