From 7284f2d9e259628c468cd2495606dee52e57174b Mon Sep 17 00:00:00 2001 From: Martin Aspeli Date: Sat, 23 May 2009 15:50:21 +0000 Subject: [PATCH] Safer fallback - this method will have a different return signature when implemented svn path=/plone.app.discussion/trunk/; revision=27064 --- plone/app/discussion/conversation.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/plone/app/discussion/conversation.py b/plone/app/discussion/conversation.py index 980a444..4551c9c 100644 --- a/plone/app/discussion/conversation.py +++ b/plone/app/discussion/conversation.py @@ -108,7 +108,7 @@ class Conversation(Traversable, Persistent, Explicit): """Get threaded comments """ # TODO - build threads - return self._comments.values() + raise NotImplemented def addComment(self, comment): """Add a new comment. The parent id should have been set already. The @@ -255,8 +255,6 @@ class ConversationReplies(object): def __contains__(self, key): return long(key) in self.children - # TODO: Should __getitem__, get, __iter__, values(), items() and iter* return aq-wrapped comments? - def __getitem__(self, key): """Get an item by its long key """