feeds failes rendering if the rss item doesn't have summary

RESOLVED FIXED in Firefox 2 alpha2

Status

()

Firefox
RSS Discovery and Preview
--
major
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: masayuki, Assigned: masayuki)

Tracking

Trunk
Firefox 2 alpha2
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

See this rss.
http://www.d-toybox.com/studio/weblog/show.php?mode=rss&ver=2.0

Fx cannot render all items in the RSS file.
Created attachment 221222 [details] [diff] [review]
Patch rv1.0
Attachment #221222 - Flags: review?
Created attachment 221223 [details] [diff] [review]
Patch rv1.0 (-w)
(Assignee)

Updated

12 years ago
Status: NEW → ASSIGNED
(Assignee)

Updated

12 years ago
Attachment #221222 - Flags: review? → review?(bugs)
(Assignee)

Updated

12 years ago
Attachment #221222 - Flags: approval-branch-1.8.1?(bugs)
You could just use |if (summary && summary.length > MAX_CHARS)|
(In reply to comment #3)
> You could just use |if (summary && summary.length > MAX_CHARS)|

yeah, but the creating text node is not needed if the summary is null.
(Assignee)

Updated

12 years ago
Attachment #221222 - Flags: review?(bugs)
Attachment #221222 - Flags: approval-branch-1.8.1?(bugs)
You need to log in before you can comment on or make changes to this bug.