Fix use of Much-None

Author: bouwew

plugwise/helper.py

@@ -267,7 +267,7 @@ def _all_appliances(self) -> None:
         self._all_locations()
 
         for appliance in self._domain_objects.findall("./appliance"):
-            appl: Munch | None = Munch()
+            appl = Munch()
             appl.pwclass = appliance.find("type").text
             # Don't collect data for the OpenThermGateway appliance
             if appl.pwclass == "open_therm_gateway":
@@ -309,8 +309,8 @@ def _all_appliances(self) -> None:
             appl.zigbee_mac = None
             appl.vendor_name = None
 
-            # Collect appliance info
-            if (appl := self._appliance_info_finder(appl, appliance)) is None:
+            # Collect appliance info, skip orphaned/removed devices
+            if not (appl := self._appliance_info_finder(appl, appliance)):
                 continue
 
             # P1: for gateway and smartmeter switch device_id - part 1
@@ -374,7 +374,7 @@ def _p1_smartmeter_info_finder(self, appl: Munch) -> None:
 
         self._create_gw_devices(appl)
 
-    def _appliance_info_finder(self, appl: Munch, appliance: etree) -> Munch | None:
+    def _appliance_info_finder(self, appl: Munch, appliance: etree) -> Munch:
         """Collect info for all appliances found."""
         match appl.pwclass:
             case "gateway":
@@ -389,7 +389,7 @@ def _appliance_info_finder(self, appl: Munch, appliance: etree) -> Munch | None:
                 self._appl_dhw_mode_info(appl, appliance)
                 # Skip orphaned heater_central (Core Issue #104433)
                 if appl.dev_id != self._heater_id:
-                    return None
+                    return Munch()
                 return appl
             case _ as s if s.endswith("_plug"):
                 # Collect info from plug-types (Plug, Aqara Smart Plug)
@@ -398,7 +398,7 @@ def _appliance_info_finder(self, appl: Munch, appliance: etree) -> Munch | None:
                 module_data = self._get_module_data(appliance, locator, mod_type)
                 # A plug without module-data is orphaned/ no present
                 if not module_data["contents"]:
-                    return None
+                    return Munch()
 
                 appl.firmware = module_data["firmware_version"]
                 appl.hardware = module_data["hardware_version"]