Web   ·   Wiki   ·   Activities   ·   Blog   ·   Lists   ·   Chat   ·   Meeting   ·   Bugs   ·   Git   ·   Translate   ·   Archive   ·   People   ·   Donate
diff options
authorerick <erick@sugar-dev-erick.(none)>2009-10-30 15:03:49 (GMT)
committer erick <erick@sugar-dev-erick.(none)>2009-10-30 15:03:49 (GMT)
commit424e44e487d3f1387840a971464fad6e5b627018 (patch)
parent60109dea12383f8ea2f6d4251952ccc3859feec5 (diff)
Added comments from code review of vince
1 files changed, 6 insertions, 1 deletions
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