From: wolfgang <wolfgang@d1b88da0-ebda-0310-925b-ed51d893ca5b>
Date: Fri, 1 Feb 2008 22:32:24 +0000 (+0000)
Subject: git-svn-id: http://svn.opengroupware.org/SOGo/inverse/trunk@1348 d1b88da0-ebda-0310... 
X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cf4173fbdc9c581a8febc7c031384ed698462f96;p=scalable-opengroupware.org

git-svn-id: http://svn.opengroupware.org/SOGo/inverse/trunk@1348 d1b88da0-ebda-0310-925b-ed51d893ca5b
---

diff --git a/ChangeLog b/ChangeLog
index b492fefd..f9f799d1 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2008-02-01  Wolfgang Sourdeau  <wsourdeau@inverse.ca>
+
+	* UI/MailerUI/UIxMailMainFrame.m ([UIxMailMainFrame
+	-quotaSupport]): new json string that lists the quota support
+	among the user's mail accounts.
+
+	* SoObjects/Mailer/SOGoMailAccount.m ([SOGoMailAccount
+	-supportsQuotas]): new method that returns whether the current
+	account server has support for quotas.
+
 2008-01-29  Wolfgang Sourdeau  <wsourdeau@inverse.ca>
 
 	* SoObjects/SOGo/SOGoGCSFolder.m ([SOGoGCSFolder
diff --git a/SoObjects/Mailer/SOGoMailAccount.h b/SoObjects/Mailer/SOGoMailAccount.h
index 8c720e1f..e907d8cd 100644
--- a/SoObjects/Mailer/SOGoMailAccount.h
+++ b/SoObjects/Mailer/SOGoMailAccount.h
@@ -48,6 +48,8 @@
   SOGoTrashFolder *trashFolder;
 }
 
+- (BOOL) supportsQuotas;
+
 /* folder pathes */
 
 - (NSArray *) allFolderPaths;
diff --git a/SoObjects/Mailer/SOGoMailAccount.m b/SoObjects/Mailer/SOGoMailAccount.m
index 4f2f5911..8b355986 100644
--- a/SoObjects/Mailer/SOGoMailAccount.m
+++ b/SoObjects/Mailer/SOGoMailAccount.m
@@ -31,6 +31,8 @@
 #import <NGExtensions/NSObject+Logs.h>
 #import <NGExtensions/NSNull+misc.h>
 #import <NGImap4/NGImap4Connection.h>
+#import <NGImap4/NGImap4Client.h>
+#import <NGImap4/NGImap4Context.h>
 
 #import <SoObjects/SOGo/NSArray+Utilities.h>
 #import <SoObjects/SOGo/SOGoUser.h>
@@ -170,6 +172,15 @@ static NSString *otherUsersFolderName = @""; // TODO: add English default
   return folders;
 }
 
+- (BOOL) supportsQuotas
+{
+  NGImap4Client *imapClient;
+
+  imapClient = [[self imap4Connection] client];
+
+  return [[imapClient context] canQuota];
+}
+
 /* hierarchy */
 
 - (SOGoMailAccount *) mailAccountFolder
diff --git a/SoObjects/Mailer/SOGoMailAccounts.h b/SoObjects/Mailer/SOGoMailAccounts.h
index 623ffcb2..afd7404c 100644
--- a/SoObjects/Mailer/SOGoMailAccounts.h
+++ b/SoObjects/Mailer/SOGoMailAccounts.h
@@ -41,6 +41,8 @@
 
 @interface SOGoMailAccounts : SOGoFolder
 
+- (NSArray *) toManyRelationshipKeys;
+
 @end
 
 #endif /* __Mailer_SOGoMailAccounts_H__ */
diff --git a/SoObjects/Mailer/SOGoMailAccounts.m b/SoObjects/Mailer/SOGoMailAccounts.m
index 515c6652..3188052c 100644
--- a/SoObjects/Mailer/SOGoMailAccounts.m
+++ b/SoObjects/Mailer/SOGoMailAccounts.m
@@ -76,6 +76,7 @@ static NSString *AgenorShareLoginMarker  = @".-.";
   }
   
   ct = [[ctClass alloc] initWithName:_key inContainer:self];
+
   return [ct autorelease];
 }
 
