← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/mail_client.py

Merge up bzr.dev.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/mail_client.py
1
 
# Copyright (C) 2007-2010 Canonical Ltd
 
1
# Copyright (C) 2007 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
 
 
17
 
from __future__ import absolute_import
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
18
16
 
19
17
import errno
20
18
import os
24
22
 
25
23
import bzrlib
26
24
from bzrlib import (
27
 
    config as _mod_config,
28
25
    email_message,
29
26
    errors,
30
27
    msgeditor,
31
28
    osutils,
32
29
    urlutils,
33
 
    registry
34
30
    )
35
31
 
36
 
mail_client_registry = registry.Registry()
37
 
 
38
32
 
39
33
class MailClient(object):
40
34
    """A mail client that can send messages with attachements."""
43
37
        self.config = config
44
38
 
45
39
    def compose(self, prompt, to, subject, attachment, mime_subtype,
46
 
                extension, basename=None, body=None):
 
40
                extension, basename=None):
47
41
        """Compose (and possibly send) an email message
48
42
 
49
43
        Must be implemented by subclasses.
63
57
        """
64
58
        raise NotImplementedError
65
59
 
66
 
    def compose_merge_request(self, to, subject, directive, basename=None,
67
 
                              body=None):
 
60
    def compose_merge_request(self, to, subject, directive, basename=None):
68
61
        """Compose (and possibly send) a merge request
69
62
 
70
63
        :param to: The address to send the request to
77
70
        prompt = self._get_merge_prompt("Please describe these changes:", to,
78
71
                                        subject, directive)
79
72
        self.compose(prompt, to, subject, directive,
80
 
            'x-patch', '.patch', basename, body)
 
73
            'x-patch', '.patch', basename)
81
74
 
82
75
    def _get_merge_prompt(self, prompt, to, subject, attachment):
83
76
        """Generate a prompt string.  Overridden by Editor.
91
84
 
92
85
 
93
86
class Editor(MailClient):
94
 
    __doc__ = """DIY mail client that uses commit message editor"""
95
 
 
96
 
    supports_body = True
 
87
    """DIY mail client that uses commit message editor"""
97
88
 
98
89
    def _get_merge_prompt(self, prompt, to, subject, attachment):
99
90
        """See MailClient._get_merge_prompt"""
104
95
                         attachment.decode('utf-8', 'replace')))
105
96
 
106
97
    def compose(self, prompt, to, subject, attachment, mime_subtype,
107
 
                extension, basename=None, body=None):
 
98
                extension, basename=None):
108
99
        """See MailClient.compose"""
109
100
        if not to:
110
101
            raise errors.NoMailAddressSpecified()
111
 
        body = msgeditor.edit_commit_message(prompt, start_message=body)
 
102
        body = msgeditor.edit_commit_message(prompt)
112
103
        if body == '':
113
104
            raise errors.NoMessageSupplied()
114
105
        email_message.EmailMessage.send(self.config,
115
 
                                        self.config.get('email'),
 
106
                                        self.config.username(),
116
107
                                        to,
117
108
                                        subject,
118
109
                                        body,
119
110
                                        attachment,
120
111
                                        attachment_mime_subtype=mime_subtype)
121
 
mail_client_registry.register('editor', Editor,
122
 
                              help=Editor.__doc__)
123
 
 
124
 
 
125
 
class BodyExternalMailClient(MailClient):
126
 
 
127
 
    supports_body = True
 
112
 
 
113
 
 
114
class ExternalMailClient(MailClient):
 
115
    """An external mail client."""
128
116
 
129
117
    def _get_client_commands(self):
130
118
        """Provide a list of commands that may invoke the mail client"""
135
123
            return self._client_commands
136
124
 
137
125
    def compose(self, prompt, to, subject, attachment, mime_subtype,
138
 
                extension, basename=None, body=None):
 
126
                extension, basename=None):
139
127
        """See MailClient.compose.
140
128
 
141
129
        Writes the attachment to a temporary file, invokes _compose.
142
130
        """
143
131
        if basename is None:
144
132
            basename = 'attachment'
145
 
        pathname = osutils.mkdtemp(prefix='bzr-mail-')
 
133
        pathname = tempfile.mkdtemp(prefix='bzr-mail-')
146
134
        attach_path = osutils.pathjoin(pathname, basename + extension)
147
135
        outfile = open(attach_path, 'wb')
148
136
        try:
149
137
            outfile.write(attachment)
150
138
        finally:
