custom header search is non-functional ('Run search on server' of Edit/Find/Search Messages doesn't work any more)
Categories
(Thunderbird :: Search, defect)
Tracking
(thunderbird_esr78 fixed, thunderbird83 affected)
People
(Reporter: stan, Assigned: infofrommozilla)
References
(Blocks 1 open bug, Regression)
Details
(Keywords: regression, Whiteboard: [datalossy])
Attachments
(1 file, 2 obsolete files)
1.47 KB,
patch
|
infofrommozilla
:
review+
wsmwk
:
approval-comm-esr78+
|
Details | Diff | Splinter Review |
Reporter | ||
Updated•16 years ago
|
Assignee | ||
Comment 1•15 years ago
|
||
Comment 2•15 years ago
|
||
Updated•15 years ago
|
Comment 3•15 years ago
|
||
Assignee | ||
Comment 5•15 years ago
|
||
Reporter | ||
Comment 6•15 years ago
|
||
Comment 7•15 years ago
|
||
Reporter | ||
Comment 8•15 years ago
|
||
Reporter | ||
Comment 9•15 years ago
|
||
Comment 10•15 years ago
|
||
Assignee | ||
Comment 11•15 years ago
|
||
Comment 12•15 years ago
|
||
Reporter | ||
Comment 13•15 years ago
|
||
Assignee | ||
Comment 14•15 years ago
|
||
Comment 15•15 years ago
|
||
Comment 16•15 years ago
|
||
Reporter | ||
Comment 17•15 years ago
|
||
Comment 18•15 years ago
|
||
Comment 19•15 years ago
|
||
Updated•15 years ago
|
Updated•14 years ago
|
Comment 21•12 years ago
|
||
Reporter | ||
Comment 22•12 years ago
|
||
Comment 23•12 years ago
|
||
Reporter | ||
Comment 24•12 years ago
|
||
Comment 25•12 years ago
|
||
Comment 26•11 years ago
|
||
Comment 27•11 years ago
|
||
Comment 28•10 years ago
|
||
Comment 30•10 years ago
|
||
Updated•10 years ago
|
Comment 31•10 years ago
|
||
Comment 32•10 years ago
|
||
Updated•10 years ago
|
Comment 33•10 years ago
|
||
Comment 34•10 years ago
|
||
Comment 35•10 years ago
|
||
Comment 36•10 years ago
|
||
Assignee | ||
Comment 37•5 years ago
|
||
Summary: 'Run search on server' of Edit/Find/Search Messages doesn't work any more, unless Body is also used
This bug has become a bit confused. As far as I can see, the main problem is that TB does not respect the Run search on server
option when searching.
As I already wrote in comment #5 this is caused by bug 537820. Adding a "body" search term is only a workaround.
Bug 537820 prefers an offline search for quick search if offline and online search is available. But the bug overdoes it.
TB respects Run search on server
for saved search folders, so it should work the same way for the search dialog.
Assignee | ||
Comment 38•5 years ago
|
||
In order not to break bug 537820 again, we have to distinguish from where the search was started.
If anyone sees an elegant way to do so: Patches are welcome.
Assignee | ||
Comment 39•5 years ago
|
||
(In reply to Alfred Peters from comment #38)
In order not to break bug 537820 again, we have to distinguish from where the search was started.
This patch is a bit ugly. It checks the existence of this.owner._enteredFolder
.
It is only available if the search is connected to an folder but not with the dialogue.
Comment 40•5 years ago
|
||
Updated•5 years ago
|
Assignee | ||
Comment 41•5 years ago
|
||
(In reply to Magnus Melin [:mkmelin] from comment #40)
But, should this be !this.displayedFolder instead of checking _enteredFolder? Or this.isVirtual?
_ | this.owner.isVirtual | this.owner.displayedFolder | !this.owner.displayedFolder |
---|---|---|---|
QuickSerach: | false | [xpconnect wrapped (nsISupports, nsIMsgFolder, nsIMsgImapMailFolder)] | false |
SavedSearch: | true | [xpconnect wrapped (nsISupports, nsIMsgFolder, nsISupportsWeakReference)] | false |
SearchDialog: | false | null | true |
OK - !this.owner.displayedFolder
will do.
Assignee | ||
Comment 42•5 years ago
|
||
(In reply to Magnus Melin [:mkmelin] from comment #40)
// If both scopes work, honor the onlineSearch request, unless we're // filtering (quick search and/or a view selected).
Should add some comments above to explain.
Isn't that old comment wrong anyway?
'selected view' is a saved search right?
But we do (and should) honor the 'onlineSearch request' there.
Comment 43•5 years ago
|
||
I think selected view may refer to e.g. what we have in View | Folders | Unified
Assignee | ||
Comment 44•5 years ago
|
||
I changed the condition to !this.owner.displayedFolder
and adapted the Comment.
Comment 46•5 years ago
|
||
Updated•5 years ago
|
Assignee | ||
Comment 47•5 years ago
|
||
(In reply to Magnus Melin [:mkmelin] from comment #46)
Comment on attachment 9184968 [details] [diff] [review]
Thanks, looks good, but needs linting. r=mkmelin
done
Assignee | ||
Updated•5 years ago
|
Comment 48•5 years ago
|
||
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/2410a7cea917
Take in account 'Run search on server' of Edit/Find/Search Messages r=mkmelin
Comment 49•5 years ago
|
||
@mkmelin
fyi, after upgrade to
Name Thunderbird
Version 78.4.1
Build ID 20201104231226
'body' searches exec'd 'on server' DO appear to exec on the server. at least, at 1st check, I see the queries in my backend Solr logs.
otoh, 'subject' searches -- or any other header for that matter -- similarly exec'd 'on server' do NOT hit the backend at all; there's NO traffic from TB to the Solr instance.
Comment 50•5 years ago
|
||
This bug has been fixed for tb 84+. I don't know what you're reporting, but if needed, file a new bug about it.
Comment 51•5 years ago
|
||
I don't know what you're reporting,
A bug was filed. Previously. Against v78.
It's here:
https://bugzilla.mozilla.org/show_bug.cgi?id=1673928
You're CC'd on it.
It was closed, with comment:
"A fix for this is upcoming."
and marked as a DUPLICATE of this bug, 6 days ago ... immediately above your comment,
Comment 52•5 years ago
•
|
||
(In reply to pgnet.dev from comment #51)
I don't know what you're reporting,
A bug was filed. Previously. Against v78.
Because you reporting the same issue against an unfixed version, we don't need bug 1673928.
The patch next needs to go through beta, which will be beta 84.
And after that into version 78, perhaps 78.6.0 (it's not likely to make 78.5.0)
Comment 53•5 years ago
|
||
Comment on attachment 9185766 [details] [diff] [review]
Take in account 'Run search on server' of Edit/Find/Search Messages
[Approval Request Comment]
User impact if declined: per bug description
Testing completed (on c-c, etc.): cc and beta
Risk to taking this patch (and alternatives if risky): not too much risk, fairly confined
Comment 54•5 years ago
|
||
Comment on attachment 9185766 [details] [diff] [review]
Take in account 'Run search on server' of Edit/Find/Search Messages
[Triage Comment]
Approved for esr78
Comment 55•5 years ago
|
||
bugherder uplift |
Thunderbird 78.5.1:
https://hg.mozilla.org/releases/comm-esr78/rev/c536b88adc03
Updated•5 years ago
|
Description
•