It would be good to have a way to opt-out of including comments in the feed.

Attached patch does that.

CommentFileSizeAuthor
#7 comment-rss1.patch2.33 KBandypost
#5 comment-rss.patch1.19 KBandypost
drupal-comment.diff1.4 KBhexmode

Comments

damien tournoud’s picture

damien tournoud commented
Version: 5.x-dev » 7.x-dev
Status: Needs review » Patch (to be ported)

Both Drupal 5 and Drupal 6 are feature frozen. Feature requests can only go in the current development version (Drupal 7-dev).

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Interesting idea. I think it'd be better to have this as a checkbox along with #335929: Option to anonymize RSS feeds in admin/content/rss-publishing implemented by comment.module using hook_form_alter. Better yet, this would possibly be great with #230710: Centralized module to control feeds and feed types.

andypost’s picture

andypost
he/him
Primary language Russian
commented

Marked as duplicate #253391: Option to include comments into feeds

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Patch (to be ported) » Needs work

Well. This isn't really to be ported. It's needs work because it's not written for D7.

andypost’s picture

andypost
he/him
Primary language Russian
commented
StatusFileSize
new comment-rss.patch1.19 KB

First of all there should be no comments if they are hidden
Next tests should be fixed

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Issue tags: +Needs tests

Yep, seems like it just needs some tests.

andypost’s picture

andypost
he/him
Primary language Russian
commented
StatusFileSize
new comment-rss1.patch2.33 KB

This patch with improved tests but I need someone to correct comments and messages...

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs work » Needs review

Start testing

dries’s picture

dries commented
Status: Needs review » Fixed

Looks great. Committed to CVS. Thanks for the tests! :)

Status: Fixed » Closed (fixed)
Issue tags: -Needs tests

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +Needs tests

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.