Closed Bug 350365 Opened 14 years ago Closed 2 years ago
Malformed feeds look empty
When you open a malformed feed, you get an error from here: http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/toolkit/components/feeds/src/FeedProcessor.js&rev=1.13&mark=1177#1176 reported to Error console, and the displayed feed page looks broken. There should be a user-visible error message displayed instead.
IE7 refuses to display this, good for them. We shouldn't have JS Console errors here. It's up to management to decide whether we want to show a partial feed or throw our hands up similarly.
Assignee: nobody → sayrer
Mike, could we get an opinion on what to do here? We have a reliable indication that the feed has fatal errors.
Attachment #237659 - Flags: review?(mconnor) → review+
Upping to MAJOR. It is plain bad if we leave a user staring at an empty prettyprint page without informing him the feed broke.
Severity: normal → major
*** Bug 363490 has been marked as a duplicate of this bug. ***
Robert: Is there a reason your patch did not get checked in? I would err on the side of displaying something letting the user know that the feed is malformed.
(In reply to comment #8) > Robert: Is there a reason your patch did not get checked in? I would err on the > side of displaying something letting the user know that the feed is malformed. Yeah, this needs to get fixed. The patch is checked in, it just makes the situation not as bad.
(In reply to comment #9) > (In reply to comment #8) > > Robert: Is there a reason your patch did not get checked in? I would err on the > > side of displaying something letting the user know that the feed is malformed. > > Yeah, this needs to get fixed. The patch is checked in, it just makes the > situation not as bad. > As mentioned in Bug 381799, we need to fix this single line. http://lxr.mozilla.org/mozilla1.8/source/browser/components/feeds/content/subscribe.xhtml#47 From my point of view this are two bugs, fixing the js console error and fixing the user interface. I suggest to reopen 381799 for interface change and using this to resolve js console issues.
Seems that we simply never reach the following line which would remove the style="display: none;" mentioned by comment #11: http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/browser/components/feeds/src/FeedWriter.js&rev=220.127.116.11&mark=366-370#343 Fixing the error from comment #0 might/should already make this happen.
This will need l10n strings, most likely. Assigning to beltzner or his delegate, so we can figure out what should happen for feeds we can't parse. IE7 and Safari both show "we give up" screens.
Assignee: sayrer → beltzner
Target Milestone: --- → Firefox 3
Blocking (and tackling!)
Flags: blocking-firefox3? → blocking-firefox3+
When adding http://news.hereisthecity.com/news/business_news_rss091.xml Live Bookmark tells me the feed was empty. Since the feed is working properly with other RSS-readers I was wondering whether it is part of this bug. PS: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a7pre) Gecko/2007071711 Minefield/3.0a7pre
Alex, can you pull together some sort of UI for this failure state? Not a feature, doesn't need to make M8
Assignee: beltzner → faaborg
Target Milestone: Firefox 3 M8 → Firefox 3 M9
I'll work on the UI for this after M8
Priority: P5 → P4
Target Milestone: Firefox 3 Mx → Firefox 3 M11
Taking off the "this._reader has no properties" since it's no longer true, and aligning the milestone with the reality that we didn't choose a few words of text before freezing.
Summary: Malformed feeds look empty; "Error: this._reader has no properties" in JS console → Malformed feeds look empty
Target Milestone: Firefox 3 beta3 → Future
Any update on this bug? Alex?
OS: Windows XP → All
Hardware: x86 → All
This fell off my radar, not sure if we can fix it given the string freeze, but I'll get the text ready so we can fix this on trunk.
Alex, what is the status on this?
in the middle of a unfortunately very large uiwanted queue. Just put up an error icon (the red one with the X), and some temporary text, and we can clean the text up in ui-review.
Assignee: faaborg → nobody
The feed preview code has been removed by bug 1477669.
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.