Closed
Bug 266536
Opened 20 years ago
Closed 20 years ago
selecting a local mail folder with an out of date db doesn't display contents of folder
Categories
(MailNews Core :: Database, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Bienvenu, Assigned: Bienvenu)
Details
(Keywords: fixed-aviary1.0, regression)
Attachments
(1 file)
22.13 KB,
patch
|
mscott
:
superreview+
|
Details | Diff | Splinter Review |
If you click on a local folder with an out of date db, we're supposed to reparse the folder and then display the contents of the folder. We currently reparse the folder but we don't display the contents until you click away and click back.
Assignee | ||
Updated•20 years ago
|
Status: NEW → ASSIGNED
Keywords: regression
Assignee | ||
Comment 1•20 years ago
|
||
this patch reworks our handling of out of date local folder summary files. You now have to explicitly call GetDatabaseWithReparse in order to get a db to reparse - the new interface also allows you to attach a listener to this process so you can get notified of when the reparsing is done, and then use the db. I've fixed a couple more problems with out of date db's - one was that we weren't restoring the sort order and sort type correctly, and the other was that if you start up with an out of date Inbox.msf, we weren't waiting for the folder to get reparsed before we tried to download mail into it, producing an alert to the effect that the folder is busy. And I've cleaned up some other little problems as well...
Assignee | ||
Comment 2•20 years ago
|
||
Comment on attachment 163744 [details] [diff] [review] proposed fix I've also changed the name of the aUpgrading param to nsMsgDatabase::Open - it's now called aLeaveInvalidDB, which means you might want to pull data out of it... this work is also part of fixing the deletion of rss feeds on linux/mac.
Attachment #163744 -
Flags: superreview?(mscott)
Comment 3•20 years ago
|
||
Comment on attachment 163744 [details] [diff] [review] proposed fix david, do you need to change mailnews\commandglue.js too?
Attachment #163744 -
Flags: superreview?(mscott) → superreview+
Assignee | ||
Comment 4•20 years ago
|
||
yes, I do. I'm not finished with the seamonkey fixes yet, but it should be the exact same fix for seamonkey.
Assignee | ||
Updated•20 years ago
|
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•