fix(mail(js)): add missing library to save msg from popup window

Fixes #4879
Alinto · Nov 18, 2019 · 7298022 · 7298022
1 parent e458a78
commit 7298022
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/UI/Templates/MailerUI/UIxMailPopupView.wox b/UI/Templates/MailerUI/UIxMailPopupView.wox
@@ -9,7 +9,7 @@
     className="UIxPageFrame"
     title="title"
     const:popup="YES"
-    const:jsFiles="Common.js, Preferences.services.js, Contacts.services.js, Scheduler.services.js, Mailer.app.popup.js, Mailer.services.js, vendor/ckeditor/ckeditor.js, vendor/ckeditor/ck.js, vendor/angular-file-upload.min.js">
+    const:jsFiles="Common.js, Preferences.services.js, Contacts.services.js, Scheduler.services.js, Mailer.app.popup.js, Mailer.services.js, vendor/ckeditor/ckeditor.js, vendor/ckeditor/ck.js, vendor/angular-file-upload.min.js, vendor/FileSaver.min.js">
 
   <main class="layout-fill" ng-controller="navController">
     <div id="detailView" class="view-detail md-default-theme md-background md-bg md-hue-2"

diff --git a/UI/WebServerResources/js/Common/Resource.service.js b/UI/WebServerResources/js/Common/Resource.service.js
@@ -191,6 +191,10 @@
     if (action) path.push(action);
     path = _.compact(_.flatten(path)).join('/');
 
+    if (typeof saveAs == 'undefined') {
+      throw new Error('To use Resource.download, FileSaver.js must be loaded.');
+    }
+
     function getFileNameFromHeader(header) {
       var result;
 
@@ -224,12 +228,7 @@
         else {
           getFileNameFromHeader(headers('content-disposition'));
         }
-        if (!saveAs) {
-          throw new Error('To use Resource.download, FileSaver.js must be loaded.');
-        }
-        else {
-          saveAs(blob, fileName);
-        }
+        saveAs(blob, fileName);
       }
     });
   };