TW-252

- TW-252 task done - Doesn't stop task before marking complete (thanks to
         Renato Alves).

ChangeLog

@@ -44,6 +44,8 @@
 - TW-250 Opening parenthesis in description gets padded in task 2.0.0 bet 2
          (thanks to Michelle Crane).
 - TW-251 extra spaces added after ( and / (thanks to Andy Spiegl).
+- TW-252 task done - Doesn't stop task before marking complete (thanks to
+         Renato Alves).
 - TW-253 Unrecognized taskwarrior file format. in
          /Users/user/Dropbox/.task/completed.data at line 1 (thanks to Kosta
          Harlan).

src/commands/CmdDone.cpp

@@ -85,9 +85,12 @@ int CmdDone::execute (std::string& output)
         task->setEnd ();
 
       // Stop the task, if started.
-      if (task->has ("start") &&
-          context.config.getBoolean ("journal.time"))
-        task->addAnnotation (context.config.get ("journal.time.stop.annotation"));
+      if (task->has ("start"))
+      {
+        task->remove ("start");
+        if (context.config.getBoolean ("journal.time"))
+          task->addAnnotation (context.config.get ("journal.time.stop.annotation"));
+      }
 
       if (permission (*task, taskDifferences (before, *task) + question, filtered.size ()))
       {