#2290 closed defect (fixed)
Invalid RSS feed for timeline
Reported by: | 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: | |||
Internal 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->name ^ line 177, column 118: XML parsing error: <unknown>:177:118: mismatched tag ... Moraes Holschuh <hmh@debian.org></title> ^
The offending item:
<item> <title>Ticket <em title="ath/if_ath.c: Incorrect use of dev->name hal; regdomain likely 96 country code 62">#96 (defect) created by Henrique de Moraes Holschuh <hmh@debian.org></title> <author>Henrique de Moraes Holschuh <hmh@debian.org></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-&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)
Change History (6)
by , 19 years ago
Attachment: | timeline-rss.txt added |
---|
comment:1 by , 19 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 , 19 years ago
Milestone: | 0.9.1 → 0.9.2 |
---|
comment:3 by , 19 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 , 19 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
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 , 19 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Fixed in 0.9-stable in [2724].
plain text version of the RSS feed that was tested by feedvalidator.org