Commit Graph

85 Commits

Author SHA1 Message Date
Timo Stollenwerk
c47689af42 Fix handling of comments with invalid transforms. Write an error msg to the log and just return the untransformed text. 2012-07-12 16:03:02 +02:00
Timo Stollenwerk
702f7e1f1d Make 'text/plain' the default mime type for comments and make sure the default type is set properly when creating a new comment. 2012-07-12 16:01:07 +02:00
Timo Stollenwerk
0535e659ff Refactor the comment creator/author_name to comply with the Plone core convention to store the username on the creator attribute and not the fullname. 2012-07-12 15:39:27 +02:00
Elizabeth Leddy
2971974f26 revert modification date since this is fixed in p.a.caching now 2012-02-25 11:59:30 +01:00
Elizabeth Leddy
07b8ebc5df make sure commenting works when caching with modified headers 2012-01-24 18:28:02 -08:00
Timo Stollenwerk
1f01a71aac Whitespace. 2012-01-09 16:31:52 +01:00
Timo Stollenwerk
6a7c89ec78 Do not raise an error if the comment text is None. 2011-11-30 13:05:59 +01:00
David Glick
2bf5b43df5 uncatalog comments from the old location of the item being moved, not its parent
svn path=/plone.app.discussion/trunk/; revision=51692
2011-08-20 02:37:24 +00:00
David Glick
2b10fd06e2 fix failing test in CMFPlone. some things are contentish but not adaptable to IConversation (such as collection criteria)
svn path=/plone.app.discussion/trunk/; revision=51682
2011-08-19 18:54:44 +00:00
Timo Stollenwerk
3798338549 No need to manually put comments from conversation.getComments in an acquisition chain since getComments now returns acquisition wrapped comments.
svn path=/plone.app.discussion/trunk/; revision=51642
2011-08-18 07:31:00 +00:00
Timo Stollenwerk
cdf2beda40 Make sure only comments to the content object are removed from the catalog when the content object is moved.
svn path=/plone.app.discussion/trunk/; revision=51527
2011-08-04 15:53:21 +00:00
Timo Stollenwerk
0e47299c75 Make sure the comment brains are updated properly when the content object is renamed. Thanks hannosch for the code review.
svn path=/plone.app.discussion/trunk/; revision=51526
2011-08-04 14:13:01 +00:00
Timo Stollenwerk
50641ac6cc Remove unnecessary lines from notify_content_object_moved event subscriber function.
svn path=/plone.app.discussion/trunk/; revision=50944
2011-07-04 07:07:42 +00:00
Timo Stollenwerk
b7dad192d5 Update comment brains in zcatalog when moving a content object with comments. This fixes http://dev.plone.org/plone/ticket/11331. Refs #11331
svn path=/plone.app.discussion/trunk/; revision=50937
2011-07-03 18:36:02 +00:00
Timo Stollenwerk
e75685d5c0 Add links to delete/approve a comment in the moderator notification email.
Remove the unnecessary workflow_action parameter from the PublishComments request.

svn path=/plone.app.discussion/trunk/; revision=49045
2011-04-22 17:09:09 +00:00
Timo Stollenwerk
b84211ddf0 Remove ImportError exceptions for Plone < 4.1 code and plone.z3cform < 0.6.0.
svn path=/plone.app.discussion/trunk/; revision=49033
2011-04-22 14:59:59 +00:00
Timo Stollenwerk
31ecf38129 Provide the comment body text in the email notification.
svn path=/plone.app.discussion/trunk/; revision=49032
2011-04-22 13:34:50 +00:00
Timo Stollenwerk
f8c39c1b87 Fix comment link in email notification. This fixes http://dev.plone.org/plone/ticket/11413
svn path=/plone.app.discussion/trunk/; revision=49031
2011-04-22 12:47:29 +00:00
Timo Stollenwerk
49504c1ddd Redirect to the comment itself when notifying a user about a new comment.
svn path=/plone.app.discussion/trunk/; revision=49030
2011-04-22 08:54:35 +00:00
Ross Patterson
7354ca4298 Fix unindexing of comments when deleting content resulting from
iterating over a BTree while modifying it. Fixes #11402.

svn path=/plone.app.discussion/trunk/; revision=48845
2011-04-14 21:30:32 +00:00
David Glick
6d11fa6d5e merge c48523 from 1.x branch
svn path=/plone.app.discussion/trunk/; revision=48527
2011-04-07 21:31:56 +00:00
David Glick
9d83b4f0e6 merge -r46947 from davisagli-features: preserve legacy titles on migrated comments
svn path=/plone.app.discussion/trunk/; revision=48357
2011-04-02 21:16:24 +00:00
David Glick
eb004aab44 merge r46437 and r46946 from davisagli-features: respect the per-comment mime_type setting, and use the old cooked text/html from legacy comments when migrating
svn path=/plone.app.discussion/trunk/; revision=48356
2011-04-02 19:51:37 +00:00
David Glick
ebbf2d9b7b fix deprecation warning
svn path=/plone.app.discussion/trunk/; revision=48234
2011-03-26 02:44:49 +00:00
Patrick Gerken
79701cce7e Take care of properly converting titles to unicode
svn path=/plone.app.discussion/trunk/; revision=47991
2011-03-08 20:15:52 +00:00
Timo Stollenwerk
bf96d0082f Remove "Plone 3 only" code; Do not monkey patch the BAD_TYPES vocabulary or plone.app.vocabularies anymore.
svn path=/plone.app.discussion/trunk/; revision=47311
2011-02-08 09:28:51 +00:00
Timo Stollenwerk
0a0e69d47d 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
2011-02-07 11:48:24 +00:00
Timo Stollenwerk
4ab29000a6 Import Owned from OFS.owner to avoid deprecation warnings.
svn path=/plone.app.discussion/trunk/; revision=46977
2011-01-20 20:38:50 +00:00
Maurits van Rees
a2a17085a3 Nuke trailing white space
svn path=/plone.app.discussion/trunk/; revision=46366
2010-12-15 23:52:56 +00:00
Maurits van Rees
aff8a3709c Avoid sending multiple notification emails to the same person when
he has commented multiple times.

