WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 5 years ago

#4643 closed defect (bug) (fixed)

RDF feeds contain invalid tags, "category".

Reported by: kztk Owned by:
Milestone: 2.3 Priority: lowest
Severity: normal Version: 2.2.1
Component: General Keywords: has-patch commit dev-reviewed
Focuses: Cc:

Description

RSS1.0 feeds contain invalid tags, "category".
For example, a RSS1.0 feed includes following.

  <dc:subject><![CDATA[SomeCategory]]></dc:subject>
  <category><![CDATA[SomeCategory]]></category>

Attachments (1)

wp-includes_feed.php.diff (666 bytes) - added by kztk 7 years ago.
A patch to "wp-includes/feed.php"

Download all attachments as: .zip

Change History (9)

kztk7 years ago

A patch to "wp-includes/feed.php"

comment:1 Nazgul7 years ago

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

You would be correct if we'd be generating RSS 1.0 feeds, but we're generating RSS 0.92 feeds, which do allow <category> tags.

See: http://backend.userland.com/rss092

comment:2 kztk7 years ago

But, this function is called with an argument "rdf" from "wp-include/wp-rdf.php" to generate RSS1.0 feed.
I think it is a problem.

In case of generation of RSS0.92 feed,
get_the_category_rss("rss") is called to generate "category" tags.
So, I think, my change does not change the generation of RSS 0.92 feeds.

comment:3 kztk7 years ago

  • Resolution invalid deleted
  • Status changed from closed to reopened

comment:4 Nazgul7 years ago

  • Milestone set to 2.3 (trunk)

You're right. I hadn't taken RDF into account.

comment:5 Nazgul7 years ago

  • Summary changed from RSS1.0 feeds contain invalid tags, "category". to RDF feeds contain invalid tags, "category".

comment:6 Nazgul7 years ago

  • Keywords commit added

comment:7 mdawaffe7 years ago

  • Keywords dev-reviewed added

+1

comment:8 westi7 years ago

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

(In [6001]) Fix RDF feeds to not contain invalid tag "category". Fixed #4643 props kztk

Note: See TracTickets for help on using tickets.