diff options
| author | Koni Marti <koni.marti@gmail.com> | 2024-02-01 23:52:55 +0100 |
|---|---|---|
| committer | Robin Jarry <robin@jarry.cc> | 2024-02-11 22:03:56 +0100 |
| commit | e7c26e02bb326f08c071e9f37ebf80952a0cca97 (patch) | |
| tree | b20503b2df2b1fafd751aa7abc998fca24fa66a4 /commands/account/search.go | |
| parent | 05874f867c76c8e9b8c1e67a57dc702e63d6d54b (diff) | |
| download | aerc-e7c26e02bb326f08c071e9f37ebf80952a0cca97.tar.gz | |
filter: allow workers to combine filter terms
Allow the backend workers to combine the filter terms. Currently, the
consecutive filters are joined in the message store with a space (" ").
This works well for most backends, but makes the filter combination for
notmuch confusing.
Example:
Issuing two consecutive filter commands in notmuch
:filter not tag:list
:filter tag:list
would create the following filter query
'not tag:list tag:list'
This is not what users would expect; they expect:
'(not tag:list) and (tag:list)'
Note that the notmuch backend works correctly for the given query, but
produced a query that does not match the user's expectation. This patch
fixes this.
The combination of filter terms in other backends remains the same.
Reported-by: Ángel Castañeda <angel@acsq.me>
Signed-off-by: Koni Marti <koni.marti@gmail.com>
Tested-by: Inwit <inwit@sindominio.net>
Acked-by: Robin Jarry <robin@jarry.cc>
Diffstat (limited to 'commands/account/search.go')
| -rw-r--r-- | commands/account/search.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/commands/account/search.go b/commands/account/search.go index 0246a9eb..cd449dfa 100644 --- a/commands/account/search.go +++ b/commands/account/search.go @@ -154,7 +154,7 @@ func (s SearchFilter) Execute(args []string) error { EndDate: s.EndDate, SearchBody: s.Body, SearchAll: s.All, - Terms: s.Terms, + Terms: []string{s.Terms}, } if args[0] == "filter" { |