Clean up

Author: bouwew

plugwise/legacy/smile.py

@@ -280,7 +280,7 @@ async def set_switch_state(
         data = f"<{switch.func_type}><state>{state}</state></{switch.func_type}>"
         if members is not None:
             return await self._set_groupswitch_member_state(
-                appl_id, data, members, state, switch
+                data, members, state, switch
             )
 
         # Handle individual relay switches
@@ -293,14 +293,13 @@ async def set_switch_state(
         return requested_state
 
     async def _set_groupswitch_member_state(
-        self, appl_id: str, data: str, members: list[str], state: str, switch: Munch
+        self, data: str, members: list[str], state: str, switch: Munch
     ) -> bool:
         """Helper-function for set_switch_state().
 
         Set the requested state of the relevant switch within a group of switches.
         Return the current group-state when none of the switches has changed its state, the requested state otherwise.
         """
-        current_state = self.gw_entities[appl_id]["switches"]["relay"]
         end_states: list[bool] = []
         requested_state = state == STATE_ON
         for member in members:

plugwise/smile.py

@@ -418,7 +418,7 @@ async def set_switch_state(
 
         if members is not None:
             return await self._set_groupswitch_member_state(
-                appl_id, data, members, state, switch
+                data, members, state, switch
             )
 
         locator = f'appliance[@id="{appl_id}"]/{switch.actuator}/{switch.func_type}'
@@ -444,14 +444,13 @@ async def set_switch_state(
         return requested_state
 
     async def _set_groupswitch_member_state(
-        self, appl_id: str, data: str, members: list[str], state: str, switch: Munch
+        self, data: str, members: list[str], state: str, switch: Munch
     ) -> bool:
         """Helper-function for set_switch_state().
 
         Set the requested state of the relevant switch within a group of switches.
         Return the current group-state when none of the switches has changed its state, the requested state otherwise.
         """
-        current_state = self.gw_entities[appl_id]["switches"]["relay"]
         end_states: list[bool] = []
         requested_state = state == STATE_ON
         for member in members: