From cefc129e9a2e4cdad4df09d8db521285e436883f Mon Sep 17 00:00:00 2001
From: Paul Beckingham <paul@beckingham.net>
Date: Sun, 1 Mar 2015 21:36:56 -0500
Subject: [PATCH] Task

- Delegated the modifiability check to Column::modifiable.
---
 src/Task.cpp | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/src/Task.cpp b/src/Task.cpp
index 0a58455b7..0deb5b652 100644
--- a/src/Task.cpp
+++ b/src/Task.cpp
@@ -1960,17 +1960,13 @@ 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];
 
+          // Some columns are not modifiable.
+          if (! column->modifiable ())
+            throw format (STRING_INVALID_MOD, name, value);
+
           // Dependencies are specified as IDs.
           if (name == "depends")
           {