WordPress.org
Get WordPress

Make WordPress Core

Opened 16 years ago

Closed 11 years ago

#15610 closed defect (bug) (fixed)

Comments on attachments don't appear in site-wide comments RSS feed

Reported by: markel Owned by: stevenkword
Priority: normal Milestone: 4.5
Component: Feeds Version: 3.1
Severity: minor Keywords: has-patch has-unit-tests
Cc: Focuses:

Description

Tested against r16950.

Comments that are entered against an attachment do not appear in the site-wide comments RSS feed (e.g., ​http://sitename.com/comments/feed/).

To test:

  1. Add an attachment of any kind (usually image), then View the attachment.
  2. Post a comment on the attachment page.
  3. Check the site's comments RSS feed - the comment will not appear.

Attachments (1)

15610.patch​ (1.7 KB ) - added by jbrinley 12 years ago.

Download all attachments as: .zip

Change History (6)

#1 @nacin
16 years ago

  • Milestone Awaiting Review → Future Release

#2 @chriscct7
12 years ago

  • Keywords needs-patch added

@jbrinley
12 years ago

#3 @jbrinley
12 years ago

  • Keywords has-patch dev-feedback added; needs-patch removed

15610.patch adds a test and changes the query for comments to include comments on attachments. I wonder if there needs to be any check on the post status for the attachment's parent post.

#4 @stevenkword
11 years ago

  • Keywords has-unit-tests added; dev-feedback removed
  • Milestone Future Release → 4.5
  • Owner set to stevenkword
  • Status new → assigned

Patch still applies cleanly and the unit tests pass. However, I think I'd rather see this test in the "feed" group. Otherwise, this looks ready for commit to me.

#5 @rachelbaker
11 years ago

  • Resolution → fixed
  • Status assigned → closed

In 36138:

Feeds: Comments on attachments display in the site-wide comments RSS feed.

Individual attachment pages already have their own RSS comment feed, and would be expected to be included in the site-wide comments RSS feed.

Props jbrinley.
Fixes #15610

Note: See TracTickets for help on using tickets.