Closed
Bug 287291
Opened 19 years ago
Closed 19 years ago
Removing then adding RSS account with same details gives blank account name
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)
1.07 KB,
patch
|
mscott
:
review+
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
Using BuildID 2005031105 on WinXP SP2 Steps to reproduce 1. Have a News & Blog Account already set up 2. Go into account settings 3. Highlight and remove News & Blog account 4. Add new News & Blog account with same details 5. Highlight new News & Blog account Expected results 1. Fields are correctly populated Actual results 1. No fields are populated until account settings is closed and reopened
Problem here is that because the entry for the account is set to null then the serverId still exists in the array thus getValueArrayFor(currentServerId) returns null instead of creating a non-null new entry in the array and returning that. This patch deletes the entry instead of setting to null.
Attachment #178295 -
Flags: review?(mscott)
Updated•19 years ago
|
Attachment #178295 -
Flags: review?(mscott) → review+
Attachment #178295 -
Flags: superreview?(bienvenu)
Updated•19 years ago
|
Attachment #178295 -
Flags: superreview?(bienvenu) → superreview+
Comment on attachment 178295 [details] [diff] [review] Patch v0.1 (Checked in) Checking in AccountManager.js; /cvsroot/mozilla/mailnews/base/prefs/resources/content/AccountManager.js,v <-- AccountManager.js new revision: 1.112; previous revision: 1.111 done
Attachment #178295 -
Attachment description: Patch v0.1 → Patch v0.1 (Checked in)
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•