Opened 18 years ago
Closed 17 years ago
#4321 closed defect (bug) (fixed)
Feed validation error with html in categories
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 2.7 | Priority: | low |
Severity: | minor | Version: | 2.2 |
Component: | General | Keywords: | rss has-patch |
Focuses: | Cc: |
Description
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. & Régl.]]></category>
Attachments (2)
Change History (11)
#2
@
18 years ago
- Milestone set to 2.3
- Priority changed from normal to low
- Severity changed from normal to minor
#5
@
18 years ago
- Keywords needs-patch added; has-patch removed
- Owner changed from anonymous to westi
- Status changed from new to assigned
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&somethingelse]]></category>
Should we be converting the & back to & here?
#7
@
17 years ago
- Keywords has-patch added; needs-patch removed
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.
Note: See
TracTickets for help on using
tickets.
That particular error is only a warning, not a feed breaker. It doesn't make it fail validation, just makes it give the warning.