[Emacs-diffs] /srv/bzr/emacs/trunk r103000: mml2015.el (mml2015-epg-sign, mml2015-epg-encrypt): Give mml2015-signers higher precedence over mml2015-sign-with-sender.

[Katsumi Yamaoka]

------------------------------------------------------------
revno: 103000
author: Daiki Ueno <address@hidden>
committer: Katsumi Yamaoka <address@hidden>
branch nick: trunk
timestamp: Fri 2011-01-28 01:34:09 +0000
message:
  mml2015.el (mml2015-epg-sign, mml2015-epg-encrypt): Give mml2015-signers 
higher precedence over mml2015-sign-with-sender.
modified:
  lisp/gnus/ChangeLog
  lisp/gnus/mml2015.el

=== modified file 'lisp/gnus/ChangeLog'
--- a/lisp/gnus/ChangeLog       2011-01-28 00:14:08 +0000
+++ b/lisp/gnus/ChangeLog       2011-01-28 01:34:09 +0000
@@ -1,3 +1,8 @@
+2011-01-28  Daiki Ueno  <address@hidden>
+
+       * mml2015.el (mml2015-epg-sign, mml2015-epg-encrypt): Give
+       mml2015-signers higher precedence over mml2015-sign-with-sender.
+
 2011-01-27  Lars Ingebrigtsen  <address@hidden>
 
        * gnus-group.el (gnus-group-refresh-group): Refresh even non-visible

=== modified file 'lisp/gnus/mml2015.el'
--- a/lisp/gnus/mml2015.el      2011-01-28 00:14:08 +0000
+++ b/lisp/gnus/mml2015.el      2011-01-28 01:34:09 +0000
@@ -116,7 +116,8 @@
   :type 'integer)
 
 (defcustom mml2015-signers nil
-  "A list of your own key ID which will be used to sign a message."
+  "A list of your own key ID which will be used to sign a message.
+If set, it overrides the setting of `mml2015-sign-with-sender'."
   :group 'mime-security
   :type '(repeat (string :tag "Key ID")))
 
@@ -965,8 +966,10 @@
   (let* ((inhibit-redisplay t)
         (context (epg-make-context))
         (boundary (mml-compute-boundary cont))
-        (sender (when mml2015-sign-with-sender
-                   (message-options-get 'message-sender)))
+        (sender (message-options-get 'message-sender))
+        (signer-names (or mml2015-signers
+                          (if (and mml2015-sign-with-sender sender)
+                              (list (concat "<" sender ">")))))
         signer-key
         (signers
          (or (message-options-get 'mml2015-epg-signers)
@@ -976,10 +979,7 @@
                   (epa-select-keys context "\
 Select keys for signing.
 If no one is selected, default secret key is used.  "
-                                   (if sender
-                                       (cons (concat "<" sender ">")
-                                             mml2015-signers)
-                                     mml2015-signers)
+                                   signer-names
                                    t)
                 (if (or sender mml2015-signers)
                     (delq nil
@@ -995,10 +995,7 @@
                                            signer)))
                                (error "No secret key for %s" signer))
                              signer-key)
-                           (if sender
-                               (cons (concat "<" sender ">")
-                                     mml2015-signers)
-                             mml2015-signers))))))))
+                           signer-names)))))))
         signature micalg)
     (epg-context-set-armor context t)
     (epg-context-set-textmode context t)
@@ -1038,15 +1035,18 @@
     (goto-char (point-max))))
 
 (defun mml2015-epg-encrypt (cont &optional sign)
-  (let ((inhibit-redisplay t)
-       (context (epg-make-context))
-       (config (epg-configuration))
-       (sender (when mml2015-sign-with-sender
-                  (message-options-get 'message-sender)))
-       (recipients (message-options-get 'mml2015-epg-recipients))
-       cipher signers
-       (boundary (mml-compute-boundary cont))
-       recipient-key signer-key)
+  (let* ((inhibit-redisplay t)
+        (context (epg-make-context))
+        (boundary (mml-compute-boundary cont))
+        (config (epg-configuration))
+        (recipients (message-options-get 'mml2015-epg-recipients))
+        cipher
+        (sender (message-options-get 'message-sender))
+        (signer-names (or mml2015-signers
+                          (if (and mml2015-sign-with-sender sender)
+                              (list (concat "<" sender ">")))))
+        signers
+        recipient-key signer-key)
     (unless recipients
       (setq recipients
            (apply #'nconc
@@ -1060,12 +1060,9 @@
                                              (read-string "Recipients: ")))
                     "[ \f\t\n\r\v,]+"))))
       (when mml2015-encrypt-to-self
-       (unless (or sender mml2015-signers)
-         (error "Message sender and mml2015-signers not set"))
-       (setq recipients (nconc recipients (if sender
-                                              (cons (concat "<" sender ">")
-                                                    mml2015-signers)
-                                            mml2015-signers))))
+       (unless signer-names
+         (error "Neither message sender nor mml2015-signers are set"))
+       (setq recipients (nconc recipients signer-names)))
       (if (eq mm-encrypt-option 'guided)
          (setq recipients
                (epa-select-keys context "\
@@ -1098,10 +1095,7 @@
                     (epa-select-keys context "\
 Select keys for signing.
 If no one is selected, default secret key is used.  "
-                                     (if sender
-                                         (cons (concat "<" sender ">")
-                                               mml2015-signers)
-                                       mml2015-signers)
+                                     signer-names
                                      t)
                   (if (or sender mml2015-signers)
                       (delq nil
@@ -1117,9 +1111,7 @@
                                              signer)))
                                  (error "No secret key for %s" signer))
                                signer-key)
-                             (if sender
-                                 (cons (concat "<" sender ">") mml2015-signers)
-                               mml2015-signers))))))))
+                             signer-names)))))))
       (epg-context-set-signers context signers))
     (epg-context-set-armor context t)
     (epg-context-set-textmode context t)