svn path=/plone.app.discussion/trunk/; revision=46365
2010-12-15 23:41:57 +00:00
Timo Stollenwerk
b26d5a5a62 Fix SMTP exception when an email is send to the moderator.
svn path=/plone.app.discussion/trunk/; revision=46013
2010-11-29 08:27:09 +00:00
Timo Stollenwerk
7dbcc9e2be Use safe_unicode to decode the title of the content. encode("utf-9") caused Dexterity based content types to raise a unicode decode error. This fixes http://dev.plone.org/plone/ticket/11292. Refs #11292
svn path=/plone.app.discussion/trunk/; revision=45915
2010-11-23 14:03:47 +00:00
Timo Stollenwerk
cbd2edf932 Catch SMTP exceptions when sending email notifications.
svn path=/plone.app.discussion/trunk/; revision=45199
2010-11-04 15:56:12 +00:00
Timo Stollenwerk
fa6e380eaf Use secureSend for user email notification on Plone 3
svn path=/plone.app.discussion/trunk/; revision=44736
2010-10-31 10:34:58 +00:00
Timo Stollenwerk
82afd3ef15 New feature: As a logged-in user, I can enable/disable email notification of additional comments on this content object.
svn path=/plone.app.discussion/trunk/; revision=40949
2010-10-30 15:02:05 +00:00
Timo Stollenwerk
81dcdeaea3 Disable the plone.app.registry check on schema elements, so no error is raised on upgrades. This fixes https://dev.plone.org/plone/ticket/11195. Refs #11195
svn path=/plone.app.discussion/trunk/; revision=40868
2010-10-29 10:43:46 +00:00
Hanno Schlichting
aa8a1ea934 Fixed handling of non-ascii member data, like fullname and email.
svn path=/plone.app.discussion/trunk/; revision=40789
2010-10-22 10:14:07 +00:00
Timo Stollenwerk
324d2274bd Check for a 'pending' state in the current comment workflow instead of checking for the 'comment_review' workflow. Refs #11184. Merge last three submits in CHANGES.txt.
svn path=/plone.app.discussion/trunk/; revision=40570
2010-10-08 10:37:10 +00:00
Timo Stollenwerk
1db321a044 Make pylint happy.
svn path=/plone.app.discussion/trunk/; revision=40527
2010-10-06 09:02:26 +00:00
Timo Stollenwerk
9841ba7229 Remove unnecessary decode statement from creator.
svn path=/plone.app.discussion/trunk/; revision=40503
2010-10-04 13:48:54 +00:00
Timo Stollenwerk
b06ba43a75 Ignore currently unused code from code coverage. Overall code coverage is up to 95%.
svn path=/plone.app.discussion/trunk/; revision=40497
2010-10-03 19:54:59 +00:00
Timo Stollenwerk
e0055d0ba7 Make sure comment titles work with special characters as well.
svn path=/plone.app.discussion/trunk/; revision=40486
2010-10-02 18:16:49 +00:00
Timo Stollenwerk
22db0b1c78 Fix comment title for anonymous commentators that do not provide a name.
svn path=/plone.app.discussion/trunk/; revision=40435
2010-09-29 10:52:11 +00:00
Timo Stollenwerk
f67c7cde6d Remove title field from the comment form and replace it with an auto-generated title ("John Doe on Welcome to Plone").
This also fixes http://dev.plone.org/plone/ticket/11098

svn path=/plone.app.discussion/trunk/; revision=40431
2010-09-29 07:56:36 +00:00
Patrick Gerken
a7b1eb7c10 Change all datetime creations from now to utcnow
svn path=/plone.app.discussion/trunk/; revision=39726
2010-09-07 12:03:07 +00:00
Vincent Fretin
4b15b31e51 Translate the subject and be sure to set the charset to utf-8 for the mail.
svn path=/plone.app.discussion/trunk/; revision=39587
2010-09-01 07:32:28 +00:00
Vincent Fretin
d5baae3eec Fixed variables in the template for use with the i18n machinery
svn path=/plone.app.discussion/trunk/; revision=39582
2010-09-01 07:19:51 +00:00
Vincent Fretin
f5bed31131 internationalize notification mail
svn path=/plone.app.discussion/trunk/; revision=39572
2010-08-31 18:07:51 +00:00
Timo Stollenwerk
e12d19a6e9 Fix bugs that have been introduced with the latest pylint "optimizations".
svn path=/plone.app.discussion/trunk/; revision=39269
2010-08-28 16:07:44 +00:00
Timo Stollenwerk
9264d2d42d Make pylint happy.
svn path=/plone.app.discussion/trunk/; revision=39265
2010-08-28 15:35:21 +00:00