Leave for-loops when done, use constant

Author: bouwew

plugwise/__init__.py

@@ -303,6 +303,7 @@ async def _smile_detect_legacy(
                 locator = f"./{network}/mac"
                 if (net_locator := system.find(locator)) is not None:
                     self.smile_mac_address = net_locator.text
+                    break
         # P1 legacy:
         elif dsmrmain is not None:
             status = await self._request(STATUS)

plugwise/data.py

@@ -69,6 +69,7 @@ def _update_gw_entities(self) -> None:
             if entity_id == self._gateway_id:
                 mac_list = self._detect_low_batteries()
                 self._add_or_update_notifications(entity_id, entity, data)
+                break  # one gateway present
 
             entity.update(data)
             is_battery_low = (
@@ -228,6 +229,7 @@ def _check_availability(
                 for msg in item.values():
                     if message in msg:
                         data["available"] = False
+                        break
 
     def _get_adam_data(self, entity: GwEntityData, data: GwEntityData) -> None:
         """Helper-function for _get_entity_data().
@@ -334,11 +336,13 @@ def _get_schedule_states_with_off(
             loc_schedule_states[schedule] = "off"
             if schedule == selected and data["climate_mode"] == "auto":
                 loc_schedule_states[schedule] = "on"
+
         self._schedule_old_states[location] = loc_schedule_states
 
         all_off = True
         for state in self._schedule_old_states[location].values():
             if state == "on":
                 all_off = False
+
         if all_off:
             data["select_schedule"] = OFF

plugwise/helper.py

@@ -195,6 +195,7 @@ def _sort_gw_entities(self) -> None:
                     other_entities = self.gw_entities
                     priority_entities = {entity_id: priority_entity}
                     self.gw_entities = {**priority_entities, **other_entities}
+                    break
 
     def _all_locations(self) -> None:
         """Collect all locations."""

plugwise/legacy/helper.py

@@ -22,6 +22,7 @@
     LIMITS,
     NONE,
     OFF,
+    PRIORITY_DEVICE_CLASSES ,
     P1_LEGACY_MEASUREMENTS,
     TEMP_CELSIUS,
     THERMOSTAT_CLASSES,
@@ -130,14 +131,15 @@ def _all_appliances(self) -> None:
             self._create_gw_entities(appl)
 
         # Place the gateway and optional heater_central devices as 1st and 2nd
-        for dev_class in ("heater_central", "gateway"):
+        for dev_class in PRIORITY_DEVICE_CLASSES :
             for entity_id, entity in dict(self.gw_entities).items():
                 if entity["dev_class"] == dev_class:
                     tmp_entity = entity
                     self.gw_entities.pop(entity_id)
                     cleared_dict = self.gw_entities
                     add_to_front = {entity_id: tmp_entity}
                     self.gw_entities = {**add_to_front, **cleared_dict}
+                    break
 
     def _all_locations(self) -> None:
         """Collect all locations."""