292229 - virtualFolders.dat lost if Thunderbird is opened from mailto: link

Status: RESOLVED FIXED

(Thunderbird :: General, defect)

Description

[Anders Betnér]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.7) Gecko/20050414 Firefox/1.0.3
Build Identifier: Thunderbird version 1.0.2 (20050317) (english)

If Thunderbird is closed and you click on a mailto: link in Firefox the
virtualFolders.dat gets truncated.

Reproducible: Always

Steps to Reproduce:
1. Make sure Thunderbird is closed and set as you default mailer in windows.
2. Click on a "mailto:" link on a page in Firefox.
3. Close the message and choose "Don't save"

Actual Results:  
virtualFolders.dat contains the single line "version=1"

Expected Results:  
virtualFolders.dat should contain all my folder definitions

Comment 1

[Anders Betnér]

I've just tried the same procedure on the latest trunk build (2005-05-27) and it
shows exactly the same behaviour. All folders are removed from
virutalFolders.dat. This really should be fixed before release. I have two
different imap accounts with different hosts defined and there are mails in
local folders imported from outlook 2003.

Comment 2

[Scott MacGregor]

wow I was able to reproduce this. I didn't think I would :).

Using 05/27 nightly build. Create a saved search folder. Quit. Click on a mailto
url, then close the compose window after it comes up. The next time you start up
thunderbird, your saved search folder is gone.

Comment 3

[David :Bienvenu]

Anders, I hate to ask, but can you try with an even more recent trunk build?

Comment 4

[Anders Betnér]

No problem.
Still happens with trunk 2005-06-14

Comment 5

[David :Bienvenu]

I can recreate this - it's "only" imap virtual folders that are lost, so the
problem seems to be that we haven't discovered imap folders at all in this
scenario, and thus don't write them out.

Comment 6

[David :Bienvenu]

Attachment: proposed fix

set a bool that says we've loaded the virtual folders, and if we haven't, don't
write out virtual folders. There are probably less direct ways of doing this,
like making sure virtual folders are always loaded when we start up, but this
seems the most robust. I checked that this works when you start up w/o virtual
folders as well...