Closed
Bug 455932
Opened 17 years ago
Closed 17 years ago
Add nsIMsgFolderListener notifications for news messages
Categories
(MailNews Core :: Backend, defect)
MailNews Core
Backend
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.9.1b1
People
(Reporter: rain1, Assigned: rain1)
References
Details
Attachments
(2 files)
|
8.62 KB,
patch
|
Bienvenu
:
review+
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
|
2.56 KB,
patch
|
Bienvenu
:
review+
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
As it says on the tin :)
- msgAdded is covered in the patch, as is msgsDeleted
- msgsMoveCopyCompleted is handled in the dest folder, and a news folder can't be a dest folder
- folderDeleted is covered by nsMsgDBFolder
|
Assignee | |
Updated•17 years ago
|
Attachment #339315 -
Flags: superreview?(bienvenu)
Attachment #339315 -
Flags: review?(bienvenu)
|
||
Updated•17 years ago
|
Attachment #339315 -
Flags: superreview?(bienvenu)
Attachment #339315 -
Flags: superreview+
Attachment #339315 -
Flags: review?(bienvenu)
Attachment #339315 -
Flags: review+
|
|
Assignee | |
Updated•17 years ago
|
Keywords: checkin-needed
|
|
Assignee | |
Updated•17 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
|
||
Updated•17 years ago
|
Target Milestone: --- → mozilla1.9.1b1
|
|
Assignee | |
Comment 2•17 years ago
|
||
I noticed that there are still more notifications than there should be. This patch batches an entire cleanup of expired/cancelled messages into one notification.
|
|
Assignee | |
Updated•17 years ago
|
Attachment #339449 -
Flags: superreview?(bienvenu)
Attachment #339449 -
Flags: review?(bienvenu)
|
|
||
Comment 3•17 years ago
|
||
Comment on attachment 339449 [details] [diff] [review]
[checked in] Batch more notifications together
it was handy that debug code was there :-)
Attachment #339449 -
Flags: superreview?(bienvenu)
Attachment #339449 -
Flags: superreview+
Attachment #339449 -
Flags: review?(bienvenu)
Attachment #339449 -
Flags: review+
|
|
Assignee | |
Updated•17 years ago
|
Keywords: checkin-needed
|
||
Updated•17 years ago
|
Attachment #339315 -
Attachment description: patch v1 → [checked in] patch v1
|
|
||
Comment 4•17 years ago
|
||
Comment on attachment 339449 [details] [diff] [review]
[checked in] Batch more notifications together
Checked in, changeset id: 385:7c6c4b32c099
Attachment #339449 -
Attachment description: Batch more notifications together → [checked in] Batch more notifications together
|
|
||
Updated•17 years ago
|
Keywords: checkin-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•