rework unpack_unit_value

handle the `None` case, simplify logic

src/ifcopenshell-python/ifcopenshell/api/pset/edit_pset.py

@@ -405,10 +405,14 @@ class Usecase:
 
     @staticmethod
     def unpack_unit_value(value_candidate):
-        unit = None
+        """
+        Returns tuple of the format: (Unit, NominalValue)
+        NOTE: Unit fallbacks to None
+        """
+        if value_candidate is None:
+            return (None, None)
+
         if isinstance(value_candidate, dict):  # Custom IfcUnits can be passed in a dict along with the pset value
-            unit = value_candidate["Unit"]
+            return (value_candidate["Unit"], value_candidate["NominalValue"])
-            value = value_candidate["NominalValue"]
+
-        else:
+        return (None, value_candidate)
-            value = value_candidate
-        return unit, value