Opened 3 years ago

Closed 3 years ago

#14451 closed defect (bug) (fixed)

RSS2 feed generates invalid category markup when 'default_feed' is 'atom'

Reported by: benward Owned by: benward
Priority: normal Milestone: 3.1
Component: Feeds Version: 3.0
Severity: normal Keywords: has-patch
Cc:

Description

If you override the 'default_feed' filter to return 'atom' (to make Atom format feeds the WordPress default at /feed/), the RSS2 format is broken because it does not explicitly declare 'rss2' as its category format, instead implying the default from get_default_feed() in get_the_category_rss(). When 'default_feed' is set to 'atom', Atom formatted categories are returned in RSS.

Attached is a one line patch for feed-rss2.php, which explicitly declares the format when including categories.

(Credit for this catch to my colleague Michael Lodick at Yahoo, whose feed importer I broke with the format religiosity that exposed this bug.)

Attachments (1)

bugfix-rss2-category-format.diff (636 bytes) - added by benward 3 years ago.
One line patch to make 'the_category_rss' explicitly declare 'rss2' as format

Download all attachments as: .zip

Change History (4)

benward3 years ago

One line patch to make 'the_category_rss' explicitly declare 'rss2' as format

  • Keywords has-patch added; feeds, rss2, default_feed, atom removed
  • Milestone changed from Awaiting Review to 3.1

Is this an issue for other types of data, and other feed types? Or just categories in the RSS2 feed.

It looks like it's only the_category_rss() that gets called without arguments and depends on 'rss2' being the default type.

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

(In [15984]) Specify feed type in the_category_rss() call. props benward, fixes #14451.

Note: See TracTickets for help on using tickets.