Open
Bug 350868
Opened 18 years ago
Updated 2 years ago
saved search folders show up for 'run now' in messages filters, but doesn't work
Categories
(MailNews Core :: Filters, defect)
MailNews Core
Filters
Tracking
(Not tracked)
REOPENED
People
(Reporter: johnathan.conley, Assigned: aceman)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
(deleted),
patch
|
mconley
:
review+
bwinton
:
ui-review+
rkent
:
feedback+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
aceman
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1b2) Gecko/20060831 BonEcho/2.0b2
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1b2) Gecko/20060831 BonEcho/2.0b2
saved search folders show up as potential targets of 'run now' for message filters, however message filters will not work on 'saved search' folders
this is already done in another area: saved search folders are removed in the saved search builder (since saved searches cannot be based on saved searches)
Reproducible: Always
Steps to Reproduce:
1. create a saved search and a message filter
2. attempt to run that message filter manually on a saved search (nothing happens)
Expected Results:
I wouldn't expect that message filters would work on saved searches, so I would expect that saved searches not show up as an option for manually running a messages filter via the 'run now' button.
Comment 1•18 years ago
|
||
Confirmed on Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1b2) Gecko/20060831 Thunderbird/2.0a1 ID:2006083103
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows XP → All
might consider with enhancements (bug 350871) and (bug 347190)
Updated•17 years ago
|
Updated•16 years ago
|
Assignee: mscott → nobody
version 3.0a2 (2008072418) vista
-if select a virtual folder, tools/run filters.. is grayed, as expected ..
-in the filter manager in the folder dropdown it shows saved searches. with no
effect. wrong
so I guess it's still here
Version: 2.0 → Trunk
Updated•15 years ago
|
Component: General → Filters
Product: Thunderbird → MailNews Core
QA Contact: general → filters
Adding mode=filing to the folder picker seems to filter out virtual folders. But it also hides Outbox folder.
Is this the correct mode to set or do we have to define some new one?
Assignee: nobody → acelists
I am not sure if Seamonkey needs this too. It has a different picker I think.
The menu item Run filters on folder is also disabled when Outbox is selected so the patch behaves consistently with that.
Attachment #655680 -
Flags: ui-review?(bwinton)
Attachment #655680 -
Flags: feedback?(kent)
Comment 6•12 years ago
|
||
Comment on attachment 655680 [details] [diff] [review]
patch
Yeah, this seems okay to me, and consistent with the menu item…
Attachment #655680 -
Flags: ui-review?(bwinton) → ui-review+
Comment 7•12 years ago
|
||
Comment on attachment 655680 [details] [diff] [review]
patch
Seems reasonable to me.
Attachment #655680 -
Flags: feedback?(kent) → feedback+
Comment on attachment 655680 [details] [diff] [review]
patch
Is this relevant for Seamonkey?
Attachment #655680 -
Flags: review?(mconley)
Attachment #655680 -
Flags: feedback?(jh)
Comment 9•12 years ago
|
||
Comment on attachment 655680 [details] [diff] [review]
patch
(In reply to :aceman from comment #8)
> Is this relevant for Seamonkey?
The file you are changing is TB-only (all files under mail/ are) so your patch is not affecting SM. The copy of our code doesn't include the line you are changing, so I guess it's best to fix this on our end in a follow-up bug since we need a different strategy of unknown complexity. For now I'll just mark this one TODO by adding it to our to-port meta bug.
Attachment #655680 -
Flags: feedback?(jh)
Comment 10•12 years ago
|
||
Comment on attachment 655680 [details] [diff] [review]
patch
Review of attachment 655680 [details] [diff] [review]:
-----------------------------------------------------------------
Good stuff! Thanks aceman!
Attachment #655680 -
Flags: review?(mconley) → review+
Keywords: checkin-needed
Comment 11•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 18.0
Assignee | ||
Comment 12•12 years ago
|
||
Suddenly in TB18 I get the menu item "Run filters on folder" enabled on a Saved search folder. Can anybody see it too?
I have also tested a filter on such a folder and it worked.
Assignee | ||
Comment 13•12 years ago
|
||
Also, this causes the folder list dropdown in the filter list to be empty for news accounts and throwing an exception. Running filters after the fact seems to work on news accounts/folders so the folder picker should show them.
How do I request backout of this patch?
Comment 14•12 years ago
|
||
I suppose you can attach a reverse of the patch, reopen and mark it checkin-needed.
You seriously need to get checkin rights yourself though. Please file a bug about that, cc me and a few more people who's done reviews for your patches so we can vouch for you.
Assignee | ||
Comment 15•12 years ago
|
||
Checkin rights will not help me, only faster reviews :)
Assignee | ||
Comment 16•12 years ago
|
||
Backout patch for this feature, please land.
Attachment #660339 -
Flags: review+
Comment 17•12 years ago
|
||
Aceman that's where having test would help you and the reviewers ...
Assignee | ||
Comment 18•12 years ago
|
||
Test?
Assignee | ||
Comment 19•12 years ago
|
||
There should have been a test in existence before we started to work on this. So that we could know what is the correct behaviour. We still do not know it now so I can't create one now. E.g. a comment in the code says filters after the fact (manual) do not work on news so it tries to disable the folder picker (but fails to do so). However, I talked to rkent and jcranmer on IRC and we found those filters do work on news. I even found that they work on a Saved search folder and the menu item in main menu is suddenly enabled on such a folder. So I am not sure what the proper behaviour here is. Maybe if we find it out we can make a test.
Can't mozilla arrange for a person creating tests full time? :)
Comment 20•12 years ago
|
||
https://hg.mozilla.org/comm-central/rev/b5b960f438ed
Leaving open for a better fix. Please close as WONTFIX if that's not the case.
Keywords: checkin-needed
Target Milestone: Thunderbird 18.0 → ---
Assignee | ||
Comment 21•12 years ago
|
||
Can we add a new attribute on nsIMsgFolder for this? So that we can easily disable filters to run on types of folders if needed in the future.
Flags: needinfo?(kent)
Comment 22•12 years ago
|
||
(In reply to :aceman from comment #21)
> Can we add a new attribute on nsIMsgFolder for this? So that we can easily
> disable filters to run on types of folders if needed in the future.
In bug 272490 comment 10 I asked whether we should try to avoid creating additional new attributes to nsIMsgFolder and nsIMsgIncomingServer, and instead use existing methods with a text identifier.
A few years back I added the inheritable string properties to the folder. Internally, it is currently only used to represent booleans (for applying junk processing, and applying incoming filters). Personally I would prefer if new attributes are added in that way rather than expanding the definitions of nsIMsgFolder and nsIMsgIncomingServer, but in the past I have been the only proponent of that position. Perhaps it would be a good tb-planning topic?
Flags: needinfo?(kent)
Assignee | ||
Comment 23•12 years ago
|
||
Yes, or a GSoC project that I proposed on the wiki page.
Comment 24•9 years ago
|
||
Removing myslef on all the bugs I'm cced on. Please NI me if you need something on MailNews Core bugs from me.
Updated•2 years ago
|
Summary: saved search folders show up for 'run now' in messages filters → saved search folders show up for 'run now' in messages filters, but doesn't work
Updated•2 years ago
|
Severity: minor → S4
You need to log in
before you can comment on or make changes to this bug.
Description
•