Fix temperature for AtlanticElectricalTowelDryer (#704)

* Create linked device property

* Fix current temperature retrieval

* Use relative import

* Remove property decorator

Property does not take any arguments... I was tired.

* Fix wrong returned type

* Remove useless else block

Co-authored-by: Mick Vleeshouwer <[email protected]>

Author: tetienne

custom_components/tahoma/climate_devices/atlantic_electrical_towel_dryer.py

@@ -13,7 +13,9 @@
     PRESET_NONE,
 )
 from homeassistant.const import ATTR_TEMPERATURE, TEMP_CELSIUS
+from pyoverkiz.enums import OverkizState
 
+from ..coordinator import OverkizDataUpdateCoordinator
 from ..entity import OverkizEntity
 
 COMMAND_SET_TARGET_TEMPERATURE = "setTargetTemperature"
@@ -68,6 +70,11 @@ class AtlanticElectricalTowelDryer(OverkizEntity, ClimateEntity):
     _attr_supported_features = SUPPORT_PRESET_MODE | SUPPORT_TARGET_TEMPERATURE
     _attr_temperature_unit = TEMP_CELSIUS
 
+    def __init__(self, device_url: str, coordinator: OverkizDataUpdateCoordinator):
+        """Init method."""
+        super().__init__(device_url, coordinator)
+        self.temperature_device = self.executor.linked_device(7)
+
     @property
     def hvac_mode(self) -> str:
         """Return hvac operation ie. heat, cool mode."""
@@ -103,13 +110,14 @@ def target_temperature(self) -> None:
         """Return the temperature."""
         if self.hvac_mode == HVAC_MODE_AUTO:
             return self.executor.select_state(IO_EFFECTIVE_TEMPERATURE_SETPOINT_STATE)
-        else:
-            return self.executor.select_state(CORE_TARGET_TEMPERATURE_STATE)
+        return self.executor.select_state(CORE_TARGET_TEMPERATURE_STATE)
 
     @property
-    def current_temperature(self):
+    def current_temperature(self) -> float:
         """Return current temperature."""
-        return self.executor.select_state(CORE_COMFORT_ROOM_TEMPERATURE_STATE)
+        return float(
+            self.temperature_device.states.get(OverkizState.CORE_TEMPERATURE).value
+        )
 
     async def async_set_temperature(self, **kwargs) -> None:
         """Set new temperature."""

custom_components/tahoma/executor.py

@@ -26,6 +26,10 @@ def device(self) -> Device:
         """Return Overkiz device linked to this entity."""
         return self.coordinator.data[self.device_url]
 
+    def linked_device(self, index) -> Device:
+        """Return Overkiz device sharing the same base url."""
+        return self.coordinator.data[f"{self.base_device_url}#{index}"]
+
     def select_command(self, *commands: str) -> str | None:
         """Select first existing command in a list of commands."""
         existing_commands = self.device.definition.commands