Legacy

- Created legacy checking function to determine whether deprecated or obsolete
  attributes are being used.

src/commands/Command.cpp

@@ -36,6 +36,7 @@
 #include <i18n.h>
 #include <Command.h>
 #include <cmake.h>
+#include <main.h>
 
 #include <CmdAdd.h>
 #include <CmdAnnotate.h>
@@ -431,56 +432,62 @@ void Command::modify_task (
         Column* column = context.columns[name];
 
         // All values must be eval'd first.
-        A3 fragment;
+        A3 value_tokens;
-        fragment.capture (value);
+        value_tokens.capture (value);
-        fragment = fragment.postfix (fragment.tokenize (fragment));
+        value_tokens = value_tokens.postfix (value_tokens.tokenize (value_tokens));
 
-        E9 e (fragment);
+        E9 e (value_tokens);
         std::string result = e.evalExpression (task);
         context.debug (std::string ("Eval '") + value + "' --> '" + result + "'");
 
-        // Dependencies must be resolved to UUIDs.
+        if (result == "")
-        if (name == "depends")
         {
-          // Convert ID to UUID.
+          task.remove (name);
-          std::vector <std::string> deps;
-          split (deps, result, ',');
-
-          // Apply or remove dendencies in turn.
-          std::vector <std::string>::iterator i;
-          for (i = deps.begin (); i != deps.end (); i++)
-          {
-            int id = strtol (i->c_str (), NULL, 10);
-            if (id < 0)
-              task.removeDependency (-id);
-            else
-              task.addDependency (id);
-          }
         }
-
-        // Dates are special, maybe.
-        else if (column->type () == "date")
-        {
-          // If the date value is less than 5 years, it is a duration, not a
-          // date, therefore add 'now'.
-          long l = strtol (result.c_str (), NULL, 10);
-          if (labs (l) < 5 * 365 * 86400)
-          {
-            Date now;
-            now += l;
-            task.set (name, now.toEpochString ());
-          }
-          else
-            task.set (name, result);
-        }
-
-        // By default, just add it.
         else
-          task.set (name, result);
+        {
+          // Dependencies must be resolved to UUIDs.
+          if (name == "depends")
+          {
+            // Convert ID to UUID.
+            std::vector <std::string> deps;
+            split (deps, result, ',');
 
-        // Legacy checks.
+            // Apply or remove dendencies in turn.
-        if (name == "fg" || name == "bg")
+            std::vector <std::string>::iterator i;
-          context.footnote (format (STRING_LEGACY_FEATURE, name));
+            for (i = deps.begin (); i != deps.end (); i++)
+            {
+              int id = strtol (i->c_str (), NULL, 10);
+              if (id < 0)
+                task.removeDependency (-id);
+              else
+                task.addDependency (id);
+            }
+          }
+
+          // Dates are special, maybe.
+          else if (column->type () == "date")
+          {
+            // If the date value is less than 5 years, it is a duration, not a
+            // date, therefore add 'now'.
+            long l = strtol (result.c_str (), NULL, 10);
+            if (labs (l) < 5 * 365 * 86400)
+            {
+              Date now;
+              now += l;
+              task.set (name, now.toEpochString ());
+            }
+            else
+              task.set (name, result);
+          }
+
+          // By default, just add/remove it.
+          else
+              task.set (name, result);
+
+          // Warn about deprecated/obsolete attribute usage.
+          legacyAttributeCheck (name);
+        }
       }
       else
         throw format (STRING_CMD_ADD_BAD_ATTRIBUTE, name);

src/legacy.cpp

@@ -25,11 +25,21 @@
 //
 ////////////////////////////////////////////////////////////////////////////////
 
+#define L10N                                           // Localization complete.
+
 #include <Context.h>
+#include <text.h>
+#include <i18n.h>
 
 extern Context context;
 
 ////////////////////////////////////////////////////////////////////////////////
+void legacyAttributeCheck (const std::string& name)
+{
+  // Legacy checks.
+  if (name == "fg" || name == "bg")
+    context.footnote (format (STRING_LEGACY_FEATURE, name));
+}
 
 ////////////////////////////////////////////////////////////////////////////////
 

src/main.h

@@ -78,6 +78,9 @@ std::string feedback (const Task&, const Task&);
 // sort.cpp
 void sort_tasks (std::vector <Task>&, std::vector <int>&, const std::string&);
 
+// legacy.cpp
+void legacyAttributeCheck (const std::string&);
+
 // list template
 ///////////////////////////////////////////////////////////////////////////////
 template <class T> bool listDiff (const T& left, const T& right)