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

RESOLVED FIXED

Status

MailNews Core
Database
RESOLVED FIXED
13 years ago
10 years ago

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Tracking

({fixed-aviary1.0, regression})

Trunk
x86
Windows 2000
fixed-aviary1.0, regression

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

13 years ago
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

13 years ago
Status: NEW → ASSIGNED
Keywords: regression
(Assignee)

Comment 1

13 years ago
Created attachment 163744 [details] [diff] [review]
proposed fix

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

13 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

13 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

13 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

13 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 13 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.