Monotone-Parent: 3d8ebb80a2f9e64a537f7a46aaadd8fcc1d801e8

Monotone-Revision: 4bc7d1bcd8cf57366a454fb7814accb19d6121c8

Monotone-Author: wsourdeau@inverse.ca
Monotone-Date: 2007-06-28T20:37:27
Monotone-Branch: ca.inverse.sogo

UI/Contacts/UIxContactsListView.h

@@ -31,7 +31,6 @@
 
 @interface UIxContactsListView : UIxComponent
 {
-  NSString *searchText;
   NSDictionary *currentContact;
   NSString *selectorComponentClass;
 }

UI/Contacts/UIxContactsListView.m

@@ -42,13 +42,6 @@
   return self;
 }
 
-- (void) dealloc
-{
-  if (searchText)
-    [searchText release];
-  [super dealloc];
-}
-
 /* accessors */
 
 - (void) setCurrentContact: (NSDictionary *) _contact
@@ -61,19 +54,6 @@
   return currentContact;
 }
 
-- (void) setSearchText: (NSString *) _txt
-{
-  ASSIGNCOPY (searchText, _txt);
-}
-
-- (id) searchText
-{
-  if (!searchText)
-    [self setSearchText: [self queryParameterForKey:@"search"]];
-
-  return searchText;
-}
-
 - (id <WOActionResults>) mailerContactsAction
 {
   selectorComponentClass = @"UIxContactsMailerSelection";
@@ -126,45 +106,33 @@
   NSString *s;
   
   s = [self queryParameterForKey: @"sort"];
-  if ([s length] == 0)
+  if (![s length])
     s = [self defaultSortKey];
 
   return s;
 }
 
-- (NSComparisonResult) sortOrdering
-{
-  return ([[self queryParameterForKey:@"desc"] boolValue]
-          ? NSOrderedDescending
-          : NSOrderedAscending);
-}
-
 - (NSArray *) contactInfos
 {
   id <SOGoContactFolder> folder;
+  NSString *ascending, *searchText, *valueText;
+  NSComparisonResult ordering;
 
   folder = [self clientObject];
 
-  return [folder lookupContactsWithFilter: [self searchText]
+  ascending = [self queryParameterForKey: @"asc"];
+  ordering = ((![ascending length] || [ascending boolValue])
+	      ? NSOrderedAscending : NSOrderedDescending);
+
+  searchText = [self queryParameterForKey: @"search"];
+  if ([searchText length] > 0)
+    valueText = [self queryParameterForKey: @"value"];
+  else
+    valueText = nil;
+
+  return [folder lookupContactsWithFilter: valueText
                  sortBy: [self sortKey]
-                 ordering: [self sortOrdering]];
-}
-
-/* notifications */
-
-- (void) sleep
-{
-  if (searchText)
-    {
-      [searchText release];
-      searchText = nil;
-    }
-  currentContact = nil;
-//   [allRecords release];
-//   allRecords = nil;
-//   [filteredRecords release];
-//   filteredRecords = nil;
-  [super sleep];
+                 ordering: ordering];
 }
 
 /* actions */

UI/Templates/ContactsUI/UIxContactsListView.wox

@@ -13,32 +13,18 @@
       <tbody>
         <tr class="tableview">
           <!-- localize -->
