fix nm_query_type=messages + query type=threads

If the default query type is 'messages', and the specific query type is
'threads', make sure the query type is actually 'threads'.

update the header colors after modifying tags

The header color depends on tags. The message header color must be
updated after modifying message tags.

fix DB transaction balancing

Signed-off-by: Karel Zak <kzak@redhat.com>

call notmuch_query_count_messages() more carefully

 - it seems that notmuch is sensitive when notmuch_query_count_messages()
   is called, let's call it when we really need the number for progress bar

 - initialize mutt progress bar when we really want to use it

References: https://github.com/karelzak/mutt-kz/issues/57
Signed-off-by: Karel Zak <kzak@redhat.com>

use exclude tags for entrie-thread too

Signed-off-by: Karel Zak <kzak@redhat.com>

get <entire-thread> work from pager context

I found annoying that we cant unfold whole thread from pager.
Implementation is pretty straight-forward.

Signed-off-by: Kirill A. Shutemov <kirill@shutemov.name>

fix getting count of new messages for non-trivial query

Currtly, we ask for count of new messages by appending tag:unread (or
whatever NotmuchUnreadTag has) to db query. It works if your query is
very simple: no 'OR', 'NOT', etc.

If you will ask new messages for query like
 "to:foo@example.com or to:bar@example.net"
mutt will query db for count of
 "to:foo@example.com or to:bar@example.net tag:unread"
and notmuch will understand it as
 "to:foo@example.com or (to:bar@example.net tag:unread)"
which, obviously, is not what you wanted.

Optimizations like
if (strstr(db_query, NotmuchUnreadTag))
*new = all ? *all : count_query(db, db_query);
doesn't work for the same reason.

Let's fix all this by construct query with proper parenthesis:
 "(to:foo@example.com or to:bar@example.net) tag:unread"

Signed-off-by: Kirill A. Shutemov <kirill@shutemov.name>

apply NotmuchExcludeTags to count queries

We have support of tag excluding, but it doesn't cover count queries.
Let's fix this.

Signed-off-by: Kirill A. Shutemov <kirill@shutemov.name>

make %N optional for notmuch browser format

It will allow to have vfolder_format option like this:

set vfolder_format="%?N?*& ?%20f %6N %10n"

This format will mark all vfolders with new messages with '*':

                inbox      0          0
*            linux/mm      2        183
     linux/mm/commits      0         39
...

Without the patch all vforlders will be marked.

Signed-off-by: Kirill A. Shutemov <kirill@shutemov.name>

docs: add UPDATING.kz

docs: update README.notmuch

Signed-off-by: Karel Zak <kzak@redhat.com>

search by vfolder name in browser

Reported-by: Vladimir Marek <Vladimir.Marek@oracle.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

don't close opened message after <modify-label>

Addresses: https://github.com/karelzak/mutt-kz/issues/48
Signed-off-by: Karel Zak <kzak@redhat.com>

<change-vfolder> default to the next unread folder

It means that for example:

 macro index R <change-vfolder><enter>

will for as expected.

Signed-off-by: Karel Zak <kzak@redhat.com>

extend <change-vfolder> to accept folder name/uri

Suvayu Ali <fatkasuvayu+linux@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

update DB on save-command

Signed-off-by: Karel Zak <kzak@redhat.com>

add CH_VIRTUAL to allow to specify when use virtual headers

For example don't add "Tags:" header to the message when copy the
message to another folder. The virtual headers make sense only when
we display the message to user.

Signed-off-by: Karel Zak <kzak@redhat.com>

improve entire-thread

 - don't support mutt tags, always work with the current message only

 - support collapsed threads

 - add "No more messages in the thread" message

Signed-off-by: Karel Zak <kzak@redhat.com>

add entire-thread command, cleanup progress bar code

Signed-off-by: Karel Zak <kzak@redhat.com>

always use notmuch message ID

It's better to use notmuch message Id than rely on Id from parsed
message, because notmuch ensures that Id exists and it generates
a fake Id for broken messages without the Id.

Reported-by: Michael Zeltner <m@niij.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

fix buffer overflow, improve tag_list usage

add nm_exclude_tags

Based on patch from: Raghavendra D Prabhu <rprabhu@wnohang.net>

 This makes use of notmuch_query_add_tag_exclude in notmuch API, to
 exclude messages tagged with tags from nm_exclude_tags from  query
 results by default unless specified explicitly.

Signed-off-by: Raghavendra D Prabhu <rprabhu@wnohang.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

clean up tags related code, improve %G

 - clean up code
 - allows to use %G (and transform-tags) for unhidden tags too

Signed-off-by: Karel Zak <kzak@redhat.com>

added some documentation

port the old index-color mutt patch to mutt-kz

added support for individual tag coloring added "Tags:" header output in
pager showing all tags added documentation for index object highlighting

fix corrupt viewing of messages

The mutt records offset of email body when parse email header.

Unfortunately when we check notmuch DB changes when we don't ask for
email path in the same way. So it's possible that for emails that are
stored in multiple folders (you sent the same email with the same ID
to multiple mailing lists) we get a different path, then notmuch mutt
code updates the path, but no the offset... result is corrupted body.

