If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

shouldn't show deferred accounts in search dialog

RESOLVED FIXED

Status

Thunderbird
General
RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: nicolas chevreux, Assigned: Bienvenu)

Tracking

({fixed-aviary1.0})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

13 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040707 Firefox/0.9.2
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040707 Firefox/0.9.2

When using the new Thunderbird builds with the global inbox, the list of folders
to seach in contains folders that do not exist.
For example, if all acount resolves to the "Local Folders" folder (i.e., if each
account does *not* have his own folder), each account is still listed, with his
"sent", "draft"... subfolders. And this is the case even when these folders
actually do not exist.

Reproducible: Always
Steps to Reproduce:
1. get the Thunderbird build with the global inbox
2. create 2 accounts
3. Tools -> Account Settings -> account1 - > Server settings -> advanced: click
on "Global inbox".
4. Tools -> Account Settings -> account1 - > Server settings -> advanced: click
on "Global inbox".
5. organize your folders so that you have only "Local folders", and no folder
for account1 and account2.
6. right click on a folder -> search messages
Actual Results:  
the drop down menu on top of the search windows displays "account1 - inbox,
drafts, sent...", "acount2 - inbox, drafts, sent...", "Local folders - inbox,
drafts, sent...", even though the account1 and account2 folders do not exists.

Expected Results:  
only existing folders should be displayed.

Comment 1

13 years ago
I can confim this.

It also shows up non-existing folders in the junk mail filter dialog.

Comment 2

13 years ago
Using version 0.8 (20040902) by the way.

Comment 3

13 years ago
David, does it make sense for us to hide the hidden folders for folder menu
lists like the search dialog just like we do for the folder pane?
(Assignee)

Comment 4

13 years ago
yes, we should hide them. I can look into that. They do exist, and can contain
messages, but we'll not show them to you, so we should just pretend they don't
exist.

Comment 5

13 years ago
cool, you might find:

msgFolderPickerOverlay.xul to be of help. I think all of our folder list widgets
get stuff from this overlay.
(Assignee)

Comment 6

13 years ago
I'm not sure about the junk mail dialog, however - those accounts do exist, and
we might use the per-account junk mail controls, but I think we use the local
folders junk controls. We don't show the deferred accounts as possible move
targets, which is correct.
Summary: global inbox: "folders to searh in" badly displayed → shouldn't show deferred accounts in search dialog
(Assignee)

Comment 7

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

why change xul when I can change c++? :-) We want deferred servers to not be
searchable in general...
(Assignee)

Updated

13 years ago
Assignee: mscott → bienvenu
Status: UNCONFIRMED → ASSIGNED
(Assignee)

Updated

13 years ago
Attachment #157836 - Flags: superreview?(mscott)

Comment 8

13 years ago
Comment on attachment 157836 [details] [diff] [review]
proposed fix

Let's put this on aviary too
Attachment #157836 - Flags: superreview?(mscott) → superreview+
(Assignee)

Updated

13 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Keywords: fixed-aviary1.0
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.