homebridge-xbox-tv 2.7.5 → 2.7.6

package/index.js

@@ -719,13 +719,13 @@ class XBOXDEVICE {
 			const input = inputs[i];
 
 			//get title Id
-			const inputTitleId = input.titleId || 'undefined';
+			const inputTitleId = input.titleId || 'Not set';
 
 			//get input reference
-			const inputReference = input.reference || 'undefined';
+			const inputReference = input.reference || 'Not set';
 
 			//get input oneStoreProductId
-			const inputOneStoreProductId = this.webApiEnabled ? input.oneStoreProductId || 'undefined' : 'undefined';
+			const inputOneStoreProductId = this.webApiEnabled ? input.oneStoreProductId || 'Not set' : 'Not set';
 
 			//get input name
 			const inputName = savedInputsNames[inputTitleId] ? savedInputsNames[inputTitleId] : savedInputsNames[inputReference] ? savedInputsNames[inputReference] : savedInputsNames[inputOneStoreProductId] ? savedInputsNames[inputOneStoreProductId] : input.name;
@@ -743,21 +743,24 @@ class XBOXDEVICE {
 			const inputService = new Service.InputSource(inputName, `Input ${i}`);
 			inputService
 				.setCharacteristic(Characteristic.Identifier, i)
-				.setCharacteristic(Characteristic.ConfiguredName, inputName)
+				.setCharacteristic(Characteristic.Name, inputName)
 				.setCharacteristic(Characteristic.IsConfigured, isConfigured)
 				.setCharacteristic(Characteristic.InputSourceType, inputType)
 				.setCharacteristic(Characteristic.CurrentVisibilityState, currentVisibility)
-				.setCharacteristic(Characteristic.TargetVisibilityState, targetVisibility);
 
-			inputService
-				.getCharacteristic(Characteristic.ConfiguredName)
+			inputService.getCharacteristic(Characteristic.ConfiguredName)
+				.onGet(async () => {
+					return inputName;
+				})
 				.onSet(async (name) => {
 					try {
 						const nameIdentifier = inputTitleId ? inputTitleId : inputReference ? inputReference : inputOneStoreProductId ? inputOneStoreProductId : false;
 						savedInputsNames[nameIdentifier] = name;
+
 						const newCustomName = JSON.stringify(savedInputsNames, null, 2);
-						fs.writeFileSync(this.inputsNamesFile, newCustomName);
-						const logInfo = this.enableDebugMode || this.firstRun ? false : this.log(`Device: ${this.host} ${accessoryName}, saved new Input name: ${name}, one store product id: ${inputOneStoreProductId}`);
+						await fsPromises.writeFile(this.inputsNamesFile, newCustomName);
+						const logDebug = this.enableDebugMode ? this.log(`Device: ${this.host} ${accessoryName}, saved new Input name: ${name}, one store product id: ${inputOneStoreProductId}`) : false;
+						inputService.setCharacteristic(Characteristic.Name, inputName);
 					} catch (error) {
 						this.log.error(`Device: ${this.host} ${accessoryName}, new Input name save error: ${error}`);
 					}
@@ -765,13 +768,17 @@ class XBOXDEVICE {
 
 			inputService
 				.getCharacteristic(Characteristic.TargetVisibilityState)
+				.onGet(async () => {
+					return targetVisibility;
+				})
 				.onSet(async (state) => {
 					try {
 						const targetVisibilityIdentifier = inputTitleId ? inputTitleId : inputReference ? inputReference : inputOneStoreProductId ? inputOneStoreProductId : false;
 						savedInputsTargetVisibility[targetVisibilityIdentifier] = state;
+
 						const newTargetVisibility = JSON.stringify(savedInputsTargetVisibility, null, 2);
-						fs.writeFileSync(this.inputsTargetVisibilityFile, newTargetVisibility);
-						const logInfo = this.enableDebugMode || this.firstRun ? false : this.log(`Device: ${this.host} ${accessoryName}, saved new Input: ${inputName} target visibility state: ${state ? 'HIDEN' : 'SHOWN'}`);
+						await fsPromises.writeFile(this.inputsTargetVisibilityFile, newTargetVisibility);
+						const logDebug = this.enableDebugMode ? this.log(`Device: ${this.host} ${accessoryName}, saved new Input: ${inputName} target visibility state: ${state ? 'HIDEN' : 'SHOWN'}`) : false;
 						inputService.setCharacteristic(Characteristic.CurrentVisibilityState, state);
 					} catch (error) {
 						this.log.error(`Device: ${this.host} ${accessoryName}, new target visibility state save error: ${error}`);
@@ -856,7 +863,7 @@ class XBOXDEVICE {
 					buttonService.getCharacteristic(Characteristic.On)
 						.onGet(async () => {
 							const state = false;
-							const logInfo = this.disableLogInfo || this.firstRun ? false : this.log(`Device: ${this.host} ${accessoryName}, get Button state successful: ${state}`);
+							const logDebug = this.disableLogInfo ? this.log(`Device: ${this.host} ${accessoryName}, get Button state successful: ${state}`) : false;
 							return state;
 						})
 						.onSet(async (state) => {
@@ -908,7 +915,7 @@ class XBOXDEVICE {
 											}
 											break;
 									}
-									const logInfo = this.enableDebugMode || this.firstRun || buttonMode === -1 ? false : this.log(`Device: ${this.host} ${accessoryName}, set button successful, name:  ${buttonName}, command: ${buttonCommand}`);
+									const logDebug = this.enableDebugMode ? this.log(`Device: ${this.host} ${accessoryName}, set button successful, name:  ${buttonName}, command: ${buttonCommand}`) : false;
 								}
 								await new Promise(resolve => setTimeout(resolve, 300));
 								const setChar = this.power && state ? buttonService.updateCharacteristic(Characteristic.On, false) : false;
@@ -925,4 +932,4 @@ class XBOXDEVICE {
 		this.api.publishExternalAccessories(PLUGIN_NAME, [accessory]);
 		const debug3 = this.enableDebugMode ? this.log(`Device: ${this.host} ${accessoryName}, published as external accessory.`) : false;
 	}
-};
+};

package/package.json

@@ -1,7 +1,7 @@
 {
   "displayName": "Xbox TV",
   "name": "homebridge-xbox-tv",
-  "version": "2.7.5",
+  "version": "2.7.6",
   "description": "Homebridge plugin (https://github.com/homebridge/homebridge) to control Xbox game consoles.",
   "license": "MIT",
   "author": "grzegorz914",