[Trac-tickets] [The Trac Project] #2580: RSS feed validation in 0.9.3

[The Trac Project]

#2580: RSS feed validation in 0.9.3
----------------------+-----------------------------------------------------
 Reporter:  osimons   |       Owner:  jonas
     Type:  defect    |      Status:  new  
 Priority:  normal    |   Milestone:       
Component:  timeline  |     Version:  0.9.2
 Severity:  normal    |    Keywords:       
----------------------+-----------------------------------------------------
 Upgraded to 0.9.3 and have found 2 problems with gettings timeline feeds
 validated in both RSS Bandit and Sage - and both Internet Explorer and
 Firefox also confirm "XML Parsing Error: not well-formed".

 1. /trunk/trac/wiki/formatter.py in line 685-687 (end of !OneLineFormatter
 format method): Three dots are converted to {{{…}}}. Some feed
 readers do not like this, especially if they use validating xml
 components, as there is no DTD for the Trac RSS XML, and the hellip entity
 is not recognized. A quick workaround(fix?) is to use the numeric entity
 reference instead, which is: …. Works for us. As far as I can see,
 this code has not changed from 0.9.2-0.9.3, but new Marker/escape/sanitise
 handling have probably introduced new ways of handling the output.

 2. That fixed all my projects except one. The problem here is located to
 be the an entry in a Subversion log message (a changeset timeline event)
 that is based on a log link to a start-stop revision list for a folder
 {{{[log:..mysource/folder at 34:45]}}}. The HTML for this gets converted to
 {{{href="..mysource/folder?rev=45&stop_rev=34}}} without escaping the HTML
 ampersand. The feed will not validate. Manually replacing {{{&}}} with
 {{{&}}} makes the feed validate without errors. The output from the
 log link seems to start with _format_link method at the end of
 [source:/trunk/trac/versioncontrol/web_ui/log.py], but with Href,
 Formatter and Markup etc. on it's path to output I have to leave it to
 someone who knows the source better.

 PS! Wanted to, but cannot, set version to 0.9.3 for this ticket :-)

-- 
Ticket URL: <http://projects.edgewall.com/trac/ticket/2580>
The Trac Project <http://trac.edgewall.com/>