151
139
            outfile.close()
152
 
        if body is not None:
153
 
            kwargs = {'body': body}
154
 
        else:
155
 
            kwargs = {}
156
140
        self._compose(prompt, to, subject, attach_path, mime_subtype,
157
 
                      extension, **kwargs)
 
141
                      extension)
158
142
 
159
143
    def _compose(self, prompt, to, subject, attach_path, mime_subtype,
160
 
                 extension, body=None, from_=None):
 
144
                extension):
161
145
        """Invoke a mail client as a commandline process.
162
146
 
163
147
        Overridden by MAPIClient.
168
152
            "text", but the precise subtype can be specified here
169
153
        :param extension: A file extension (including period) associated with
170
154
            the attachment type.
171
 
        :param body: Optional body text.
172
 
        :param from_: Optional From: header.
173
155
        """
174
156
        for name in self._get_client_commands():
175
157
            cmdline = [self._encode_path(name, 'executable')]
176
 
            if body is not None:
177
 
                kwargs = {'body': body}
178
 
            else:
179
 
                kwargs = {}
180
 
            if from_ is not None:
181
 
                kwargs['from_'] = from_
182
158
            cmdline.extend(self._get_compose_commandline(to, subject,
183
 
                                                         attach_path,
184
 
                                                         **kwargs))
 
159
                                                         attach_path))
185
160
            try:
186
161
                subprocess.call(cmdline)
187
162
            except OSError, e:
192
167
        else:
193
168
            raise errors.MailClientNotFound(self._client_commands)
194
169
 
195
 
    def _get_compose_commandline(self, to, subject, attach_path, body):
 
170
    def _get_compose_commandline(self, to, subject, attach_path):
196
171
        """Determine the commandline to use for composing a message
197
172
 
198
173
        Implemented by various subclasses
211
186
        :return:    encoded string if u is unicode, u itself otherwise.
212
187
        """
213
188
        if isinstance(u, unicode):
214
 
            return u.encode(osutils.get_user_encoding(), 'replace')
 
189
            return u.encode(bzrlib.user_encoding, 'replace')
215
190
        return u
216
191
 
217
192
    def _encode_path(self, path, kind):
225
200
        """
226
201
        if isinstance(path, unicode):
227
202
            try:
228
 
                return path.encode(osutils.get_user_encoding())
 
203
                return path.encode(bzrlib.user_encoding)
229
204
            except UnicodeEncodeError:
230
205
                raise errors.UnableEncodePath(path, kind)
231
206
        return path
232
207
 
233
208
 
234
 
class ExternalMailClient(BodyExternalMailClient):
235
 
    __doc__ = """An external mail client."""
236
 
 
237
 
    supports_body = False
238
 
 
239
 
 
240
 
class Evolution(BodyExternalMailClient):
241
 
    __doc__ = """Evolution mail client."""
 
209
class Evolution(ExternalMailClient):
 
210
    """Evolution mail client."""
242
211
 
243
212
    _client_commands = ['evolution']
244
213
 
245
 
    def _get_compose_commandline(self, to, subject, attach_path, body=None):
 
214
    def _get_compose_commandline(self, to, subject, attach_path):
246
215
        """See ExternalMailClient._get_compose_commandline"""
247
216
        message_options = {}
248
217
        if subject is not None:
249
218
            message_options['subject'] = subject
250
219
        if attach_path is not None:
251
220
            message_options['attach'] = attach_path
252
 
        if body is not None:
253
 
            message_options['body'] = body
254
221
        options_list = ['%s=%s' % (k, urlutils.escape(v)) for (k, v) in
255
222
                        sorted(message_options.iteritems())]
256
223
        return ['mailto:%s?%s' % (self._encode_safe(to or ''),
257
224
            '&'.join(options_list))]
258
 
mail_client_registry.register('evolution', Evolution,
259
 
                              help=Evolution.__doc__)
260
 
 
261
 
 
262
 
class Mutt(BodyExternalMailClient):
263
 
    __doc__ = """Mutt mail client."""
 
225
 
 
226
 
 
227
class Mutt(ExternalMailClient):
 
228
    """Mutt mail client."""
264
229
 
265
230
    _client_commands = ['mutt']
266
231
 
267
 
    def _get_compose_commandline(self, to, subject, attach_path, body=None):
 
232
    def _get_compose_commandline(self, to, subject, attach_path):
268
233
        """See ExternalMailClient._get_compose_commandline"""
269
234
        message_options = []
270
235
        if subject is not None:
272
237
        if attach_path is not None:
273
238
            message_options.extend(['-a',
274
239
                self._encode_path(attach_path, 'attachment')])
275
 
        if body is not None:
276
 
            # Store the temp file object in self, so that it does not get
277
 
            # garbage collected and delete the file before mutt can read it.
278
 
            self._temp_file = tempfile.NamedTemporaryFile(
279
 
                prefix="mutt-body-", suffix=".txt")
280
 
            self._temp_file.write(body)
281
 
            self._temp_file.flush()
282
 
            message_options.extend(['-i', self._temp_file.name])
283
240
        if to is not None:
284
 
            message_options.extend(['--', self._encode_safe(to)])
 
241
            message_options.append(self._encode_safe(to))
285
242
        return message_options
286
 
mail_client_registry.register('mutt', Mutt,
287
 
                              help=Mutt.__doc__)
288
 
 
289
 
 
290
 
class Thunderbird(BodyExternalMailClient):
291
 
    __doc__ = """Mozilla Thunderbird (or Icedove)
 
