Revert double, combine

Author: bouwew

plugwise/helper.py

@@ -258,7 +258,6 @@ def __init__(self) -> None:
         self._on_off_device = False
         self._opentherm_device = False
         self._outdoor_temp: float
-        self._presets_list: list[str] = []
         self._smile_legacy = False
         self._stretch_v2 = False
         self._stretch_v3 = False

plugwise/smile.py

@@ -144,11 +144,11 @@ def _device_data_climate(
         device_data["preset_modes"] = None
         device_data["active_preset"] = None
         if presets := self._presets(loc_id):
-            self._presets_list = list(presets)
+            presets_list = list(presets)
             # Adam does not show vacation preset anymore, issue #185
             if self.smile_name == "Adam":
-                self._presets_list.remove("vacation")
-            device_data["preset_modes"] = self._presets_list
+                presets_list.remove("vacation")
+            device_data["preset_modes"] = presets_list
 
             device_data["active_preset"] = self._preset(loc_id)
 
@@ -553,7 +553,7 @@ async def _set_preset_legacy(self, preset: str) -> None:
 
     async def set_preset(self, loc_id: str, preset: str) -> None:
         """Set the given Preset on the relevant Thermostat - from LOCATIONS."""
-        if preset not in self._presets_list:
+        if preset not in self._presets(loc_id):
             raise PlugwiseError("Plugwise: invalid preset.")
 
         if self._smile_legacy:
@@ -564,9 +564,6 @@ async def set_preset(self, loc_id: str, preset: str) -> None:
         location_name = current_location.find("name").text
         location_type = current_location.find("type").text
 
-        if preset not in self._presets(loc_id):
-            raise PlugwiseError("Plugwise: invalid preset.")
-
         uri = f"{LOCATIONS};id={loc_id}"
         data = (
             "<locations><location"