-          <td class="tbtv_headercell headerName">
-            <var:component className="UIxSortableTableHeader"
-              label:label="Name"
-              const:sortKey="cn"
-              const:href="view"
-              var:queryDictionary="context.request.formValues"
-              />
-          </td>
-          <td class="tbtv_headercell headerEmail">
-            <var:component className="UIxSortableTableHeader"
-              label:label="Email"
-              const:sortKey="mail"
-              const:href="view"
-              var:queryDictionary="context.request.formValues"
-              />
-          </td>
-          <td class="tbtv_headercell">
-            <var:string label:value="Screen Name" />
-          </td>
-          <td class="tbtv_headercell">
-            <var:string label:value="Organization" />
-          </td>
-          <td class="tbtv_headercell">
-            <var:string label:value="Work Phone" />
-          </td>
-        </tr>
+          <td class="tbtv_headercell" id="nameHeader"
+	    ><var:string label:value="Name"
+	      /></td
+	    ><td class="tbtv_headercell" id="mailHeader"
+	    ><var:string label:value="Email"/></td
+	    ><td class="tbtv_headercell" id="screenNameHeader"
+	    ><var:string label:value="Screen Name" /></td
+	    ><td class="tbtv_headercell" id="orgHeader"
+	    ><var:string label:value="Organization" /></td
+	    ><td class="tbtv_headercell" id="phoneHeader"
+	    ><var:string label:value="Work Phone" /></td
+	    ></tr>
 
         <var:foreach list="contactInfos" item="currentContact">
           <tr class="tableview"
@@ -48,12 +34,12 @@
             onclick="return onContactRowClick(event, this);"
             ondblclick="return onContactRowDblClick(event, this);"
             oncontextmenu="return onContactContextMenu(event, this);">
-            <td onmousedown="return false;"><img rsrc:src="abcard.gif"
+            <td><img rsrc:src="abcard.gif"
 		/><var:string value="displayName" const:escapeHTML="YES" /></td>
-            <td onmousedown="return false;"><var:string value="currentContact.mail"/></td>
-            <td onmousedown="return false;"><var:string value="currentContact.screenname"/></td>
-            <td onmousedown="return false;"><var:string value="currentContact.o"/></td>
-            <td onmousedown="return false;"><var:string value="currentContact.telephonenumber"/></td>
+            <td><var:string value="currentContact.mail"/></td>
+            <td><var:string value="currentContact.screenname"/></td>
+            <td><var:string value="currentContact.o"/></td>
+            <td><var:string value="currentContact.telephonenumber"/></td>
           </tr>
         </var:foreach>
       </tbody>

UI/WebServerResources/ContactsUI.css

@@ -179,8 +179,8 @@ TABLE#contactsList TD
 { white-space: nowrap;
   width: 20%; }
 
-TABLE#contactsList TD.headerName,
-TABLE#contactsList TD.headerEmail
+TD#nameHeader,
+TD#mailHeader
 { width: 30%; }
 
 TABLE#contactsList TD IMG

UI/WebServerResources/ContactsUI.js

@@ -1,7 +1,7 @@
 /* JavaScript for SOGoContacts */
 
 var cachedContacts = new Array();
-var currentContactFolder = '/personal';
+var currentContactFolder = null;
 
 var usersRightsWindowHeight = 200;
 var usersRightsWindowWidth = 450;
@@ -30,8 +30,22 @@ function validateEditorInput(sender) {
   return true;
 }
 