243
 
 
244
 
 
245
class Thunderbird(ExternalMailClient):
 
246
    """Mozilla Thunderbird (or Icedove)
292
247
 
293
248
    Note that Thunderbird 1.5 is buggy and does not support setting
294
249
    "to" simultaneously with including a attachment.
298
253
    """
299
254
 
300
255
    _client_commands = ['thunderbird', 'mozilla-thunderbird', 'icedove',
301
 
        '/Applications/Mozilla/Thunderbird.app/Contents/MacOS/thunderbird-bin',
302
 
        '/Applications/Thunderbird.app/Contents/MacOS/thunderbird-bin']
 
256
        '/Applications/Mozilla/Thunderbird.app/Contents/MacOS/thunderbird-bin']
303
257
 
304
 
    def _get_compose_commandline(self, to, subject, attach_path, body=None):
 
258
    def _get_compose_commandline(self, to, subject, attach_path):
305
259
        """See ExternalMailClient._get_compose_commandline"""
306
260
        message_options = {}
307
261
        if to is not None:
311
265
        if attach_path is not None:
312
266
            message_options['attachment'] = urlutils.local_path_to_url(
313
267
                attach_path)
314
 
        if body is not None:
315
 
            options_list = ['body=%s' % urlutils.quote(self._encode_safe(body))]
316
 
        else:
317
 
            options_list = []
318
 
        options_list.extend(["%s='%s'" % (k, v) for k, v in
319
 
                        sorted(message_options.iteritems())])
 
268
        options_list = ["%s='%s'" % (k, v) for k, v in
 
269
                        sorted(message_options.iteritems())]
320
270
        return ['-compose', ','.join(options_list)]
321
 
mail_client_registry.register('thunderbird', Thunderbird,
322
 
                              help=Thunderbird.__doc__)
323
271
 
324
272
 
325
273
class KMail(ExternalMailClient):
326
 
    __doc__ = """KDE mail client."""
 
274
    """KDE mail client."""
327
275
 
328
276
    _client_commands = ['kmail']
329
277
 
338
286
        if to is not None:
339
287
            message_options.extend([self._encode_safe(to)])
340
288
        return message_options
341
 
mail_client_registry.register('kmail', KMail,
342
 
                              help=KMail.__doc__)
343
 
 
344
 
 
345
 
class Claws(ExternalMailClient):
346
 
    __doc__ = """Claws mail client."""
347
 
 
348
 
    supports_body = True
349
 
 
350
 
    _client_commands = ['claws-mail']
351
 
 
352
 
    def _get_compose_commandline(self, to, subject, attach_path, body=None,
353
 
                                 from_=None):
354
 
        """See ExternalMailClient._get_compose_commandline"""
355
 
        compose_url = []
356
 
        if from_ is not None:
357
 
            compose_url.append('from=' + urlutils.quote(from_))
358
 
        if subject is not None:
359
 
            # Don't use urlutils.quote_plus because Claws doesn't seem
360
 
            # to recognise spaces encoded as "+".
361
 
            compose_url.append(
362
 
                'subject=' + urlutils.quote(self._encode_safe(subject)))
363
 
        if body is not None:
364
 
            compose_url.append(
365
 
                'body=' + urlutils.quote(self._encode_safe(body)))
366
 
        # to must be supplied for the claws-mail --compose syntax to work.
367
 
        if to is None:
368
 
            raise errors.NoMailAddressSpecified()
