Extend internal-use selfs

Author: bouwew

plugwise/data.py

@@ -36,7 +36,7 @@ def _all_entity_data(self) -> None:
         self._update_gw_entities()
         if self.smile(ADAM):
             self._update_zones()
-            self.gw_entities.update(self.zones)
+            self.gw_entities.update(self._zones)
 
         self.gw_data.update(
             {
@@ -55,9 +55,9 @@ def _all_entity_data(self) -> None:
     def _update_zones(self) -> None:
         """Helper-function for _all_entity_data() and async_update().
 
-        Collect data for each zone/location and add to self.zones.
+        Collect data for each zone/location and add to self._zones.
         """
-        for location_id, zone in self.zones.items():
+        for location_id, zone in self._zones.items():
             data = self._get_location_data(location_id)
             zone.update(data)
 
@@ -158,7 +158,7 @@ def _get_location_data(self, loc_id: str) -> GwEntityData:
 
         Provide entity-data, based on Location ID (= loc_id).
         """
-        zone = self.zones[loc_id]
+        zone = self._zones[loc_id]
         data = self._get_zone_data(loc_id)
         if ctrl_state := self._control_state(loc_id):
             data["control_state"] = ctrl_state

plugwise/helper.py

@@ -224,6 +224,7 @@ def __init__(self) -> None:
         self._is_thermostat: bool
         self._last_active: dict[str, str | None]
         self._last_modified: dict[str, str] = {}
+        self._loc_data: dict[str, ThermoLoc]
         self._notifications: dict[str, dict[str, str]] = {}
         self._on_off_device: bool
         self._opentherm_device: bool
@@ -251,7 +252,6 @@ def __init__(self) -> None:
         self.gateway_id: str
         self.gw_data: GatewayData = {}
         self.gw_entities: dict[str, GwEntityData] = {}
-        self.loc_data: dict[str, ThermoLoc]
         self.smile_fw_version: Version | None
         self.smile_hw_version: str | None
         self.smile_mac_address: str | None
@@ -261,7 +261,7 @@ def __init__(self) -> None:
         self.smile_type: str
         self.smile_zigbee_mac_address: str | None
         self.therms_with_offset_func: list[str] = []
-        self.zones: dict[str, GwEntityData] = {}
+        self._zones: dict[str, GwEntityData] = {}
         SmileCommon.__init__(self)
 
     def _all_appliances(self) -> None:
@@ -354,11 +354,11 @@ def _all_locations(self) -> None:
             if loc.name == "Home":
                 self._home_location = loc.loc_id
 
-            self.loc_data[loc.loc_id] = {"name": loc.name}
+            self._loc_data[loc.loc_id] = {"name": loc.name}
 
     def _p1_smartmeter_info_finder(self, appl: Munch) -> None:
         """Collect P1 DSMR SmartMeter info."""
-        loc_id = next(iter(self.loc_data.keys()))
+        loc_id = next(iter(self._loc_data.keys()))
         location = self._domain_objects.find(f'./location[@id="{loc_id}"]')
         locator = MODULE_LOCATOR
         module_data = self._get_module_data(location, locator)
@@ -488,7 +488,7 @@ def _get_zone_data(self, loc_id: str) -> GwEntityData:
         Collect the location-data based on location id.
         """
         data: GwEntityData = {"sensors": {}}
-        zone = self.zones[loc_id]
+        zone = self._zones[loc_id]
         measurements = ZONE_MEASUREMENTS
         if (
             location := self._domain_objects.find(f'./location[@id="{loc_id}"]')
@@ -861,7 +861,7 @@ def _scan_thermostats(self) -> None:
 
         for loc_id, loc_data in list(self._thermo_locs.items()):
             if loc_data["primary_prio"] != 0:
-                self.zones[loc_id] = {
+                self._zones[loc_id] = {
                     "dev_class": "climate",
                     "name": loc_data["name"],
                     "thermostats": {"primary": loc_data["primary"], "secondary": loc_data["secondary"]}
@@ -874,7 +874,7 @@ def _match_locations(self) -> dict[str, ThermoLoc]:
         Match appliances with locations.
         """
         matched_locations: dict[str, ThermoLoc] = {}
-        for location_id, location_details in self.loc_data.items():
+        for location_id, location_details in self._loc_data.items():
             for appliance_details in self.gw_entities.values():
                 if appliance_details["location"] == location_id:
                     location_details.update(

plugwise/smile.py

@@ -126,7 +126,7 @@ async def async_update(self) -> PlugwiseData:
         """Perform an incremental update for updating the various device states."""
         self.gw_data: GatewayData = {}
         self.gw_entities: dict[str, GwEntityData] = {}
-        self.zones: dict[str, GwEntityData] = {}
+        self._zones: dict[str, GwEntityData] = {}
         try:
             await self.full_xml_update()
             self.get_all_gateway_entities()