openhab / org.openhab.binding.zwave

openHAB binding for Z-Wave
Eclipse Public License 2.0
170 stars 202 forks source link

Fibaro FGT001 Thermostat: wrong channel for 2nd endpoint battery level #1821

Open Sail-Away opened 1 year ago

Sail-Away commented 1 year ago

If the problem is associated with a device, please provide the following -:

The channel battery-level2 is supposed to report the battery level of the additional external temperature sensor, if coupled. However, it reports the same value as battery-level and battery-level1, which reflect the thermostats internal battery. In the attached generated xml file you can see that indeed different values are reported for the two endpoints.

The issue also was mentioned in the openhab forum some time ago, without replies.

I suppose this issue could be related to the channel definition in the thing-description file for this device in the repository. In line 115, shouldn't it read <property name="binding:*:PercentType">COMMAND_CLASS_BATTERY:2</property> instead of <property name="binding:*:PercentType">COMMAND_CLASS_BATTERY</property>?

Is there any way to fix this without an update of the binding?

Sail-Away commented 1 year ago

Update: I was able to fix this for me. I changed the thing-description file fgt001_0_0.xml the way indicated in my first post and replaced the version in the binding in my current installation. After deleting (not removing from the controller) and re-adding the device the channel for endpoint 2 now reports the external sensor's battery level as expected.

For reference, the changes can be found in my repository.

However, I'm not sure if and how this could be corrected from the database.

Sail-Away commented 1 year ago

@cdjackson Chris, do you you have any thoughts on this? Not a big issue for me, but for future updates it would be great getting around without editing the jar file.

Thanks, Christian