Skip loading node_info from cache when Node is available

 - the data is already available from a NodeInfoRequest

Author: bouwew

plugwise_usb/nodes/node.py

@@ -50,6 +50,7 @@
 CACHE_NODE_TYPE = "node_type"
 CACHE_HARDWARE = "hardware"
 CACHE_NODE_INFO_TIMESTAMP = "node_info_timestamp"
+CACHE_RELAY = "relay"
 
 class PlugwiseBaseNode(FeaturePublisher, ABC):
     """Abstract Base Class for a Plugwise node."""
@@ -459,44 +460,50 @@ async def node_info_update(
             _LOGGER.debug("No response for node_info_update() for %s", self.mac)
             await self._available_update_state(False)
             return self._node_info
+
         await self._available_update_state(True, node_info.timestamp)
         await self.update_node_details(
             firmware=node_info.firmware,
-            node_type=node_info.node_type,
             hardware=node_info.hardware,
-            timestamp=node_info.timestamp,
-            relay_state=node_info.relay_state,
             logaddress_pointer=node_info.current_logaddress_pointer,
+            node_type=node_info.node_type,
+            relay_state=node_info.relay_state,
+            timestamp=node_info.timestamp,
         )
         return self._node_info
 
     async def _node_info_load_from_cache(self) -> bool:
         """Load node info settings from cache."""
+        if self._available:
+            # Skip loading this data from cache when the Node is available
+            return True
+
         firmware = self._get_cache_as_datetime(CACHE_FIRMWARE)
         hardware = self._get_cache(CACHE_HARDWARE)
-        timestamp = self._get_cache_as_datetime(CACHE_NODE_INFO_TIMESTAMP)
         node_type: NodeType | None = None
         if (node_type_str := self._get_cache(CACHE_NODE_TYPE)) is not None:
             node_type = NodeType(int(node_type_str))
 
+        relay_state = self._get_cache(CACHE_RELAY)
+        timestamp = self._get_cache_as_datetime(CACHE_NODE_INFO_TIMESTAMP)
         return await self.update_node_details(
             firmware=firmware,
             hardware=hardware,
+            logaddress_pointer=None,
             node_type=node_type,
+            relay_state=relay_state,
             timestamp=timestamp,
-            relay_state=None,
-            logaddress_pointer=None,
         )
 
     # pylint: disable=too-many-arguments
     async def update_node_details(
         self,
         firmware: datetime | None,
         hardware: str | None,
+        logaddress_pointer: int | None,
         node_type: NodeType | None,
-        timestamp: datetime | None,
         relay_state: bool | None,
-        logaddress_pointer: int | None,
+        timestamp: datetime | None,
     ) -> bool:
         """Process new node info and return true if all fields are updated."""
         _LOGGER.debug(