Move vacation-preset removal to _presets()

Author: bouwew

plugwise/helper.py

@@ -691,6 +691,10 @@ def _presets(self, loc_id: str) -> dict[str, list[float]]:
                         float(preset.get("cooling_setpoint")),
                     ]
 
+        # Adam does not show vacation preset anymore, issue #185
+        if self.smile_name == "Adam":
+            presets.pop("vacation")
+
         return presets
 
     def _rule_ids_by_name(self, name: str, loc_id: str) -> dict[str, str]:

plugwise/smile.py

@@ -144,12 +144,7 @@ def _device_data_climate(
         device_data["preset_modes"] = None
         device_data["active_preset"] = None
         if presets := self._presets(loc_id):
-            LOGGER.debug("HOI 1 %s", presets)
             presets_list = list(presets)
-            LOGGER.debug("HOI 2 %s", presets_list)
-            # Adam does not show vacation preset anymore, issue #185
-            if self.smile_name == "Adam":
-                presets_list.remove("vacation")
             device_data["preset_modes"] = presets_list
 
             device_data["active_preset"] = self._preset(loc_id)