Implement more none module_data guarding

Author: bouwew

plugwise/common.py

@@ -101,6 +101,8 @@ def _appl_heater_central_info(
         module_data = self._get_module_data(xml_1, locator_1, xml_3)
         if not module_data["contents"]:
             module_data = self._get_module_data(xml_1, locator_2, xml_3)
+            if not module_data["contents"]:
+                return Munch()  # no module-data present means the device has been removed
         appl.vendor_name = module_data["vendor_name"]
         appl.hardware = module_data["hardware_version"]
         appl.model_id = module_data["vendor_model"] if not legacy else None

plugwise/helper.py

@@ -171,8 +171,8 @@ def _get_p1_smartmeter_info(self) -> None:
         locator = MODULE_LOCATOR
         module_data = self._get_module_data(self._home_location, locator)
         if not module_data["contents"]:
-            LOGGER.error("No module data found for SmartMeter")  # pragma: no cover
-            return  # pragma: no cover
+            return Munch()  # no module-data present means the device has been removed
+
         appl.available = None
         appl.entity_id = self._gateway_id
         appl.firmware = module_data["firmware_version"]

plugwise/legacy/helper.py

@@ -214,6 +214,9 @@ def _energy_entity_info_finder(self, appliance: etree, appl: Munch) -> Munch:
             module_data = self._get_module_data(
                 appliance, locator, self._modules, legacy=True
             )
+            if not module_data["contents"]:
+                return Munch()  # no module-data present means the device has been removed
+
             appl.zigbee_mac = module_data["zigbee_mac_address"]
             # Filter appliance without zigbee_mac, it's an orphaned device
             if appl.zigbee_mac is None and self.smile.type != "power":