Skip to:

Opened 6 years ago

Closed 5 years ago

#1181 closed defect (duplicate)

RSS feeds not validating

Reported by: tomdebruin Owned by: chrishajer
Milestone: 1.0.3 Priority: normal
Severity: normal Version: 1.0.2
Component: Front-end Keywords: has-patch tested


There's an issue with RSS feeds not validating. The problem appears to stem from » in the titles (rss.php) so I swapped them all with ::.
It now validates. It did work fine in Safari, but Firefox and IE threw up errors.


There's also a recommendation to not use <textInput>.

Attachments (1)

full-rss-filters.patch (5.7 KB) - added by sambauers 6 years ago.
Reverses [2374] and implements ent2ncr() via filters.

Download all attachments as: .zip

Change History (21)

#1 @sambauers
6 years ago

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

(In [2374]) Put the RSS feed description into CDATA. Fixes #1181

#2 @sambauers
6 years ago

  • Milestone set to 1.0.3

#3 @Nightgunner5
6 years ago

Suggestion: Change all &raquo; to &#187; (and other non-numeric entities to their numeric counterpart)

#4 @_ck_
6 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

I would highly suggest a better solution than CDATA (which is a lazy workaround at best).

CDATA looks ugly in some readers or isn't parsed.
Doesn't bbPress already have a few entity encoding routines from wordpress/backpress? Is CDATA how WordPress solves it?

6 years ago

Reverses [2374] and implements ent2ncr() via filters.

#5 @GautamGupta
6 years ago

  • Keywords has-patch needs-testing added
  • Version changed from 1.0-rc-3 (trunk) to 1.0.2

#6 @chrishajer
6 years ago

  • Owner set to chrishajer
  • Status changed from reopened to new

#7 @chrishajer
6 years ago

  • Status changed from new to assigned

#8 @chrishajer
6 years ago

What would be a good way to test this?

#9 @GautamGupta
6 years ago

When you have applied this patch on your install, then post some entities in your post text like &raquo; etc., full list can be found here (the left side values), and then check your feed from here -

#10 @GautamGupta
6 years ago

  • Keywords tested added; needs-testing removed

#11 @chrishajer
6 years ago

Without the patch, my trunk feed validates. Is there somewhere I can see this feed as invalid? This also mentions "title" not post body.

#12 @GautamGupta
6 years ago

Did you reverse [2374] on your install?

#13 @chrishajer
6 years ago

Not explicitly, I don't think. I just applied this patch:

Is there more to do?

#15 @chrishajer
6 years ago

I think there are still issues with this one.

  1. CDATA is still being used in several places after applying this patch. Check here:

If it is a lazy workaround as _ck_ suggests, why isn't it lazy to keep it in these places too? Why not remove it everywhere?

  1. Is there a good reason to leave in the <textInput>?

  1. I am getting some weird characters in that feed. It looks like something is converting the opening paragraph tags to

and closing ones to


I imagine if I had other HTML in the replies that the open and close (greater and less than) would be converted, incorrectly as well. Also, what about bbcode if the forum allows that? Maybe that's handled properly by being processed first and converted the html, but then I think the < and > would be converted. to 60 and 62

yeah, check this, it's a mess:

This is not good enough yet I don't think.

#16 @chrishajer
6 years ago

We could always just remove raquo from all the php files and replace with the numeric entity. Would that resolve this ticket instead of the existing patch? I could test using html and entities in a post and see what happens.

#17 @GautamGupta
6 years ago

  • Resolution set to duplicate
  • Status changed from assigned to closed

Merging feed tickets into one ticket: #1261

#18 @chrishajer
6 years ago

  • Resolution changed from duplicate to fixed

(In [2420]) RSS feed validation code changes. (You will need to update any custom theme.) Fixes #1181. Props sambauers, GautamGupta

#19 @GautamGupta
5 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

#20 @GautamGupta
5 years ago

  • Resolution set to duplicate
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.