Numeric marshaled values are rounded to the nearest integer about taskw

[sjbronner]

I just ran across this while debugging the same issue in telch. I believe the problem is in fields/numeric.py in the function NumericField.deserialize. Deserialization as an integer is preferred there over deserialization as a float, regardless of the presence of decimal digits or a decimal point. This basically ensures that all numbers output from task appear rounded in python.

In my opinion, a somewhat more complex check would be in order: if float(value) != int(value): return float(value), otherwise if '.' in value: return float(value), otherwise return int(value). The conditions for None and a non-numeric value should be kept as they are.

[sjbronner]

It turns out the following patch is enough to alleviate this specific problem:

--- /usr/lib/python3/dist-packages/taskw/fields/numeric.py.orig 2015-01-19 16:12:46.000000000 +0100
+++ /usr/lib/python3/dist-packages/taskw/fields/numeric.py      2020-12-06 23:34:32.081430072 +0100
@@ -11,7 +11,7 @@
         return value

     def deserialize(self, value):
-        if value is None:
+        if value is None or isinstance(value, (int, float)):
             return value
         try:
             return int(value)

[coddingtonbear]

Merged a fix for this via the changes in #124