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)

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: 20 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: