Fix another typo

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

diff --git a/webcit/static/summaryview.js b/webcit/static/summaryview.js
index eb9e2ebe1119164791002c6d314f383884d70057..ad08e53ceed23a70bb0086e3c502e4030ca6d27f 100644 (file)
--- a/webcit/static/summaryview.js
+++ b/webcit/static/summaryview.js
@@ -28,11 +28,12 @@ var exitedMouseDown = false;
 var trTemplate = new Array(11);
 trTemplate[0] = "<tr id=\"";
 trTemplate[2] = "\" citadel:dropenabled=\"dropenabled\" class=\"";
-trTemplate[4] = "\" citadel:dndelement=\"summaryViewDragAndDropHandler\" citadel:msgID=\"";
-trTemplate[6] = "\"><td class=\"col1\">";
-trTemplate[8] = "</td><td class=\"col2\">";
-trTemplate[10] = "</td><td class=\"col3\">";
-trTemplate[12] = "</td></tr>";
+trTemplate[4] = "\" citadel:dndelement=\"summaryViewDragAndDropHandler\" citadel:msgid=\"";
+trTemplate[6] = "\" citadel:ctdlrowid=\"";
+trTemplate[8] = "\"><td class=\"col1\">";
+trTemplate[10] = "</td><td class=\"col2\">";
+trTemplate[12] = "</td><td class=\"col3\">";
+trTemplate[14] = "</td></tr>";
 
 var currentPage = 0;
 var sortModes = {
@@ -106,9 +107,15 @@ new Ajax.Request("roommsgs", {
        });
 }
 function evalJSON(data) {
+  var jsonData = null;
   if (typeof(JSON) === 'object' && typeof(JSON.parse) === 'function') {
-    return JSON.parse(data);
-  } else {
+    try {
+    jsonData = JSON.parse(data);
+    } catch (e) {
+      // ignore
+    }
+  }
+  if (jsonData == null) {
     return eval('('+data+')');
   }
 }
@@ -176,13 +183,14 @@ function resortAndDisplay(sortMode) {
     }
     trTemplate[3] = className;
     trTemplate[5] = currentRow[0];
-    trTemplate[7] = currentRow[1];
-    trTemplate[9] = currentRow[2];
-    trTemplate[11] = currentRow[4];
+    trTemplate[7] = x;
+    trTemplate[9] = currentRow[1];
+    trTemplate[11] = currentRow[2];
+    trTemplate[13] = currentRow[4];
     var i = x+1;
     compiled[i] = trTemplate.join("");
     } catch (e) {
-      alert("Exception on row " +  x + ":" + e);
+      WCLog("Exception on row " +  x + ":" + e);
     }
   }
   compiled[length+2] = "</tbody>";
@@ -238,7 +246,7 @@ function CtdlMessageListClick(evt) {
   // If the ctrl key modifier wasn't used, unmark all rows and load the message
   if (!event.shiftKey && !event.ctrlKey && !event.altKey) {
     unmarkAllRows();
-    markedRowId = parent.ctdlRowId;
+    markedRowId = parent.getAttribute("citadel:ctdlrowid");
     document.getElementById("preview_pane").innerHTML = "";
     new Ajax.Updater('preview_pane', 'msg/'+msgId, {method: 'get'});
     markRow(parent);