369
 
        compose_url = 'mailto:%s?%s' % (
370
 
            self._encode_safe(to), '&'.join(compose_url))
371
 
        # Collect command-line options.
372
 
        message_options = ['--compose', compose_url]
373
 
        if attach_path is not None:
374
 
            message_options.extend(
375
 
                ['--attach', self._encode_path(attach_path, 'attachment')])
376
 
        return message_options
377
 
 
378
 
    def _compose(self, prompt, to, subject, attach_path, mime_subtype,
379
 
                 extension, body=None, from_=None):
380
 
        """See ExternalMailClient._compose"""
381
 
        if from_ is None:
382
 
            from_ = self.config.get('email')
383
 
        super(Claws, self)._compose(prompt, to, subject, attach_path,
384
 
                                    mime_subtype, extension, body, from_)
385
 
 
386
 
 
387
 
mail_client_registry.register('claws', Claws,
388
 
                              help=Claws.__doc__)
389
 
 
390
 
 
391
 
class XDGEmail(BodyExternalMailClient):
392
 
    __doc__ = """xdg-email attempts to invoke the user's preferred mail client"""
 
289
 
 
290
 
 
291
class XDGEmail(ExternalMailClient):
 
292
    """xdg-email attempts to invoke the user's preferred mail client"""
393
293
 
394
294
    _client_commands = ['xdg-email']
395
295
 
396
 
    def _get_compose_commandline(self, to, subject, attach_path, body=None):
 
296
    def _get_compose_commandline(self, to, subject, attach_path):
397
297
        """See ExternalMailClient._get_compose_commandline"""
398
298
        if not to:
399
299
            raise errors.NoMailAddressSpecified()
403
303
        if attach_path is not None:
404
304
            commandline.extend(['--attach',
405
305
                self._encode_path(attach_path, 'attachment')])
406
 
        if body is not None:
407
 
            commandline.extend(['--body', self._encode_safe(body)])
408
306
        return commandline
409
 
mail_client_registry.register('xdg-email', XDGEmail,
410
 
                              help=XDGEmail.__doc__)
411
307
 
412
308
 
413
309
class EmacsMail(ExternalMailClient):
414
 
    __doc__ = """Call emacsclient to have a mail buffer.
 
310
    """Call emacsclient to have a mail buffer.
415
311
 
416
312
    This only work for emacs >= 22.1 due to recent -e/--eval support.
417
313
 
426
322
 
427
323
    _client_commands = ['emacsclient']
428
324
 
429
 
    def __init__(self, config):
430
 
        super(EmacsMail, self).__init__(config)
431
 
        self.elisp_tmp_file = None
432
 
 
433
325
    def _prepare_send_function(self):
434
326
        """Write our wrapper function into a temporary file.
435
327
 
436
328
        This temporary file will be loaded at runtime in
437
329
        _get_compose_commandline function.
438
330
 
439
 
        This function does not remove the file.  That's a wanted
 
331
        FIXME: this function does not remove the file. That's a wanted
440
332
        behaviour since _get_compose_commandline won't run the send
441
333
        mail function directly but return the eligible command line.
442
334
        Removing our temporary file here would prevent our sendmail
443
 
        function to work.  (The file is deleted by some elisp code
444
 
        after being read by Emacs.)
 
335
        function to work.
 
336
 
 
337
        A possible workaround could be to load the function here with
 
338
        emacsclient --eval '(load temp)' but this is not robust since
 
339
        emacs could have been stopped between here and the call to
 
340
        mail client.
445
341
        """
446
342
 
