Homebridge latest 1.3 beta issues

Hey, Thanks for taking care of the plugin and maintaining it. I’m seeing an issue on the latest Homebridge beta… it’s not a real issue now but it would be in the future.

getting a lot of this:

[22/12/2020, 13:29:51] [BroadlinkRM] RMmini Learn getSwitchState: undefined
[22/12/2020, 13:29:51] Received warning for the plugin 'homebridge-broadlink-rm-pro' from the characteristic 'On': characteristic was supplied illegal value: undefined! This might throw errors in the future!
[On@Homebridge D688 BF96] characteristic was supplied illegal value: undefined! This might throw errors in the future!

I’m not sure if you are familiar with it but it’s due to the fact that the new beta does not like getting invalid values as states… therefore it’s important to verify that the value is valid before returning it with the callback or updateValue.

any change you’ll take care of it? thanks

[horaciopaulo]

(sorry. removed cached accessories…problem solved)

…not so solved…investigating, will get back here

[masterfordev]

About this…

One more warn, this with my STB with no pingIPAddress (i thinks that maybe the issue, because my others tvs with pingIPAddress works fine):

[Caja del Cuarto 26D1@Caja del Cuarto@Active] characteristic was supplied illegal value: undefined! This might throw errors in the future!

@kiwi-cam maybe the issue is here where you should set 0 or 1. Maybe… 😅 considering that the documentation indicates that the allowed values are 0 or 1.

Maybe fix:

const value = active ? Characteristic.Active.ACTIVE : Characteristic.Active.INACTIVE;

Or maybe this, dont know. I dont know, i have not yet fully understood the logic of the plugin 😅 but I try to help.