Closed
Bug 410553
Opened 18 years ago
Closed 18 years ago
Message Filters that only "Tag Message" aren't entered in filterlog.html
Categories
(MailNews Core :: Filters, defect)
MailNews Core
Filters
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.9beta3
People
(Reporter: my1stgig, Assigned: mkmelin)
Details
Attachments
(1 file)
|
1.56 KB,
patch
|
mnyromyr
:
review+
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.11) Gecko/20071128 Firefox/2.0.0.4
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.11) Gecko/20071128 Firefox/2.0.0.4
When "Perform these actions:" section of "Filter Rules" dialog contains only a "Tag Message" action, the action is not logged in filterlog.html even though the tag is applied.
Reproducible: Always
Steps to Reproduce:
2.Compose a message with a unique subject and "Save"
3.Copy message from Draft folder to a temporary folder
1.Create message filter "Test" that only applies a tag when subject in (1) is matched
4.From "Tolls"->"Message Filters" dialog, select filter just created and "Run selected filter(s) on" the temporary folder
Actual Results:
no entry in filter log
Expected Results:
Applied filter "Test" to message from ... at 2008-01-02 14:01:31 Tagged as ...
Oops... numbers in 'Steps to Reproduce' should be 1,2,3,4 NOT 2,3,1,4
| Assignee | ||
Comment 2•18 years ago
|
||
Confirming, moving to core.
Assignee: mail → nobody
Status: UNCONFIRMED → NEW
Component: MailNews: Main Mail Window → MailNews: Filters
Ever confirmed: true
OS: Windows XP → All
Product: Mozilla Application Suite → Core
QA Contact: filters
Hardware: PC → All
| Assignee | ||
Comment 3•18 years ago
|
||
The localization value for filterAction17 was missing.
Assignee: nobody → mkmelin+mozilla
Status: NEW → ASSIGNED
Attachment #295276 -
Flags: superreview?(bienvenu)
Attachment #295276 -
Flags: review?(mnyromyr)
| Assignee | ||
Comment 4•18 years ago
|
||
The other "tagging" in the file is for the old label version.
Comment 5•18 years ago
|
||
Comment on attachment 295276 [details] [diff] [review]
proposed fix
thx, Magnus, yes, that was the problem.
Attachment #295276 -
Flags: superreview?(bienvenu) → superreview+
Updated•18 years ago
|
Attachment #295276 -
Flags: review?(mnyromyr) → review+
| Assignee | ||
Comment 6•18 years ago
|
||
Checking in mail/locales/en-US/chrome/messenger/filter.properties;
/cvsroot/mozilla/mail/locales/en-US/chrome/messenger/filter.properties,v <-- filter.properties
new revision: 1.7; previous revision: 1.6
done
Checking in suite/locales/en-US/chrome/mailnews/filter.properties;
/cvsroot/mozilla/suite/locales/en-US/chrome/mailnews/filter.properties,v <-- filter.properties
new revision: 1.23; previous revision: 1.22
done
->FIXED
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M11
Updated•17 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•