Enhancement - modify command

- Implemented modify command

src/Context.cpp

@@ -188,9 +188,6 @@ std::string Context::dispatch ()
   else if (cmd.command == "calendar")      { out = handleReportCalendar  (); }
   else if (cmd.command == "timesheet")     { out = handleReportTimesheet (); }
   else if (cmd.command == "add")           { out = handleAdd             (); }
-/*
-  else if (cmd.command == "" && task.getId ()) { out = handleModify          (); }
-*/
   else if (cmd.command == "append")        { out = handleAppend          (); }
   else if (cmd.command == "annotate")      { out = handleAnnotate        (); }
   else if (cmd.command == "done")          { out = handleDone            (); }
@@ -207,6 +204,8 @@ std::string Context::dispatch ()
 /*
   else if (cmd.command == "edit")          { out = handleEdit            (); }
 */
+  else if (cmd.command == "" &&
+           sequence.size ())               { out = handleModify          (); }
 
   // Command that display IDs and therefore need TDB::gc first.
   else if (cmd.command == "next")          { if (!inShadow) gcMod = tdb.gc (); out = handleReportNext    (); }

src/command.cpp

@@ -761,57 +761,66 @@ std::string handleExport ()
 ////////////////////////////////////////////////////////////////////////////////
 std::string handleModify ()
 {
-/*
   int count = 0;
-*/
   std::stringstream out;
-/*
-  std::vector <T> all;
-  tdb.allPendingT (all);
 
-  std::vector <T> filtered = all;
+  std::vector <Task> tasks;
-  filterSequence (filtered, task);
+  context.tdb.lock (context.config.get ("locking", true));
-  foreach (seq, filtered)
+  context.tdb.loadPending (tasks, context.filter);
+  handleRecurrence (tasks);
+
+  // Filter sequence.
+  std::vector <Task> all = tasks;
+  context.filter.applySequence (tasks, context.sequence);
+
+  foreach (task, tasks)
   {
     // Perform some logical consistency checks.
-    if (task.has ("recur") &&
+    if (context.task.has ("recur") &&
-        !task.has ("due")  &&
+        !context.task.has ("due")  &&
-        !seq->has ("due"))
+        !task->has ("due"))
       throw std::string ("You cannot specify a recurring task without a due date.");
 
-    if (task.has ("until")  &&
+    if (context.task.has ("until")  &&
-        !task.has ("recur") &&
+        !context.task.has ("recur") &&
-        !seq->has ("recur"))
+        !task->has ("recur"))
       throw std::string ("You cannot specify an until date for a non-recurring task.");
 
     // Make all changes.
     foreach (other, all)
     {
-      if (other->getId ()               == seq->getId ()                   || // Self
+      if (other->id             == task->id               || // Self
-          (seq->has ("parent") &&
+          (task->has ("parent") &&
-           seq->getAttribute ("parent") == other->getAttribute ("parent")) || // Sibling
+           task->get ("parent") == other->get ("parent")) || // Sibling
-          other->getUUID ()             == seq->getAttribute ("parent"))      // Parent
+          other->get ("uuid")   == task->get ("parent"))     // Parent
       {
         // A non-zero value forces a file write.
         int changes = 0;
 
         // Apply other deltas.
-        changes += deltaDescription   (*other, task);
+        changes += deltaDescription (*other);
-        changes += deltaTags          (*other, task);
+        context.task.remove ("description");
-        changes += deltaAttributes    (*other, task);
+
-        changes += deltaSubstitutions (*other, task);
+        changes += deltaTags (*other);
+        context.task.remove ("tags");
+
+        changes += deltaAttributes (*other);
+        changes += deltaSubstitutions (*other);
 
         if (changes)
-          tdb.modifyT (*other);
+          context.tdb.update (*other);
 
         ++count;
       }
     }
   }
 
+  context.tdb.commit ();
+  context.tdb.unlock ();
+
   if (context.config.get ("echo.command", true))
     out << "Modified " << count << " task" << (count == 1 ? "" : "s") << std::endl;
-*/
+
   return out.str ();
 }