Replace getattr() as suggested

bouwew · bouwew · commit d4cf635f4fdc · 2025-06-08T08:35:09.000+02:00
diff --git a/plugwise_usb/nodes/circle.py b/plugwise_usb/nodes/circle.py
@@ -637,7 +637,7 @@ async def set_relay(self, state: bool) -> bool:
                 f"Changing state of relay is not supported for node {self.mac}"
             )
 
-        if getattr(self._relay_lock, "state"):
+        if self._relay_lock.state:
             raise NodeError("Changing state of relay failed, it is locked")
 
         _LOGGER.debug("set_relay() start")

Original file line number	Diff line number	Diff line change
`@@ -637,7 +637,7 @@ async def set_relay(self, state: bool) -> bool:`
`637`	`637`	`f"Changing state of relay is not supported for node {self.mac}"`
`638`	`638`	`)`
`639`	`639`
`640`		`- if getattr(self._relay_lock, "state"):`
	`640`	`+ if self._relay_lock.state:`
`641`	`641`	`raise NodeError("Changing state of relay failed, it is locked")`
`642`	`642`
`643`	`643`	`_LOGGER.debug("set_relay() start")`