From 6b5253e6a067a98852cce91eca029219506f684b Mon Sep 17 00:00:00 2001
From: Cory Donnelly <cory@onryou.com>
Date: Mon, 31 Jan 2011 18:31:57 -0500
Subject: [PATCH] Bug #669

- A test for due date < wait date in command.cpp only sometimes added
  an extra context.footnote complaining about the condition and was removed.
---
 src/command.cpp | 23 -----------------------
 1 file changed, 23 deletions(-)

diff --git a/src/command.cpp b/src/command.cpp
index 781e05830..d3ad4c5eb 100644
--- a/src/command.cpp
+++ b/src/command.cpp
@@ -1985,29 +1985,6 @@ int handleModify (std::string& outs)
         context.task.get ("recur") == "")
       throw std::string ("You cannot remove the recurrence from a recurring task.");
 
-    // This looks unnecessarily complex, but isn't.  "due" and "wait" are
-    // independent and may exist without the other, but if both exist then wait
-    // must be before due.
-    if ((task->has ("wait")       &&
-         context.task.has ("due") &&
-         Date (context.task.get ("due")) < Date (task->get ("wait")))
-        ||
-        (context.task.has ("wait") &&
-         !context.task.has ("due") &&
-         task->has ("due")         &&
-         Date (task->get ("due")) < Date (context.task.get ("wait")))
-        ||
-        (context.task.has ("wait") &&
-         context.task.has ("due")  &&
-         Date (context.task.get ("due")) < Date (context.task.get ("wait")))
-        ||
-        (task->has ("wait") &&
-         task->has ("due")  &&
-         Date (task->get ("due")) < Date (task->get ("wait"))))
-    {
-      context.footnote ("Warning: the wait date falls after the due date.");
-    }
-
     // Make all changes.
     bool warned = false;
     foreach (other, all)