447
343
        _defun = r"""(defun bzr-add-mime-att (file)
448
 
  "Attach FILE to a mail buffer as a MIME attachment."
 
344
  "Attach FILe to a mail buffer as a MIME attachment."
449
345
  (let ((agent mail-user-agent))
 
346
    (mail-text)
 
347
    (newline)
450
348
    (if (and file (file-exists-p file))
451
349
        (cond
452
350
         ((eq agent 'sendmail-user-agent)
453
 
          (progn
454
 
            (mail-text)
455
 
            (newline)
456
 
            (if (functionp 'etach-attach)
457
 
              (etach-attach file)
458
 
              (mail-attach-file file))))
459
 
         ((or (eq agent 'message-user-agent)
460
 
              (eq agent 'gnus-user-agent)
461
 
              (eq agent 'mh-e-user-agent))
462
 
          (progn
463
 
            (mml-attach-file file "text/x-patch" "BZR merge" "inline")))
464
 
         ((eq agent 'mew-user-agent)
465
 
          (progn
466
 
            (mew-draft-prepare-attachments)
467
 
            (mew-attach-link file (file-name-nondirectory file))
468
 
            (let* ((nums (mew-syntax-nums))
469
 
                   (syntax (mew-syntax-get-entry mew-encode-syntax nums)))
470
 
              (mew-syntax-set-cd syntax "BZR merge")
471
 
              (mew-encode-syntax-print mew-encode-syntax))
472
 
            (mew-header-goto-body)))
 
351
          (etach-attach file))
 
352
         ((or (eq agent 'message-user-agent)(eq agent 'gnus-user-agent))
 
353
          (mml-attach-file file "text/x-patch" "BZR merge" "attachment"))
473
354
         (t
474
 
          (message "Unhandled MUA, report it on bazaar@lists.canonical.com")))
 
355
          (message "Unhandled MUA")))
475
356
      (error "File %s does not exist." file))))
476
357
"""
477
358
 
490
371
        _subject = "nil"
491
372
 
492
373
        if to is not None:
493
 
            _to = ("\"%s\"" % self._encode_safe(to).replace('"', '\\"'))
 
374
            _to = ("\"%s\"" % self._encode_safe(to))
494
375
        if subject is not None:
495
 
            _subject = ("\"%s\"" %
496
 
                        self._encode_safe(subject).replace('"', '\\"'))
 
376
            _subject = ("\"%s\"" % self._encode_safe(subject))
497
377
 
498
378
        # Funcall the default mail composition function
499
379
        # This will work with any mail mode including default mail-mode
505
385
        # Try to attach a MIME attachment using our wrapper function
506
386
        if attach_path is not None:
507
387
            # Do not create a file if there is no attachment
508
 
            elisp = self._prepare_send_function()
509
 
            self.elisp_tmp_file = elisp
510
 
            lmmform = '(load "%s")' % elisp
 
388
            lmmform = '(load "%s")' % self._prepare_send_function()
511
389
            mmform  = '(bzr-add-mime-att "%s")' % \
512
390
                self._encode_path(attach_path, 'attachment')
513
 
            rmform = '(delete-file "%s")' % elisp
514
391
            commandline.append(lmmform)
515
392
            commandline.append(mmform)
516
 
            commandline.append(rmform)
517
393
 
518
394
        return commandline
519
 
mail_client_registry.register('emacsclient', EmacsMail,
520
 
                              help=EmacsMail.__doc__)
521
 
 
522
 
 
523
 
class MAPIClient(BodyExternalMailClient):
524
 
    __doc__ = """Default Windows mail client launched using MAPI."""
 
395
 
 
396
 
 
397
class MAPIClient(ExternalMailClient):
 
398
    """Default Windows mail client launched using MAPI."""
525
399
 
526
400
    def _compose(self, prompt, to, subject, attach_path, mime_subtype,
527
 
                 extension, body=None):
 
401
                 extension):
528
402
        """See ExternalMailClient._compose.
529
403
 
530
404
        This implementation uses MAPI via the simplemapi ctypes wrapper
531
405
        """
532
406
        from bzrlib.util import simplemapi
533
407
        try:
534
 
            simplemapi.SendMail(to or '', subject or '', body or '',
535
 
                                attach_path)
 
408
            simplemapi.SendMail(to or '', subject or '', '', attach_path)
536
409
        except simplemapi.MAPIError, e:
537
410
            if e.code != simplemapi.MAPI_USER_ABORT:
538
411
                raise errors.MailClientNotFound(['MAPI supported mail client'
539
412
                                                 ' (error %d)' % (e.code,)])
540
 
mail_client_registry.register('mapi', MAPIClient,
541
 
                              help=MAPIClient.__doc__)
542
 
 
543
 
 
544
 
class MailApp(BodyExternalMailClient):
545
 
    __doc__ = """Use MacOS X's Mail.app for sending email messages.
546
 
 
547
 
    Although it would be nice to use appscript, it's not installed
548
 
    with the shipped Python installations.  We instead build an
549
 
    AppleScript and invoke the script using osascript(1).  We don't
550
 
    use the _encode_safe() routines as it's not clear what encoding
551
 
    osascript expects the script to be in.
552
 
    """
553
 
 
554
 
    _client_commands = ['osascript']
555
 
 
556
 
    def _get_compose_commandline(self, to, subject, attach_path, body=None,
557
 
                                from_=None):
