Limit length of Commands sent to citserver; do multible json requests if neccessary

[citadel.git] / webcit / static / summaryview.js

diff --git a/webcit/static/summaryview.js b/webcit/static/summaryview.js
index edcb3e2b81ce2f00b222e2712f6c21f5b3d76fc0..0dffd2416482ea75d493a79596532b6ff56aa010 100644 (file)
--- a/webcit/static/summaryview.js
+++ b/webcit/static/summaryview.js
@@ -364,10 +364,10 @@ function getSortMode(toggleElem) {
 function removeOldSortClass() {
        if (currentSorterToggle) {
                var classes = currentSorterToggle.className;
-               /* classes = classes.replace("current_sort_mode","");
-                  classes = classes.replace("sort_ascending","");
-                  classes = classes.replace("sort_descending",""); */
-               currentSorterToggle.className = "";
+               classes = classes.replace("current_sort_mode","");
+               classes = classes.replace("sort_ascending","");
+               classes = classes.replace("sort_descending","");
+               currentSorterToggle.className = classes;
        }
 }
 function markRow(row) {
@@ -408,26 +408,46 @@ function deleteAllMarkedRows() {
        msgs = newMsgs;
        resortAndDisplay(null);
 }
-
 function deleteAllSelectedMessages() {
-       var pa = "";
-       for(msgId in currentlyMarkedRows) {
-               if (!room_is_trash) {
-                       pa = encodeURI("g_cmd=MOVE " + msgId + "|_TRASH_|0");
-               }
-               else {
-                       pa = encodeURI("g_cmd=DELE " + msgId);
+    var mvCommand = "";
+    var msgIds = "";
+    for(msgId in currentlyMarkedRows) {
+       msgIds += ","+msgId;
+
+       if (msgIds.length > 800) {
+           if (!room_is_trash) {
+               mvCommand = encodeURI("g_cmd=MOVE " + msgIds + "|_TRASH_|0");
+           }
+           else {
+               mvCommand = encodeURI("g_cmd=DELE " + msgIds);
+           }
+           new Ajax.Request("ajax_servcmd", {
+               parameters: mvCommand,
+               method: 'post',
+               onSuccess: function(transport) {
+                   WCLog(transport.responseText);
                }
-               new Ajax.Request("ajax_servcmd", {
-                       parameters: pa,
-                       method: 'post',
-                       onSuccess: function(transport) {
-                               WCLog(transport.responseText);
-                       }
-               });
+           });
+           msgIds = "";
        }
-       document.getElementById("preview_pane").innerHTML = "";
-       deleteAllMarkedRows();
+    }
+
+    if (!room_is_trash) {
+       mvCommand = encodeURI("g_cmd=MOVE " + msgIds + "|_TRASH_|0");
+    }
+    else {
+       mvCommand = encodeURI("g_cmd=DELE " + msgIds);
+    }
+    new Ajax.Request("ajax_servcmd", {
+       parameters: mvCommand,
+       method: 'post',
+       onSuccess: function(transport) {
+           WCLog(transport.responseText);
+       }
+    });
+
+    document.getElementById("preview_pane").innerHTML = "";
+    deleteAllMarkedRows();
 }
 
 function CtdlMessageListKeyUp(event) {
@@ -483,7 +503,7 @@ function sizePreviewPane() {
        var heightOfViewPort = document.viewport.getHeight() // prototypejs method
                var bannerHeight = banner.offsetHeight;
        var contentViewPortHeight = heightOfViewPort-banner.offsetHeight-message_list_hdr.offsetHeight;
-       contentViewPortHeight = 0.98 * contentViewPortHeight; // leave some error
+       contentViewPortHeight = 0.95 * contentViewPortHeight; // leave some error (especially for FF3!!)
        // Set summary_view to 20%;
        var summary_height = ctdlLocalPrefs.readPref("svheight");
        if (summary_height == null) {