From 86f08a2380147bf0cb1ed3f618aa90b116dc838f Mon Sep 17 00:00:00 2001
From: Francis Lachapelle <flachapelle@inverse.ca>
Date: Thu, 19 Aug 2021 17:15:12 -0400
Subject: [PATCH] fix(mail(js)): show "Download all attachments" menu option

---
 UI/Templates/MailerUI/UIxMailViewTemplate.wox    |  4 ++--
 .../js/Mailer/Message.service.js                 | 16 ++++++++++++++++
 2 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/UI/Templates/MailerUI/UIxMailViewTemplate.wox b/UI/Templates/MailerUI/UIxMailViewTemplate.wox
index 5ac46300d..7d93e1299 100644
--- a/UI/Templates/MailerUI/UIxMailViewTemplate.wox
+++ b/UI/Templates/MailerUI/UIxMailViewTemplate.wox
@@ -41,7 +41,7 @@
             <md-icon>reply</md-icon>
           </md-button>
           <md-button class="sg-icon-button"
-                     ng-show="viewer.message.allowReplyAll()"
+                     ng-show="::viewer.message.allowReplyAll()"
                      ng-click="viewer.replyAll($event)"
                      label:aria-label="Reply All">
             <md-tooltip md-direction="bottom"><var:string label:value="Reply to sender and all recipients"/></md-tooltip>
@@ -92,7 +92,7 @@
                   <var:string label:value="Download message (zip)"/>
                 </md-button>
               </md-menu-item>
-              <md-menu-item ng-show="::viewer.message.attachmentAttrs.length">
+              <md-menu-item ng-show="::viewer.message.hasAttachments()">
                 <md-button label:aria-label="Download all attachments"
                            ng-click="viewer.message.downloadAttachmentsArchive()">
                   <var:string label:value="Download all attachments"/>
diff --git a/UI/WebServerResources/js/Mailer/Message.service.js b/UI/WebServerResources/js/Mailer/Message.service.js
index d2dc05e22..bd690c71c 100644
--- a/UI/WebServerResources/js/Mailer/Message.service.js
+++ b/UI/WebServerResources/js/Mailer/Message.service.js
@@ -556,6 +556,22 @@
     return Message.$$resource.post(this.$absolutePath(), 'sendMDN');
   };
 
+  /**
+   * @function hasAttachments
+   * @memberof Message.prototype
+   * @returns true if there's one ore more attached files
+   */
+  Message.prototype.hasAttachments = function(content) {
+    var _this = this;
+
+    return !!_.find(content || this.parts.content, function(part) {
+      if (angular.isArray(part.content)) {
+        return _this.hasAttachments(part.content);
+      }
+      return part.type == 'UIxMailPartLinkViewer' || part.type == 'UIxMailPartImageViewer';
+    });
+  };
+
   /**
    * @function $deleteAttachment
    * @memberof Message.prototype