Merge pull request #189 from plugwise/alt_fix_pw_beta_280

Move solution to pw-beta #280 into backend

Author: bouwew

CHANGELOG.md

@@ -1,5 +1,7 @@
 # Changelog
 
+# v0.18.3: Smile: move solution for pw-beta #280 into backend
+
 # v0.18.2: Smile: fix for https://github.com/plugwise/python-plugwise/issues/187
 
 # v0.18.1: Smile Adam: don't show vacation-preset, as not shown in the Plugwise App or on the local Adam-website

plugwise/__init__.py

@@ -1,6 +1,6 @@
 """Plugwise module."""
 
-__version__ = "0.18.2"
+__version__ = "0.18.3"
 
 from plugwise.smile import Smile
 from plugwise.stick import Stick

plugwise/smile.py

@@ -482,10 +482,18 @@ async def _set_schedule_state_legacy(self, name: str, status: str) -> None:
 
         await self._request(uri, method="put", data=data)
 
-    async def set_schedule_state(self, loc_id: str, name: str, state: str) -> None:
-        """Set the Schedule, with the given name, on the relevant Thermostat.
+    async def set_schedule_state(
+        self, loc_id: str, name: str | None, state: str
+    ) -> None:
+        """Activate/deactivate the Schedule, with the given name, on the relevant Thermostat.
         Determined from - DOMAIN_OBJECTS.
+        In HA Core used to set the hvac_mode: in practice switch between schedule on - off.
         """
+        # Do nothing when name == None, meaning no schedule to activate / deactivate
+        # Also, don't show an error, as doing nothing is the correct action in this scenario.
+        if name is None:
+            return
+
         if self._smile_legacy:
             await self._set_schedule_state_legacy(name, state)
             return