Use LOGGER from homewizard.const instead per-file loggers (home-assis…

…tant#135320)
tdelaux · Jan 10, 2025 · 00c3b8c · 00c3b8c
1 parent bf747bb
commit 00c3b8c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 16 deletions.
diff --git a/homeassistant/components/homewizard/config_flow.py b/homeassistant/components/homewizard/config_flow.py
@@ -3,7 +3,6 @@
 from __future__ import annotations
 
 from collections.abc import Mapping
-import logging
 from typing import Any, NamedTuple
 
 from homewizard_energy import HomeWizardEnergyV1
@@ -25,10 +24,9 @@
     CONF_PRODUCT_TYPE,
     CONF_SERIAL,
     DOMAIN,
+    LOGGER,
 )
 
-_LOGGER = logging.getLogger(__name__)
-
 
 class DiscoveryData(NamedTuple):
     """User metadata."""
@@ -55,7 +53,7 @@ async def async_step_user(
             try:
                 device_info = await self._async_try_connect(user_input[CONF_IP_ADDRESS])
             except RecoverableError as ex:
-                _LOGGER.error(ex)
+                LOGGER.error(ex)
                 errors = {"base": ex.error_code}
             else:
                 await self.async_set_unique_id(
@@ -122,7 +120,7 @@ async def async_step_dhcp(
         try:
             device = await self._async_try_connect(discovery_info.ip)
         except RecoverableError as ex:
-            _LOGGER.error(ex)
+            LOGGER.error(ex)
             return self.async_abort(reason="unknown")
 
         await self.async_set_unique_id(
@@ -147,7 +145,7 @@ async def async_step_discovery_confirm(
             try:
                 await self._async_try_connect(self.discovery.ip)
             except RecoverableError as ex:
-                _LOGGER.error(ex)
+                LOGGER.error(ex)
                 errors = {"base": ex.error_code}
             else:
                 return self.async_create_entry(
@@ -190,7 +188,7 @@ async def async_step_reauth_confirm(
             try:
                 await self._async_try_connect(reauth_entry.data[CONF_IP_ADDRESS])
             except RecoverableError as ex:
-                _LOGGER.error(ex)
+                LOGGER.error(ex)
                 errors = {"base": ex.error_code}
             else:
                 await self.hass.config_entries.async_reload(reauth_entry.entry_id)
@@ -208,7 +206,7 @@ async def async_step_reconfigure(
                 device_info = await self._async_try_connect(user_input[CONF_IP_ADDRESS])
 
             except RecoverableError as ex:
-                _LOGGER.error(ex)
+                LOGGER.error(ex)
                 errors = {"base": ex.error_code}
             else:
                 await self.async_set_unique_id(
@@ -253,7 +251,7 @@ async def _async_try_connect(ip_address: str) -> Device:
             ) from ex
 
         except UnsupportedError as ex:
-            _LOGGER.error("API version unsuppored")
+            LOGGER.error("API version unsuppored")
             raise AbortFlow("unsupported_api_version") from ex
 
         except RequestError as ex:
@@ -262,7 +260,7 @@ async def _async_try_connect(ip_address: str) -> Device:
             ) from ex
 
         except Exception as ex:
-            _LOGGER.exception("Unexpected exception")
+            LOGGER.exception("Unexpected exception")
             raise AbortFlow("unknown_error") from ex
 
         finally:

diff --git a/homeassistant/components/homewizard/coordinator.py b/homeassistant/components/homewizard/coordinator.py
@@ -2,8 +2,6 @@
 
 from __future__ import annotations
 
-import logging
-
 from homewizard_energy import HomeWizardEnergy, HomeWizardEnergyV1
 from homewizard_energy.errors import DisabledError, RequestError
 from homewizard_energy.models import CombinedModels as DeviceResponseEntry
@@ -14,9 +12,7 @@
 from homeassistant.helpers.aiohttp_client import async_get_clientsession
 from homeassistant.helpers.update_coordinator import DataUpdateCoordinator, UpdateFailed
 
-from .const import DOMAIN, UPDATE_INTERVAL
-
-_LOGGER = logging.getLogger(__name__)
+from .const import DOMAIN, LOGGER, UPDATE_INTERVAL
 
 
 class HWEnergyDeviceUpdateCoordinator(DataUpdateCoordinator[DeviceResponseEntry]):
@@ -32,7 +28,7 @@ def __init__(
         hass: HomeAssistant,
     ) -> None:
         """Initialize update coordinator."""
-        super().__init__(hass, _LOGGER, name=DOMAIN, update_interval=UPDATE_INTERVAL)
+        super().__init__(hass, LOGGER, name=DOMAIN, update_interval=UPDATE_INTERVAL)
         self.api = HomeWizardEnergyV1(
             self.config_entry.data[CONF_IP_ADDRESS],
             clientsession=async_get_clientsession(hass),