Closed Bug 266536 Opened 21 years ago Closed 21 years ago

selecting a local mail folder with an out of date db doesn't display contents of folder

Categories

(MailNews Core :: Database, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Details

(Keywords: fixed-aviary1.0, regression)

Attachments

(1 file)

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.
Status: NEW → ASSIGNED
Keywords: regression
Attached patch proposed fixSplinter Review
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...
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 on attachment 163744 [details] [diff] [review] proposed fix david, do you need to change mailnews\commandglue.js too?
Attachment #163744 - Flags: superreview?(mscott) → superreview+
yes, I do. I'm not finished with the seamonkey fixes yet, but it should be the exact same fix for seamonkey.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Keywords: fixed-aviary1.0
Resolution: --- → FIXED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: