From: Tomas Babej <tomasbabej@gmail.com>
Date: Sat, 3 Jan 2015 11:16:31 +0000 (+0100)
Subject: Task: Check that we are unable to set read only values through __init__
X-Git-Url: https://git.madduck.net/etc/taskwarrior.git/commitdiff_plain/faa9531194cd3657ce1e2f40e6e8067ac546958c?ds=inline;hp=228ef69b3f641c8bb81d7377977b9b57855f8572

Task: Check that we are unable to set read only values through __init__
---

diff --git a/tasklib/task.py b/tasklib/task.py
index f7afbf2..3a58e50 100644
--- a/tasklib/task.py
+++ b/tasklib/task.py
@@ -132,6 +132,11 @@ class Task(TaskResource):
     def __init__(self, warrior, **kwargs):
         self.warrior = warrior
 
+        # Check that user is not able to set read-only value in __init__
+        for key in kwargs.keys():
+            if key in self.read_only_fields:
+                raise RuntimeError('Field \'%s\' is read-only' % key)
+
         # We serialize the data in kwargs so that users of the library
         # do not have to pass different data formats via __setitem__ and
         # __init__ methods, that would be confusing