From 16d02455bdcb5e8d2e2c5b4dc160b294895da82c Mon Sep 17 00:00:00 2001
From: Alexandre Cloutier <acloutier@inverse.ca>
Date: Mon, 7 Jul 2014 10:00:16 -0400
Subject: [PATCH] Serialization of the whiteList

---
 UI/PreferencesUI/UIxPreferences.m             | 20 +++++++++++++++++++
 UI/Templates/PreferencesUI/UIxPreferences.wox | 12 ++++++-----
 UI/WebServerResources/UIxPreferences.js       | 20 +++++++++----------
 3 files changed, 36 insertions(+), 16 deletions(-)

diff --git a/UI/PreferencesUI/UIxPreferences.m b/UI/PreferencesUI/UIxPreferences.m
index 096ae7b97..96e91f0e6 100644
--- a/UI/PreferencesUI/UIxPreferences.m
+++ b/UI/PreferencesUI/UIxPreferences.m
@@ -639,6 +639,26 @@ static NSArray *reminderValues = nil;
   return [userDefaults busyOffHours];
 }
 
+- (NSString *) whiteListValue
+{
+  SOGoUserSettings *us;
+  id *whiteListValue;
+  
+  us = [user userSettings];
+  whiteListValue = [us objectForKey:@"whiteListInvitations"];
+
+  return whiteListValue;
+}
+
+- (void) setWhiteListValue: (NSArray *) whiteList
+{
+  SOGoUserSettings *us;
+  
+  us = [user userSettings];
+  [us setObject: whiteList forKey: @"whiteListInvitations"];
+  [us synchronize];
+}
+
 - (void) setPreventInvitations: (BOOL) preventInvitations
 {
   SOGoUserSettings *us;
diff --git a/UI/Templates/PreferencesUI/UIxPreferences.wox b/UI/Templates/PreferencesUI/UIxPreferences.wox
index 287029d40..fcef5e7ff 100644
--- a/UI/Templates/PreferencesUI/UIxPreferences.wox
+++ b/UI/Templates/PreferencesUI/UIxPreferences.wox
@@ -19,7 +19,7 @@
   <div class="popupMenu" id="contactsMenu">
     <ul></ul>
   </div>
-
+  
   <div id="colorPickerDialog" style="display: none;" class="dialog right bottom">
     <div>
       <span class="blc-FFFFFF"><!-- --></span>
@@ -223,7 +223,6 @@
                 var:checked="preventInvitations" />
                 <var:string label:value="Prevent from being invited to appointments"/></dd>
             </dl>
-            
             <label id="whiteListLabel"><var:string label:value="White list for appointments invitations:"/></label>
             <div id="appointmentsWhiteListWrapper" class="listWrapper">
               <table class="tableview" id="tableViewWhiteList">
@@ -231,7 +230,8 @@
                   <tr class="tableview">
                     <th const:class="tbtv_headercell" const:id="whiteListTableHeader">
                       <var:string label:value="Contacts names"/></th>
-                  </tr></thead>
+                  </tr>
+                </thead>
                 <tbody>
                   <var:foreach list="appointmentsWhiteList" item="contact">
                     <tr const:class="whiteListRow">
@@ -244,10 +244,12 @@
             </div>
             <div id="whiteListToolbar">
               <a const:id="appointmentsWhiteListAdd" href="#">
-                  <img rsrc:src="add-icon.png" label:title="Add" id="WhiteListAdd"/></a>
+                <img rsrc:src="add-icon.png" label:title="Add" id="WhiteListAdd"/></a>
               <a const:id="appointmentsWhiteListDelete" href="#">
-                  <img rsrc:src="remove-icon.png" label:title="Delete" id="WhiteListDelete" /></a>
+                <img rsrc:src="remove-icon.png" label:title="Delete" id="WhiteListDelete" /></a>
             </div>
+            <input type="hidden" const:id="whiteListValue"
+            const:name="whiteListValue" var:value="whiteListValue"/>
             
             <label id="calendarCategoriesLabel"><var:string label:value="Categories"/></label>
             <div id="calendarCategoriesListWrapper" class="listWrapper">
diff --git a/UI/WebServerResources/UIxPreferences.js b/UI/WebServerResources/UIxPreferences.js
index dba91eea2..8086879f3 100644
--- a/UI/WebServerResources/UIxPreferences.js
+++ b/UI/WebServerResources/UIxPreferences.js
@@ -9,8 +9,8 @@ function savePreferences(sender) {
   if (sigList)
     sigList.disabled = false;
   
- // if ($("appointmentsWhiteListWrapper"))
- //   serializeAppointmentsWhiteList();
+  if ($("appointmentsWhiteListWrapper"))
+    serializeAppointmentsWhiteList();
   
   if ($("calendarCategoriesListWrapper"))
     serializeCalendarCategories();
@@ -236,10 +236,9 @@ function initPreferences() {
     table.multiselect = true;
     $("appointmentsWhiteListAdd").observe("click", onAppointmentsWhiteListAdd);
     $("appointmentsWhiteListDelete").observe("click", onAppointmentsWhiteListDelete);
-    whiteList.observe("scroll", onBodyClickHandler);
   }
   
-  // Calendar categories
+  // Calender categories
   var wrapper = $("calendarCategoriesListWrapper");
   if (wrapper) {
     var table = wrapper.childNodesWithTag("table")[0];
@@ -1095,13 +1094,12 @@ function serializeAppointmentsWhiteList() {
   
   var values = [];
   for (var i = 0; i < r.length; i++) {
-    var tds = r[i].childElements();
-    var name  = $(tds.first()).innerHTML;
-    var email = "";
-    values.push("\"" + name + "\": \"" + color + "\"");
+    var tds = r[i].childElements().first().down("INPUT");
+    var uid  = tds.getAttribute("uid");
+    values.push(uid);
   }
   
-  //$("calendarCategoriesValue").value = "{ " + values.join(",\n") + "}";
+  $("whiteListValue").value = values;
 }
 
 function onCalendarCategoryAdd(e) {
@@ -1143,7 +1141,7 @@ function serializeCalendarCategories() {
     var values = [];
     for (var i = 0; i < r.length; i++) {
         var tds = r[i].childElements();
-        var name  = $(tds.first()).innerHTML;
+        var name  = $(tds.first()).innerHTML.trim();
         var color = $(tds.last().childElements().first()).readAttribute('data-color');
         values.push("\"" + name + "\": \"" + color + "\"");
     }
@@ -1271,7 +1269,7 @@ function onContactsCategoryAdd(e) {
     var list = $('contactsCategoriesListWrapper').down("TABLE").down("TBODY");
     list.appendChild(row);
 
-    resetContactsTableActions ();
+    resetContactsTableActions();
     nametd.editionController.startEditing();
 }