Signed-off-by: Karel Zak <kzak@redhat.com>

add support for text transforms of tag names

tag names visible with '%g' in the index and pager formats

sidebar: extend sidebar sort methods, add sort_sidebar=

 sort_sidebar=(desc|path|count|new|unsorted)

Signed-off-by: Karel Zak <kzak@redhat.com>

add $record support

Signed-off-by: Karel Zak <kzak@redhat.com>

hide signed and encrypted tags by default

Mutt shows signed/encrypted status in another way and occupies only 1
char in index line. Hide the tags by default and leave more space for
other tags.

Signed-off-by: David Sterba <dsterba@suse.cz>

reuse connection in complete_all_nm_tags()

Signed-off-by: Karel Zak <kzak@redhat.com>

Fix trivial documentation typoes

Various trivial typographical errors in notmuch-specific code, are now fixed.

Add TAB completion of tag names in <modify-labels>*

<modify-labels> and <modify-labels-then-hide> can now use TAB completion of tag
names supplied after + and -.

Add TAB completion of tag: names in <vfolder-from-query>

<vfolder-from-query>'s 'Query' prompt can now use TAB completion of tag names
supplied to the 'tag:' parameter.

Add nm_get_all_tags()

Add nm_get_all_tags(), a mechanism to return a list of all notmuch tags in the
database.

Plug a leaked notmuch DB handle in get_count()

nm_nonctx_get_count() was leaking its notmuch database handle if NOTMUCH_API_3
was in use. Looks like it missed the same treatment that the other call sites
got to do this.

Add 'label' pattern modifier '~Y'

Add the new '~Y' pattern modifier. Use it in e.g. 'limit' and 'color' commands
to match a regexp against notmuch message labels.

Fix auto_tag=yes for modify-labels* and quasi-delete

When 'auto_tag' was 'yes', <modify-labels>* and <quasi-delete> still only
applied to the currently-selected message. Now they apply to all tagged
messages, as they should.

add %g examples

Signed-off-by: Karel Zak <kzak@redhat.com>

count messages for mutt progress bar

Signed-off-by: Karel Zak <kzak@redhat.com>

docs: add link to notmuch mutttips

Signed-off-by: Karel Zak <kzak@redhat.com>

add notmuch_{query,message}_destroy to DB checker

Reported-by: Chris Mason <chris.mason@fusionio.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

add nm_query_type config file option

Signed-off-by: Karel Zak <kzak@redhat.com>

add note about nm_db_limit to README

Signed-off-by: Karel Zak <kzak@redhat.com>

mutt_nm_db_limit option defines a default limit

print read progress of a query

Signed-off-by: David Sterba <dsterba@suse.cz>

format: support optional modifier for g (notmuch labels)

Used in index_format like this %?g?(%g)? will show nothing if there are
no notmuch labels, eg. in mixed mail setups.

Signed-off-by: David Sterba <dsterba@suse.cz>

sync flags for all duplicate messages

Signed-off-by: Karel Zak <kzak@redhat.com>

use the Maildir if NotmuchDefaultUri is unset

[kzak@redhat.com: - cleanup code, add note to README.notmuch]

Signed-off-by: Karel Zak <kzak@redhat.com>

use notmuch_status_t for DB open

Signed-off-by: Karel Zak <kzak@redhat.com>

use the new notmuch api if available

As of notmuch 0.13 libnotmuch's api has changed. This makes use of it if the
new api was detected.

fix nm_check_database to count new messages correctly

.. otherwise mx_update_context() might ends with SIGSEGV

Signed-off-by: Karel Zak <kzak@redhat.com>

allow to read whole-threads

This patch allows to specify type of query for notmuch DB, supported
are "threads" and "messages", the default is "messages". For example:

   notmuch://?type=threads&query=tag:inbox"

will read all matching threads.

Based on prototype from Chris Mason <chris.mason@fusionio.com>.

Signed-off-by: Karel Zak <kzak@redhat.com>

don't rename in DB before mx commit

Signed-off-by: Karel Zak <kzak@redhat.com>

add NOTMUCH in show_version()

make filename changes more robust

 * use code from 'notmuch new' for add/remove messages
 * use the last added filename for duplicate emails

Signed-off-by: Karel Zak <kzak@redhat.com>

fix new->cur maildir move

Signed-off-by: Karel Zak <kzak@redhat.com>

fix prefixes in README

Signed-off-by: Karel Zak <kzak@redhat.com>

add nm_open_timeout

Signed-off-by: Karel Zak <kzak@redhat.com>

don't use uninitialized HEADER in nm_check_database()

Signed-off-by: Karel Zak <kzak@redhat.com>

fix msg remove code

Signed-off-by: Karel Zak <kzak@redhat.com>

fix sync

Signed-off-by: Karel Zak <kzak@redhat.com>

remove extra slashes

Signed-off-by: Karel Zak <kzak@redhat.com>

