WordPress.org

Make WordPress Core

Opened 9 years ago

Last modified 3 months ago

#8828 new enhancement

Tags and Categories are undifferentiable in Atom and RSS feeds

Reported by: znarfor Owned by:
Milestone: Future Release Priority: low
Severity: minor Version: 2.7
Component: Feeds Keywords: has-patch needs-refresh
Focuses: Cc:

Description (last modified by DrewAPicture)

In RSS feeds this can be achieved using the 'domain' attributes of the category element. This is already done in WordPress export through the wxr_post_taxonomy function, so we just have to do the same in RSS feeds.

In Atom feeds this can be achieved using the 'scheme' attributes of the category element. The Atom standard is not explicit about what to put in this scheme attributes. There is an interesting topic on Edward O’Connor blog. In my proposed patch, i decided to use the 1st representation and i also considered seriously the 2nd.

In the patch, i'm also adding the 'label' attribute to the atom category element and changed the 'term' attribute to be the tag slug. Seems this are currently the best practices in the Atom community.

Attachments (2)

wp-feed-category.diff (2.1 KB) - added by znarfor 9 years ago.
i tried to not modify too much the function to improve the patch readability, but i think this function can be refactorised somehow after.
wp-feed-category-2.diff (2.3 KB) - added by znarfor 9 years ago.
update of my previous patch

Download all attachments as: .zip

Change History (13)

@znarfor
9 years ago

i tried to not modify too much the function to improve the patch readability, but i think this function can be refactorised somehow after.

#1 @znarfor
9 years ago

  • Keywords has-patch added

#2 @peaceablewhale
9 years ago

#9236 attempts to fix the same issue.

#3 @Denis-de-Bernardy
9 years ago

  • Keywords needs-patch added; has-patch removed
  • Milestone changed from 2.8 to Future Release

broken patch

#4 @Denis-de-Bernardy
9 years ago

  • Component changed from General to Feeds
  • Owner anonymous deleted

#5 @peaceablewhale
9 years ago

  • Keywords needs-patch removed

This report will be fixed if the patch from #9510 is accepted.

@znarfor
9 years ago

update of my previous patch

#6 @znarfor
9 years ago

  • Cc znarf@… added
  • Keywords has-patch needs-review added
  • Milestone changed from Future Release to 2.9

The patch had been updated for 2.9-rare (trunk).

#7 @janeforshort
9 years ago

  • Milestone changed from 2.9 to Future Release

Lack of community testing on new patch > punting due to being in beta.

#8 @jeremyfelt
4 years ago

  • Keywords needs-refresh added
  • Priority changed from normal to low
  • Severity changed from normal to minor

I was doing something with RSS feeds recently that required the addition of the domain attribute to pass terms from a custom taxonomy. It would be nice if the default taxonomies fell in line with this as well. I would imagine that leaving domain off of categories makes sense, as the element name is <category>. We could add domain="post_tag" to tags that are output in the feed.

I haven't tested the last patch on this ticket, which I'm guessing needs to be reworked a bit. Seems fairly straight forward and the RSS side of things.

#9 @chriscct7
3 years ago

  • Keywords needs-review removed

#10 @stevenkword
11 months ago

  • Keywords good-first-bug added

#11 @DrewAPicture
3 months ago

  • Description modified (diff)
  • Keywords good-first-bug removed
Note: See TracTickets for help on using tickets.