Rename variable for consistency.

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

diff --git a/tools/htdocs/mkws.js b/tools/htdocs/mkws.js
index b5e9908..bb96eac 100644 (file)
--- a/tools/htdocs/mkws.js
+++ b/tools/htdocs/mkws.js
@@ -226,17 +226,7 @@ function team($, teamName) {
     debug("start running MKWS");
 
     m_sort = mkws_config.sort_default;
-    debug("copied mkws_config.sort_default '" + mkws_config.sort_default + "' to m_sort");
-
-    if (!isNaN(parseInt(mkws_config.perpage_default))) {
-       m_perpage = parseInt(mkws_config.perpage_default);
-    }
-
-    // protocol independent link for pazpar2: "//mkws/sp" -> "https://mkws/sp"
-    if (mkws_config.pazpar2_url.match(/^\/\//)) {
-       mkws_config.pazpar2_url = document.location.protocol + mkws_config.pazpar2_url;
-       debug("adjust protocol independent links: " + mkws_config.pazpar2_url);
-    }
+    m_perpage = mkws_config.perpage_default;
 
     debug("Create main pz2 object");
     // create a parameters array and pass it to the pz2's constructor
@@ -269,7 +259,7 @@ function team($, teamName) {
     function onBytarget(data, teamName) {
        debug("target");
        var targetDiv = findnode('.mkwsBytarget');
-       if (!targetDiv) {
+       if (targetDiv.length === 0) {
            return;
        }
 
@@ -296,11 +286,11 @@ function team($, teamName) {
 
     function onStat(data, teamName) {
        debug("stat");
-       var stat = findnode('.mkwsStat');
-       if (stat.length === 0)
+       var statDiv = findnode('.mkwsStat');
+       if (statDiv.length === 0)
            return;
 
-       stat.html('<span class="head">' + M('Status info') + '</span>' +
+       statDiv.html('<span class="head">' + M('Status info') + '</span>' +
            ' -- ' +
            '<span class="clients">' + M('Active clients') + ': ' + data.activeclients + '/' + data.clients + '</span>' +
            ' -- ' +
@@ -1432,6 +1422,12 @@ function team($, teamName) {
            $(document).ready(function() { mkws.resizePage() });
        }
 
+       // protocol independent link for pazpar2: "//mkws/sp" -> "https://mkws/sp"
+       if (mkws_config.pazpar2_url.match(/^\/\//)) {
+           mkws_config.pazpar2_url = document.location.protocol + mkws_config.pazpar2_url;
+           debug("adjust protocol independent links: " + mkws_config.pazpar2_url);
+       }
+
        // Backwards compatibility: set new magic class names on any
        // elements that have the old magic IDs.
        var ids = [ "Switch", "Lang", "Search", "Pager", "Navi",