Closed
Bug 101640
Opened 23 years ago
Closed 23 years ago
if necessary, update selection in three pane after an account is removed
Categories
(SeaMonkey :: MailNews: Message Display, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: sspitzer, Assigned: sspitzer)
References
Details
Attachments
(2 files)
7.90 KB,
patch
|
Details | Diff | Splinter Review | |
8.05 KB,
patch
|
cavin
:
review+
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
if necessary, update selection in three pane after an account is removed select a folder on a server, then go remove that account. we should update the folder pane selection. we should go select the default account. (cavin needs this fix for bug #14295). when that lands, he can use my fix to adjust selection if the username / hostname on an imap server)
Assignee | ||
Comment 1•23 years ago
|
||
Assignee | ||
Comment 2•23 years ago
|
||
cavin, after this lands, you'll need to do something like this for #14295, in nsMsgIncomingServer.cpp, +#include "nsIMsgAccountManager.h" then, in SetRealHostname() and SetRealUsername(): + // because the real username (or real hostname) has changed + // we need to notify any listeners + nsCOMPtr<nsIMsgAccountManager> accountManager = + do_GetService(NS_MSGACCOUNTMANAGER_CONTRACTID, &rv); + NS_ENSURE_SUCCESS(rv, rv); + + rv = accountManager->NotifyServerChanged(this); + NS_ENSURE_SUCCESS(rv, rv); + cavin / bienvenu, can you review my patch?
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•23 years ago
|
||
Updated•23 years ago
|
Attachment #50802 -
Flags: superreview+
Comment 4•23 years ago
|
||
Comment on attachment 50802 [details] [diff] [review] updated patch, change selection for both imap and nntp servers r=cavin.
Attachment #50802 -
Flags: review+
Assignee | ||
Comment 5•23 years ago
|
||
fixed. this should unblock cavin now so he can finish #14295
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 6•23 years ago
|
||
I just tried the fix on my tree and it worked perfectly! Seth, thanks for the support.
Using build 2001-09-26 on winxp, linux and mac I don't see the fix in the branch. This was fixed on the trunk only, verify on trunk.
Comment 8•23 years ago
|
||
Trunk build 2001-11-13: WinMe Trunk build 2001-11-09: Linux RH7.1, Mac 9.1 Verified Fixed.
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•