558
 
       """See ExternalMailClient._get_compose_commandline"""
559
 
 
560
 
       fd, self.temp_file = tempfile.mkstemp(prefix="bzr-send-",
561
 
                                         suffix=".scpt")
562
 
       try:
563
 
           os.write(fd, 'tell application "Mail"\n')
564
 
           os.write(fd, 'set newMessage to make new outgoing message\n')
565
 
           os.write(fd, 'tell newMessage\n')
566
 
           if to is not None:
567
 
               os.write(fd, 'make new to recipient with properties'
568
 
                   ' {address:"%s"}\n' % to)
569
 
           if from_ is not None:
570
 
               # though from_ doesn't actually seem to be used
571
 
               os.write(fd, 'set sender to "%s"\n'
572
 
                   % sender.replace('"', '\\"'))
573
 
           if subject is not None:
574
 
               os.write(fd, 'set subject to "%s"\n'
575
 
                   % subject.replace('"', '\\"'))
576
 
           if body is not None:
577
 
               # FIXME: would be nice to prepend the body to the
578
 
               # existing content (e.g., preserve signature), but
579
 
               # can't seem to figure out the right applescript
580
 
               # incantation.
581
 
               os.write(fd, 'set content to "%s\\n\n"\n' %
582
 
                   body.replace('"', '\\"').replace('\n', '\\n'))
583
 
 
584
 
           if attach_path is not None:
585
 
               # FIXME: would be nice to first append a newline to
586
 
               # ensure the attachment is on a new paragraph, but
587
 
               # can't seem to figure out the right applescript
588
 
               # incantation.
589
 
               os.write(fd, 'tell content to make new attachment'
590
 
                   ' with properties {file name:"%s"}'
591
 
                   ' at after the last paragraph\n'
592
 
                   % self._encode_path(attach_path, 'attachment'))
593
 
           os.write(fd, 'set visible to true\n')
594
 
           os.write(fd, 'end tell\n')
595
 
           os.write(fd, 'end tell\n')
596
 
       finally:
597
 
           os.close(fd) # Just close the handle but do not remove the file.
598
 
       return [self.temp_file]
599
 
mail_client_registry.register('mail.app', MailApp,
600
 
                              help=MailApp.__doc__)
601
413
 
602
414
 
603
415
class DefaultMail(MailClient):
604
 
    __doc__ = """Default mail handling.  Tries XDGEmail (or MAPIClient on Windows),
 
416
    """Default mail handling.  Tries XDGEmail (or MAPIClient on Windows),
605
417
    falls back to Editor"""
606
418
 
607
 
    supports_body = True
608
 
 
609
419
    def _mail_client(self):
610
420
        """Determine the preferred mail client for this platform"""
611
421
        if osutils.supports_mapi():
614
424
            return XDGEmail(self.config)
615
425
 
616
426
    def compose(self, prompt, to, subject, attachment, mime_subtype,
617
 
                extension, basename=None, body=None):
 
427
                extension, basename=None):
618
428
        """See MailClient.compose"""
619
429
        try:
620
430
            return self._mail_client().compose(prompt, to, subject,
621
 
                                               attachment, mime_subtype,
622
 
                                               extension, basename, body)
 
431
                                               attachment, mimie_subtype,
 
432
                                               extension, basename)
623
433
        except errors.MailClientNotFound:
624
434
            return Editor(self.config).compose(prompt, to, subject,
625
 
                          attachment, mime_subtype, extension, body)
 
435
                          attachment, mimie_subtype, extension)
626
436
 
627
 
    def compose_merge_request(self, to, subject, directive, basename=None,
628
 
                              body=None):
 
437
    def compose_merge_request(self, to, subject, directive, basename=None):
629
438
        """See MailClient.compose_merge_request"""
630
439
        try:
631
440
            return self._mail_client().compose_merge_request(to, subject,
632
 
                    directive, basename=basename, body=body)
 
441
                    directive, basename=basename)
633
442
        except errors.MailClientNotFound:
634
443
            return Editor(self.config).compose_merge_request(to, subject,
635
 
                          directive, basename=basename, body=body)
636
 
mail_client_registry.register('default', DefaultMail,
637
 
                              help=DefaultMail.__doc__)
638
 
mail_client_registry.default_key = 'default'
639
 
 
640
 
opt_mail_client = _mod_config.RegistryOption('mail_client',
641
 
        mail_client_registry, help='E-mail client to use.', invalid='error')
 
444
                          directive, basename=basename)