Tags and Categories are undifferentiable in Atom and RSS feeds

[znarfor]

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.

[znarfor]

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

[peaceablewhale]

#9236 attempts to fix the same issue.

[Denis-de-Bernardy]

broken patch

[peaceablewhale]

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

[znarfor]

update of my previous patch

[znarfor]

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

[janeforshort]

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

[jeremyfelt]

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.

This code change updates original changes from the trac ticket below, to match the current code from WordPress Trunk. Everything should match what the original patch submitter added, with the exception of switching from attribute_escape() to esc_attr() because of function deprecation.

Trac ticket: See https://core.trac.wordpress.org/ticket/8828

[tw2113]

Atom Feed <category> element before:

<category scheme="https://wpcore.test" term="test category" />

Atom Feed <category> element after:

<category scheme="https://wpcore.test/category/" term="test category" label="test category" />

RSS Feed <category> element before:

<category><![CDATA[test category]]></category>

RSS Feed <category> element after:

<category domain="category"><![CDATA[test category]]></category>