Closed Bug 287284 Opened 19 years ago Closed 19 years ago

Invalid RSS Feeds are not removed correctly

Categories

(MailNews Core :: Feed Reader, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

References

Details

Attachments

(1 file)

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)
Status: NEW → ASSIGNED
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)
Blocks: 255834
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Component: RSS → Feed Reader
Product: Thunderbird → MailNews Core
Component: RSS → Feed Reader
Product: Thunderbird → MailNews Core
You need to log in before you can comment on or make changes to this bug.