Feature #608

- Added feature #608, and now completing a task, with journal.time turned on
  will stop the task first (thanks to Andy Kriger).

AUTHORS

@@ -72,4 +72,5 @@ suggestions:
   Erlan Sergaziev
   Eric Fluger
   Steve Rader
+  Andy Kriger
 

ChangeLog

@@ -30,6 +30,8 @@
    month to be displayed in the calendar report (thanks to Michelle Crane).
  + Added feature #574, default due dates (thanks to Erlan Sergaziev).
  + Added feature #575, including Danish holidays (thanks to Irfan Siddiqui).
+ + Added feature #608, and now completing a task, with journal.time turned on
+   will stop the task first (thanks to Andy Kriger).
  + Eliminated dependency on ncurses.
  + The dependency columns are now right-justified (thanks to Eric Fluger).
  + Fixed bug that caused the 'done' command to always exit with a non-zero

src/command.cpp

@@ -1862,6 +1862,11 @@ int handleDone (std::string& outs)
         // Change status.
         task->setStatus (Task::completed);
 
+        // Stop the task, if started.
+        if (task->has ("start") &&
+            context.config.getBoolean ("journal.time"))
+          task->addAnnotation (context.config.get ("journal.time.stop.annotation"));
+
         // Only allow valid tasks.
         task->validate ();