From bb8d7424236b670b2cabdd5132967e4bd7ebb338 Mon Sep 17 00:00:00 2001
From: Christopher Fenner <9592452+CFenner@users.noreply.github.com>
Date: Sat, 3 Feb 2024 12:53:22 +0100
Subject: [PATCH] Ignore gateway devices in ViCare integration (#106477)

* filter unsupported devices

* Update __init__.py

* use debug

* remove dead code
---
 homeassistant/components/vicare/__init__.py   | 23 +++++++++---
 .../components/vicare/config_flow.py          |  2 +-
 homeassistant/components/vicare/sensor.py     | 35 -------------------
 3 files changed, 19 insertions(+), 41 deletions(-)

diff --git a/homeassistant/components/vicare/__init__.py b/homeassistant/components/vicare/__init__.py
index 603a42bae41..a2b2f3ac769 100644
--- a/homeassistant/components/vicare/__init__.py
+++ b/homeassistant/components/vicare/__init__.py
@@ -10,6 +10,7 @@ from typing import Any
 
 from PyViCare.PyViCare import PyViCare
 from PyViCare.PyViCareDevice import Device
+from PyViCare.PyViCareDeviceConfig import PyViCareDeviceConfig
 from PyViCare.PyViCareUtils import (
     PyViCareInvalidConfigurationError,
     PyViCareInvalidCredentialsError,
@@ -85,15 +86,16 @@ def setup_vicare_api(hass: HomeAssistant, entry: ConfigEntry) -> None:
     """Set up PyVicare API."""
     vicare_api = vicare_login(hass, entry.data)
 
-    for device in vicare_api.devices:
-        _LOGGER.info(
+    device_config_list = get_supported_devices(vicare_api.devices)
+
+    for device in device_config_list:
+        _LOGGER.debug(
             "Found device: %s (online: %s)", device.getModel(), str(device.isOnline())
         )
 
     # Currently we only support a single device
-    device_list = vicare_api.devices
-    device = device_list[0]
-    hass.data[DOMAIN][entry.entry_id][VICARE_DEVICE_CONFIG_LIST] = device_list
+    device = device_config_list[0]
+    hass.data[DOMAIN][entry.entry_id][VICARE_DEVICE_CONFIG_LIST] = device_config_list
     hass.data[DOMAIN][entry.entry_id][VICARE_DEVICE_CONFIG] = device
     hass.data[DOMAIN][entry.entry_id][VICARE_API] = getattr(
         device,
@@ -113,3 +115,14 @@ async def async_unload_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
         )
 
     return unload_ok
+
+
+def get_supported_devices(
+    devices: list[PyViCareDeviceConfig],
+) -> list[PyViCareDeviceConfig]:
+    """Remove unsupported devices from the list."""
+    return [
+        device_config
+        for device_config in devices
+        if device_config.getModel() not in ["Heatbox1", "Heatbox2_SRC"]
+    ]
diff --git a/homeassistant/components/vicare/config_flow.py b/homeassistant/components/vicare/config_flow.py
index 87bfcf7b146..32ae4af0fe7 100644
--- a/homeassistant/components/vicare/config_flow.py
+++ b/homeassistant/components/vicare/config_flow.py
@@ -118,7 +118,7 @@ class ConfigFlow(config_entries.ConfigFlow, domain=DOMAIN):
     async def async_step_dhcp(self, discovery_info: dhcp.DhcpServiceInfo) -> FlowResult:
         """Invoke when a Viessmann MAC address is discovered on the network."""
         formatted_mac = format_mac(discovery_info.macaddress)
-        _LOGGER.info("Found device with mac %s", formatted_mac)
+        _LOGGER.debug("Found device with mac %s", formatted_mac)
 
         await self.async_set_unique_id(formatted_mac)
         self._abort_if_unique_id_configured()
diff --git a/homeassistant/components/vicare/sensor.py b/homeassistant/components/vicare/sensor.py
index 6c794b548ad..a8a21c7e787 100644
--- a/homeassistant/components/vicare/sensor.py
+++ b/homeassistant/components/vicare/sensor.py
@@ -692,41 +692,6 @@ COMPRESSOR_SENSORS: tuple[ViCareSensorEntityDescription, ...] = (
 )
 
 
-def _build_entity(
-    vicare_api,
-    device_config: PyViCareDeviceConfig,
-    entity_description: ViCareSensorEntityDescription,
-):
-    """Create a ViCare sensor entity."""
-    if is_supported(entity_description.key, entity_description, vicare_api):
-        return ViCareSensor(
-            vicare_api,
-            device_config,
-            entity_description,
-        )
-    return None
-
-
-async def _entities_from_descriptions(
-    hass: HomeAssistant,
-    entities: list[ViCareSensor],
-    sensor_descriptions: tuple[ViCareSensorEntityDescription, ...],
-    iterables,
-    config_entry: ConfigEntry,
-) -> None:
-    """Create entities from descriptions and list of burners/circuits."""
-    for description in sensor_descriptions:
-        for current in iterables:
-            entity = await hass.async_add_executor_job(
-                _build_entity,
-                current,
-                hass.data[DOMAIN][config_entry.entry_id][VICARE_DEVICE_CONFIG],
-                description,
-            )
-            if entity:
-                entities.append(entity)
-
-
 def _build_entities(
     device: PyViCareDevice,
     device_config: PyViCareDeviceConfig,