-function openContactsFolder(contactsFolder, params) {
-  if (contactsFolder != currentContactFolder || params) {
+function openContactsFolder(contactsFolder, reload, idx) {
+  if (contactsFolder != currentContactFolder || reload) {
+     currentContactFolder = contactsFolder;
+     var url = URLForFolderID(currentContactFolder) +
+	"/view?noframe=1";
+
+     var searchValue = search["value"];
+     if (searchValue && searchValue.length > 0)
+	url += ("&search=" + search["criteria"]
+		+ "&value=" + searchValue);
+     var sortAttribute = sorting["attribute"];
+     if (sortAttribute && sortAttribute.length > 0)
+	url += ("&sort=" + sorting["attribute"]
+		+ "&asc=" + sorting["ascending"]);
+
+     var selection;
      if (contactsFolder == currentContactFolder) {
         var contactsList = $("contactsList");
         if (contactsList)
@@ -40,21 +54,14 @@ function openContactsFolder(contactsFolder, params) {
            window.alert("no contactsList");
      }
      else
-      selection = null;
+	selection = null;
 
-    currentContactFolder = contactsFolder;
-    var url = URLForFolderID(currentContactFolder) +
-       "/view?noframe=1&sort=cn&desc=0";
-    if (params)
-      url += '&' + params;
-
-    var selection;
-    if (document.contactsListAjaxRequest) {
-      document.contactsListAjaxRequest.aborted = true;
-      document.contactsListAjaxRequest.abort();
-    }
-    document.contactsListAjaxRequest
-      = triggerAjaxRequest(url, contactsListCallback, selection);
+     if (document.contactsListAjaxRequest) {
+	document.contactsListAjaxRequest.aborted = true;
+	document.contactsListAjaxRequest.abort();
+     }
+     document.contactsListAjaxRequest
+	= triggerAjaxRequest(url, contactsListCallback, selection);
   }
 }
 
@@ -70,6 +77,15 @@ function openContactsFolderAtIndex(element) {
     = triggerAjaxRequest(url, contactsListCallback);
 }
 
+function configureContactsListHeaders(cells) {
+   for (var i = 0; i < cells.length; i++) {
+      var currentCell = $(cells[i]);
+      Event.observe(currentCell, "click",
+		    onHeaderClick.bindAsEventListener(currentCell));
+      Event.observe(currentCell, "mousedown", listRowMouseDownHandler);
+   }
+}
+
 function contactsListCallback(http) {
   var div = $("contactsListContent");
 
@@ -77,12 +93,44 @@ function contactsListCallback(http) {
       && http.status == 200) {
     document.contactsListAjaxRequest = null;
     div.innerHTML = http.responseText;
+
+    var table = $("contactsList");
+    if (table)
+       configureContactsListHeaders(table.tBodies[0].rows[0].cells);
+
+    if (sorting["attribute"] && sorting["attribute"].length > 0) {
+       var sortHeader;
+       if (sorting["attribute"] == "cn")
+	  sortHeader = $("nameHeader");
+       else if (sorting["attribute"] == "mail")
+	  sortHeader = $("mailHeader");
+       else if (sorting["attribute"] == "screenname")
+	  sortHeader = $("screenNameHeader");
+       else if (sorting["attribute"] == "o")
+	  sortHeader = $("orgHeader");
+       else if (sorting["attribute"] == "telephonenumber")
+	  sortHeader = $("phoneHeader");
+       else
+	  sortHeader = null;
+       
+       if (sortHeader) {
+	  var sortImage = createElement("img", "messageSortImage", "sortImage");
+	  sortHeader.insertBefore(sortImage, sortHeader.firstChild);
+	  if (sorting["ascending"])
+	     sortImage.src = ResourcesURL + "/title_sortdown_12x12.png";
+	  else
+	     sortImage.src = ResourcesURL + "/title_sortup_12x12.png";
+       }
+    }
+
     var selected = http.callbackData;
     if (selected) {
-        for (var i = 0; i < selected.length; i++)
-          $(selected[i]).select();
+       for (var i = 0; i < selected.length; i++) {
+	  var row = $(selected[i]);
+	  if (row)
+	     row.select();
+       }
     }
-    configureSortableTableHeaders();
   }
   else
     log ("ajax problem 1");
@@ -332,19 +380,31 @@ function newEmailTo(sender) {
 }
 
 function onHeaderClick(event) {
-  if (document.contactsListAjaxRequest) {
-    document.contactsListAjaxRequest.aborted = true;
-    document.contactsListAjaxRequest.abort();
-  }
-  url = URLForFolderID(currentContactFolder);
-// //   log("url: " + url);
-//   var url = "" + this.href;
-  if (url.indexOf("noframe=", 0) == -1)
-     url += "&noframe=1";
-  document.contactsListAjaxRequest
-     = triggerAjaxRequest(url, contactsListCallback);
+   var headerId = this.getAttribute("id");
+   var newSortAttribute;
+   if (headerId == "nameHeader")
+      newSortAttribute = "cn";
+   else if (headerId == "mailHeader")
+      newSortAttribute = "mail";
+   else if (headerId == "screenNameHeader")
+      newSortAttribute = "screenname";
+   else if (headerId == "orgHeader")
+      newSortAttribute = "o";
+   else if (headerId == "phoneHeader")
+      newSortAttribute = "telephonenumber";
 
-  event.preventDefault();
+   log("header: " + headerId);
+
+   if (sorting["attribute"] == newSortAttribute)
+      sorting["ascending"] = !sorting["ascending"];
+   else {
+      sorting["attribute"] = newSortAttribute;
+      sorting["ascending"] = true;
+   }
+
+   refreshCurrentFolder();
+
+   preventDefault(event);
 }
 
 function registerDraggableMessageNodes() {
@@ -367,17 +427,17 @@ function onFolderSelectionChange() {
       var div = $("contactsListContent");
       div.innerHTML = "";
    }
-   else
-      openContactsFolder(nodes[0].getAttribute("id"), null);
+   else {
+      search = {};
+      sorting = {};
+      $("searchValue").value = "";
+      initCriteria();
+      openContactsFolder(nodes[0].getAttribute("id"));
+   }
 }
 
-function onSearchFormSubmit() {
-  var searchValue = $("searchValue");
-
-  openContactsFolder(currentContactFolder,
-		     "search=" + searchValue.value);
-
-  return false;
+function refreshCurrentFolder() {
+   openContactsFolder(currentContactFolder, true);
 }
 
 function onConfirmContactSelection(event) {
@@ -409,7 +469,7 @@ function onConfirmContactSelection(event) {
        && selectorList.value != initialValues)
       selector.changeNotification("addition");
 
-   event.preventDefault();
+   preventDefault(event);
 }
 
 function onContactMailTo(node) {
@@ -417,11 +477,11 @@ function onContactMailTo(node) {
 }
 
 function refreshContacts(contactId) {
-  openContactsFolder(currentContactFolder, "reload=true");
-  cachedContacts[currentContactFolder + "/" + contactId] = null;
-  loadContact(contactId);
+   refreshCurrentFolder();
+   cachedContacts[currentContactFolder + "/" + contactId] = null;
+   loadContact(contactId);
 
-  return false;
+   return false;
 }
 
 function onAddressBookNew(event) {
@@ -465,7 +525,7 @@ function newUserFolderCallback(folderData) {
 function onAddressBookAdd(event) {
    openUserFolderSelector(newUserFolderCallback, "contact");
 
-   event.preventDefault();
+   preventDefault(event);
 }
 
 function onFolderUnsubscribeCB(folderId) {
@@ -494,7 +554,7 @@ function onAddressBookRemove(event) {
      }
   }
 
-  event.preventDefault();
+  preventDefault(event);
 }
 
 function deletePersonalAddressBook(folderId) {
@@ -594,13 +654,15 @@ function configureContactFolders() {
 
     var personalFolder = $("/personal");
     personalFolder.select();
+    openContactsFolder("/personal");
   }
 }
 
 function setEventsOnContactFolder(node) {
-  Event.observe(node, "mousedown", listRowMouseDownHandler, false);
-  Event.observe(node, "click", onRowClick, false);
-  Event.observe(node, "contextmenu", onContactFoldersContextMenu.bindAsEventListener(node), false);
+   Event.observe(node, "mousedown", listRowMouseDownHandler, false);
+   Event.observe(node, "click", onRowClick, false);
+   Event.observe(node, "contextmenu",
+		 onContactFoldersContextMenu.bindAsEventListener(node), false);
 }
 
 function onMenuSharing(event) {
@@ -630,7 +692,8 @@ function configureSelectionButtons() {
    if (container) {
       var buttons = container.childNodesWithTag("input");
       for (var i = 0; i < buttons.length; i++)
-	Event.observe(buttons[i], "click", onConfirmContactSelection.bindAsEventListener(buttons[i]), false);
+	Event.observe(buttons[i], "click",
+		      onConfirmContactSelection.bindAsEventListener(buttons[i]));
    }
 }