Resolve dataclass issue by forcing default of None which will always get overriden and resolve setting deprecated property step on numberentitydescription

Author: Sven ten Raa

custom_components/victron/number.py

@@ -71,15 +71,13 @@ async def async_setup_entry(
                             name=register_name.replace('_', ' '),
                             slave=slave,
                             native_unit_of_measurement=registerInfo.unit,
-                            # native_min_value=registerInfo.writeType.lowerLimit,
-                            # native_max_value=registerInfo.writeType.upperLimit,
                             mode=NumberMode.SLIDER if config_entry.options[CONF_USE_SLIDERS] else NumberMode.BOX,
                             native_min_value=determine_min_value(registerInfo.unit, config_entry.options, registerInfo.entityType.powerType, registerInfo.entityType.negative),
                             native_max_value=determine_max_value(registerInfo.unit, config_entry.options, registerInfo.entityType.powerType),
                             entity_category=EntityCategory.CONFIG,
                             address=registerInfo.register,
                             scale = registerInfo.scale,
-                            step = registerInfo.step
+                            native_step = registerInfo.step
                         ))
 
     entities = []
@@ -145,14 +143,17 @@ def determine_max_value(unit, config_entry:config_entries.ConfigEntry, powerType
 @dataclass
 class VictronNumberMixin:
     """A class that describes number entities."""
-    scale: int
-    mode: bool
+    scale: int | None  = None
+    mode: bool | None  = None
 
+@dataclass
 class VictronNumberStep:
-    step: float = 0
+    native_step: float = 0
 
 @dataclass
 class VictronEntityDescription(NumberEntityDescription, VictronWriteBaseEntityDescription, VictronNumberMixin, VictronNumberStep):
+    #Overwrite base entitydescription property to resolve automatic property ordering issues when a mix of non-default and default properties are used
+    key: str | None  = None
     """Describes victron number entity."""
 
 class VictronNumber(NumberEntity):
@@ -207,8 +208,8 @@ def native_value(self) -> float:
     def native_step(self) -> float | None:
         if not self.description.mode == NumberMode.SLIDER: # allow users to skip stepping in case of box mode
             return None
-        if self.description.step > 0:
-            return self.description.step
+        if self.description.native_step > 0:
+            return self.description.native_step
         max = self.native_max_value
         min = self.native_min_value
         gap = len(list(range(int(min), int(max), 1)))