Edgewall Software
Modify

Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#2290 closed defect (fixed)

Invalid RSS feed for timeline

Reported by: mrenzmann@… Owned by: Christopher Lenz
Priority: normal Milestone: 0.9.3
Component: timeline Version: 0.9b2
Severity: normal Keywords: rss timeline
Cc: Branch:
Release Notes:
API Changes:

Description

I noticed that the following ticket broke RSS feed validation: http://madwifi.org/ticket/96

Feedvalidator.org reported the following problems:

line 176, column 18: Undefined title element: em

    <title>Ticket <em title="ath/if_ath.c: Incorrect use of dev-&gt;name
                  ^
line 177, column 118: XML parsing error: <unknown>:177:118: mismatched tag

... Moraes Holschuh &lt;hmh@debian.org&gt;</title>
                                             ^

The offending item:

   <item>
    <title>Ticket <em title="ath/if_ath.c: Incorrect use of dev-&gt;name

hal; regdomain likely 96 country code 62">#96 (defect) created by Henrique de Moraes Holschuh &lt;hmh@debian.org&gt;</title>
     <author>Henrique de Moraes Holschuh &lt;hmh@debian.org&gt;</author>
    <pubDate>Mon, 31 Oct 2005 22:00:05 GMT</pubDate>
    <link>http://madwifi.org/ticket/96</link>
    <description>ath/if_ath.c: Incorrect use of dev-&amp;gt;name

hal; regdomain likely 96 country code 62</description>
   </item>

First guess from IRC:

<cmlenz> otaku42: I think that's an error in how the timeline RSS feed removes HTML tags :-/

Attachments (1)

timeline-rss.txt (11.1 KB ) - added by mrenzmann@… 14 years ago.
plain text version of the RSS feed that was tested by feedvalidator.org

Download all attachments as: .zip

Change History (6)

by mrenzmann@…, 14 years ago

Attachment: timeline-rss.txt added

plain text version of the RSS feed that was tested by feedvalidator.org

comment:1 by Christopher Lenz, 14 years ago

The ticket summary causes this is

“ath/if_ath.c: Incorrect use of dev→name hal; regdomain likely 96 country code 62”.

I think the / and > characters are confusing the regexp that removes HTML tags for the RSS feed.

comment:2 by Christopher Lenz, 14 years ago

Milestone: 0.9.10.9.2

comment:3 by Matthew Good, 14 years ago

Actually, it's the newline in the ticket summary that's the offender here. Were these tickets imported from another source? Normally the summaries wouldn't have any newlines since they're entered in a single line input box.

comment:4 by Christopher Lenz, 14 years ago

Owner: changed from Jonas Borgström to Christopher Lenz
Status: newassigned

This should be fixed in trunk since [2685]. I'll mark this ticket as closed once we've merged those changes intro 0.9-stable.

comment:5 by Christopher Lenz, 14 years ago

Resolution: fixed
Status: assignedclosed

Fixed in 0.9-stable in [2724].

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christopher Lenz.
The resolution will be deleted. Next status will be 'reopened'.
to as closed The owner will be changed from Christopher Lenz to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.