Bug 676754 - Detaching a single attachment doesn't work; r=protz,a=Standard8
authorJim Porter <squibblyflabbetydoo@gmail.com>
Tue, 09 Aug 2011 19:54:43 -0500
changeset 8373 4eb158b9555d8b6e2d516b3daf7cdb77ed1d5954
parent 8372 9ef10c2815edbd84f226af283f1edffc81214aa1
child 8374 c2b9aaff6ff58508c47db9d5a0a76e2e9aaf2e94
push idunknown
push userunknown
push dateunknown
reviewersprotz, Standard8
bugs676754
Bug 676754 - Detaching a single attachment doesn't work; r=protz,a=Standard8
mail/base/content/msgHdrViewOverlay.js
--- a/mail/base/content/msgHdrViewOverlay.js
+++ b/mail/base/content/msgHdrViewOverlay.js
@@ -2339,17 +2339,17 @@ function HandleMultipleAttachments(attac
                                      attachmentDisplayNameArray,
                                      attachmentMessageUriArray);
         return;
       case "detach":
         // 'detach' on a multiple selection of attachments is so far not really
         // supported. As a workaround, resort to normal detach-'all'. See also
         // the comment on 'detaching a multiple selection of attachments' below.
         if (attachments.length == 1)
-          detachAttachment(attachments[0], true);
+          attachments[0].detach(true);
         else
           messenger.detachAllAttachments(attachmentContentTypeArray.length,
                                          attachmentContentTypeArray,
                                          attachmentUrlArray,
                                          attachmentDisplayNameArray,
                                          attachmentMessageUriArray,
                                          true); // save
         return;