Closed
Bug 347080
Opened 18 years ago
Closed 18 years ago
Open cross-folder saved search folder doesn't show new messages that arrive in folders that didn't have matching hits when view opened
Categories
(MailNews Core :: Backend, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Bienvenu, Assigned: Bienvenu)
Details
(Keywords: fixed1.8.1)
Attachments
(1 file, 1 obsolete file)
10.67 KB,
patch
|
mscott
:
superreview+
|
Details | Diff | Splinter Review |
If you open a cross folder saved search folder, and one of the searched folders doesn't have any hits when you open the saved search folder, if new messages are added to the searched folder that should match the saved search, those messages aren't displayed in the saved search view until you go to an other folder and back to the saved search. The problem is that we don't add a db listener for the folders w/o hits when we open the folder.
For saved searches that search over a bunch of folders, but usually only have hits on a small subset of folders, we might want to use a pending listener, so that we don't hold the db's open...but that would require adding the listeners after the search is done.
Assignee | ||
Comment 1•18 years ago
|
||
use pending listeners for all the folders - this is a slightly scary change, so I'm going to run with it. The nice thing about pending listeners is that they allow you to passively listen to changes for a folder, w/o holding open the db. The db will close if no one is holding onto it, and your listener will re-attach when someone opens the db.
Assignee | ||
Comment 2•18 years ago
|
||
as described earlier, this uses pending listeners for virtual folders. It also cleans up some comments, and other miscellaneous stuff.
Attachment #231863 -
Attachment is obsolete: true
Attachment #232827 -
Flags: superreview?(mscott)
Updated•18 years ago
|
Attachment #232827 -
Flags: superreview?(mscott) → superreview+
Assignee | ||
Comment 3•18 years ago
|
||
fixed on trunk and branch - this might solve some mysteries cross-folder virtual folders were seeing.
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•