Skip to content

Commit

Permalink
airzone_cloud: use entity_id on errors/logs
Browse files Browse the repository at this point in the history
Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
  • Loading branch information
Noltari committed Oct 30, 2023
1 parent eacf72f commit 1cfc888
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 16 deletions.
3 changes: 1 addition & 2 deletions homeassistant/components/airzone_cloud/climate.py
Expand Up @@ -20,7 +20,6 @@
AZD_MASTER,
AZD_MODE,
AZD_MODES,
AZD_NAME,
AZD_NUM_DEVICES,
AZD_NUM_GROUPS,
AZD_POWER,
Expand Down Expand Up @@ -388,5 +387,5 @@ async def async_set_hvac_mode(self, hvac_mode: HVACMode) -> None:

if slave_raise:
raise HomeAssistantError(
f"Mode can't be changed on slave zone {self.get_airzone_value(AZD_NAME)}"
f"Mode can't be changed on slave zone {self.entity_id}"
)
22 changes: 8 additions & 14 deletions homeassistant/components/airzone_cloud/entity.py
Expand Up @@ -80,16 +80,14 @@ def get_airzone_value(self, key: str) -> Any:

async def _async_update_params(self, params: dict[str, Any]) -> None:
"""Send Aidoo parameters to Cloud API."""
_LOGGER.debug(
"aidoo=%s: update_params=%s", self.get_airzone_value(AZD_NAME), params
)
_LOGGER.debug("aidoo=%s: update_params=%s", self.entity_id, params)
try:
await self.coordinator.airzone.api_set_aidoo_id_params(
self.aidoo_id, params
)
except AirzoneCloudError as error:
raise HomeAssistantError(
f"Failed to set {self.get_airzone_value(AZD_NAME)} params: {error}"
f"Failed to set {self.entity_id} params: {error}"
) from error

self.coordinator.async_set_updated_data(self.coordinator.airzone.data())
Expand Down Expand Up @@ -124,16 +122,14 @@ def get_airzone_value(self, key: str) -> Any:

async def _async_update_params(self, params: dict[str, Any]) -> None:
"""Send Group parameters to Cloud API."""
_LOGGER.debug(
"group=%s: update_params=%s", self.get_airzone_value(AZD_NAME), params
)
_LOGGER.debug("group=%s: update_params=%s", self.entity_id, params)
try:
await self.coordinator.airzone.api_set_group_id_params(
self.group_id, params
)
except AirzoneCloudError as error:
raise HomeAssistantError(
f"Failed to set {self.get_airzone_value(AZD_NAME)} params: {error}"
f"Failed to set {self.entity_id} params: {error}"
) from error

self.coordinator.async_set_updated_data(self.coordinator.airzone.data())
Expand Down Expand Up @@ -170,7 +166,7 @@ async def _async_update_params(self, params: dict[str, Any]) -> None:
"""Send Installation parameters to Cloud API."""
_LOGGER.debug(
"installation=%s: update_params=%s",
self.get_airzone_value(AZD_NAME),
self.entity_id,
params,
)
try:
Expand All @@ -179,7 +175,7 @@ async def _async_update_params(self, params: dict[str, Any]) -> None:
)
except AirzoneCloudError as error:
raise HomeAssistantError(
f"Failed to set {self.get_airzone_value(AZD_NAME)} params: {error}"
f"Failed to set {self.entity_id} params: {error}"
) from error

self.coordinator.async_set_updated_data(self.coordinator.airzone.data())
Expand Down Expand Up @@ -275,14 +271,12 @@ def get_airzone_value(self, key: str) -> Any:

async def _async_update_params(self, params: dict[str, Any]) -> None:
"""Send Zone parameters to Cloud API."""
_LOGGER.debug(
"zone=%s: update_params=%s", self.get_airzone_value(AZD_NAME), params
)
_LOGGER.debug("zone=%s: update_params=%s", self.entity_id, params)
try:
await self.coordinator.airzone.api_set_zone_id_params(self.zone_id, params)
except AirzoneCloudError as error:
raise HomeAssistantError(
f"Failed to set {self.get_airzone_value(AZD_NAME)} params: {error}"
f"Failed to set {self.entity_id} params: {error}"
) from error

self.coordinator.async_set_updated_data(self.coordinator.airzone.data())

0 comments on commit 1cfc888

Please sign in to comment.