From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by arlo.cworth.org (Postfix) with ESMTP id 85E3E6DE108D for ; Thu, 4 Apr 2019 16:01:43 -0700 (PDT) X-Virus-Scanned: Debian amavisd-new at cworth.org X-Spam-Flag: NO X-Spam-Score: -2.692 X-Spam-Level: X-Spam-Status: No, score=-2.692 tagged_above=-999 required=5 tests=[AWL=-0.190, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_MED=-2.3, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001] autolearn=disabled Received: from arlo.cworth.org ([127.0.0.1]) by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 6cggXEOhvylT for ; Thu, 4 Apr 2019 16:01:43 -0700 (PDT) Received: from smtp-4.sys.kth.se (smtp-4.sys.kth.se [130.237.48.193]) by arlo.cworth.org (Postfix) with ESMTPS id 60CBF6DE104E for ; Thu, 4 Apr 2019 16:01:41 -0700 (PDT) Received: from smtp-4.sys.kth.se (localhost.localdomain [127.0.0.1]) by smtp-4.sys.kth.se (Postfix) with ESMTP id F1B626895 for ; Fri, 5 Apr 2019 01:01:39 +0200 (CEST) X-Virus-Scanned: by amavisd-new at kth.se Received: from smtp-4.sys.kth.se ([127.0.0.1]) by smtp-4.sys.kth.se (smtp-4.sys.kth.se [127.0.0.1]) (amavisd-new, port 10024) with LMTP id sT58j5-n8Ytn; Fri, 5 Apr 2019 01:01:39 +0200 (CEST) X-KTH-Auth: ekeberg [85.229.204.229] DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kth.se; s=default; t=1554418899; bh=/ys2lksdUuuPiLefF5Ux5YBlABmINrCY1KWLrmVyh1g=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=aBDdwrUBVMz/zB/+dSl4S6I4y4sFvFJqeVXKRxAm0FDiSZ78//njpPhsc2Ye3wByQ e0jtf63hanRfXi2O+dFJ7T9HQ1OGFhsYuysqDkLSq7jB42KvXOX+TR9JFtMy3g4efc ZeEo4qSbkWmkgIPrDdWATBuWm4/spV0t08ZcENVk= X-KTH-mail-from: ekeberg@kth.se Received: from swing.localdomain (c-e5cce555.09-723-73746f39.bbcust.telenor.se [85.229.204.229]) by smtp-4.sys.kth.se (Postfix) with ESMTPSA id 427C328E8; Fri, 5 Apr 2019 01:01:39 +0200 (CEST) From: =?UTF-8?q?=C3=96rjan=20Ekeberg?= To: notmuch@notmuchmail.org Subject: [PATCH v2 2/4] emacs: Use a buffer-local variable to update tags when sending replies Date: Fri, 5 Apr 2019 01:01:24 +0200 Message-Id: <20190404230126.4283-3-ekeberg@kth.se> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20190404230126.4283-1-ekeberg@kth.se> References: <20190404230126.4283-1-ekeberg@kth.se> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: notmuch@notmuchmail.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "Use and development of the notmuch mail system." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 04 Apr 2019 23:01:43 -0000 Instead of relying on the "In-Reply-To" header, use a buffer-local variable, notmuch-message-queued-tag-changes, to add and remove tags to affected messages when the message-send-hook is triggered. --- emacs/notmuch-message.el | 24 ++++++++++++++++-------- emacs/notmuch-mua.el | 5 +++++ 2 files changed, 21 insertions(+), 8 deletions(-) diff --git a/emacs/notmuch-message.el b/emacs/notmuch-message.el index 55e4cfee..e7615998 100644 --- a/emacs/notmuch-message.el +++ b/emacs/notmuch-message.el @@ -34,18 +34,26 @@ will be removed from the message being replied to. For example, if you wanted to add a \"replied\" tag and remove the \"inbox\" and \"todo\" tags, you would set: - (\"+replied\" \"-inbox\" \"-todo\"\)" + (\"+replied\" \"-inbox\" \"-todo\")" :type '(repeat string) :group 'notmuch-send) -(defun notmuch-message-mark-replied () - ;; get the in-reply-to header and parse it for the message id. - (let ((rep (mail-header-parse-addresses (message-field-value "In-Reply-To")))) - (when (and notmuch-message-replied-tags rep) - (notmuch-tag (notmuch-id-to-query (car (car rep))) - (notmuch-tag-change-list notmuch-message-replied-tags))))) +(defconst notmuch-message-queued-tag-changes nil + "List of messages and corresponding tag-changes to be applied when sending a message. -(add-hook 'message-send-hook 'notmuch-message-mark-replied) +This variable is overridden by buffer-local versions in message +buffers where tag changes should be triggered when sending off +the message. Each item in this list is a list of strings, where +the first is a notmuch query and the rest are the tag changes to +be applied to the matching messages.") + +(defun notmuch-message-apply-queued-tag-changes () + ;; Apply the tag changes queued in the buffer-local variable notmuch-message-queued-tag-changes. + (dolist (query-and-tags notmuch-message-queued-tag-changes) + (notmuch-tag (car query-and-tags) + (cdr query-and-tags)))) + +(add-hook 'message-send-hook 'notmuch-message-apply-queued-tag-changes) (provide 'notmuch-message) diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el index 23f3d8b1..0af9c71b 100644 --- a/emacs/notmuch-mua.el +++ b/emacs/notmuch-mua.el @@ -259,6 +259,11 @@ Typically this is added to `notmuch-mua-send-hook'." (notmuch-headers-plist-to-alist reply-headers) nil (notmuch-mua-get-switch-function)))) + ;; Create a buffer-local queue for tag changes triggered when sending the reply + (when notmuch-message-replied-tags + (setq-local notmuch-message-queued-tag-changes + (list (cons query-string notmuch-message-replied-tags)))) + ;; Insert the message body - but put it in front of the signature ;; if one is present, and after any other content ;; message*setup-hooks may have added to the message body already. -- 2.20.1