ylabonte / ioBroker.procon-ip

Adapter for ProCon.IP swimming pool control unit. It is intended for integration with your ioBroker home automation eg. building logic that involves other devices or to be combined with voice assistants (Alexa, Siri, Google) to give water temperature info or switch relays.
MIT License
6 stars 3 forks source link

Label for relays and sensors differ #31

Closed masterflai closed 2 years ago

masterflai commented 2 years ago

Version 1.2.1

My current instance shows always the label of the following element. For example relais 7 named as "Heat pump" within Procon-IP is also named "Heat Pump", but the text label shows the valua of the following relay 8. Same for sensors.

Can you reproduce this issue?

What is correct in this case? 2022-04-15 23_17_54-javascript - Pi4b

ylabonte commented 2 years ago

This should be an unintended side-effect of a change in the api library (https://github.com/ylabonte/procon-ip#release-v131). The object categories (sensors, relays, etc.) started counting with 1 in the past. Since this library change they are counting from 0. I could resolve this by working around the change in this adapter, but this could affect people that installed the adapter after that change or found a workaround be their own.

The solution to this problem is to delete the procon-ip object tree in your ioBroker installation and restart the adapter. All objects will automatically be recreated on adapter (re)start.

Can you confirm that this solution worked for you?

masterflai commented 2 years ago

Thanks for the fast reply Yannik. The issue was solved by removing the objects, instance and the adapter and their reinstallation. Important: If someone is running scripts on these objects they need also a rework to point to the correct objects.

ylabonte commented 2 years ago

Thanks for the advice! :)