diff --git a/UI/MailerUI/UIxMailMainFrame.h b/UI/MailerUI/UIxMailMainFrame.h
index 1e4ecd55..2147d5f1 100644
--- a/UI/MailerUI/UIxMailMainFrame.h
+++ b/UI/MailerUI/UIxMailMainFrame.h
@@ -30,6 +30,13 @@
   NSUserDefaults *ud;
   NSMutableDictionary *moduleSettings;
 }
+
+- (WOResponse *) getDragHandlesStateAction;
+- (WOResponse *) getFoldersStateAction;
+
+- (WOResponse *) saveDragHandlesStateAction;
+- (WOResponse *) saveFoldersStateAction;
+
 @end
 
 #endif /* UIXMAILMAINFRAME_H */
diff --git a/UI/MailerUI/UIxMailMainFrame.m b/UI/MailerUI/UIxMailMainFrame.m
index e45d9d30..83f24614 100644
--- a/UI/MailerUI/UIxMailMainFrame.m
+++ b/UI/MailerUI/UIxMailMainFrame.m
@@ -20,6 +20,7 @@
 */
 
 #import <Foundation/NSUserDefaults.h>
+#import <Foundation/NSValue.h>
 
 #import <NGObjWeb/WOContext.h>
 #import <NGObjWeb/WORequest.h>
@@ -28,6 +29,7 @@
 #import <NGExtensions/NSString+misc.h>
 
 #import <SoObjects/Mailer/SOGoMailObject.h>
+#import <SoObjects/Mailer/SOGoMailAccount.h>
 #import <SoObjects/Mailer/SOGoMailAccounts.h>
 #import <SoObjects/SOGo/NSDictionary+URL.h>
 #import <SoObjects/SOGo/NSArray+Utilities.h>
@@ -71,6 +73,29 @@
   return [accountNames jsonRepresentation];
 }
 
