Bug #595

- Fixed bug #595, where taskwarrior ignored changes to the wait date during
  the edit command, consequently not changing task status (thanks to Eric
  Fluger).

ChangeLog

@@ -59,6 +59,9 @@
    Steve Rader).
  + Fixed bug #589, where the man page did not adequately describe searching
    or usage of attribute modifiers (thanks to Steve Rader).
+ + Fixed bug #595, where taskwarrior ignored changes to the wait date during
+   the edit command, consequently not changing task status (thanks to Eric
+   Fluger).
 
 ------ old releases ------------------------------
 

src/edit.cpp

@@ -427,12 +427,14 @@ static void parseTask (Task& task, const std::string& after)
       {
         std::cout << "Wait date modified.\n";
         task.set ("wait", value);
+        task.setStatus (Task::waiting);
       }
     }
     else
     {
       std::cout << "Wait date modified.\n";
       task.set ("wait", value);
+      task.setStatus (Task::waiting);
     }
   }
   else
@@ -441,6 +443,7 @@ static void parseTask (Task& task, const std::string& after)
     {
       std::cout << "Wait date removed.\n";
       task.remove ("wait");
+      task.setStatus (Task::pending);
     }
   }
 

src/util.cpp

@@ -557,7 +557,8 @@ std::string taskInfoDifferences (const Task& before, const Task& after)
 std::string renderAttribute (const std::string& name, const std::string& value)
 {
   Att a;
-  if (a.type (name) == "date")
+  if (a.type (name) == "date" &&
+      value != "")
   {
     Date d ((time_t)::atoi (value.c_str ()));
     return d.toString (context.config.get ("dateformat"));