Feed validation error with html in categories

[devil1591]

when I put a "&" in a category name, the feed fails validation on ​http://feedvalidator.org/

example:

line 46, column 41: category should not contain HTML: & [help]
<category><![CDATA[Form. &amp; Régl.]]></category>

[Otto42]

That particular error is only a warning, not a feed breaker. It doesn't make it fail validation, just makes it give the warning.

[Nazgul]

Simple patch attached

[westi]

Nazguls patch does not fix the warning on the validator for me:

line 23, column 51: category should not contain HTML: & [help]
<category><![CDATA[something&#38;somethingelse]]></category>

Should we be converting the & back to & here?

[lildude]

This patch is against the 2.5.1 source

[lildude]

The problem is the category name has had things like the ampersand converted into their HTML Entities. So to clear the warning, it should be sufficient to decode the $cat_name when producing the feed.

Attached patch (4321-new.diff) does exactly that and allows the feed to validate without any warnings.

[lloydbudd]

Replying to Otto42:

That particular error is only a warning, not a feed breaker. It doesn't make it fail validation, just makes it give the warning.

According to #7815, it appears this is a real issue in IE8. Seems like time to finally fix this.

[matt]

(In [9148]) Don't encoded category element contents - fixes #4321 and #7815.