Enhancement - nag

- Improved nagging with respect to multiple tasks.

src/command.cpp

@@ -568,6 +568,7 @@ std::string handleStart ()
   // Filter sequence.
   context.filter.applySequence (tasks, context.sequence);
 
+  bool nagged = false;
   foreach (task, tasks)
   {
     if (! task->has ("start"))
@@ -585,7 +586,8 @@ std::string handleStart ()
             << task->get ("description")
             << "'"
             << std::endl;
-      nag (*task);
+      if (!nagged)
+        nagged = nag (*task);
     }
     else
     {
@@ -664,6 +666,7 @@ std::string handleDone ()
   std::vector <Task> all = tasks;
   context.filter.applySequence (tasks, context.sequence);
 
+  bool nagged = false;
   foreach (task, tasks)
   {
     if (task->getStatus () == Task::pending)
@@ -692,7 +695,8 @@ std::string handleDone ()
             << std::endl;
 
       updateRecurrenceMask (all, *task);
-      nag (*task);
+      if (!nagged)
+        nagged = nag (*task);
 
       ++count;
     }

src/main.h

@@ -53,7 +53,7 @@ Date getNextRecurrence (Date&, std::string&);
 bool generateDueDates (Task&, std::vector <Date>&);
 void updateRecurrenceMask (std::vector <Task>&, Task&);
 int getDueState (const std::string&);
-void nag (Task&);
+bool nag (Task&);
 
 // command.cpp
 std::string handleAdd ();

src/recur.cpp

@@ -397,7 +397,7 @@ int getDueState (const std::string& due)
 }
 
 ////////////////////////////////////////////////////////////////////////////////
-void nag (Task& task)
+bool nag (Task& task)
 {
   std::string nagMessage = context.config.get ("nag", "");
   if (nagMessage != "")
@@ -448,15 +448,18 @@ void nag (Task& task)
     }
 
     // General form is "if there are no more deserving tasks", suppress the nag.
-    if (isOverdue                                         ) return;
-    if (pri == 'H' && !overdue                            ) return;
-    if (pri == 'M' && !overdue && !high                   ) return;
-    if (pri == 'L' && !overdue && !high && !medium        ) return;
-    if (pri == ' ' && !overdue && !high && !medium && !low) return;
+    if (isOverdue                                         ) return false;
+    if (pri == 'H' && !overdue                            ) return false;
+    if (pri == 'M' && !overdue && !high                   ) return false;
+    if (pri == 'L' && !overdue && !high && !medium        ) return false;
+    if (pri == ' ' && !overdue && !high && !medium && !low) return false;
 
     // All the excuses are made, all that remains is to nag the user.
     context.message (nagMessage);
+    return true;
   }
+
+  return false;
 }
 
 ////////////////////////////////////////////////////////////////////////////////