homebridge-denon-tv 4.4.2 → 4.4.4

package/package.json

@@ -1,7 +1,7 @@
 {
   "displayName": "Denon TV",
   "name": "homebridge-denon-tv",
-  "version": "4.4.2",
+  "version": "4.4.4",
   "description": "Homebridge plugin to control Denon/Marantz AV Receivers.",
   "license": "MIT",
   "author": "grzegorz914",

package/src/denon.js

@@ -224,6 +224,12 @@ class DENON extends EventEmitter {
                     //select reference
                     const reference = zoneControl <= 2 ? input : soundMode;
 
+                    //update only if value change
+                    if (power === this.power && reference === this.reference && volume === this.volume && volumeDisplay === this.volumeDisplay && mute === this.mute && pictureMode === this.pictureMode && soundMode === this.soundMode) {
+                        this.checkState();
+                        return;
+                    };
+
                     this.power = power;
                     this.reference = reference;
                     this.volume = volume;
@@ -232,7 +238,7 @@ class DENON extends EventEmitter {
                     this.pictureMode = pictureMode;
                     this.soundMode = soundMode;
 
-                    //emit state
+                    //emit state changes
                     const emitConnected = !this.connected ? this.emit('message', `Connected.`) : false;
                     this.connected = true;
                     this.emit('stateChanged', power, reference, volume, volumeDisplay, mute, pictureMode);
@@ -249,7 +255,7 @@ class DENON extends EventEmitter {
 
                     this.checkState();
                 } catch (error) {
-                    const debug = disableLogConnectError ? false : this.emit('error', `State error: ${error}, reconnect in 15s.`);
+                    const debug = disableLogConnectError ? false : this.emit('error', `State error: ${error}, reconnect in ${this.refreshInterval}s.`);
                     this.emit('disconnect');
                 };
             })

package/src/denondevice.js

@@ -83,7 +83,7 @@ class DenonDevice extends EventEmitter {
         this.volumeControlType = ';';
         this.mute = true;
         this.mediaState = false;
-        this.inputIdentifier = 0;
+        this.inputIdentifier = 1;
         this.supportPictureMode = false;
         this.pictureMode = 0;
         this.brightness = 0;
@@ -793,25 +793,24 @@ class DenonDevice extends EventEmitter {
                 const inputsCount = inputs.length;
                 const possibleInputsCount = 85 - this.allServices.length;
                 const maxInputsCount = inputsCount >= possibleInputsCount ? possibleInputsCount : inputsCount;
-                let inputIdentifier = 0;
                 for (let i = 0; i < maxInputsCount; i++) {
                     //input
                     const input = inputs[i];
-
-                    //get input reference
-                    const inputReference = input.reference;
+                    input.identifier = i + 1;
 
                     //get input name
                     const name = input.name ?? 'App/Input';
-                    const savedInputsNames = this.savedInputsNames[inputReference] ?? false;
+                    const savedInputsNames = this.savedInputsNames[input.reference] ?? false;
                     const inputName = !savedInputsNames ? name : savedInputsNames;
+                    input.name = inputName;
+
+                    //get input reference
+                    const inputReference = input.reference;
 
                     //get visibility
                     const currentVisibility = this.savedInputsTargetVisibility[inputReference] ?? 0;
-
-                    input.identifier = inputIdentifier++;
-                    input.name = inputName;
                     input.visibility = currentVisibility;
+
                     this.inputsConfigured.push(input);
                 }