Fix none-type attributes

pyhon/appliance.py

@@ -7,6 +7,7 @@ from typing import TYPE_CHECKING
 
 from pyhon import helper, exceptions
 from pyhon.commands import HonCommand
+from pyhon.parameter.base import HonParameter
 from pyhon.parameter.fixed import HonParameterFixed
 
 if TYPE_CHECKING:
@@ -33,6 +34,7 @@ class HonAppliance:
         self._zone: int = zone
         self._additional_data: Dict[str, Any] = {}
         self._last_update = None
+        self._default_setting = HonParameter("", {}, "")
 
         try:
             self._extra = importlib.import_module(
@@ -142,7 +144,9 @@ class HonAppliance:
             if last is None:
                 continue
             parameters = command_history[last].get("command", {}).get("parameters", {})
-            if command.categories:
+            if command.categories and (
+                parameters.get("program") or parameters.get("category")
+            ):
                 if parameters.get("program"):
                     command.category = parameters.pop("program").split(".")[-1].lower()
                 else:
@@ -198,7 +202,7 @@ class HonAppliance:
     async def load_commands(self):
         raw = await self.api.load_commands(self)
         self._appliance_model = raw.pop("applianceModel")
-        raw.pop("dictionaryId")
+        raw.pop("dictionaryId", None)
         self._commands = self._get_commands(raw)
         await self._recover_last_command_states()
 
@@ -227,7 +231,7 @@ class HonAppliance:
         result = {}
         for name, command in self._commands.items():
             for key in command.setting_keys:
-                setting = command.settings.get(key)
+                setting = command.settings.get(key, self._default_setting)
                 result[f"{name}.{key}"] = setting
         if self._extra:
             return self._extra.settings(result)
@@ -254,18 +258,21 @@ class HonAppliance:
             return self._extra.data(result)
         return result
 
-    def diagnose(self, whitespace="\u200B \u200B "):
+    def diagnose(self, whitespace="\u200B \u200B ", command_only=False):
         data = {
             "attributes": self.attributes.copy(),
             "appliance": self.info,
             "additional_data": self._additional_data,
         }
+        if command_only:
+            data.pop("attributes")
+            data.pop("appliance")
         data |= {n: c.parameter_groups for n, c in self._commands.items()}
         extra = {n: c.data for n, c in self._commands.items() if c.data}
         if extra:
             data |= {"extra_command_data": extra}
         for sensible in ["PK", "SK", "serialNumber", "code", "coords"]:
-            data["appliance"].pop(sensible, None)
+            data.get("appliance", {}).pop(sensible, None)
         result = helper.pretty_print({"data": data}, whitespace=whitespace)
         result += helper.pretty_print(
             {"commands": helper.create_command(self.commands)},

pyhon/commands.py

@@ -85,8 +85,8 @@ class HonCommand:
             self._parameters[name] = HonParameterProgram(name, self, "custom")
 
     async def send(self) -> bool:
-        params = self.parameter_groups["parameters"]
-        ancillary_params = self.parameter_groups["ancillary_parameters"]
+        params = self.parameter_groups.get("parameters", {})
+        ancillary_params = self.parameter_groups.get("ancillaryParameters", {})
         return await self._api.send_command(
             self._appliance, self._name, params, ancillary_params
         )
@@ -103,7 +103,8 @@ class HonCommand:
 
     @category.setter
     def category(self, category: str) -> None:
-        self._appliance.commands[self._name] = self.categories[category]
+        if category in self.categories:
+            self._appliance.commands[self._name] = self.categories[category]
 
     @property
     def setting_keys(self) -> List[str]:

pyhon/parameter/program.py

@@ -13,9 +13,9 @@ class HonParameterProgram(HonParameterEnum):
         super().__init__(key, {}, group)
         self._command = command
         if "PROGRAM" in command.category:
-            self._value: str = command.category.split(".")[-1].lower()
+            self._value = command.category.split(".")[-1].lower()
         else:
-            self._value: str = command.category
+            self._value = command.category
         self._programs: Dict[str, "HonCommand"] = command.categories
         self._typology: str = "enum"
 

pyhon/parameter/range.py

@@ -32,6 +32,8 @@ class HonParameterRange(HonParameter):
 
     @property
     def step(self) -> float:
+        if not self._step:
+            return 1
         return self._step
 
     @property

setup.py

@@ -7,7 +7,7 @@ with open("README.md", "r") as f:
 
 setup(
     name="pyhOn",
-    version="0.10.2",
+    version="0.10.6",
     author="Andre Basche",
     description="Control hOn devices with python",
     long_description=long_description,