diff --git a/mu4e/mu4e-compose.el b/mu4e/mu4e-compose.el index 86d00d39..98d5e8f0 100644 --- a/mu4e/mu4e-compose.el +++ b/mu4e/mu4e-compose.el @@ -1,7 +1,6 @@ -;; -*-mode: emacs-lisp; tab-width: 8; indent-tabs-mode: t -*- ;; mu4e-compose.el -- part of mu4e, the mu mail user agent for emacs ;; -;; Copyright (C) 2011-2016 Dirk-Jan C. Binnema +;; Copyright (C) 2011-2019 Dirk-Jan C. Binnema ;; Author: Dirk-Jan C. Binnema ;; Maintainer: Dirk-Jan C. Binnema @@ -105,14 +104,14 @@ which takes no arguments, and which should return one of the mentioned symbols, for example: (setq mu4e-sent-messages-behavior (lambda () - (if (string= (message-sendmail-envelope-from) \"foo@example.com\") - 'delete 'sent))) + (if (string= (message-sendmail-envelope-from) \"foo@example.com\") + 'delete 'sent))) The various `message-' functions from `message-mode' are available for querying the message information." :type '(choice (const :tag "move message to mu4e-sent-folder" sent) - (const :tag "move message to mu4e-trash-folder" trash) - (const :tag "delete message" delete)) + (const :tag "move message to mu4e-trash-folder" trash) + (const :tag "delete message" delete)) :group 'mu4e-compose) (defcustom mu4e-compose-context-policy 'ask @@ -131,13 +130,13 @@ contexts match, we have the following choices: Also see `mu4e-context-policy'." :type '(choice - (const :tag "Always ask what context to use" always-ask) - (const :tag "Ask if none of the contexts match" ask) - (const :tag "Ask when there's no context yet" ask-if-none) - (const :tag "Pick the first context if none match" pick-first) - (const :tag "Don't change the context when none match" nil) - :safe 'symbolp - :group 'mu4e-compose)) + (const :tag "Always ask what context to use" always-ask) + (const :tag "Ask if none of the contexts match" ask) + (const :tag "Ask when there's no context yet" ask-if-none) + (const :tag "Pick the first context if none match" pick-first) + (const :tag "Don't change the context when none match" nil) + :safe 'symbolp + :group 'mu4e-compose)) (defcustom mu4e-compose-crypto-reply-encrypted-policy 'sign-and-encrypt "Policy for signing/encrypting replies to encrypted messages. @@ -148,12 +147,12 @@ We have the following choices: - `encrypt': encrypt the reply, but don't sign it. - anything else: do nothing." :type '(choice - (const :tag "Sign the reply" sign) - (const :tag "Sign and encrypt the reply" sign-and-encrypt) - (const :tag "Encrypt the reply" encrypt) - (const :tag "Don't do anything" nil) - :safe 'symbolp - :group 'mu4e-compose)) + (const :tag "Sign the reply" sign) + (const :tag "Sign and encrypt the reply" sign-and-encrypt) + (const :tag "Encrypt the reply" encrypt) + (const :tag "Don't do anything" nil) + :safe 'symbolp + :group 'mu4e-compose)) (defcustom mu4e-compose-crypto-reply-plain-policy nil "Policy for signing/encrypting replies to messages received unencrypted. @@ -164,28 +163,24 @@ We have the following choices: - `encrypt': encrypt the reply, but don't sign it. - anything else: do nothing." :type '(choice - (const :tag "Sign the reply" sign) - (const :tag "Sign and encrypt the reply" sign-and-encrypt) - (const :tag "Encrypt the reply" encrypt) - (const :tag "Don't do anything" nil) - :safe 'symbolp - :group 'mu4e-compose)) + (const :tag "Sign the reply" sign) + (const :tag "Sign and encrypt the reply" sign-and-encrypt) + (const :tag "Encrypt the reply" encrypt) + (const :tag "Don't do anything" nil) + :safe 'symbolp + :group 'mu4e-compose)) -(defcustom mu4e-compose-crypto-reply-policy nil "The use of the - 'mu4e-compose-crypto-reply-policy' variable is deprecated. - 'mu4e-compose-crypto-reply-plain-policy' and - 'mu4e-compose-crypto-reply-encrypted-policy' should be used instead") (make-obsolete-variable 'mu4e-compose-crypto-reply-policy "The use of the 'mu4e-compose-crypto-reply-policy' variable is deprecated. 'mu4e-compose-crypto-reply-plain-policy' and 'mu4e-compose-crypto-reply-encrypted-policy' should be used instead" - "2017-09-02") + "2017-09-02") (defcustom mu4e-compose-format-flowed nil - "Whether to compose messages to be sent as format=flowed (or - with long lines if `use-hard-newlines' is set to nil). The - variable `fill-flowed-encode-column' lets you customize the - width beyond which format=flowed lines are wrapped." + "Whether to compose messages to be sent as format=flowed. +\(Or with long lines if variable `use-hard-newlines' is set to +nil). The variable `fill-flowed-encode-column' lets you customize +the width beyond which format=flowed lines are wrapped." :type 'boolean :safe 'booleanp :group 'mu4e-compose) @@ -206,8 +201,8 @@ place to do that." :group 'mu4e-compose) (defvar mu4e-compose-type nil - "The compose-type for this buffer, which is a symbol, `new', - `forward', `reply' or `edit'.") + "The compose-type for this buffer. +This is a symbol, `new', `forward', `reply' or `edit'.") ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;; @@ -249,24 +244,25 @@ Messages are captured with `mu4e-action-capture-message'." "Maybe setup Fcc, based on `mu4e-sent-messages-behavior'. If needed, set the Fcc header, and register the handler function." (let* ((sent-behavior - ;; Note; we cannot simply use functionp here, since at least - ;; delete is a function, too... - (if (member mu4e-sent-messages-behavior '(delete trash sent)) - mu4e-sent-messages-behavior - (if (functionp mu4e-sent-messages-behavior) - (funcall mu4e-sent-messages-behavior) - mu4e-sent-messages-behavior))) - (mdir - (cl-case sent-behavior - (delete nil) - (trash (mu4e-get-trash-folder mu4e-compose-parent-message)) - (sent (mu4e-get-sent-folder mu4e-compose-parent-message)) - (otherwise - (mu4e-error "unsupported value '%S' `mu4e-sent-messages-behavior'." - mu4e-sent-messages-behavior)))) - (fccfile (and mdir - (concat mu4e-maildir mdir "/cur/" - (mu4e~draft-message-filename-construct "S"))))) + ;; Note; we cannot simply use functionp here, since at least + ;; delete is a function, too... + (if (member mu4e-sent-messages-behavior '(delete trash sent)) + mu4e-sent-messages-behavior + (if (functionp mu4e-sent-messages-behavior) + (funcall mu4e-sent-messages-behavior) + mu4e-sent-messages-behavior))) + (mdir + (cl-case sent-behavior + (delete nil) + (trash (mu4e-get-trash-folder mu4e-compose-parent-message)) + (sent (mu4e-get-sent-folder mu4e-compose-parent-message)) + (otherwise + (mu4e-error "Unsupported value '%S' + `mu4e-sent-messages-behavior'" + mu4e-sent-messages-behavior)))) + (fccfile (and mdir + (concat mu4e-maildir mdir "/cur/" + (mu4e~draft-message-filename-construct "S"))))) ;; if there's an fcc header, add it to the file (when fccfile (message-add-header (concat "Fcc: " fccfile "\n")) @@ -274,17 +270,17 @@ If needed, set the Fcc header, and register the handler function." ;; etc. if you run it after mu4e so, (hack hack) we reset it to the old ;; handler after we've done our thing. (setq message-fcc-handler-function - (lexical-let ((maildir mdir) (old-handler message-fcc-handler-function)) - (lambda (file) - (setq message-fcc-handler-function old-handler) ;; reset the fcc handler - (let ((mdir-path (concat mu4e-maildir maildir))) - ;; Create the full maildir structure for the sent folder if it doesn't exist. - ;; `mu4e~proc-mkdir` runs asynchronously but no matter whether it runs before or after - ;; `write-file`, the sent maildir ends up in the correct state. - (unless (file-exists-p mdir-path) - (mu4e~proc-mkdir mdir-path))) - (write-file file) ;; writing maildirs files is easy - (mu4e~proc-add file (or maildir "/")))))))) ;; update the database + (lexical-let ((maildir mdir) (old-handler message-fcc-handler-function)) + (lambda (file) + (setq message-fcc-handler-function old-handler) ;; reset the fcc handler + (let ((mdir-path (concat mu4e-maildir maildir))) + ;; Create the full maildir structure for the sent folder if it doesn't exist. + ;; `mu4e~proc-mkdir` runs asynchronously but no matter whether it runs before or after + ;; `write-file`, the sent maildir ends up in the correct state. + (unless (file-exists-p mdir-path) + (mu4e~proc-mkdir mdir-path))) + (write-file file) ;; writing maildirs files is easy + (mu4e~proc-add file (or maildir "/")))))))) ;; update the database (defvar mu4e-compose-hidden-headers `("^References:" "^Face:" "^X-Face:" @@ -300,27 +296,31 @@ If needed, set the Fcc header, and register the handler function." "^\\(To\\|B?Cc\\|Reply-To\\|From\\):") (defun mu4e~compose-register-message-save-hooks () - "Just before saving, we remove the mail-header-separator; just -after saving we restore it; thus, the separator should never + "Just before saving, we remove the `mail-header-separator'. +Just after saving we restore it; thus, the separator should never appear on disk." (add-hook 'before-save-hook - 'mu4e~draft-remove-mail-header-separator nil t) + (lambda() + (save-match-data + (mu4e~draft-remove-mail-header-separator))) nil t) (add-hook 'after-save-hook (lambda () - (mu4e~compose-set-friendly-buffer-name) - (mu4e~draft-insert-mail-header-separator) - ;; hide some headers again - (mu4e~compose-hide-headers) - (widen) - (set-buffer-modified-p nil) - (mu4e-message "Saved (%d lines)" (count-lines (point-min) (point-max))) - ;; update the file on disk -- ie., without the separator - (mu4e~proc-add (buffer-file-name) mu4e~draft-drafts-folder)) nil t)) + (save-match-data + (mu4e~compose-set-friendly-buffer-name) + (mu4e~draft-insert-mail-header-separator) + ;; hide some headers again + (mu4e~compose-hide-headers) + (widen) + (set-buffer-modified-p nil) + (mu4e-message "Saved (%d lines)" (count-lines (point-min) (point-max))) + ;; update the file on disk -- ie., without the separator + (mu4e~proc-add (buffer-file-name) mu4e~draft-drafts-folder))) nil t)) ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;; ;; address completion; inspired by org-contacts.el and ;; https://github.com/nordlow/elisp/blob/master/mine/completion-styles-cycle.el (defun mu4e~compose-complete-handler (str pred action) + "Complete address STR with predication PRED for ACTION." (cond ((eq action nil) (try-completion str mu4e~contacts pred)) @@ -330,29 +330,29 @@ appear on disk." ;; our contacts are already sorted - just need to tell the ;; completion machinery not to try to undo that... '(metadata - (display-sort-function . identity) - (cycle-sort-function . identity))))) + (display-sort-function . identity) + (cycle-sort-function . identity))))) (defun mu4e~compose-complete-contact (&optional start) "Complete the text at START with a contact. Ie. either 'name ' or 'email')." (interactive) (let ((mail-abbrev-mode-regexp mu4e~compose-address-fields-regexp) - (eoh ;; end-of-headers - (save-excursion - (goto-char (point-min)) - (search-forward-regexp mail-header-separator nil t)))) + (eoh ;; end-of-headers + (save-excursion + (goto-char (point-min)) + (search-forward-regexp mail-header-separator nil t)))) ;; try to complete only when we're in the headers area, ;; looking at an address field. (when (and eoh (> eoh (point)) (mail-abbrev-in-expansion-header-p)) (let* ((end (point)) - (start - (or start - (save-excursion - (re-search-backward "\\(\\`\\|[\n:,]\\)[ \t]*") - (goto-char (match-end 0)) - (point))))) - (list start end 'mu4e~compose-complete-handler))))) + (start + (or start + (save-excursion + (re-search-backward "\\(\\`\\|[\n:,]\\)[ \t]*") + (goto-char (match-end 0)) + (point))))) + (list start end 'mu4e~compose-complete-handler))))) (defun mu4e~compose-setup-completion () "Set up auto-completion of addresses." @@ -363,9 +363,9 @@ Ie. either 'name ' or 'email')." 'mu4e~compose-complete-contact nil t)) (defun mu4e~remove-refs-maybe () - "Remove the References: header if the In-Reply-To header is -missing. This allows the user to effectively start a new -message-thread by removing the In-Reply-To header." + "Remove References: if In-Reply-To: is missing. +This allows the user to effectively start a new message-thread by +removing the In-Reply-To header." (unless (message-fetch-field "in-reply-to") (message-remove-header "References"))) @@ -382,10 +382,11 @@ message-thread by removing the In-Reply-To header." map))) (defun mu4e-fill-paragraph (&optional region) - "If `use-hard-newlines', takes a multi-line paragraph and makes -it into a single line of text. Assume paragraphs are separated -by blank lines. If `use-hard-newlines' is not enabled, this -simply executes `fill-paragraph'." + "Re-layout either the whole message or REGION. +If variable `use-hard-newlines', takes a multi-line paragraph and +makes it into a single line of text. Assume paragraphs are +separated by blank lines. If variable `use-hard-newlines' is not +set, this simply executes `fill-paragraph'." ;; Inspired by https://www.emacswiki.org/emacs/UnfillParagraph (interactive (progn (barf-if-buffer-read-only) '(t))) (if mu4e-compose-format-flowed @@ -404,7 +405,8 @@ simply executes `fill-paragraph'." (turn-on-auto-fill))) (defun mu4e~compose-remap-faces () - "Our parent `message-mode' uses font-locking for the compose + "Remap `message-mode' faces to mu4e ones. +Our parent `message-mode' uses font-locking for the compose buffers; lets remap its faces so it uses the ones for mu4e." ;; normal headers (face-remap-add-relative 'message-header-name @@ -456,68 +458,68 @@ buffers; lets remap its faces so it uses the ones for mu4e." ;; offer completion for e-mail addresses (when mu4e-compose-complete-addresses (unless mu4e~contacts ;; work-around for https://github.com/djcb/mu/issues/1016 - (mu4e~request-contacts-maybe)) + (mu4e~request-contacts-maybe)) (mu4e~compose-setup-completion)) (if mu4e-compose-format-flowed - (progn - (turn-off-auto-fill) - (setq truncate-lines nil - word-wrap t - mml-enable-flowed t - use-hard-newlines t) - (visual-line-mode t)) + (progn + (turn-off-auto-fill) + (setq truncate-lines nil + word-wrap t + mml-enable-flowed t + use-hard-newlines t) + (visual-line-mode t)) (setq mml-enable-flowed nil)) (let ((keymap (lookup-key message-mode-map [menu-bar text]))) (when keymap - (define-key-after - keymap - [mu4e-hard-newlines] - '(menu-item "Format=flowed" mu4e-toggle-use-hard-newlines - :button (:toggle . use-hard-newlines) - :help "Toggle format=flowed" - :visible (eq major-mode 'mu4e-compose-mode) - :enable mu4e-compose-format-flowed) - 'sep) + (define-key-after + keymap + [mu4e-hard-newlines] + '(menu-item "Format=flowed" mu4e-toggle-use-hard-newlines + :button (:toggle . use-hard-newlines) + :help "Toggle format=flowed" + :visible (eq major-mode 'mu4e-compose-mode) + :enable mu4e-compose-format-flowed) + 'sep) - (define-key-after - keymap - [mu4e-electric-quote-mode] - '(menu-item "Electric quote" electric-quote-local-mode - :button (:toggle . electric-quote-mode) - :help "Toggle Electric quote mode" - :visible (and (eq major-mode 'mu4e-compose-mode) - (functionp 'electric-quote-local-mode))) - 'mu4e-hard-newlines))) + (define-key-after + keymap + [mu4e-electric-quote-mode] + '(menu-item "Electric quote" electric-quote-local-mode + :button (:toggle . electric-quote-mode) + :help "Toggle Electric quote mode" + :visible (and (eq major-mode 'mu4e-compose-mode) + (functionp 'electric-quote-local-mode))) + 'mu4e-hard-newlines))) (when (lookup-key mml-mode-map [menu-bar Attachments]) (define-key-after - (lookup-key mml-mode-map [menu-bar Attachments]) - [mu4e-compose-attach-captured-message] - '(menu-item "Attach captured message" - mu4e-compose-attach-captured-message - :help "Attach message captured in Headers View (with 'a c')" - :visible (eq major-mode 'mu4e-compose-mode)) - (quote Attach\ External...))) + (lookup-key mml-mode-map [menu-bar Attachments]) + [mu4e-compose-attach-captured-message] + '(menu-item "Attach captured message" + mu4e-compose-attach-captured-message + :help "Attach message captured in Headers View (with 'a c')" + :visible (eq major-mode 'mu4e-compose-mode)) + (quote Attach\ External...))) ;; setup the fcc-stuff, if needed (add-hook 'message-send-hook (lambda () ;; mu4e~compose-save-before-sending - ;; when in-reply-to was removed, remove references as well. - (when (eq mu4e-compose-type 'reply) - (mu4e~remove-refs-maybe)) - (when use-hard-newlines - (mu4e-send-harden-newlines)) - ;; for safety, always save the draft before sending - (set-buffer-modified-p t) - (save-buffer) - (mu4e~compose-setup-fcc-maybe) - (widen)) nil t) + ;; when in-reply-to was removed, remove references as well. + (when (eq mu4e-compose-type 'reply) + (mu4e~remove-refs-maybe)) + (when use-hard-newlines + (mu4e-send-harden-newlines)) + ;; for safety, always save the draft before sending + (set-buffer-modified-p t) + (save-buffer) + (mu4e~compose-setup-fcc-maybe) + (widen)) nil t) ;; when the message has been sent. (add-hook 'message-sent-hook (lambda () ;; mu4e~compose-mark-after-sending - (setq mu4e-sent-func 'mu4e-sent-handler) - (mu4e~proc-sent (buffer-file-name) mu4e~draft-drafts-folder)) nil t)) + (setq mu4e-sent-func 'mu4e-sent-handler) + (mu4e~proc-sent (buffer-file-name) mu4e~draft-drafts-folder)) nil t)) ;; mark these two hooks as permanent-local, so they'll survive mode-changes ;; (put 'mu4e~compose-save-before-sending 'permanent-local-hook t) (put 'mu4e~compose-mark-after-sending 'permanent-local-hook t)) @@ -533,36 +535,37 @@ buffers; lets remap its faces so it uses the ones for mu4e." "Maximum length of the mu4e-send-buffer-name.") (defun mu4e~compose-set-friendly-buffer-name (&optional compose-type) - "Set some user-friendly buffer name based on the compose type." + "Set some user-friendly buffer name based on the COMPOSE-TYPE." (let* ((subj (message-field-value "subject")) - (subj (unless (and subj (string-match "^[:blank:]*$" subj)) subj)) - (str (or subj - (cl-case compose-type - (reply "*reply*") - (forward "*forward*") - (otherwise "*draft*"))))) + (subj (unless (and subj (string-match "^[:blank:]*$" subj)) subj)) + (str (or subj + (cl-case compose-type + (reply "*reply*") + (forward "*forward*") + (otherwise "*draft*"))))) (rename-buffer (generate-new-buffer-name - (truncate-string-to-width str - mu4e~compose-buffer-max-name-length - nil nil t) - (buffer-name))))) + (truncate-string-to-width str + mu4e~compose-buffer-max-name-length + nil nil t) + (buffer-name))))) (defun mu4e~compose-crypto-reply (parent compose-type) - "When composing a reply to an encrypted message, we can + "Possibly encrypt or sign a message based on PARENT and COMPOSE-TYPE. +When composing a reply to an encrypted message, we can automatically encrypt that reply. When the message is unencrypted, we can decide what we want to do." (if (and (eq compose-type 'reply) - (and parent (member 'encrypted (mu4e-message-field parent :flags)))) - (cl-case mu4e-compose-crypto-reply-encrypted-policy - (sign (mml-secure-message-sign)) - (encrypt (mml-secure-message-encrypt)) - (sign-and-encrypt (mml-secure-message-sign-encrypt)) - (message "Do nothing")) - (cl-case mu4e-compose-crypto-reply-plain-policy - (sign (mml-secure-message-sign)) - (encrypt (mml-secure-message-encrypt)) - (sign-and-encrypt (mml-secure-message-sign-encrypt)) - (message "Do nothing"))) + (and parent (member 'encrypted (mu4e-message-field parent :flags)))) + (cl-case mu4e-compose-crypto-reply-encrypted-policy + (sign (mml-secure-message-sign)) + (encrypt (mml-secure-message-encrypt)) + (sign-and-encrypt (mml-secure-message-sign-encrypt)) + (message "Do nothing")) + (cl-case mu4e-compose-crypto-reply-plain-policy + (sign (mml-secure-message-sign)) + (encrypt (mml-secure-message-encrypt)) + (sign-and-encrypt (mml-secure-message-sign-encrypt)) + (message "Do nothing"))) ) @@ -602,8 +605,8 @@ tempfile)." (condition-case nil (mu4e-draft-open compose-type original-msg) (quit (set-window-configuration winconf) - (mu4e-message "Operation aborted") - (cl-return-from mu4e~compose-handler)))) + (mu4e-message "Operation aborted") + (cl-return-from mu4e~compose-handler)))) ;; insert mail-header-separator, which is needed by message mode to separate ;; headers and body. will be removed before saving to disk (mu4e~draft-insert-mail-header-separator) @@ -615,10 +618,10 @@ tempfile)." (goto-char (point-max)) ;; put attachments at the end (if (and (eq compose-type 'forward) mu4e-compose-forward-as-attachment) - (mu4e-compose-attach-message original-msg) + (mu4e-compose-attach-message original-msg) (dolist (att includes) - (mml-attach-file - (plist-get att :file-name) (plist-get att :mime-type))))) + (mml-attach-file + (plist-get att :file-name) (plist-get att :mime-type))))) (mu4e~compose-set-friendly-buffer-name compose-type) @@ -628,10 +631,10 @@ tempfile)." ;; otherwise, it depends... (cl-case message-cite-reply-position ((above traditional) - (message-goto-body)) + (message-goto-body)) (t - (when (message-goto-signature) - (forward-line -2))))) + (when (message-goto-signature) + (forward-line -2))))) ;; bind to `mu4e-compose-parent-message' of compose buffer (set (make-local-variable 'mu4e-compose-parent-message) original-msg) @@ -660,16 +663,16 @@ tempfile)." "Try to go back to some previous buffer, in the order view->headers->main." (unless (eq mu4e-split-view 'single-window) (if (buffer-live-p (mu4e-get-view-buffer)) - (switch-to-buffer (mu4e-get-view-buffer)) + (switch-to-buffer (mu4e-get-view-buffer)) (if (buffer-live-p (mu4e-get-headers-buffer)) - (switch-to-buffer (mu4e-get-headers-buffer)) - ;; if all else fails, back to the main view - (when (fboundp 'mu4e) (mu4e)))))) + (switch-to-buffer (mu4e-get-headers-buffer)) + ;; if all else fails, back to the main view + (when (fboundp 'mu4e) (mu4e)))))) (defun mu4e-sent-handler (docid path) - "Handler function, called with DOCID and PATH for the just-sent -message. For Forwarded ('Passed') and Replied messages, try to set -the appropriate flag at the message forwarded or replied-to." + "Handler called with DOCID and PATH for the just-sent message. +For Forwarded ('Passed') and Replied messages, try to set the +appropriate flag at the message forwarded or replied-to." (mu4e~compose-set-parent-flag path) (when (file-exists-p path) ;; maybe the draft was not saved at all (mu4e~proc-remove docid)) @@ -677,9 +680,9 @@ the appropriate flag at the message forwarded or replied-to." ;; this seems a bit hamfisted... (dolist (buf (buffer-list)) (when (and (buffer-file-name buf) - (string= (buffer-file-name buf) path)) + (string= (buffer-file-name buf) path)) (if message-kill-buffer-on-exit - (kill-buffer buf)))) + (kill-buffer buf)))) (mu4e~switch-back-to-mu4e-buffer) (mu4e-message "Message sent")) @@ -693,12 +696,13 @@ It restores mu4e window layout after killing the compose-buffer." (when (not (equal current-buffer (current-buffer))) ;; Restore mu4e (if mu4e-compose-in-new-frame - (delete-frame) - (mu4e~switch-back-to-mu4e-buffer))))) + (delete-frame) + (mu4e~switch-back-to-mu4e-buffer))))) (defun mu4e~compose-set-parent-flag (path) - "Set the 'replied' \"R\" flag on messages we replied to, and the -'passed' \"F\" flag on message we have forwarded. + "Set flags for replied-t and forwarded for the message at PATH. +That is, set the 'replied' \"R\" flag on messages we replied to, +and the 'passed' \"F\" flag on message we have forwarded. If a message has an 'in-reply-to' header, it is considered a reply to the message with the corresponding message id. If it does not @@ -717,31 +721,31 @@ buffer." (let ((buf (find-file-noselect path))) (when buf (with-current-buffer buf - (message-narrow-to-headers-or-head) - (let ((in-reply-to (message-fetch-field "in-reply-to")) - (forwarded-from) - (references (message-fetch-field "references"))) - (unless in-reply-to - (when references - (with-temp-buffer ;; inspired by `message-shorten-references'. - (insert references) - (goto-char (point-min)) - (let ((refs)) - (while (re-search-forward "<[^ <]+@[^ <]+>" nil t) - (push (match-string 0) refs)) - ;; the last will be the first - (setq forwarded-from (first refs)))))) - ;; remove the <> - (when (and in-reply-to (string-match "<\\(.*\\)>" in-reply-to)) - (mu4e~proc-move (match-string 1 in-reply-to) nil "+R-N")) - (when (and forwarded-from (string-match "<\\(.*\\)>" forwarded-from)) - (mu4e~proc-move (match-string 1 forwarded-from) nil "+P-N"))))))) + (message-narrow-to-headers-or-head) + (let ((in-reply-to (message-fetch-field "in-reply-to")) + (forwarded-from) + (references (message-fetch-field "references"))) + (unless in-reply-to + (when references + (with-temp-buffer ;; inspired by `message-shorten-references'. + (insert references) + (goto-char (point-min)) + (let ((refs)) + (while (re-search-forward "<[^ <]+@[^ <]+>" nil t) + (push (match-string 0) refs)) + ;; the last will be the first + (setq forwarded-from (first refs)))))) + ;; remove the <> + (when (and in-reply-to (string-match "<\\(.*\\)>" in-reply-to)) + (mu4e~proc-move (match-string 1 in-reply-to) nil "+R-N")) + (when (and forwarded-from (string-match "<\\(.*\\)>" forwarded-from)) + (mu4e~proc-move (match-string 1 forwarded-from) nil "+P-N"))))))) (defun mu4e-compose (compose-type) - "Start composing a message of COMPOSE-TYPE, where COMPOSE-TYPE -is a symbol, one of `reply', `forward', `edit', `resend' -`new'. All but `new' take the message at point as input. Symbol -`edit' is only allowed for draft messages." + "Start composing a message of COMPOSE-TYPE. +COMPOSE-TYPE is a symbol, one of `reply', `forward', `edit', +`resend' `new'. All but `new' take the message at point as input. +Symbol `edit' is only allowed for draft messages." (let ((msg (mu4e-message-at-point 'noerror))) ;; some sanity checks (unless (or msg (eq compose-type 'new)) @@ -749,7 +753,7 @@ is a symbol, one of `reply', `forward', `edit', `resend' (unless (member compose-type '(reply forward edit resend new)) (mu4e-error "Invalid compose type '%S'" compose-type)) (when (and (eq compose-type 'edit) - (not (member 'draft (mu4e-message-field msg :flags)))) + (not (member 'draft (mu4e-message-field msg :flags)))) (mu4e-warn "Editing is only allowed for draft messages")) ;; 'new is special, since it takes no existing message as arg; therefore, we @@ -758,22 +762,22 @@ is a symbol, one of `reply', `forward', `edit', `resend' (mu4e~compose-handler 'new) ;; otherwise, we need the doc-id (let* ((docid (mu4e-message-field msg :docid)) - ;; decrypt (or not), based on `mu4e-decryption-policy'. - (decrypt - (and (member 'encrypted (mu4e-message-field msg :flags)) - (if (eq mu4e-decryption-policy 'ask) - (yes-or-no-p (mu4e-format "Decrypt message?")) - mu4e-decryption-policy)))) - ;; if there's a visible view window, select that before starting - ;; composing a new message, so that one will be replaced by the compose - ;; window. The 10-or-so line headers buffer is not a good place to write - ;; it... - (unless (eq mu4e-split-view 'single-window) - (let ((viewwin (get-buffer-window (mu4e-get-view-buffer)))) - (when (window-live-p viewwin) - (select-window viewwin)))) - ;; talk to the backend - (mu4e~proc-compose compose-type decrypt docid))))) + ;; decrypt (or not), based on `mu4e-decryption-policy'. + (decrypt + (and (member 'encrypted (mu4e-message-field msg :flags)) + (if (eq mu4e-decryption-policy 'ask) + (yes-or-no-p (mu4e-format "Decrypt message?")) + mu4e-decryption-policy)))) + ;; if there's a visible view window, select that before starting + ;; composing a new message, so that one will be replaced by the compose + ;; window. The 10-or-so line headers buffer is not a good place to write + ;; it... + (unless (eq mu4e-split-view 'single-window) + (let ((viewwin (get-buffer-window (mu4e-get-view-buffer)))) + (when (window-live-p viewwin) + (select-window viewwin)))) + ;; talk to the backend + (mu4e~proc-compose compose-type decrypt docid))))) (defun mu4e-compose-reply () "Compose a reply for the message at point in the headers buffer." @@ -809,11 +813,41 @@ draft message." ;;;###autoload (defun mu4e~compose-mail (&optional to subject other-headers continue - switch-function yank-action send-actions return-action) - "This is mu4e's implementation of `compose-mail'." + switch-function yank-action send-actions return-action) + "This is mu4e's implementation of `compose-mail'. +Quoting its docstring: +Start composing a mail message to send. +This uses the user’s chosen mail composition package +as selected with the variable ‘mail-user-agent’. +The optional arguments TO and SUBJECT specify recipients +and the initial Subject field, respectively. - ;; create a new draft message 'resetting' (as below) is not actually needed in - ;; this case, but let's prepare for the re-edit case as well +OTHER-HEADERS is an alist specifying additional +header fields. Elements look like (HEADER . VALUE) where both +HEADER and VALUE are strings. + +CONTINUE, if non-nil, says to continue editing a message already +being composed. Interactively, CONTINUE is the prefix argument. + +SWITCH-FUNCTION, if non-nil, is a function to use to +switch to and display the buffer used for mail composition. + +YANK-ACTION, if non-nil, is an action to perform, if and when necessary, +to insert the raw text of the message being replied to. +It has the form (FUNCTION . ARGS). The user agent will apply +FUNCTION to ARGS, to insert the raw text of the original message. +\(The user agent will also run ‘mail-citation-hook’, *after* the +original text has been inserted in this way.) + +SEND-ACTIONS is a list of actions to call when the message is sent. +Each action has the form (FUNCTION . ARGS). + +RETURN-ACTION, if non-nil, is an action for returning to the +caller. It has the form (FUNCTION . ARGS). The function is +called after the mail has been sent or put aside, and the mail +buffer buried." + ;; create a new draft message 'resetting' (as below) is not actually needed in this case, but + ;; let's prepare for the re-edit case as well (mu4e~compose-handler 'new) (when (message-goto-to) ;; reset to-address, if needed @@ -880,7 +914,7 @@ Go to the end of the message (before signature) or, if already there, go to the end of the buffer." (interactive) (let ((old-position (point)) - (message-position (save-excursion (message-goto-body) (point)))) + (message-position (save-excursion (message-goto-body) (point)))) (goto-char (point-max)) (when (re-search-backward message-signature-separator message-position t) (forward-line -1)) @@ -891,7 +925,4 @@ end of the buffer." (vector 'remap 'end-of-buffer) 'mu4e-compose-goto-bottom) (provide 'mu4e-compose) - -;; Load mu4e completely even when this file was loaded through -;; autoload. -(require 'mu4e) +;;; mu4e-compose.el ends here