search

rudyberends / homebridge-loxone-proxy

Homebridge Dynamic Platform Plugin which exposes a Loxone System to Homekit
Apache License 2.0
11 stars 6 forks source link

Whitespace in Radio Buttons block All-Off description - causes Syntax error #95

Closed VVlasy closed 2 months ago

VVlasy commented 3 months ago

Hi, I have installed your plugin and added my miniserver to it.

In logs I am seeing this error log. 

[5/19/2024, 3:29:42 PM] [homebridge-loxone-proxy] Connected to Miniserver
SyntaxError: Unexpected identifier 'vypnout'
    at new Function (<anonymous>)
    at Radio.callBackHandler (/homebridge/node_modules/homebridge-loxone-proxy/src/loxone/items/Radio.ts:42:29)
    at Radio.LoxoneAccessory.callBack (/homebridge/node_modules/homebridge-loxone-proxy/src/LoxoneAccessory.ts:95:12)
    at /homebridge/node_modules/homebridge-loxone-proxy/src/loxone/LoxoneHandler.ts:69:56
    at Array.forEach (<anonymous>)
    at handleAnyEvent (/homebridge/node_modules/homebridge-loxone-proxy/src/loxone/LoxoneHandler.ts:69:36)
    at Object.socketOnEventReceived (/homebridge/node_modules/homebridge-loxone-proxy/src/loxone/LoxoneHandler.ts:96:17)
    at WebSocket._handleBinaryEvent (/homebridge/node_modules/homebridge-loxone-proxy/node_modules/lxcommunicator/modules/WebSocket.js:934:78)
    at WebSocket._binaryMessageHandler (/homebridge/node_modules/homebridge-loxone-proxy/node_modules/lxcommunicator/modules/WebSocket.js:773:26)
    at W3CWebSocket.onmessage (/homebridge/node_modules/homebridge-loxone-proxy/node_modules/lxcommunicator/vendor/WebSocketWrapper.js:107:62)

Seeing as the error originates in Radio component. I checked my only radio block in my Miniserver configuration.

image

I have used a whitespace in the 'All-Off' description field. When I removed the whitespace and restarted homebridge. The error disappeared.

rudyberends commented 3 months ago

Hi,

It should be fixed in the latest Beta version. Would you be able to confirm this?