grzegorz914 / homebridge-denon-tv

Homebridge plugin for Denon/Marantz AV Receivers, publish as independent external platform accessory.
MIT License
70 stars 15 forks source link

Error after updating to 4.8.1 #231

Closed TheTeeDot closed 5 months ago

TheTeeDot commented 5 months ago

Hi there, since updating the plugin to 4.8.1 I now receive this error in my home bridge status logs:

Device: 192.168.1.31 Living Room Denon AVR-3300w, Info error: TypeError: Cannot read properties of undefined (reading 'value'), reconnect in 15s.

Thanks, T.

Edit, debug log below:

[1/18/2024, 6:27:33 AM] [DenonTv] Device: 192.168.1.31 Living Room Denon AVR-3300w, debug: Info: { "FriendlyName": { "value": "Denon AVR-X3300W" }, "Power": { "value": "STANDBY" }, "ZonePower": { "value": "OFF" }, "RCSourceSelect": { "value": "POS" }, "RenameZone": { "value": "MAIN ZONE" }, "TopMenuLink": { "value": "ON" }, "VideoSelectDisp": { "value": "OFF" }, "VideoSelect": { "value": "" }, "VideoSelectOnOff": { "value": "OFF" }, "VideoSelectLists": { "value": [ { "#text": "On", "@_index": "ON" }, { "#text": "Off", "@_index": "OFF" }, { "#text": "CBL/SAT", "@_index": "SAT/CBL" }, { "#text": "DVD", "@_index": "DVD" }, { "#text": "Blu-ray", "@_index": "BD" }, { "#text": "Game", "@_index": "GAME" }, { "#text": "AUX1", "@_index": "AUX1" }, { "#text": "AUX2", "@_index": "AUX2" }, { "#text": "Media Player", "@_index": "MPLAY" } ] }, "ECOModeDisp": { "value": "ON" }, "ECOMode": { "value": "" }, "ECOModeLists": { "value": [ { "@_index": "ON", "@_table": "ECO : ON", "@_param": "" }, { "@_index": "OFF", "@_table": "ECO : OFF", "@_param": "" }, { "@_index": "AUTO", "@_table": "ECO : AUTO", "@_param": "" } ] }, "AddSourceDisplay": { "value": "FALSE" }, "ModelId": { "value": 3 }, "BrandId": { "value": "DENON_MODEL" }, "SalesArea": { "value": 4 }, "InputFuncSelect": { "value": "AUX2" }, "NetFuncSelect": { "value": "FAVORITES" }, "selectSurround": { "value": "Stereo" }, "VolumeDisplay": { "value": "Absolute" }, "MasterVolume": { "value": -28 }, "Mute": { "value": "off" }, "RemoteMaintenance": { "value": "" }, "SubwooferDisplay": { "value": "FALSE" }, "Zone2VolDisp": { "value": "TRUE" }, "SleepOff": { "value": "Off" } } [1/18/2024, 6:27:33 AM] [DenonTv] Device: 192.168.1.31 Living Room Denon AVR-3300w, saved device info: { "FriendlyName": { "value": "Denon AVR-X3300W" }, "Power": { "value": "STANDBY" }, "ZonePower": { "value": "OFF" }, "RCSourceSelect": { "value": "POS" }, "RenameZone": { "value": "MAIN ZONE" }, "TopMenuLink": { "value": "ON" }, "VideoSelectDisp": { "value": "OFF" }, "VideoSelect": { "value": "" }, "VideoSelectOnOff": { "value": "OFF" }, "VideoSelectLists": { "value": [ { "#text": "On", "@_index": "ON" }, { "#text": "Off", "@_index": "OFF" }, { "#text": "CBL/SAT", "@_index": "SAT/CBL" }, { "#text": "DVD", "@_index": "DVD" }, { "#text": "Blu-ray", "@_index": "BD" }, { "#text": "Game", "@_index": "GAME" }, { "#text": "AUX1", "@_index": "AUX1" }, { "#text": "AUX2", "@_index": "AUX2" }, { "#text": "Media Player", "@_index": "MPLAY" } ] }, "ECOModeDisp": { "value": "ON" }, "ECOMode": { "value": "" }, "ECOModeLists": { "value": [ { "@_index": "ON", "@_table": "ECO : ON", "@_param": "" }, { "@_index": "OFF", "@_table": "ECO : OFF", "@_param": "" }, { "@_index": "AUTO", "@_table": "ECO : AUTO", "@_param": "" } ] }, "AddSourceDisplay": { "value": "FALSE" }, "ModelId": { "value": 3 }, "BrandId": { "value": "DENON_MODEL" }, "SalesArea": { "value": 4 }, "InputFuncSelect": { "value": "AUX2" }, "NetFuncSelect": { "value": "FAVORITES" }, "selectSurround": { "value": "Stereo" }, "VolumeDisplay": { "value": "Absolute" }, "MasterVolume": { "value": -28 }, "Mute": { "value": "off" }, "RemoteMaintenance": { "value": "" }, "SubwooferDisplay": { "value": "FALSE" }, "Zone2VolDisp": { "value": "TRUE" }, "SleepOff": { "value": "Off" } }

grzegorz914 commented 5 months ago

which version was before? are you select correct generation in config?

TheTeeDot commented 5 months ago

Version: I didn't take note but probably 4.7.0 given the release history. I tend to upgrade when I see a version bump.

Yes, old generation selected in the config

grzegorz914 commented 5 months ago

can you try with middle generation

TheTeeDot commented 5 months ago

Changed - the error is no longer present and the device is now 'responding' and usable within HomeKit That appears to have resolved things

Thanks.

grzegorz914 commented 5 months ago

nice