Commit Graph

110 Commits

Author SHA1 Message Date
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
Timo Stollenwerk
a5683b0674 remove unnecessary imports.
svn path=/plone.app.discussion/trunk/; revision=37765
2010-07-13 10:40:24 +00:00
Timo Stollenwerk
cac0bae7a5 revert r37751.
svn path=/plone.app.discussion/trunk/; revision=37752
2010-07-12 13:47:53 +00:00
Timo Stollenwerk
eabf3bc890 Always use zope.component getUtility instead of queryUtility.
svn path=/plone.app.discussion/trunk/; revision=37751
2010-07-12 13:34:02 +00:00
Hanno Schlichting
7d3247ff0c Avoid deprecation warning for the Globals package.
svn path=/plone.app.discussion/trunk/; revision=37349
2010-06-18 12:01:44 +00:00
Timo Stollenwerk
400956e264 Comments added to the notification subscriber functions.
svn path=/plone.app.discussion/trunk/; revision=37329
2010-06-17 06:57:51 +00:00
Florian Friesdorf
5beebe42fa hardcoded fti_title, don't query typestool for Type()
all tests succeed

svn path=/plone.app.discussion/trunk/; revision=35290
2010-03-18 14:42:52 +00:00
Florian Friesdorf
8150617081 fixed #662654: As an administrator, I can configure a Collection to show
recent comments.

Type now correctly returns FTI type title. A catalog rebuild is
necessary.

svn path=/plone.app.discussion/trunk/; revision=35266
2010-03-18 13:22:16 +00:00
Timo Stollenwerk
52ebbe2e71 Fix #2494228: Remove comments as well when a content object is deleted.
svn path=/plone.app.discussion/trunk/; revision=34965
2010-03-16 15:06:43 +00:00
Timo Stollenwerk
4ed9d08119 fix the moderator notification for Plone 3.x.
svn path=/plone.app.discussion/trunk/; revision=34698
2010-03-11 19:23:53 +00:00
Timo Stollenwerk
be3d347471 user email notification added.
svn path=/plone.app.discussion/branches/notification/; revision=33963
2010-02-13 21:31:17 +00:00
Timo Stollenwerk
f0fb6514c7 Moderator email notification added: As a moderator, I am notified when new comments require my attention.
svn path=/plone.app.discussion/branches/notification/; revision=33921
2010-02-11 21:05:28 +00:00
Timo Stollenwerk
c4dbf971fb author_notification field added.
svn path=/plone.app.discussion/branches/notification/; revision=33867
2010-02-08 12:02:54 +00:00
Timo Stollenwerk
c9a5070681 Adding Persistent to subclasses of comment - this fixes five.intid choking on Comments in the catalog on reinstall. evilbungle branch merge.
svn path=/plone.app.discussion/trunk/; revision=30670
2009-10-17 16:29:45 +00:00
Timo Stollenwerk
e5f1a70de6 Make unit tests pass with Plone 3.3 again. Migration tests still failing.
svn path=/plone.app.discussion/trunk/; revision=30633
2009-10-16 12:11:58 +00:00
Hanno Schlichting
6e8f812eee Duh, I wanted the new CatalogAware without the OpaqueItemManager base, not the old CMFCatalogAware
svn path=/plone.app.discussion/trunk/; revision=30630
2009-10-16 00:51:58 +00:00
Hanno Schlichting
71ed707794 Get this running again for the most part under Plone 4.0 - the main trick was probably to use the new CMFCatalogAware class. Removed collective.testcaselayer dependency, as that broke totally. There's some test failures left, but indexing works in general. Maybe the custom indexing event handlers aren't all required anymore. The title_or_id test failure looks like there's an infinite loop in the __parent__ pointers never reaching the actual content object. This change breaks Plone 3.x compatibility for the moment.
svn path=/plone.app.discussion/trunk/; revision=30629
2009-10-16 00:42:08 +00:00
Carsten Senger
e946ef7979 Add an accessor method for title. As title is part of the interface
IComment for which we <require> zope2.View a permission check will raise
Unauthorized cause we can not assure that the user's userfolder is in 
the acqusition chain of the object (the attribute is a bare unicode object).
It works for methods. im_self is used to get the acquisition wrapped
object.

