From 75aaef0f3dc98a65f99add393a73351c81c0f941 Mon Sep 17 00:00:00 2001
From: Francis Lachapelle <flachapelle@inverse.ca>
Date: Wed, 22 Mar 2017 14:53:07 -0400
Subject: [PATCH] (js) Restrict special mailboxes to the top level

Fixes #6963
---
 NEWS                                               |  1 +
 UI/Templates/MailerUI/UIxMailMainFrame.wox         |  8 ++++----
 UI/WebServerResources/js/Mailer/Mailbox.service.js | 10 ++++++++++
 3 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/NEWS b/NEWS
index 81ef32b125..0c067a7847 100644
--- a/NEWS
+++ b/NEWS
@@ -8,6 +8,7 @@ Enhancements
  - [web] constrain event/task reminder to a positive number
  - [web] display year in day and week views
  - [web] split string on comma and semicolon when pasting multiple addresses (#4097)
+ - [web] restrict Draft/Sent/Trash/Junk mailboxes to the top level (#6963)
 
 Bug fixes
  - [core] handle broken CalDAV clients sending bogus SENT-BY (#3992)
diff --git a/UI/Templates/MailerUI/UIxMailMainFrame.wox b/UI/Templates/MailerUI/UIxMailMainFrame.wox
index de90f07e84..743b27b648 100644
--- a/UI/Templates/MailerUI/UIxMailMainFrame.wox
+++ b/UI/Templates/MailerUI/UIxMailMainFrame.wox
@@ -161,23 +161,23 @@
                         <var:string label:value="Search"/>
                       </md-button>
                     </md-menu-item>
-                    <md-menu-divider ng-show="folder.type == 'folder'"><!-- divider --></md-menu-divider>
+                    <md-menu-divider ng-show="folder.$canFolderAs()"><!-- divider --></md-menu-divider>
                     <md-menu-item ng-show="folder.type == 'folder'">
                       <md-button type="button" ng-click="app.setFolderAs(folder, 'Drafts')">
                         <var:string label:value="Set as Drafts"/>
                       </md-button>
                     </md-menu-item>
-                    <md-menu-item ng-show="folder.type == 'folder'">
+                    <md-menu-item ng-show="folder.$canFolderAs()">
                       <md-button type="button" ng-click="app.setFolderAs(folder, 'Sent')">
                         <var:string label:value="Set as Sent"/>
                       </md-button>
                     </md-menu-item>
-                    <md-menu-item ng-show="folder.type == 'folder'">
+                    <md-menu-item ng-show="folder.$canFolderAs()">
                       <md-button type="button" ng-click="app.setFolderAs(folder, 'Trash')">
                         <var:string label:value="Set as Trash"/>
                       </md-button>
                     </md-menu-item>
-                    <md-menu-item ng-show="folder.type == 'folder'">
+                    <md-menu-item ng-show="folder.$canFolderAs()">
                       <md-button type="button" ng-click="app.setFolderAs(folder, 'Junk')">
                         <var:string label:value="Set as Junk"/>
                       </md-button>
diff --git a/UI/WebServerResources/js/Mailer/Mailbox.service.js b/UI/WebServerResources/js/Mailer/Mailbox.service.js
index abba6073da..bb56f3f069 100644
--- a/UI/WebServerResources/js/Mailer/Mailbox.service.js
+++ b/UI/WebServerResources/js/Mailer/Mailbox.service.js
@@ -571,6 +571,16 @@
       });
   };
 
+  /**
+   * @function $canFolderAs
+   * @memberof Mailbox.prototype
+   * @desc Check if the folder can be set as Drafts/Sent/Trash
+   * @returns true if folder is eligible
+   */
+  Mailbox.prototype.$canFolderAs = function() {
+    return this.type == 'folder' && this.level === 0;
+  };
+
   /**
    * @function $setFolderAs
    * @memberof Mailbox.prototype