evlli/hass-core
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Apply late review comments for Comelit climate (#109114)

Browse source 
* Apply late review comments for Comelit climate

* remove logging
This commit is contained in:
Simone Chemelli 2024-01-31 13:13:25 +01:00 • committed by GitHub
parent dcd677fea8
commit 18a6ae081a
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 2 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

6
homeassistant/components/comelit/climate.py
View file

@ -21,7 +21,7 @@ from homeassistant.core import HomeAssistant
from homeassistant.helpers.entity_platform import AddEntitiesCallback from homeassistant.helpers.entity_platform import AddEntitiesCallback
from homeassistant.helpers.update_coordinator import CoordinatorEntity from homeassistant.helpers.update_coordinator import CoordinatorEntity
from .const import _LOGGER, DOMAIN from .const import DOMAIN
from .coordinator import ComelitSerialBridge from .coordinator import ComelitSerialBridge
@ -115,7 +115,7 @@ class ComelitClimateEntity(CoordinatorEntity[ComelitSerialBridge], ClimateEntity
@property @property
def _clima(self) -> list[Any]: def _clima(self) -> list[Any]:
"""Return clima device data.""" """Return clima device data."""
# CLIMATE has 2 turple: # CLIMATE has a 2 item tuple:
# - first for Clima # - first for Clima
# - second for Humidifier # - second for Humidifier
return self.coordinator.data[CLIMATE][self._device.index].val[0] return self.coordinator.data[CLIMATE][self._device.index].val[0]
@ -159,7 +159,6 @@ class ComelitClimateEntity(CoordinatorEntity[ComelitSerialBridge], ClimateEntity
if self._api_mode in API_STATUS: if self._api_mode in API_STATUS:
return API_STATUS[self._api_mode]["hvac_mode"] return API_STATUS[self._api_mode]["hvac_mode"]
_LOGGER.warning("Unknown API mode '%s' in hvac_mode", self._api_mode)
return None return None
@property @property
@ -175,7 +174,6 @@ class ComelitClimateEntity(CoordinatorEntity[ComelitSerialBridge], ClimateEntity
if self._api_mode in API_STATUS: if self._api_mode in API_STATUS:
return API_STATUS[self._api_mode]["hvac_action"] return API_STATUS[self._api_mode]["hvac_action"]
_LOGGER.warning("Unknown API mode '%s' in hvac_action", self._api_mode)
return None return None
async def async_set_temperature(self, **kwargs: Any) -> None: async def async_set_temperature(self, **kwargs: Any) -> None: