Task: Clean up default formatting of the attributes as arguments for modify command

In the current version, we were not properly making difference between value
which is False when converting to boolean, and proper empty value. Thus user
would be unable to save legitimate False-like values into TW, like numeric 0.

diff --git a/tasklib/task.py b/tasklib/task.py
index 6de828f92cd458702690516de6548c3a1635c8ff..a0b3c288b784f8a7ab3dca72083154fcee515e81 100644 (file)
--- a/tasklib/task.py
+++ b/tasklib/task.py
@@ -434,13 +434,21 @@ class Task(TaskResource):
         def add_field(field):
             # Add the output of format_field method to args list (defaults to
             # field:value)
-            serialized_value = self._serialize(field, self._data[field]) or ''
-            format_default = lambda: "{0}:{1}".format(
-                field,
-                "'{0}'".format(serialized_value) if serialized_value else ''
-            )
+            serialized_value = self._serialize(field, self._data[field])
+
+            # Empty values should not be enclosed in quotation marks, see
+            # TW-1510
+            if serialized_value is '':
+                escaped_serialized_value = ''
+            else:
+                escaped_serialized_value = "'{0}'".format(serialized_value)
+
+            format_default = lambda: "{0}:{1}".format(field,
+                                                      escaped_serialized_value)
+
             format_func = getattr(self, 'format_{0}'.format(field),
                                   format_default)
+
             args.append(format_func())
 
         # If we're modifying saved task, simply pass on all modified fields