+- (NSString *) quotaSupport
+{
+  NSEnumerator *accountNames;
+  NSMutableArray *quotas;
+  NSString *currentAccount;
+  SOGoMailAccounts *co;
+  BOOL supportsQuota;
+
+  co = [self clientObject];
+  accountNames = [[co toManyRelationshipKeys] objectEnumerator];
+
+  quotas = [NSMutableArray array];
+  while ((currentAccount = [accountNames nextObject]))
+    {
+      supportsQuota = [[co lookupName: currentAccount
+			   inContext: context
+			   acquire: NO] supportsQuotas];
+      [quotas addObject: [NSNumber numberWithInt: supportsQuota]];
+    }
+
+  return [quotas jsonRepresentation];
+}
+
 - (NSString *) pageFormURL
 {
   NSString *u;
@@ -127,7 +152,9 @@
   accounts = [[context activeUser] mailAccounts];
   firstAccount = [[accounts objectsForKey: @"name"] objectAtIndex: 0];
   formValues = [[context request] formValues];
-  parameters = [formValues count] > 0 ? [formValues asURLParameters] : @"?mailto=";
+  parameters = ([formValues count] > 0
+		? [formValues asURLParameters]
+		: @"?mailto=");
   newLocation = [NSString stringWithFormat: @"%@/%@/compose%@",
 			  [co baseURLInContext: context],
 			  firstAccount,
@@ -136,6 +163,16 @@
   return [self redirectToLocation: newLocation];
 }
 
+- (WOResponse *) getDragHandlesStateAction
+{
+  NSString *dragHandles;
+
+  [self _setupContext];
+  dragHandles = [moduleSettings objectForKey: @"DragHandles"];
+
+  return [self responseWithStatus: 200 andString: dragHandles];
+}
+
 - (WOResponse *) getFoldersStateAction
 {
   NSString *expandedFolders;
@@ -146,6 +183,23 @@
   return [self responseWithStatus: 200 andString: expandedFolders];
 }
 
+- (WOResponse *) saveDragHandlesStateAction
+{
+  WORequest *request;
+  NSString *dragHandles;
+  
+  [self _setupContext];
+  request = [context request];
+  dragHandles = [request formValueForKey: @"dragHandles"];
+  
+  [moduleSettings setObject: dragHandles
+		  forKey: @"DragHandles"];
+
+  [ud synchronize];
+
+  return [self responseWithStatus: 204];
+}
+
 - (WOResponse *) saveFoldersStateAction
 {
   WORequest *request;
diff --git a/UI/MailerUI/product.plist b/UI/MailerUI/product.plist
index 61714cd2..e15dd206 100644
--- a/UI/MailerUI/product.plist
+++ b/UI/MailerUI/product.plist
@@ -330,11 +330,21 @@
 	  pageName    = "UIxMailMainFrame";
 	  actionName  = "compose";
 	};
+	dragHandlesState = {
+	  protectedBy = "View";
+	  pageName    = "UIxMailMainFrame";
+	  actionName  = "getDragHandlesState";
+	};
 	foldersState = {
 	  protectedBy = "View";
 	  pageName    = "UIxMailMainFrame";
 	  actionName  = "getFoldersState";
 	};
+	saveDragHandlesState = {
+	  protectedBy = "View";
+	  pageName    = "UIxMailMainFrame";
+	  actionName  = "saveDragHandlesState";
+	};
 	saveFoldersState = {
 	  protectedBy = "View";
 	  pageName    = "UIxMailMainFrame";
diff --git a/UI/Templates/MailerUI/UIxMailMainFrame.wox b/UI/Templates/MailerUI/UIxMailMainFrame.wox
index 355c8999..cd847feb 100644
--- a/UI/Templates/MailerUI/UIxMailMainFrame.wox
+++ b/UI/Templates/MailerUI/UIxMailMainFrame.wox
@@ -10,6 +10,7 @@
   const:jsFiles="dtree.js,MailerUI+dTree.js">
   <script type="text/javascript">
     var textMailAccounts = '<var:string value="mailAccounts" const:escapeHTML="NO"/>';
+    var textQuotaSupport = '<var:string value="quotaSupport" const:escapeHTML="NO"/>';
   </script>
   <div class="menu" id="accountIconMenu">
     <ul>
diff --git a/UI/WebServerResources/ContactsUI.js b/UI/WebServerResources/ContactsUI.js
index bb30036a..03eae4a4 100644
--- a/UI/WebServerResources/ContactsUI.js
+++ b/UI/WebServerResources/ContactsUI.js
@@ -763,6 +763,10 @@ function folderRenameCallback(http) {
   }
 }
 
+function saveDragHandlesState() {
+  // Call from SOGoDragHandles.js
+}
+
 function onMenuSharing(event) {
   if ($(this).hasClassName("disabled"))
     return;
diff --git a/UI/WebServerResources/MailerUI.js b/UI/WebServerResources/MailerUI.js
index 9604045a..83e7a0ff 100644
--- a/UI/WebServerResources/MailerUI.js
+++ b/UI/WebServerResources/MailerUI.js
@@ -2,12 +2,19 @@
 var accounts = {};
 var mailboxTree;
 var mailAccounts;
+var quotaSupport;
 if (typeof textMailAccounts != 'undefined') {
   if (textMailAccounts.length > 0)
     mailAccounts = textMailAccounts.evalJSON(true);
   else
     mailAccounts = new Array();
 }
+if (typeof textQuotaSupport != 'undefined') {
+  if (textQuotaSupport.length > 0)
+    quotaSupport = textQuotaSupport.evalJSON(true);
+  else
+    quotaSupport = new Array();
+}
 
 var Mailer = {
  currentMailbox: null,
@@ -179,15 +186,14 @@ function mailListMarkMessage(event) {
 }
 
 function mailListMarkMessageCallback(http) {
-  if (http.readyState == 4)
-    if (isHttpStatus204(http.status)) {
-      var data = http.callbackData;
-      markMailInWindow(data["window"], data["msguid"], data["markread"]);
-    }
-    else {
-      alert("Message Mark Failed (" + http.status + "): " + http.statusText);
-      window.location.reload();
-    }
+  if (isHttpStatus204(http.status)) {
+    var data = http.callbackData;
+    markMailInWindow(data["window"], data["msguid"], data["markread"]);
+  }
+  else {
+    alert("Message Mark Failed (" + http.status + "): " + http.statusText);
+    window.location.reload();
+  }
 }
 
 /* maillist row highlight */
@@ -235,31 +241,29 @@ function deleteSelectedMessages(sender) {
 }
 
 function deleteSelectedMessagesCallback(http) {
-  if (http.readyState == 4) {
-    if (isHttpStatus204(http.status)) {
-      var data = http.callbackData;
-      deleteCachedMessage(data["messageId"]);
-      deleteMessageRequestCount--;
-      if (Mailer.currentMailbox == data["mailbox"]) {
-	var div = $('messageContent');
-	if (Mailer.currentMessages[Mailer.currentMailbox] == data["id"]) {
-	  div.update();
-	  Mailer.currentMessages[Mailer.currentMailbox] = null;	
-	}
+  if (isHttpStatus204(http.status)) {
+    var data = http.callbackData;
+    deleteCachedMessage(data["messageId"]);
+    deleteMessageRequestCount--;
+    if (Mailer.currentMailbox == data["mailbox"]) {
+      var div = $('messageContent');
+      if (Mailer.currentMessages[Mailer.currentMailbox] == data["id"]) {
+	div.update();
+	Mailer.currentMessages[Mailer.currentMailbox] = null;	
+      }
 
-	var row = $("row_" + data["id"]);
-	var nextRow = row.next("tr");
-	if (!nextRow)
-	  nextRow = row.previous("tr");
-	row.parentNode.removeChild(row);
+      var row = $("row_" + data["id"]);
+      var nextRow = row.next("tr");
+      if (!nextRow)
+	nextRow = row.previous("tr");
+      row.parentNode.removeChild(row);
 //	row.addClassName("deleted"); // when we'll offer "mark as deleted"
       
-	if (deleteMessageRequestCount == 0) {
-	  if (nextRow) {
-	    Mailer.currentMessages[Mailer.currentMailbox] = nextRow.getAttribute("id").substr(4);
-	    nextRow.select();
-	    loadMessage(Mailer.currentMessages[Mailer.currentMailbox]);
-	  }
+      if (deleteMessageRequestCount == 0) {
+	if (nextRow) {
+	  Mailer.currentMessages[Mailer.currentMailbox] = nextRow.getAttribute("id").substr(4);
+	  nextRow.select();
+	  loadMessage(Mailer.currentMessages[Mailer.currentMailbox]);
 	}
       }
     }
@@ -487,9 +491,11 @@ function openMailbox(mailbox, reload, idx) {
       = triggerAjaxRequest(url, messageListCallback,
 			   currentMessage);
 
-    var quotasUrl = ApplicationBaseURL + mailbox + "/quotas";
-    document.quotasAjaxRequest
-      = triggerAjaxRequest(quotasUrl, quotasCallback);
+    var account = Mailer.currentMailbox.split("/")[1];
+    if (accounts[account].supportsQuotas) {
+      var quotasUrl = ApplicationBaseURL + mailbox + "/quotas";
+      triggerAjaxRequest(quotasUrl, quotasCallback);
+    }
   }
 }
 
@@ -503,8 +509,7 @@ function messageListCallback(http) {
   var div = $('mailboxContent');
   var table = $('messageList');
   
-  if (http.readyState == 4
-      && http.status == 200) {
+  if (http.status == 200) {
     document.messageListAjaxRequest = null;
 
     if (table) {
@@ -571,8 +576,7 @@ function messageListCallback(http) {
 }
 
 function quotasCallback(http) {
-  if (http.readyState == 4
-      && http.status == 200) {
+  if (http.status == 200) {
     var hasQuotas = false;
 
     if (http.responseText.length > 0) {
@@ -629,7 +633,7 @@ function onMessageContextMenuHide(event) {
 
 function onFolderMenuClick(event) {
   var onhide, menuName;
-   
+
   var menutype = this.parentNode.getAttribute("datatype");
   if (menutype) {
     if (menutype == "inbox") {
@@ -822,18 +826,16 @@ function onICalendarButtonClick(event) {
 }
 
 function ICalendarButtonCallback(http) {
-  if (http.readyState == 4)
-    if (isHttpStatus204(http.status)) {
-      var oldMsg = http.callbackData;
-      var msg = Mailer.currentMailbox + "/" + Mailer.currentMessages[Mailer.currentMailbox];
-      if (oldMsg == msg) {
-	deleteCachedMessage(oldMsg);
-	loadMessage(Mailer.currentMessages[Mailer.currentMailbox]);
-      }
-    }
-    else {
-      window.alert("received code: " + http.status);
+  if (isHttpStatus204(http.status)) {
+    var oldMsg = http.callbackData;
+    var msg = Mailer.currentMailbox + "/" + Mailer.currentMessages[Mailer.currentMailbox];
+    if (oldMsg == msg) {
+      deleteCachedMessage(oldMsg);
+      loadMessage(Mailer.currentMessages[Mailer.currentMailbox]);
     }
+  }
+  else
+    window.alert("received code: " + http.status);
 }
 
 function resizeMailContent() {
@@ -878,8 +880,7 @@ function onImageClick(event) {
 function messageCallback(http) {
   var div = $('messageContent');
 
-  if (http.readyState == 4
-      && http.status == 200) {
+  if (http.status == 200) {
     document.messageAjaxRequest = null;
     div.update(http.responseText);
     configureLinksInMessage();
@@ -1161,7 +1162,8 @@ function configureMessageListBodyEvents(table) {
     if ($(cell).hasClassName("tbtv_navcell")) {
       var anchors = $(cell).childNodesWithTag("a");
       for (var i = 0; i < anchors.length; i++)
-	Event.observe(anchors[i], "click", openMailboxAtIndex.bindAsEventListener(anchors[i]));
+	Event.observe(anchors[i], "click",
+		      openMailboxAtIndex.bindAsEventListener(anchors[i]));
     }
 
     rows = table.tBodies[0].rows;
@@ -1251,6 +1253,7 @@ function initMailer(event) {
 //     initDnd();
     initMailboxTree();
     initMessageCheckTimer();
+    getDragHandlesState();
   }
   
   // Default sort options
@@ -1418,8 +1421,7 @@ function updateMailboxMenus() {
 }
 
 function onLoadMailboxesCallback(http) {
-  if (http.readyState == 4
-      && http.status == 200) {
+  if (http.status == 200) {
     checkAjaxRequestsState();
     if (http.responseText.length > 0) {
       var newAccount = buildMailboxes(http.callbackData,
@@ -1456,6 +1458,10 @@ function onLoadMailboxesCallback(http) {
 
 function buildMailboxes(accountName, encoded) {
   var account = new Mailbox("account", accountName);
+
+  var accountIndex = mailAccounts.indexOf(accountName);
+  account.supportsQuotas = (quotaSupport[accountIndex] != 0);
+
   var data = encoded.evalJSON(true);
   for (var i = 0; i < data.length; i++) {
     var currentNode = account;
@@ -1481,6 +1487,28 @@ function buildMailboxes(accountName, encoded) {
   return account;
 }
 
+function getDragHandlesState() {
+  var urlstr =  ApplicationBaseURL + "dragHandlesState";
+  triggerAjaxRequest(urlstr, getDragHandlesStateCallback);
+}
+
+function getDragHandlesStateCallback(http) {
+  if (http.status == 200) {
+    if (http.responseText.length > 0) {
+      // The response text is a JSON array
+      // of the right and top offsets.
+      var data = http.responseText.evalJSON(true);
+
+      $("leftPanel").setStyle({ width: data[0] });
+      $("rightPanel").setStyle({ left: data[0] });
+      $("verticalDragHandle").setStyle({ left: data[0] });
+      $("mailboxContent").setStyle({ height: data[1] });
+      $("messageContent").setStyle({ top: data[1] });
+      $("rightDragHandle").setStyle({ top: data[1] });
+    }
+  }
+}
+
 function getFoldersState() {
   if (mailAccounts.length > 0) {
     var urlstr =  ApplicationBaseURL + "foldersState";
@@ -1489,10 +1517,9 @@ function getFoldersState() {
 }
 
 function getFoldersStateCallback(http) {
-  if (http.readyState == 4
-      && http.status == 200) {
+  if (http.status == 200) {
     if (http.responseText.length > 0) {
-      // The response text is a JSOn representation
+      // The response text is a JSON array
       // of the folders that were left opened.
       var data = http.responseText.evalJSON(true);
       for (var i = 1; i < mailboxTree.aNodes.length; i++) {
@@ -1501,8 +1528,28 @@ function getFoldersStateCallback(http) {
 	  mailboxTree.o(i);
       }
     }
+    mailboxTree.autoSync();
+  }
+}
+
+function saveDragHandlesState() {
+  // Call from SOGoDragHandles.js
+  var leftBlock = $("leftPanel");
+  var upperBlock = $("mailboxContent");
+
+  if (leftBlock && upperBlock) {
+    var dragHandlesState = new Array(leftBlock.getStyle("width"), 
+				     upperBlock.getStyle("height"));
+    var urlstr =  ApplicationBaseURL + "saveDragHandlesState" + "?dragHandles=" + dragHandlesState.toJSON();
+    triggerAjaxRequest(urlstr, saveDragHandlesStateCallback);
+  }
+}
+
+function saveDragHandlesStateCallback(http) {
+  if (http.readyState == 4
+      && isHttpStatus204(http.status)) {
+    log ("drag handles state saved");
   }
-  mailboxTree.autoSync();
 }
 
 function saveFoldersState() {
@@ -1514,8 +1561,7 @@ function saveFoldersState() {
 }
 
 function saveFoldersStateCallback(http) {
-  if (http.readyState == 4
-      && isHttpStatus204(http.status)) {
+  if (isHttpStatus204(http.status)) {
     log ("folders state saved");
   }
 }
diff --git a/UI/WebServerResources/SOGoDragHandles.js b/UI/WebServerResources/SOGoDragHandles.js
index d36a45e5..b81b281d 100644
--- a/UI/WebServerResources/SOGoDragHandles.js
+++ b/UI/WebServerResources/SOGoDragHandles.js
@@ -88,6 +88,8 @@ var SOGoDragHandlesInterface = {
     document.body.setAttribute('style', '');
     
     Event.stop(event);
+
+    saveDragHandlesState();
   },
   move: function (event) {
     if (!this.dhType)
diff --git a/UI/WebServerResources/SchedulerUI.js b/UI/WebServerResources/SchedulerUI.js
index 52b46a90..510d0cb6 100644
--- a/UI/WebServerResources/SchedulerUI.js
+++ b/UI/WebServerResources/SchedulerUI.js
@@ -1722,6 +1722,10 @@ function deletePersonalCalendarCallback(http) {
     log ("ajax problem 5: " + http.status);
 }
 
+function saveDragHandlesState() {
+  // Call from SOGoDragHandles.js
+}
+
 function configureLists() {
   var list = $("tasksList");
   list.multiselect = true;
diff --git a/UI/WebServerResources/UIxAttendeesEditor.js b/UI/WebServerResources/UIxAttendeesEditor.js
index 21b4ae46..9484f18b 100644
--- a/UI/WebServerResources/UIxAttendeesEditor.js
+++ b/UI/WebServerResources/UIxAttendeesEditor.js
@@ -322,7 +322,8 @@ function displayFreeBusyForNode(input) {
       }
       var sd = $('startTime_date').valueAsShortDateString();
       var ed = $('endTime_date').valueAsShortDateString();
-      var urlstr = ( UserFolderURL + "../" + input.uid + "/freebusy.ifb/ajaxRead?"
+      var urlstr = ( UserFolderURL + "../" + input.uid
+		     + "/freebusy.ifb/ajaxRead?"
 		     + "sday=" + sd + "&eday=" + ed + "&additional=" +
 		     additionalDays );
       document.contactFreeBusyAjaxRequest
@@ -601,7 +602,8 @@ function prepareAttendees() {
 	 $(row).addClassName(attendeesEditor.states[i]);
 	 var input = $(row).down("input");
 	 var value = "";
-	 if (attendeesEditor.names[i].length > 0 && attendeesEditor.names[i] != attendeesEditor.emails[i])
+	 if (attendeesEditor.names[i].length > 0
+	     && attendeesEditor.names[i] != attendeesEditor.emails[i])
 	    value += attendeesEditor.names[i] + " ";
 	 value += "<" + attendeesEditor.emails[i] + ">";
 	 input.value = value;
@@ -623,6 +625,7 @@ function prepareAttendees() {
       attendeesEditor.names = new Array();
       attendeesEditor.UIDs = new Array();
       attendeesEditor.emails = new Array();
+      newAttendee(null);
    }
 
    var inputs = tableAttendees.getElementsByTagName("input");
diff --git a/UI/WebServerResources/generic.js b/UI/WebServerResources/generic.js
index 98b92030..d25ea667 100644
--- a/UI/WebServerResources/generic.js
+++ b/UI/WebServerResources/generic.js
@@ -309,39 +309,47 @@ function appendDifferentiator(url) {
   return url_nocache;
 }
 
+function onAjaxRequestStateChange(http) {
+  try {
+    if (http.readyState == 4
+	&& activeAjaxRequests > 0) {
+      if (!http.aborted)
+	http.callback(http);
+      activeAjaxRequests--;
+      checkAjaxRequestsState();
+      http.onreadystatechange = Prototype.emptyFunction;
+      http.callback = Prototype.emptyFunction;
+      http.callbackData = null;
+    }
+  }
+  catch (e) {
+    activeAjaxRequests--;
+    checkAjaxRequestsState();
+    http.onreadystatechange = Prototype.emptyFunction;
+    http.callback = Prototype.emptyFunction;
+    http.callbackData = null;
+    log("AJAX Request, Caught Exception: " + e.name);
+    log(e.message);
+    log(backtrace());
+  }
+}
+
 function triggerAjaxRequest(url, callback, userdata, content, headers) {
   var http = createHTTPClient();
 
-  activeAjaxRequests += 1;
-  document.animTimer = setTimeout("checkAjaxRequestsState();", 50);
+  activeAjaxRequests++;
+  document.animTimer = setTimeout("checkAjaxRequestsState();", 250);
   //url = appendDifferentiator(url);
 
   if (http) {
     http.open("POST", url, true);
     http.url = url;
-    http.onreadystatechange
-      = function() {
-//       log ("state changed (" + http.readyState + "): " + url);
-      try {
-	if (http.readyState == 4
-	    && activeAjaxRequests > 0) {
-	  if (!http.aborted) {
-	    if (userdata)
-	      http.callbackData = userdata;
-	    callback(http);
-	  }
-	  activeAjaxRequests -= 1;
-	  checkAjaxRequestsState();
-	}
-      }
-      catch (e) {
-	activeAjaxRequests -= 1;
-	checkAjaxRequestsState();
-	log("AJAX Request, Caught Exception: " + e.name);
-	log(e.message);
-	log(backtrace());
-      }
-    };
+    http.callback = callback;
+    http.callbackData = userdata;
+    http.onreadystatechange = function() { onAjaxRequestStateChange(http) };
+//       = function() {
+// //       log ("state changed (" + http.readyState + "): " + url);
+//     };
     var hasContentLength = false;
     if (headers) {
       for (var i in headers) {
@@ -367,36 +375,31 @@ function triggerAjaxRequest(url, callback, userdata, content, headers) {
 
 function startAnimation(parent, nextNode) {
   var anim = $("progressIndicator");
-  if (anim) return anim;
-  
-  anim = document.createElement("img");
-  anim = $(anim);
-  anim.id = "progressIndicator";
-  anim.src = ResourcesURL + "/busy.gif";
-  anim.setStyle({ visibility: "hidden" });
-  if (nextNode)
-    parent.insertBefore(anim, nextNode);
-  else
-    parent.appendChild(anim);
-  anim.setStyle({ visibility: "visible" });
+  if (!anim) {
+    anim = createElement("img", "progressIndicator", null,
+			 {src: ResourcesURL + "/busy.gif"});
+    anim.setStyle({ visibility: "hidden" });
+    if (nextNode)
+      parent.insertBefore(anim, nextNode);
+    else
+      parent.appendChild(anim);
+    anim.setStyle({ visibility: "visible" });
+  }
 
   return anim;
 }
 
 function checkAjaxRequestsState() {
-  var toolbar = document.getElementById("toolbar");
-  if (toolbar) {
-    if (activeAjaxRequests > 0
-        && !document.busyAnim) {
-      document.busyAnim = startAnimation(toolbar);
-    }
-    else if (activeAjaxRequests == 0
-	     && document.busyAnim
-	     && document.busyAnim.parentNode) {
-      document.busyAnim.parentNode.removeChild(document.busyAnim);
-      document.busyAnim = null;
-    }
+  var progressImage = $("progressIndicator");
+  if (activeAjaxRequests > 0
+      && !progressImage) {
+    var toolbar = $("toolbar");
+    if (toolbar)
+      startAnimation(toolbar);
   }
+  else if (!activeAjaxRequests
+	   && progressImage)
+    progressImage.parentNode.removeChild(progressImage);
 }
 
 function isSafari3() {