search

chomupashchuk / ariston-remotethermo-home-assistant-v2

Ariston NET remotethermo integration for Home Assistant based on API
MIT License
94 stars 22 forks source link

Problem setting ch_set_temperature_zone1: ch_mode #66

Open gftrotta opened 1 year ago

gftrotta commented 1 year ago

Hi, I should have found a possibile bug in the the function _preparing_setting_http_data in ariston.py file.

In particular I think that the code on line 1878

if set_temp == economy_old and self._get_sensor_value(self._PARAM_CH_MODE) == "Time program":

should be:

if set_temp == economy_old and self._get_sensor_value((self._zone_sensor_name(self._PARAM_CH_MODE, zone))) == "Time program":

Else, the function _get_sensor_value is triggered with a _PRAM_CH_MODE without a zone.

Is it correct?