Opened 4 years ago

Closed 4 years ago

#10016 closed defect (bug) (fixed)

<wfw:commentRss> may not return an RSS 2 feed

Reported by: peaceablewhale Owned by:
Priority: normal Milestone: 2.9
Component: Feeds Version: 2.8
Severity: normal Keywords: has-patch commit
Cc:

Description

When the default feed format is not RSS 2, the <wfw:commentRss> elements in the RSS 2 feed will contain links pointing to comment feeds in a different feed format. This behavior is unexpected for some applications which always look for a RSS 2 feed.

Attachments (1)

10016.patch (600 bytes) - added by peaceablewhale 4 years ago.

Download all attachments as: .zip

Change History (10)

itching to close this as a dup of #5802

  • Milestone changed from 2.8 to 2.9

punting per IRC discussion

  • Milestone changed from 2.9 to 2.8

This is not related to #5802. The feed format can be changed by a plugin. If the format is Atom, wfw:commentRss will return the Atom comment feed link because the feed format is not specified. That patch fixes to format to "rss2".

By the way, how may I join the IRC discussion?

  • Milestone changed from 2.8 to 2.9

irc.freenode.net, channel wordpress-dev

punting back to 2.9. only critical bugs should remain in 2.8.

Thanks. I finally noticed the blog post saying that only blockers should remain: http://wordpress.org/development/2009/06/summary-of-wordpress-dev-irc-meetup-for-20090603/

Unfortunately it is difficult to for me to join the IRC... 21:00 UTC is the time I am sleeping. (I am in Hong Kong, the timezone is UTC+8) I hope that discussion of bugs will remain in Trac.

  • Milestone changed from 2.9 to 2.8.1

Can this be patched in 2.8.1? I think this is a simple quick fix.

  • Keywords commit added

comment:8   ryan4 years ago

  • Milestone changed from 2.8.5 to 2.9

comment:9   ryan4 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [11864]) Force correct feed type for wfw:CommentRss element. Props peaceablewhale. fixes #10016

Note: See TracTickets for help on using tickets.