williamp/localtuya-modded
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix bug when climate entity is using default min/max temp and temp unit is F

Browse Source 
The default min/max temps are in C but when the climate entity is set up with F this will cause incorrect boundary check and an inability to change the temperature from the home assistant entity UI widget.
This commit is contained in:
nu
2023-08-17 23:59:53 -04:00
committed by GitHub
parent f06e4848e6
commit b3afa14095
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
custom_components/localtuya/climate.py
View File

@@ -343,6 +343,10 @@ class LocaltuyaClimate(LocalTuyaEntity, ClimateEntity):
"""Return the minimum temperature.""" """Return the minimum temperature."""
if self.has_config(CONF_MIN_TEMP_DP): if self.has_config(CONF_MIN_TEMP_DP):
return self.dps_conf(CONF_MIN_TEMP_DP) return self.dps_conf(CONF_MIN_TEMP_DP)
# DEFAULT_MIN_TEMP is in C
if self.temperature_unit == TEMP_FAHRENHEIT:
return DEFAULT_MIN_TEMP * 1.8 + 32
else:
return DEFAULT_MIN_TEMP return DEFAULT_MIN_TEMP
@property @property
@@ -350,6 +354,10 @@ class LocaltuyaClimate(LocalTuyaEntity, ClimateEntity):
"""Return the maximum temperature.""" """Return the maximum temperature."""
if self.has_config(CONF_MAX_TEMP_DP): if self.has_config(CONF_MAX_TEMP_DP):
return self.dps_conf(CONF_MAX_TEMP_DP) return self.dps_conf(CONF_MAX_TEMP_DP)
# DEFAULT_MAX_TEMP is in C
if self.temperature_unit == TEMP_FAHRENHEIT:
return DEFAULT_MAX_TEMP * 1.8 + 32
else:
return DEFAULT_MAX_TEMP return DEFAULT_MAX_TEMP
def status_updated(self): def status_updated(self):