From 0a0e69d47dac57e0f34f98d2d839e34e0d4aa3aa Mon Sep 17 00:00:00 2001 From: Timo Stollenwerk Date: Mon, 7 Feb 2011 11:48:24 +0000 Subject: [PATCH] Do not check for a comment review workflow when sending out a moderator email notification. This fixes http://dev.plone.org/plone/ticket/11444. Refs #11444 svn path=/plone.app.discussion/trunk/; revision=47292 --- CHANGES.txt | 3 +++ plone/app/discussion/comment.py | 7 ------- plone/app/discussion/tests/test_notifications.py | 13 ------------- 3 files changed, 3 insertions(+), 20 deletions(-) diff --git a/CHANGES.txt b/CHANGES.txt index 589c5fa..3d51b4a 100644 --- a/CHANGES.txt +++ b/CHANGES.txt @@ -22,6 +22,9 @@ Changelog Plone 4.1. [timo] +- Do not check for a comment review workflow when sending out a moderator email + notification. This fixes http://dev.plone.org/plone/ticket/11444. + 1.0RC1 (unreleased) ------------------- diff --git a/plone/app/discussion/comment.py b/plone/app/discussion/comment.py index 356cc4a..13d8260 100644 --- a/plone/app/discussion/comment.py +++ b/plone/app/discussion/comment.py @@ -284,13 +284,6 @@ def notify_moderator(obj, event): if not settings.moderator_notification_enabled: return - # Check if the current workflow implements a pending state. - wf_tool = getToolByName(obj, 'portal_workflow') - comment_workflow = wf_tool.getChainForPortalType('Discussion Item')[0] - comment_workflow = wf_tool[comment_workflow] - if 'pending' not in comment_workflow.states: - return - # Get informations that are necessary to send an email mail_host = getToolByName(obj, 'MailHost') portal_url = getToolByName(obj, 'portal_url') diff --git a/plone/app/discussion/tests/test_notifications.py b/plone/app/discussion/tests/test_notifications.py index ae78ac4..86b05ad 100644 --- a/plone/app/discussion/tests/test_notifications.py +++ b/plone/app/discussion/tests/test_notifications.py @@ -256,18 +256,5 @@ class TestModeratorNotificationUnit(PloneTestCase): self.conversation.addComment(comment) self.assertEquals(len(self.mailhost.messages), 0) - def test_do_not_notify_moderator_when_moderation_workflow_is_disabled(self): - # Disable comment moderation and make sure no email is send to the - # moderator. - self.portal.portal_types['Document'].allow_discussion = True - self.portal.portal_workflow.setChainForPortalTypes( - ('Discussion Item',), - ('one_state_workflow',)) - - comment = createObject('plone.Comment') - comment.text = 'Comment text' - self.conversation.addComment(comment) - self.assertEquals(len(self.mailhost.messages), 0) - def test_suite(): return unittest.defaultTestLoader.loadTestsFromName(__name__)