From 737d2ffccc6af16d16f47d79fe876197c2ca6217 Mon Sep 17 00:00:00 2001
From: Ludovic Marcotte <lmarcotte@inverse.ca>
Date: Fri, 22 Jan 2016 10:32:13 -0500
Subject: [PATCH] (feat) UI for previously commited new feature (junk/not junk)

---
 UI/MailerUI/Toolbars/SOGoMailFolder.toolbar | 12 ++--
 UI/MailerUI/Toolbars/SOGoMailObject.toolbar | 12 ++--
 UI/Templates/MailerUI/UIxMailMainFrame.wox  |  1 +
 UI/Templates/MailerUI/UIxMailPopupView.wox  |  1 +
 UI/WebServerResources/MailerUI.js           | 69 ++++++++++++++++++++-
 UI/WebServerResources/MailerUIdTree.js      |  6 +-
 UI/WebServerResources/UIxMailPopupView.js   | 14 +++++
 7 files changed, 100 insertions(+), 15 deletions(-)

diff --git a/UI/MailerUI/Toolbars/SOGoMailFolder.toolbar b/UI/MailerUI/Toolbars/SOGoMailFolder.toolbar
index 119b2ac9b..d052c16e3 100644
--- a/UI/MailerUI/Toolbars/SOGoMailFolder.toolbar
+++ b/UI/MailerUI/Toolbars/SOGoMailFolder.toolbar
@@ -49,12 +49,12 @@
       cssClass = "tbicon_delete";
       label = "Delete";
       tooltip = "Delete selected message or folder"; },
