Closed
Bug 287284
Opened 19 years ago
Closed 19 years ago
Invalid RSS Feeds are not removed correctly
Categories
(MailNews Core :: Feed Reader, defect)
MailNews Core
Feed Reader
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: iannbugzilla, Assigned: iannbugzilla)
References
Details
Attachments
(1 file)
869 bytes,
patch
|
mscott
:
review+
mscott
:
superreview+
|
Details | Diff | Splinter Review |
Using Build ID 2005031105 on WinXP SP2 Steps to reproduce 1. Add a News & Blogs account 2. Click on account in account settings 3. Click on Manage Subscriptions 4. Click on Add Feed 5. Enter http://www.mozillazine.org/contents.txt Actual results 1. Popup saying "http://www.mozillazine.org/contents.txt is not a valid RSS feed." 2. Following message appears in JS console: Error: url is not defined Source File: chrome://messenger-newsblog/content/Feed.js Line: 171 3. When try to add same invalid feed again, no error message pops up. Expected results 1. Popup saying "http://www.mozillazine.org/contents.txt is not a valid RSS feed." 2. Trying again gives the same popup.
Corrects url to aFeed.url
Attachment #178283 -
Flags: review?(mscott)
Updated•19 years ago
|
Attachment #178283 -
Flags: superreview+
Attachment #178283 -
Flags: review?(mscott)
Attachment #178283 -
Flags: review+
Comment on attachment 178283 [details] [diff] [review] Patch v0.1 (Checked in) Checking in Feed.js; /cvsroot/mozilla/mail/extensions/newsblog/content/Feed.js,v <-- Feed.js new revision: 1.37; previous revision: 1.36
Attachment #178283 -
Attachment description: Patch v0.1 → Patch v0.1 (Checked in)
You need to log in
before you can comment on or make changes to this bug.
Description
•