Enhancement

- Implemented tri-state confirmation for done command.

src/command.cpp

@@ -679,13 +679,21 @@ std::string handleDone ()
   std::vector <Task> all = tasks;
   context.filter.applySequence (tasks, context.sequence);
 
+  Permission permission;
+  if (context.sequence.size () > (size_t) context.config.get ("bulk", 2))
+    permission.bigSequence ();
+
   bool nagged = false;
   foreach (task, tasks)
   {
     if (task->getStatus () == Task::pending)
     {
-      // Apply deltas.
+      Task before (*task);
-      deltaDescription (*task);
+
+      // Apply other deltas.
+      if (deltaDescription (*task))
+        permission.bigChange ();
+
       deltaTags (*task);
       deltaAttributes (*task);
       deltaSubstitutions (*task);
@@ -697,21 +705,29 @@ std::string handleDone ()
 
       // Change status.
       task->setStatus (Task::completed);
-      context.tdb.update (*task);
 
-      if (context.config.get ("echo.command", true))
+      if (taskDiff (before, *task))
-        out << "Completed "
+      {
-            << task->id
+        std::string question = taskDifferences (before, *task) + "Are you sure?";
-            << " '"
+        if (permission.confirmed (question))
-            << task->get ("description")
+        {
-            << "'"
+          context.tdb.update (*task);
-            << std::endl;
+
+          if (context.config.get ("echo.command", true))
+            out << "Completed "
+                << task->id
+                << " '"
+                << task->get ("description")
+                << "'"
+                << std::endl;
+
+          ++count;
+        }
+      }
 
       updateRecurrenceMask (all, *task);
       if (!nagged)
         nagged = nag (*task);
-
-      ++count;
     }
     else
       out << "Task "