-//    { link = "#";
-//      isSafe = NO;
-//      image = "tb-mail-junk-flat-24x24.png";
-//      cssClass = "tbicon_junk";
-//      label = "Junk";
-//      tooltip = "Mark the selected messages as junk"; },
+    { link = "#";
+      isSafe = NO;
+      image = "tb-mail-junk-flat-24x24.png";
+      cssClass = "tbicon_junk";
+      label = "Junk";
+      tooltip = "Mark the selected messages as junk"; },
   ),
   (
    { link = "#";
diff --git a/UI/MailerUI/Toolbars/SOGoMailObject.toolbar b/UI/MailerUI/Toolbars/SOGoMailObject.toolbar
index 062af0643..ef0a59ef6 100644
--- a/UI/MailerUI/Toolbars/SOGoMailObject.toolbar
+++ b/UI/MailerUI/Toolbars/SOGoMailObject.toolbar
@@ -33,12 +33,12 @@
       cssClass = "tbicon_delete";
       label = "Delete";
       tooltip = "Delete selected message or folder"; },
-//    { link = "#";
-//      isSafe = NO;
-//      image = "tb-mail-junk-flat-24x24.png";
-//      cssClass = "tbicon_junk";
-//      label = "Junk";
-//      tooltip = "Mark the selected messages as junk"; },
+    { link = "#";
+      isSafe = NO;
+      image = "tb-mail-junk-flat-24x24.png";
+      cssClass = "tbicon_junk";
+      label = "Junk";
+      tooltip = "Mark the selected messages as junk"; },
   ),
   (
    { link = "#";
diff --git a/UI/Templates/MailerUI/UIxMailMainFrame.wox b/UI/Templates/MailerUI/UIxMailMainFrame.wox
index 4aaf37b3a..78f70fc4d 100644
--- a/UI/Templates/MailerUI/UIxMailMainFrame.wox
+++ b/UI/Templates/MailerUI/UIxMailMainFrame.wox
@@ -117,6 +117,7 @@
       <li><var:string label:value="Sent Messages" /></li>
       <li><var:string label:value="Drafts" /></li>
       <li><var:string label:value="Deleted Messages" /></li>
+      <li><var:string label:value="Junk Messages" /></li>
     </ul>
   </div>
   
diff --git a/UI/Templates/MailerUI/UIxMailPopupView.wox b/UI/Templates/MailerUI/UIxMailPopupView.wox
index f3d17c577..3a99ae5d4 100644
--- a/UI/Templates/MailerUI/UIxMailPopupView.wox
+++ b/UI/Templates/MailerUI/UIxMailPopupView.wox
@@ -12,6 +12,7 @@
     <script type="text/javascript">
       var messageName = '<var:string value="clientObject.relativeImap4Name"/>';
       var mailboxName = '/<var:string value="clientObject.container.container.nameInContainer"/>/<var:string value="clientObject.container.nameInContainer"/>';
+      var mailboxType = '<var:string value="clientObject.container.className"/>';
     </script>
     <var:component className="UIxMailView" />
     <div class="menu" id="addressMenu">
diff --git a/UI/WebServerResources/MailerUI.js b/UI/WebServerResources/MailerUI.js
index d584729cb..f3c3e9d22 100644
--- a/UI/WebServerResources/MailerUI.js
+++ b/UI/WebServerResources/MailerUI.js
@@ -644,6 +644,45 @@ function onMenuDeleteMessage(event) {
     preventDefault(event);
 }
 
+function onMarkOrUnmarkMessagesAsJunk(is_junk) {
+  var targetMailbox;
+  var messageList = $("messageListBody").down("TBODY");
+  var rowIds = messageList.getSelectedNodesId();
+  var uids = new Array(); // message IDs
+
+  for (var i = 0; i < rowIds.length; i++) {
+    var uid = rowIds[i].substr(4);
+    uids.push(uid);
+  }
+
+  var url;
+
+  if (is_junk)
+    url = ApplicationBaseURL + encodeURI(Mailer.currentMailbox) + "/markMessagesAsNotJunk";
+  else
+    url = ApplicationBaseURL + encodeURI(Mailer.currentMailbox) + "/markMessagesAsJunk";
+
+  var data = { "uids": uids };
+  triggerAjaxRequest(url, onMarkOrUnmarkMessagesAsJunkCallback, data, Object.toJSON(data));
+
+  return false;
+}
+
+function onMarkOrUnmarkMessagesAsJunkCallback(http) {
+  // If all went well, lets move the message
+  if (isHttpStatus204(http.status) || http.status == 200) {
+    var func, item;
+
+    if (Mailer.currentMailboxType == "junk") {
+      item = $$("[datatype=inbox]").first();
+    } else {
+      item = $$("[datatype=junk]").first();
+    }
+    func = onMailboxMenuMove.bind(item);
+    func();
+  }
+}
+
 /**
  * The following two functions are called from UIxMailPopupView
  * with window.opener.
@@ -682,9 +721,26 @@ function onPrintCurrentMessage(event) {
     preventDefault(event);
 }
 
+function toggleJunkAction(isJunk) {
+  var button = $$(".tbicon_junk").first();
+  button.stopObserving("click");
+
+  if (isJunk) {
+    button.title = "Mark the selected messages as not junk";
+    button.select('span').first().childNodes[3].nodeValue = "Not junk";
+  }
+  else {
+    button.title = "Mark the selected messages as junk";
+    button.select('span').first().childNodes[3].nodeValue = "Junk";
+  }
+
+  button.on("click", onMarkOrUnmarkMessagesAsJunk.bind(button, isJunk));
+}
+
 function onMailboxTreeItemClick(event) {
     var topNode = $("mailboxTree");
     var mailbox = this.parentNode.getAttribute("dataname");
+    var is_junk = false;
     if (topNode.selectedEntry)
         topNode.selectedEntry.deselect();
     this.selectElement();
@@ -707,6 +763,8 @@ function onMailboxTreeItemClick(event) {
         var datatype = this.parentNode.getAttribute("datatype");
         if (datatype == 'draft' || datatype == 'sent')
             toggleAddressColumn("from", "to");
+        else if (datatype == 'junk')
+            is_junk = true;
         else
             toggleAddressColumn("to", "from");
 
@@ -714,6 +772,8 @@ function onMailboxTreeItemClick(event) {
         openMailbox(mailbox);
     }
 
+    toggleJunkAction(is_junk);
+
     Event.stop(event);
 }
 
@@ -2073,6 +2133,8 @@ function initMailer(event) {
         initMailboxTree();
         initRefreshViewCheckTimer();
 
+        toggleJunkAction(false);
+
         Event.observe(document, "keydown", onDocumentKeydown);
 
         /* Perform an expunge when leaving the webmail */
@@ -2555,6 +2617,10 @@ function onMenuChangeToTrashFolder(event) {
     return _onMenuChangeToXXXFolder(event, "Trash");
 }
 
+function onMenuChangeToJunkFolder(event) {
+    return _onMenuChangeToXXXFolder(event, "Junk");
+}
+
 function onMenuToggleMessageRead(event) {
     mailListToggleMessagesRead();
 }
@@ -2956,7 +3022,8 @@ function getMenus() {
                               onMenuDeleteMessage ],
         folderTypeMenu: [ onMenuChangeToSentFolder,
                           onMenuChangeToDraftsFolder,
-                          onMenuChangeToTrashFolder  ],
+                          onMenuChangeToTrashFolder,
+                          onMenuChangeToJunkFolder ],
 
         "label-menu": [ onMenuLabelNone, "-" ],
 
diff --git a/UI/WebServerResources/MailerUIdTree.js b/UI/WebServerResources/MailerUIdTree.js
index d37149a64..c7363c625 100644
--- a/UI/WebServerResources/MailerUIdTree.js
+++ b/UI/WebServerResources/MailerUIdTree.js
@@ -4,11 +4,13 @@ var MailerUIdTreeExtension = {
                    inbox: "tbtv_inbox_17x17.png",
                    sent: "tbtv_sent_17x17.png",
                    draft: "tbtv_drafts_17x17.png",
-                   trash: "tbtv_trash_17x17.png" },
+                   trash: "tbtv_trash_17x17.png",
+                   junk: "tb-mail-junk-flat-17x17.png" },
     folderNames: { inbox: _("InboxFolderName"),
                    sent: _("SentFolderName"),
                    draft: _("DraftsFolderName"),
-                   trash: _("TrashFolderName") },
+                   trash: _("TrashFolderName"),
+                   junk: _("JunkFolderName") },
     _addFolderNode: function (parent, name, fullName, type, unseen) {
         var icon = this.folderIcons[type];
         if (icon)
diff --git a/UI/WebServerResources/UIxMailPopupView.js b/UI/WebServerResources/UIxMailPopupView.js
index e5b619375..f0cbbaec3 100644
--- a/UI/WebServerResources/UIxMailPopupView.js
+++ b/UI/WebServerResources/UIxMailPopupView.js
@@ -23,6 +23,20 @@ function initPopupMailer(event) {
     var td = $("subject");
     if (td)
         document.title = td.allTextContent();
+
+    var button = $$(".tbicon_junk").first();
+    button.stopObserving("click");
+
+    if (window.mailboxType == "SOGoJunkFolder") {
+        button.title = "Mark the selected messages as not junk";
+        button.select('span').first().childNodes[3].nodeValue = "Not junk";
+    }
+    else {
+        button.title = "Mark the selected messages as junk";
+        button.select('span').first().childNodes[3].nodeValue = "Junk";
+    }
+
+    button.on("click", window.opener.onMarkOrUnmarkMessagesAsJunk.bind(button, (window.mailboxType == "SOGoJunkFolder")));
 }
 
 function onICalendarButtonClick(event) {