Move utility function findnode() to near the end.

[mkws-moved-to-github.git] / tools / htdocs / mkws.js

diff --git a/tools/htdocs/mkws.js b/tools/htdocs/mkws.js
index 65848c9..abee7d2 100644 (file)
--- a/tools/htdocs/mkws.js
+++ b/tools/htdocs/mkws.js
@@ -252,26 +252,10 @@ function team($, teamName) {
                      "onshow": onShow,
                      "onrecord": onRecord,
                      "showtime": 500,            //each timer (show, stat, term, bytarget) can be specified this way
-                     "termlist": mkws_config.facets.join(','),
-                     "showResponseType": '' // or "json" (for debugging?)
+                     "termlist": mkws_config.facets.join(',')
                    });
 
 
-    // Finds the node of the specified class within the current team
-    // Multiple OR-clauses separated by commas are handled
-    // More complex cases may not work
-    //
-    function findnode(selector, teamName) {
-       teamName = teamName || m_teamName;
-
-       selector = selector.split(',').map(function(s) {
-           return s + '.mkwsTeam_' + teamName;
-       }).join(',');
-
-       return $(selector);
-    }
-
-
     //
     // pz2.js event handlers:
     //
@@ -282,40 +266,31 @@ function team($, teamName) {
     }
 
 
