Bug Fix - #395

- When a recurrence period is added to a pending task, the status should
  change from pending to recurring, and a mask attribute should be added.
  The lack of those changes meant that "task 1 recur:1w" did not do what
  was expected.  Thanks to T. Charles Yun.

ChangeLog

@@ -20,7 +20,9 @@
  + Fixed bug #402 which failed compilation on Arch Linux (thanks to
    Johannes Schlatow).
  + Fixed bug #401 that ignored the search.case.sensitive configuration
-   setting when filtering on project names (thank to John Florian).
+   setting when filtering on project names (thanks to John Florian).
+ + Fixed bug #395 that prevented the upgrade of a pending task to a
+   recurring task (thanks to T. Charles Yun).
 
 ------ old releases ------------------------------
 

src/command.cpp

@@ -1359,6 +1359,21 @@ int handleModify (std::string &outs)
         // A non-zero value forces a file write.
         int changes = 0;
 
+        // If a task is being made recurring, there are other cascading
+        // changes.
+        if (!task->has ("recur") &&
+            context.task.has ("recur"))
+        {
+          other->setStatus (Task::recurring);
+          other->set ("mask", "");
+          ++changes;
+
+          std::cout << "Task "
+                    << other->id
+                    << " is now a recurring task."
+                    << std::endl;
+        }
+
         // Apply other deltas.
         if (deltaDescription (*other))
         {