Closed Bug 123110 Opened 23 years ago Closed 22 years ago

labels don't stick after "building summary for X"

Categories

(MailNews Core :: Filters, defect)

x86
Windows 98
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
mozilla1.0

People

(Reporter: sahaby, Assigned: naving)

Details

Attachments

(1 file)

From Bugzilla Helper:
User-Agent: Mozilla/4.0 (compatible; MSIE 5.0; Windows 98; DigExt)
BuildID:    2002012403

after MozMail rebuilds the summary for a specific folder (like Inbox for 
instance), all of the labels assigned to messages via filters disappear. All 
messages that were labeled via a filter revert to black but all of the ones 
that I specifically assign retain their colours (and labels).

Reproducible: Always
Steps to Reproduce:
1. Create a filter that changes the label property
2. Catch a mail that triggers that filter
3. Wait for a "Building Summary of Inbox (or whatever)"

Actual Results:  The mail that triggered that filter loses its label.

Expected Results:  That e-mail should retain its label.

I left the severity as normal, but I guess it could be considered a "dataloss" 
bug, since I am technically losing data, but its not that big of data....
sounds like the filter labelling code is just changing the msgHdr flags without
setting the correct mozilla status line flags. Let me know if you want me to
look at this, Navin.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: nsbeta1
Discussed in 2/27/02 Mail & News bug meeting.  Decisions was to plus this bug.
Keywords: nsbeta1nsbeta1+
Target Milestone: --- → mozilla1.0
Attached patch proposed fixSplinter Review
The fix is to use db to set the label so that it updates the moz-status line in

mbox. david, need review.
Comment on attachment 72827 [details] [diff] [review]
proposed fix

r=bienvenu
Attachment #72827 - Flags: review+
Comment on attachment 72827 [details] [diff] [review]
proposed fix

sr=mscott
Attachment #72827 - Flags: superreview+
Comment on attachment 72827 [details] [diff] [review]
proposed fix

a=shaver for 1.0 trunk.
Attachment #72827 - Flags: approval+
fix checked in for navin.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
OK using mar26 commercial trunk: win98, linux rh6.2
OK mar26 commercial trunk build mac OS 9.2
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: