Skip to:

Opened 12 years ago

Closed 11 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 12 years ago.
Reverses [2374] and implements ent2ncr() via filters.

Download all attachments as: .zip

Change History (21)

#1 @sambauers
12 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
12 years ago

  • Milestone set to 1.0.3

#3 @Nightgunner5
12 years ago

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

#4 @_ck_
12 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?

12 years ago

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

#5 @GautamGupta
12 years ago

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

#6 @chrishajer
12 years ago

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

#7 @chrishajer
12 years ago

  • Status changed from new to assigned

#8 @chrishajer
12 years ago

What would be a good way to test this?

#9 @GautamGupta
12 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
12 years ago

  • Keywords tested added; needs-testing removed

#11 @chrishajer
12 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
12 years ago

Did you reverse [2374] on your install?

#13 @chrishajer
12 years ago

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

Is there more to do?

#15 @chrishajer
12 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
12 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
12 years ago

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

Merging feed tickets into one ticket: #1261

#18 @chrishajer
12 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
11 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

#20 @GautamGupta
11 years ago

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