From c6f17cd45cff78a378da62c15c074c6bd895dc44 Mon Sep 17 00:00:00 2001
From: Ludovic Marcotte <lmarcotte@inverse.ca>
Date: Wed, 19 Aug 2015 08:47:33 -0400
Subject: [PATCH] (feat) make use of the sieve capabilities

---
 UI/PreferencesUI/UIxPreferences.m             |  2 +-
 UI/Templates/PreferencesUI/UIxPreferences.wox |  1 +
 .../js/Preferences/FiltersDialogController.js | 31 +++++++++++++------
 .../js/Preferences/PreferencesController.js   |  6 ++--
 4 files changed, 27 insertions(+), 13 deletions(-)

diff --git a/UI/PreferencesUI/UIxPreferences.m b/UI/PreferencesUI/UIxPreferences.m
index 8ed908ebf..1d478f652 100644
--- a/UI/PreferencesUI/UIxPreferences.m
+++ b/UI/PreferencesUI/UIxPreferences.m
@@ -1064,7 +1064,7 @@ static NSArray *reminderValues = nil;
 
   if (!capabilities)
     {
-      if ([self _sieveClient])
+      if ([self isSieveScriptsEnabled] && [self _sieveClient])
         capabilities = [[self _sieveClient] capabilities];
       else
         capabilities = [NSArray array];
diff --git a/UI/Templates/PreferencesUI/UIxPreferences.wox b/UI/Templates/PreferencesUI/UIxPreferences.wox
index 68910641d..c0d575f1e 100644
--- a/UI/Templates/PreferencesUI/UIxPreferences.wox
+++ b/UI/Templates/PreferencesUI/UIxPreferences.wox
@@ -18,6 +18,7 @@
     var timeZonesList = <var:string value="timeZonesList" const:escapeHTML="NO"/>;
     var defaultEmailAddresses = '<var:string value="defaultEmailAddresses" const:escapeHTML="NO"/>';
     var forwardConstraints = <var:string value="forwardConstraints" const:escapeHTML="NO"/>;
+    var sieveCapabilities = <var:string value="sieveCapabilities" const:escapeHTML="NO"/>;
   </script>
 
   <script type="text/ng-template" id="preferences.html">
diff --git a/UI/WebServerResources/js/Preferences/FiltersDialogController.js b/UI/WebServerResources/js/Preferences/FiltersDialogController.js
index 240cee8de..ecafae7ec 100644
--- a/UI/WebServerResources/js/Preferences/FiltersDialogController.js
+++ b/UI/WebServerResources/js/Preferences/FiltersDialogController.js
@@ -7,8 +7,8 @@
   /**
    * @ngInject
    */
-  FiltersDialogController.$inject = ['$scope', '$mdDialog', 'filter', 'mailboxes', 'labels'];
-  function FiltersDialogController($scope, $mdDialog, filter, mailboxes, labels) {
+  FiltersDialogController.$inject = ['$scope', '$mdDialog', 'filter', 'mailboxes', 'labels', 'sieveCapabilities'];
+  function FiltersDialogController($scope, $mdDialog, filter, mailboxes, labels, sieveCapabilities) {
     $scope.filter = filter;
     $scope.mailboxes = mailboxes;
     $scope.labels = labels;
@@ -20,21 +20,29 @@
       "cc": l("Cc"),
       "to_or_cc": l("To or Cc"),
       "size": l("Size (Kb)"),
-      "header": l("Header"),
-      "body": l("Body")
+      "header": l("Header")
     };
 
+    if (sieveCapabilities.indexOf("body") > -1)
+      $scope.fieldLabels.body = l("Body");
+
     $scope.methodLabels = {
-      "addflag": l("Flag the message with:"),                         
       "discard": l("Discard the message"),
-      "fileinto": l("File the message in:"),
       "keep": l("Keep the message"),
       "redirect": l("Forward the message to:"),
-      "reject": l("Send a reject message:"),
       "vacation": l("Send a vacation message"),
       "stop": l("Stop processing filter rules")
     };
 
+    if (sieveCapabilities.indexOf("reject") > -1)
+      $scope.methodLabels.reject = l("Send a reject message:");
+
+    if (sieveCapabilities.indexOf("fileinto") > -1)
+      $scope.methodLabels.fileinto = l("File the message in:");
+
+    if (sieveCapabilities.indexOf("imapflags") > -1 || sieveCapabilities.indexOf("imap4flags") > -1)
+      $scope.methodLabels.addflag = l("Flag the message with:");
+
     $scope.numberOperatorLabels = {
       "under": l("is under"),
       "over": l("is over")
@@ -46,11 +54,14 @@
       "contains": l("contains"),
       "contains_not": l("does not contain"),
       "matches": l("matches"),
-      "matches_not": l("does not match"),
-      "regex": l("matches regex"),
-      "regex_not": l("does not match regex")
+      "matches_not": l("does not match")
     };
 
+    if (sieveCapabilities.indexOf("regex") > -1) {
+      $scope.textOperatorLabels.regex = l("matches regex");
+      $scope.textOperatorLabels.regex_not = l("does not match regex");
+    }
+
     $scope.flagLabels = {
       "seen": l("Seen"),
       "deleted": l("Deleted"),
diff --git a/UI/WebServerResources/js/Preferences/PreferencesController.js b/UI/WebServerResources/js/Preferences/PreferencesController.js
index 64947cf7f..0fbece5fa 100644
--- a/UI/WebServerResources/js/Preferences/PreferencesController.js
+++ b/UI/WebServerResources/js/Preferences/PreferencesController.js
@@ -137,7 +137,8 @@
         locals: {
           filter: filter,
           mailboxes: vm.mailboxes,
-          labels: vm.preferences.defaults.SOGoMailLabelsColors
+          labels: vm.preferences.defaults.SOGoMailLabelsColors,
+          sieveCapabilities: window.sieveCapabilities
         }
       });
     }
@@ -152,7 +153,8 @@
         locals: {
           filter: filter,
           mailboxes: vm.mailboxes,
-          labels: vm.preferences.defaults.SOGoMailLabelsColors
+          labels: vm.preferences.defaults.SOGoMailLabelsColors,
+          sieveCapabilities: window.sieveCapabilities
         }
       }).then(function() {
         vm.preferences.defaults.SOGoSieveFilters[index] = filter;