[Checkins] SVN: Products.CMFDefault/trunk/Products/CMFDefault/ - removed dead code

Yvo Schubbe y.2008 at wcm-solutions.de
Fri Jan 11 12:46:28 EST 2008


Log message for revision 82814:
  - removed dead code

Changed:
  U   Products.CMFDefault/trunk/Products/CMFDefault/CHANGES.txt
  U   Products.CMFDefault/trunk/Products/CMFDefault/DiscussionItem.py

-=-
Modified: Products.CMFDefault/trunk/Products/CMFDefault/CHANGES.txt
===================================================================
--- Products.CMFDefault/trunk/Products/CMFDefault/CHANGES.txt	2008-01-11 15:46:05 UTC (rev 82813)
+++ Products.CMFDefault/trunk/Products/CMFDefault/CHANGES.txt	2008-01-11 17:46:27 UTC (rev 82814)
@@ -2,6 +2,9 @@
 
   Products.CMFDefault 2.2.0 (unreleased)
 
+    - DiscussionItem: Removed 'addDiscussionItem' function.
+      This was dead code. 'createReply' is used for adding DiscussionItems.
+
     - content: Factories no longer sends add events.
 
     - profiles: Removed obsolete local import and export step registrations.

Modified: Products.CMFDefault/trunk/Products/CMFDefault/DiscussionItem.py
===================================================================
--- Products.CMFDefault/trunk/Products/CMFDefault/DiscussionItem.py	2008-01-11 15:46:05 UTC (rev 82813)
+++ Products.CMFDefault/trunk/Products/CMFDefault/DiscussionItem.py	2008-01-11 17:46:27 UTC (rev 82814)
@@ -35,36 +35,8 @@
 from permissions import ManagePortal
 from permissions import ReplyToItem
 from permissions import View
-from utils import scrubHTML
 
 
-def addDiscussionItem(self, id, title, description, text_format, text,
-                      reply_to, RESPONSE=None):
-    """ Add a discussion item
-
-    'title' is also used as the subject header
-    if 'description' is blank, it is filled with the contents of 'title'
-    'reply_to' is the object (or path to the object) which this is a reply to
-
-    Otherwise, same as addDocument
-    """
-
-    if not description: description = title
-    text = scrubHTML(text)
-    item = DiscussionItem( id )
-    item.title = title
-    item.description = description
-    item.text_format = text_format
-    item.text = text
-    item.setReplyTo(reply_to)
-
-    item._parse()
-    self._setObject(id, item)
-
-    if RESPONSE is not None:
-        RESPONSE.redirect(self.absolute_url())
-
-
 class DiscussionItem(Document):
 
     """ Class for content which is a response to other content.



More information about the Checkins mailing list