svn path=/plone.app.discussion/trunk/; revision=29199
2009-08-20 02:06:15 +00:00
Timo Stollenwerk
e2934a66b1 replace content_object.reindexObject() with event handlers.
svn path=/plone.app.discussion/trunk/; revision=27814
2009-07-04 16:18:48 +00:00
Timo Stollenwerk
fee068dc09 let comment Type method return portal_type instead of a string.
svn path=/plone.app.discussion/trunk/; revision=27655
2009-06-24 13:18:54 +00:00
Timo Stollenwerk
1cc02a3acd Type() is meant to be a method not an attribute.
svn path=/plone.app.discussion/trunk/; revision=27652
2009-06-24 08:25:11 +00:00
Timo Stollenwerk
6d414055bc set correct Type ('Discussion Item') for Comments.
svn path=/plone.app.discussion/trunk/; revision=27629
2009-06-22 12:23:19 +00:00
Timo Stollenwerk
8fb254f73d add optional workflow actions to the get_replies method of the comments viewlet.
switch from Acquisition.explicit to Acquisition.implicit for the comment class.
adjust test_comment to work with Acquisition.implicit.

svn path=/plone.app.discussion/trunk/; revision=27406
2009-06-13 16:46:37 +00:00
Timo Stollenwerk
751ae5a06a revert 27204: change import for discussion modification_date from datetime to DateTime.
svn path=/plone.app.discussion/trunk/; revision=27213
2009-05-30 06:08:44 +00:00
Timo Stollenwerk
9a91b6f9ed change import for discussion modification_date from datetime to DateTime.
svn path=/plone.app.discussion/trunk/; revision=27204
2009-05-29 06:30:55 +00:00
Martin Aspeli
8ffca9009c Ensure workflow is initialised when the comment is added. Also ensure that we won't get an error on unindex if the local utility can't be found.
svn path=/plone.app.discussion/trunk/; revision=27070
2009-05-23 16:37:10 +00:00
Martin Aspeli
8c3c4bdfa0 Add FTI for Discussion Item and test
svn path=/plone.app.discussion/trunk/; revision=27068
2009-05-23 16:12:45 +00:00
Martin Aspeli
250b556428 Refactor tests and add a lot of skeleton tests we really ought to have
svn path=/plone.app.discussion/trunk/; revision=27012
2009-05-18 15:15:36 +00:00
Martin Aspeli
22ae84735b - specify dependencies
- set profile version to 1 (profile version != package version)
- complete dict APIs and IReplies adapters (not yet fully tested)
- tidy up addComment() and __delitem__ w.r.t events
- sync interfaces with actual code
- move tests to use collective.testcaselayer

svn path=/plone.app.discussion/trunk/; revision=27010
2009-05-18 14:16:48 +00:00
Lennart Regebro
a2542ea5a8 Use long timestamps as id
svn path=/plone.app.discussion/trunk/; revision=26964
2009-05-16 11:21:23 +00:00
Martin Aspeli
51b076935e Typo fix + remove needless property
svn path=/plone.app.discussion/trunk/; revision=26963
2009-05-16 10:57:49 +00:00
Lennart Regebro
49c6c45453 First running test.
svn path=/plone.app.discussion/trunk/; revision=26962
2009-05-16 10:34:12 +00:00
Martin Aspeli
e0e313c919 Some more recent thoughts
svn path=/plone.app.discussion/trunk/; revision=26953
2009-05-16 04:46:47 +00:00
Martin Aspeli
d3a9cecf6b More notes/thoughts
svn path=/plone.app.discussion/trunk/; revision=26919
2009-05-13 15:20:02 +00:00
Martin Aspeli
3b2db86d54 Some more design notes/sketchings. Nothing that works yet.
svn path=/plone.app.discussion/trunk/; revision=26918
2009-05-13 14:54:06 +00:00
Martin Aspeli
6381b14763 check in some skeletal code + notes, nothing concrete or working yet
svn path=/plone.app.discussion/trunk/; revision=26892
2009-05-11 16:52:16 +00:00