TW-1420, TW-1421, TW-1422

- TW-1420 Modifying 'uuid' fails to generate error (thanks to Black Ops Testing).
- TW-1421 Modifying 'uuid' with bogus value can corrupt undo.data (thanks to
          Black Ops Testing).
- TW-1422 Attempt to modify 'id' attribute creates an id attribute (thanks to
          Black Ops Testing).

AUTHORS

@@ -232,3 +232,4 @@ suggestions:
   Peter Vitt
   dev-zero
   Petteri
+  Black Ops Testing

ChangeLog

@@ -159,6 +159,11 @@
 - TW-1409 Allow "1 of N tasks remaining" to be correctly localized (thanks to
           Jeremy John Reeder).
 - TW-1415 The recurrence value 'month' is not valid (thanks to Petteri).
+- TW-1420 Modifying 'uuid' fails to generate error (thanks to Black Ops Testing).
+- TW-1421 Modifying 'uuid' with bogus value can corrupt undo.data (thanks to
+          Black Ops Testing).
+- TW-1422 Attempt to modify 'id' attribute creates an id attribute (thanks to
+          Black Ops Testing).
 - TW-1428 Add support for color.uda.<name>.<value> rules.
 - Removed deprecated 'echo.command' setting, in favor of the 'header' and
   'affected' verbosity tokens.

src/Task.cpp

@@ -1938,6 +1938,14 @@ void Task::modify (modType type, bool text_required /* = false */)
         }
         else
         {
+          // Some columns are not modifiable.
+          if (name == "uuid"  ||
+              name == "id"    ||
+              name == "mask"  ||
+              name == "imask" ||
+              name == "parent")
+            throw format (STRING_INVALID_MOD, name, value);
+
           // Get the column info.
           Column* column = context.columns[name];