Closed
Bug 790554
Opened 12 years ago
Closed 12 years ago
feed-parser.js raises an error with buggy RSS feeds; Error: linkElements is null
Categories
(MailNews Core :: Feed Reader, defect)
Tracking
(thunderbird16 fixed, thunderbird17 fixed)
RESOLVED
FIXED
Thunderbird 18.0
People
(Reporter: kawamoto, Assigned: alta88)
References
()
Details
Attachments
(1 file, 1 obsolete file)
1.86 KB,
patch
|
mconley
:
review+
standard8
:
approval-comm-aurora+
standard8
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:15.0) Gecko/20100101 Firefox/15.0.1 Build ID: 20120905151427 Steps to reproduce: Following feed URL does not work anymore. There is a RSS feed of the Google Hot Trends in my News Feeds subscription list: http://www.google.com/trends/hottrends/atom/hourly This feed URL is not legitimate, invalid, or buggy, but very useful. And this URL worked well up to Thunderbird 14. Actual results: When I update the feed (File -> Get New Messages for -> Blogs & News Feeds), a following error is raised in the Error Console: Timestamp: 9/12/12 5:42:14 PM Error: linkElements is null Source File: chrome://messenger-newsblog/content/feed-parser.js Line: 597 This may be caused by the buggy URL, but we should accept various feed URLs gently. Expected results: Hourly Google Hot Trends top 10 should come! This bug will be fixed by the following patch. Enjoy!
Updated•12 years ago
|
Component: General → Feed Reader
Product: Thunderbird → MailNews Core
alta88 can you look at the patch what it is trying to accomplish? And it isn't in the proper format yet. TB18 refuses to add this feed in the Subscription dialog.
Updated•12 years ago
|
Attachment #660355 -
Attachment is patch: true
Attachment #660355 -
Flags: feedback?(alta88)
Comment 2•12 years ago
|
||
Confirming on trunk: Thu Sep 13 2012 00:22:12 Error: linkElements is null Source file: chrome://messenger-newsblog/content/feed-parser.js Line: 635
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Mac OS X → All
Hardware: x86 → All
thanks kawamoto, but the patch isn't the right way to avoid the lack of check. returning/checking for null is inherent to other parsing logic. fix included. it may be a year before it's released though.
Assignee: nobody → alta88
Attachment #660355 -
Attachment is obsolete: true
Attachment #660355 -
Flags: feedback?(alta88)
Attachment #660489 -
Flags: review?(mconley)
Reporter | ||
Comment 4•12 years ago
|
||
Hi alta88, thank you for the fix. It's OK for me about release date. Thanks again, Yoshihisa Kawamoto
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WORKSFORME
Yoshihisa Kawamoto, please do not close the bug, the correct patch was not yet accepted and merged into Thunderbird code.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Reporter | ||
Comment 6•12 years ago
|
||
Oh, I'm sorry for that.
Comment 7•12 years ago
|
||
Comment on attachment 660489 [details] [diff] [review] patch Review of attachment 660489 [details] [diff] [review]: ----------------------------------------------------------------- Looks good to me. Thanks!
Attachment #660489 -
Flags: review?(mconley) → review+
Keywords: checkin-needed
Comment 8•12 years ago
|
||
https://hg.mozilla.org/comm-central/rev/8fdbcb519211
Status: ASSIGNED → RESOLVED
Closed: 12 years ago → 12 years ago
Flags: in-testsuite?
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 18.0
See also 792657. If this is really a regression starting at TB15, would it be worth it to push into TB16/17 quickly?
Assignee | ||
Comment 11•12 years ago
|
||
Comment on attachment 660489 [details] [diff] [review] patch [Approval Request Comment] Regression caused by (bug #): User impact if declined: certain feed subscribes fail. Testing completed (on c-c, etc.): Risk to taking this patch (and alternatives if risky): zero.
Attachment #660489 -
Flags: approval-comm-beta?
Attachment #660489 -
Flags: approval-comm-aurora?
Updated•12 years ago
|
Attachment #660489 -
Flags: approval-comm-beta?
Attachment #660489 -
Flags: approval-comm-beta+
Attachment #660489 -
Flags: approval-comm-aurora?
Attachment #660489 -
Flags: approval-comm-aurora+
Comment 12•12 years ago
|
||
Checked in: https://hg.mozilla.org/releases/comm-aurora/rev/dc89f93aa4f8 https://hg.mozilla.org/releases/comm-beta/rev/15773ab9a33b
status-thunderbird16:
--- → fixed
status-thunderbird17:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•