Re: [PATCH 4/5] nmbug-status: Wrap query phrases in parentheses when and-ing together

Subject: Re: [PATCH 4/5] nmbug-status: Wrap query phrases in parentheses when and-ing together

Date: Thu, 31 Dec 2015 08:47:19 -0800

To: notmuch@notmuchmail.org

Cc: David Bremner, Tomi Ollila, Jani Nikula, Carl Worth

From: W. Trevor King

On Wed, Dec 30, 2015 at 11:16:59AM -0800, W. Trevor King wrote:
> @@ -167,7 +167,8 @@ class Page (object):
>                  view['title'], sort_key))
>          if 'query-string' not in view:
>              query = view['query']
> -            view['query-string'] = ' and '.join(query)
> +            view['query-string'] = ' and '.join(
> +                '( {} )'.format(q) for q in query)

This patch is missing a similar change around line 414 to handle:

  $ nmbug-status --get-query VIEW

I've queued that for v2.

Cheers,
Trevor

-- 
This email may be signed or encrypted with GnuPG (http://www.gnupg.org).
For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy
signature.asc (application/pgp-signature)

Thread: