From 424e44e487d3f1387840a971464fad6e5b627018 Mon Sep 17 00:00:00 2001
From: erick <erick@sugar-dev-erick.(none)>
Date: Fri, 30 Oct 2009 15:03:49 +0000
Subject: Added comments from code review of vince

---
(limited to 'tutorius/tutorial.py')

diff --git a/tutorius/tutorial.py b/tutorius/tutorial.py
index 6a6e089..0736bbb 100644
--- a/tutorius/tutorial.py
+++ b/tutorius/tutorial.py
@@ -16,6 +16,9 @@
 # Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
 
 #TODO: On update, only properties should be passed as arguments for State.update and Tutorial.update
+#TODO: On update for transition, a None value means that the value is not updated
+#TODO: Check for putting checks for validity of names as decorators
+#TODO: For notification of modifications on the Tutorial check for GObject and PyDispatcher for inspiration
 
 class Tutorial(object):
     """ This class replaces the previous Tutorial class and 
@@ -328,6 +331,8 @@ class Tutorial(object):
          
         @return A reference to the dictionary of all the states in the tutorial with state_name as key and state as value
         """
+        # Maybe we will need to change it for an immutable dictionary
+        # to make sure the internal representation is not modified
         return self._state_dict
 
     def get_following_states_dict(self, state_name):
@@ -723,7 +728,7 @@ class State(object):
         @param otherState The state that will be compared to this one
         @return True if the states are the same, False otherwise
 `       """
-        raise NotImplemented
+        raise NotImplementedError
     
 
     
--
cgit v0.9.1