search in new/cur subfolders if DB is not up-to-date

Signed-off-by: Karel Zak <kzak@redhat.com>

use context data if possible rather than directly context

This allows to use many functions outside the notmuch CONTEXT
with a dummy "struct nm_ctxdata".

Signed-off-by: Karel Zak <kzak@redhat.com>

explicitly require notmuch context

Signed-off-by: Karel Zak <kzak@redhat.com>

cleanup private notmuch structs initialization

Signed-off-by: Karel Zak <kzak@redhat.com>

fix a bug with processing mailboxes when they are all virtual.

This bug occurs when there are only virtual mailboxes defined. Prior to this
change, the sidebar descriptions of the virtual mailboxes would be displayed as
the notmuch query itself, as opposed to the user-defined description.

add note about notmuch version requirement

Signed-off-by: Karel Zak <kzak@redhat.com>

Grammar and typo fixes.

build-sys: fix compilation without --enable-notmuch

We can still leave mutt-kz compilable entirely without notmuch, just for the
case there are users out there not interested in notmuch enhancements, but I
seriously doubt that :)

Here's the patch which compiles fine without --enable-notmuch

[kzak@redhat.com: - fix s/Incomming/Incoming/ typo]

Signed-off-by: Karel Zak <kzak@redhat.com>

fix segfault when exiting with 'x' key

docs: add note about sidebar to notmuch docs

Signed-off-by: Karel Zak <kzak@redhat.com>

add support for virtual folders, add sidebar-toggle command

 * add support for virtual folders
 * add sidebar-toggle function to toggle between virtual and normal
   folders in sudebar
 * virtual folders are used by default if 'virtual_spoolfile' is 'yes'

Signed-off-by: Karel Zak <kzak@redhat.com>

add ML to README

Signed-off-by: Karel Zak <kzak@redhat.com>

fix compilation error at get_nm_message()

The shown error is:

mutt_notmuch.c: In function ‘get_nm_message’:
mutt_notmuch.c:643:2: error: too few arguments to function ‘notmuch_database_find_message’
/usr/include/notmuch.h:371:1: note: declared here
make[2]: *** [mutt_notmuch.o] Error 1

The mutt-ks is made for notmuch version 0.8, but in version 0.9 the API
changed, in particular with this commit:

http://git.notmuchmail.org/git/notmuch/commitdiff/02a30767116ad8abcbd0a3351f2e4d43bbbd655f

This patch updates the function notmuch_database_find_message() usage.

Signed-off-by: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>

improve nm_unread_tag setting

Signed-off-by: Karel Zak <kzak@redhat.com>

improve virtual_mailboxes command, update docs

Signed-off-by: Karel Zak <kzak@redhat.com>

update README

Signed-off-by: Karel Zak <kzak@redhat.com>

add debug messages

Signed-off-by: Karel Zak <kzak@redhat.com>

support default db uri

Signed-off-by: Karel Zak <kzak@redhat.com>

implement nm_check_database()

Signed-off-by: Karel Zak <kzak@redhat.com>

small refactoring

Signed-off-by: Karel Zak <kzak@redhat.com>

add virtual spoolfile

Signed-off-by: Karel Zak <kzak@redhat.com>

properly update context numbers

Signed-off-by: Karel Zak <kzak@redhat.com>

print vfolder description on status bar

Signed-off-by: Karel Zak <kzak@redhat.com>

add <modify-labels-then-hide>

Signed-off-by: Karel Zak <kzak@redhat.com>

header: add HEADER->free_cb() to free driver specific data

The mutt_free_header() need to care about driver specific data.

BTW, it seems that there is a leak in imap code -- it does not free
HEADER->data->keywords list.

Signed-off-by: Karel Zak <kzak@redhat.com>

update README

Signed-off-by: Karel Zak <kzak@redhat.com>

fix CONTEXT->data->db usage

Signed-off-by: Karel Zak <kzak@redhat.com>

update README.notmuch

Signed-off-by: Karel Zak <kzak@redhat.com>

add modify-label

Signed-off-by: Karel Zak <kzak@redhat.com>

change-vfolder, print folder info, etc.

 - allow to open email in virtual folder
 - allow to open virtual folder
 - print vfolder specific stuff in index and pager

Signed-off-by: Karel Zak <kzak@redhat.com>

add support to browser

Signed-off-by: Karel Zak <kzak@redhat.com>

add notmuch calls to mx_* API

Signed-off-by: Karel Zak <kzak@redhat.com>

don't modify virtual folders' paths

Signed-off-by: Karel Zak <kzak@redhat.com>

add basic functions

Signed-off-by: Karel Zak <kzak@redhat.com>

add basic .muttrc variables

Signed-off-by: Karel Zak <kzak@redhat.com>

add context magic

Signed-off-by: Karel Zak <kzak@redhat.com>

consolidate msg sync code

 - add mh_sync_mailbox_message() to sync just one message
 - add msg->commited_path to save info about the final message path
   (will be used keep notmuch DB up to date)

Signed-off-by: Karel Zak <kzak@redhat.com>