-    function onShow(data, teamName) {
-       debug("show");
-       m_totalRec = data.merged;
-
-       var pager = findnode(".mkwsPager");
-       if (pager.length) {
-           pager.html(drawPager(data))
+    function onBytarget(data, teamName) {
+       debug("target");
+       var targetDiv = findnode('.mkwsBytarget');
+       if (!targetDiv) {
+           return;
        }
 
-       var results = findnode(".mkwsRecords");
-       if (!results.length)
-           return;
+       var table ='<table><thead><tr>' +
+           '<td>' + M('Target ID') + '</td>' +
+           '<td>' + M('Hits') + '</td>' +
+           '<td>' + M('Diags') + '</td>' +
+           '<td>' + M('Records') + '</td>' +
+           '<td>' + M('State') + '</td>' +
+           '</tr></thead><tbody>';
 
-       var html = [];
-       for (var i = 0; i < data.hits.length; i++) {
-            var hit = data.hits[i];
-           html.push('<div class="record" id="mkwsRecdiv_' + teamName + '_' + hit.recid + '" >',
-                     renderSummary(hit),
-                     '</div>');
-           if (hit.recid == m_curDetRecId) {
-               if (m_curDetRecData)
-                   html.push(renderDetails(m_curDetRecData));
-           }
+       for (var i = 0; i < data.length; i++) {
+            table += "<tr><td>" + data[i].id +
+               "</td><td>" + data[i].hits +
+               "</td><td>" + data[i].diagnostic +
+               "</td><td>" + data[i].records +
+               "</td><td>" + data[i].state + "</td></tr>";
        }
-       results.html(html.join(''));
-    }
-
 
-    function renderSummary(hit)
-    {
-       var template = loadTemplate("Summary");
-       hit._id = "mkwsRec_" + hit.recid;
-       hit._onclick = "mkws.showDetails('" + m_teamName + "', this.id);return false;"
-       return template(hit);
+       table += '</tbody></table>';
+       targetDiv.html(table);
     }
 
 
@@ -368,6 +343,57 @@ function team($, teamName) {
     }
 
 
+    function onShow(data, teamName) {
+       debug("show");
+       m_totalRec = data.merged;
+
+       var pager = findnode(".mkwsPager");
+       if (pager.length) {
+           pager.html(drawPager(data))
+       }
+
+       var results = findnode(".mkwsRecords");
+       if (!results.length)
+           return;
+
+       var html = [];
+       for (var i = 0; i < data.hits.length; i++) {
+            var hit = data.hits[i];
+           html.push('<div class="record" id="mkwsRecdiv_' + teamName + '_' + hit.recid + '" >',
+                     renderSummary(hit),
+                     '</div>');
+           if (hit.recid == m_curDetRecId) {
+               if (m_curDetRecData)
+                   html.push(renderDetails(m_curDetRecData));
+           }
+       }
+       results.html(html.join(''));
+    }
+
+
+    function onRecord(data, args, teamName) {
+       debug("record: teamName=" + teamName + ", m_teamName=" + m_teamName);
+       // FIXME: record is async!!
+       clearTimeout(m_paz.recordTimer);
+       // in case on_show was faster to redraw element
+       var detRecordDiv = document.getElementById('mkwsDet_' + teamName + '_' + data.recid);
+       if (detRecordDiv) return;
+       m_curDetRecData = data;
+       var recordDiv = document.getElementById('mkwsRecdiv_' + teamName + '_' + m_curDetRecData.recid);
+       var html = renderDetails(m_curDetRecData);
+       recordDiv.innerHTML += html;
+    }
+
+
+    function renderSummary(hit)
+    {
+       var template = loadTemplate("Summary");
+       hit._id = "mkwsRec_" + hit.recid;
+       hit._onclick = "mkws.showDetails('" + m_teamName + "', this.id);return false;"
+       return template(hit);
+    }
+
+
     function addSingleFacet(acc, caption, data, max, pzIndex) {
        acc.push('<div class="facet mkwsFacet' + caption + ' mkwsTeam_' + m_teamName + '">');
        acc.push('<div class="termtitle">' + M(caption) + '</div>');
@@ -403,47 +429,6 @@ function team($, teamName) {
     }
 
 
-    function onRecord(data, args, teamName) {
-       debug("record: teamName=" + teamName + ", m_teamName=" + m_teamName);
-       // FIXME: record is async!!
-       clearTimeout(m_paz.recordTimer);
-       // in case on_show was faster to redraw element
-       var detRecordDiv = document.getElementById('mkwsDet_' + teamName + '_' + data.recid);
-       if (detRecordDiv) return;
-       m_curDetRecData = data;
-       var recordDiv = document.getElementById('mkwsRecdiv_' + teamName + '_' + m_curDetRecData.recid);
-       var html = renderDetails(m_curDetRecData);
-       recordDiv.innerHTML += html;
-    }
-
-
-    function onBytarget(data, teamName) {
-       debug("target");
-       var targetDiv = findnode('.mkwsBytarget');
-       if (!targetDiv) {
-           return;
-       }
-
-       var table ='<table><thead><tr>' +
-           '<td>' + M('Target ID') + '</td>' +
-           '<td>' + M('Hits') + '</td>' +
-           '<td>' + M('Diags') + '</td>' +
-           '<td>' + M('Records') + '</td>' +
-           '<td>' + M('State') + '</td>' +
-           '</tr></thead><tbody>';
-
-       for (var i = 0; i < data.length; i++) {
-            table += "<tr><td>" + data[i].id +
-               "</td><td>" + data[i].hits +
-               "</td><td>" + data[i].diagnostic +
-               "</td><td>" + data[i].records +
-               "</td><td>" + data[i].state + "</td></tr>";
-       }
-
-       table += '</tbody></table>';
-       targetDiv.html(table);
-    }
-
     ////////////////////////////////////////////////////////////////////////////////
     ////////////////////////////////////////////////////////////////////////////////
 
@@ -1180,6 +1165,21 @@ function team($, teamName) {
     mkws.M = M; // so the Handlebars helper can use it
 
 
+    // Finds the node of the specified class within the current team
+    // Multiple OR-clauses separated by commas are handled
+    // More complex cases may not work
+    //
+    function findnode(selector, teamName) {
+       teamName = teamName || m_teamName;
+
+       selector = selector.split(',').map(function(s) {
+           return s + '.mkwsTeam_' + teamName;
+       }).join(',');
+
+       return $(selector);
+    }
+
+
     // main
     (function() {
        try {