Move all template code to end of team objects.
[mkws-moved-to-github.git] / tools / htdocs / mkws.js
index 83bef63..b5e9908 100644 (file)
@@ -228,6 +228,10 @@ function team($, teamName) {
     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;
@@ -238,37 +242,18 @@ function team($, teamName) {
     // create a parameters array and pass it to the pz2's constructor
     // then register the form submit event with the pz2.search function
     // autoInit is set to true on default
-    m_paz = new pz2({ "onshow": onShow,
-                     "windowid": teamName,
-                     "showtime": 500,            //each timer (show, stat, term, bytarget) can be specified this way
+    m_paz = new pz2({ "windowid": teamName,
                      "pazpar2path": mkws_config.pazpar2_url,
+                     "usesessions" : mkws_config.use_service_proxy ? false : true,
                      "oninit": onInit,
+                     "onbytarget": onBytarget,
                      "onstat": onStat,
                      "onterm": (mkws_config.facets.length ? onTerm : undefined),
-                     "termlist": mkws_config.facets.join(','),
-                     "onbytarget": onBytarget,
-                     "usesessions" : mkws_config.use_service_proxy ? false : true,
-                     "showResponseType": '', // or "json" (for debugging?)
-                     "onrecord": onRecord });
-
-    if (!isNaN(parseInt(mkws_config.perpage_default))) {
-       m_perpage = parseInt(mkws_config.perpage_default);
-    }
-
-
-    // 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);
-    }
+                     "onshow": onShow,
+                     "onrecord": onRecord,
+                     "showtime": 500,            //each timer (show, stat, term, bytarget) can be specified this way
+                     "termlist": mkws_config.facets.join(',')
+                   });
 
 
     //
@@ -281,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);
     }
 
 
@@ -353,11 +329,11 @@ function team($, teamName) {
 
        for(var i = 0; i < facets.length; i++) {
            if (facets[i] == "xtargets") {
-               add_single_facet(acc, "Sources",  data.xtargets, 16, null);
+               addSingleFacet(acc, "Sources",  data.xtargets, 16, null);
            } else if (facets[i] == "subject") {
-               add_single_facet(acc, "Subjects", data.subject,  10, "subject");
+               addSingleFacet(acc, "Subjects", data.subject,  10, "subject");
            } else if (facets[i] == "author") {
-               add_single_facet(acc, "Authors",  data.author,   10, "author");
+               addSingleFacet(acc, "Authors",  data.author,   10, "author");
            } else {
                alert("bad facet configuration: '" + facets[i] + "'");
            }
@@ -367,7 +343,49 @@ function team($, teamName) {
     }
 
 
-    function add_single_facet(acc, caption, data, max, pzIndex) {
+    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 addSingleFacet(acc, caption, data, max, pzIndex) {
        acc.push('<div class="facet mkwsFacet' + caption + ' mkwsTeam_' + m_teamName + '">');
        acc.push('<div class="termtitle">' + M(caption) + '</div>');
        for (var i = 0; i < data.length && i < max; i++) {
@@ -377,7 +395,7 @@ function team($, teamName) {
            if (!pzIndex) {
                // Special case: target selection
                acc.push('target_id='+data[i].id+' ');
-               if (!target_filtered(data[i].id)) {
+               if (!targetFiltered(data[i].id)) {
                    action = 'mkws.limitTarget(\'' + m_teamName + '\', this.getAttribute(\'target_id\'),this.firstChild.nodeValue)';
                }
            } else {
@@ -391,7 +409,7 @@ function team($, teamName) {
     }
 
 
-    function target_filtered(id) {
+    function targetFiltered(id) {
        for (var i = 0; i < m_filters.length; i++) {
            if (m_filters[i].id === id ||
                m_filters[i].id === 'pz:id=' + id) {
@@ -402,47 +420,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);
-    }
-
     ////////////////////////////////////////////////////////////////////////////////
     ////////////////////////////////////////////////////////////////////////////////
 
@@ -466,7 +443,7 @@ function team($, teamName) {
        }
 
        m_filters = []
-       redraw_navi();
+       redrawNavi();
        resetPage();
        loadSelect();
        triggerSearch(query, sort, targets);
@@ -559,7 +536,7 @@ function team($, teamName) {
     {
        debug("limitTarget(id=" + id + ", name=" + name + ")");
        m_filters.push({ id: id, name: name });
-       redraw_navi();
+       redrawNavi();
        resetPage();
        loadSelect();
        triggerSearch();
@@ -572,7 +549,7 @@ function team($, teamName) {
     {
        debug("limitQuery(field=" + field + ", value=" + value + ")");
        m_filters.push({ field: field, value: value });
-       redraw_navi();
+       redrawNavi();
        resetPage();
        loadSelect();
        triggerSearch();
@@ -595,7 +572,7 @@ function team($, teamName) {
        }
        m_filters = newFilters;
 
-       redraw_navi();
+       redrawNavi();
        resetPage();
        loadSelect();
        triggerSearch();
@@ -620,7 +597,7 @@ function team($, teamName) {
        }
        m_filters = newFilters;
 
-       redraw_navi();
+       redrawNavi();
        resetPage();
        loadSelect();
        triggerSearch();
@@ -628,7 +605,7 @@ function team($, teamName) {
     }
 
 
-    function redraw_navi ()
+    function redrawNavi ()
     {
        var navi = findnode('.mkwsNavi');
        if (!navi) return;
@@ -789,132 +766,17 @@ function team($, teamName) {
     }
 
 
-    function renderDetails(data, marker)
-    {
-       var template = loadTemplate("Record");
-       var details = template(data);
-       return '<div class="details" id="mkwsDet_' + m_teamName + '_' + data.recid + '">' + details + '</div>';
-    }
-
-
-    function loadTemplate(name)
-    {
-       var template = m_template[name];
-
-       if (template === undefined) {
-           // Fall back to generic template if there is no team-specific one
-           var node = findnode(".mkwsTemplate_" + name);
-           if (!node.length) {
-               node = findnode(".mkwsTemplate_" + name, "ALL");
-           }
-
-           var source = node.html();
-           if (!source) {
-               source = defaultTemplate(name);
-           }
-
-           template = Handlebars.compile(source);
-           debug("compiled template '" + name + "'");
-           m_template[name] = template;
-       }
-
-       return template;
-    }
-
-
-    function defaultTemplate(name)
-    {
-       if (name === 'Record') {
-           return '\
-<table>\
-  <tr>\
-    <th>{{translate "Title"}}</th>\
-    <td>\
-      {{md-title}}\
-      {{#if md-title-remainder}}\
-       ({{md-title-remainder}})\
-      {{/if}}\
-      {{#if md-title-responsibility}}\
-       <i>{{md-title-responsibility}}</i>\
-      {{/if}}\
-    </td>\
-  </tr>\
-  {{#if md-date}}\
-  <tr>\
-    <th>{{translate "Date"}}</th>\
-    <td>{{md-date}}</td>\
-  </tr>\
-  {{/if}}\
-  {{#if md-author}}\
-  <tr>\
-    <th>{{translate "Author"}}</th>\
-    <td>{{md-author}}</td>\
-  </tr>\
-  {{/if}}\
-  {{#if md-electronic-url}}\
-  <tr>\
-    <th>{{translate "Links"}}</th>\
-    <td>\
-      {{#each md-electronic-url}}\
-       <a href="{{this}}">Link{{index1}}</a>\
-      {{/each}}\
-    </td>\
-  </tr>\
-  {{/if}}\
-  {{#if-any location having="md-subject"}}\
-  <tr>\
-    <th>{{translate "Subject"}}</th>\
-    <td>\
-      {{#first location having="md-subject"}}\
-       {{#if md-subject}}\
-         {{#commaList md-subject}}\
-           {{this}}{{/commaList}}\
-       {{/if}}\
-      {{/first}}\
-    </td>\
-  </tr>\
-  {{/if-any}}\
-  <tr>\
-    <th>{{translate "Locations"}}</th>\
-    <td>\
-      {{#commaList location}}\
-       {{attr "@name"}}{{/commaList}}\
-    </td>\
-  </tr>\
-</table>\
-';
-       } else if (name === "Summary") {
-           return '\
-<a href="#" id="{{_id}}" onclick="{{_onclick}}">\
-  <b>{{md-title}}</b>\
-</a>\
-{{#if md-title-remainder}}\
-  <span>{{md-title-remainder}}</span>\
-{{/if}}\
-{{#if md-title-responsibility}}\
-  <span><i>{{md-title-responsibility}}</i></span>\
-{{/if}}\
-';
-       }
-
-       var s = "There is no default '" + name +"' template!";
-       alert(s);
-       return s;
-    }
-
-
     /*
      * All the HTML stuff to render the search forms and
      * result pages.
      */
-    // ### This and other multi-word identifiers should be camelCase
-    function mkws_html_all() {
-       mkws_set_lang();
+    function mkwsHtmlAll() {
+       mkwsSetLang();
        if (mkws_config.show_lang)
-           mkws_html_lang();
+           mkwsHtmlLang();
 
        debug("HTML search form");
-       mkws.handle_node_with_team(findnode('.mkwsSearch'), function(tname) {
+       mkws.handleNodeWithTeam(findnode('.mkwsSearch'), function(tname) {
            this.html('\
 <form name="mkwsSearchForm" class="mkwsSearchForm mkwsTeam_' + tname + '" action="" >\
   <input class="mkwsQuery mkwsTeam_' + tname + '" type="text" size="' + mkws_config.query_width + '" />\
@@ -959,17 +821,17 @@ function team($, teamName) {
            var ranking_data = '';
            ranking_data += '<form name="mkwsSelect" class="mkwsSelect mkwsTeam_' + m_teamName + '" action="" >';
            if (mkws_config.show_sort) {
-               ranking_data +=  M('Sort by') + ' ' + mkws_html_sort() + ' ';
+               ranking_data +=  M('Sort by') + ' ' + mkwsHtmlSort() + ' ';
            }
            if (mkws_config.show_perpage) {
-               ranking_data += M('and show') + ' ' + mkws_html_perpage() + ' ' + M('per page') + '.';
+               ranking_data += M('and show') + ' ' + mkwsHtmlPerpage() + ' ' + M('per page') + '.';
            }
             ranking_data += '</form>';
 
            node.html(ranking_data);
        }
 
-       mkws_html_switch();
+       mkwsHtmlSwitch();
 
        var node;
        node = findnode('.mkwsSearchForm');
@@ -993,20 +855,7 @@ function team($, teamName) {
     }
 
 
-    // implement $.parseQuerystring() for parsing URL parameters
-    function parseQuerystring() {
-       var nvpair = {};
-       var qs = window.location.search.replace('?', '');
-       var pairs = qs.split('&');
-       $.each(pairs, function(i, v){
-           var pair = v.split('=');
-           nvpair[pair[0]] = pair[1];
-       });
-       return nvpair;
-    }
-
-
-    function mkws_set_lang()  {
+    function mkwsSetLang()  {
        var lang = parseQuerystring().lang || mkws_config.lang;
        if (!lang || !mkws.locale_lang[lang]) {
            mkws_config.lang = ""
@@ -1019,7 +868,7 @@ function team($, teamName) {
     }
 
 
-    function mkws_html_switch() {
+    function mkwsHtmlSwitch() {
        debug("HTML switch for team " + m_teamName);
 
        var node = findnode(".mkwsSwitch");
@@ -1037,7 +886,7 @@ function team($, teamName) {
     }
 
 
-    function mkws_html_sort() {
+    function mkwsHtmlSort() {
        debug("HTML sort, m_sort = '" + m_sort + "'");
        var sort_html = '<select class="mkwsSort mkwsTeam_' + m_teamName + '">';
 
@@ -1058,7 +907,7 @@ function team($, teamName) {
     }
 
 
-    function mkws_html_perpage() {
+    function mkwsHtmlPerpage() {
        debug("HTML perpage, m_perpage = " + m_perpage);
        var perpage_html = '<select class="mkwsPerpage mkwsTeam_' + m_teamName + '">';
 
@@ -1078,7 +927,7 @@ function team($, teamName) {
 
 
     /* create locale language menu */
-    function mkws_html_lang() {
+    function mkwsHtmlLang() {
        var lang_default = "en";
        var lang = mkws_config.lang || lang_default;
        var list = [];
@@ -1120,7 +969,7 @@ function team($, teamName) {
     }
 
 
-    that.run_auto_search = function() {
+    that.runAutoSearch = function() {
        // ### should check mkwsTermlist as well, for facet-only teams
        var node = findnode('.mkwsRecords');
        var query = node.attr('autosearch');
@@ -1157,6 +1006,19 @@ function team($, teamName) {
     }
 
 
+    // implement $.parseQuerystring() for parsing URL parameters
+    function parseQuerystring() {
+       var nvpair = {};
+       var qs = window.location.search.replace('?', '');
+       var pairs = qs.split('&');
+       $.each(pairs, function(i, v){
+           var pair = v.split('=');
+           nvpair[pair[0]] = pair[1];
+       });
+       return nvpair;
+    }
+
+
     // This function is taken from a StackOverflow answer
     // http://stackoverflow.com/questions/901115/how-can-i-get-query-string-values-in-javascript/901144#901144
     // ### should we unify this and parseQuerystring()?
@@ -1180,10 +1042,148 @@ 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);
+    }
+
+
+    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 renderDetails(data, marker)
+    {
+       var template = loadTemplate("Record");
+       var details = template(data);
+       return '<div class="details" id="mkwsDet_' + m_teamName + '_' + data.recid + '">' + details + '</div>';
+    }
+
+
+    function loadTemplate(name)
+    {
+       var template = m_template[name];
+
+       if (template === undefined) {
+           // Fall back to generic template if there is no team-specific one
+           var node = findnode(".mkwsTemplate_" + name);
+           if (!node.length) {
+               node = findnode(".mkwsTemplate_" + name, "ALL");
+           }
+
+           var source = node.html();
+           if (!source) {
+               source = defaultTemplate(name);
+           }
+
+           template = Handlebars.compile(source);
+           debug("compiled template '" + name + "'");
+           m_template[name] = template;
+       }
+
+       return template;
+    }
+
+
+    function defaultTemplate(name)
+    {
+       if (name === 'Record') {
+           return '\
+<table>\
+  <tr>\
+    <th>{{translate "Title"}}</th>\
+    <td>\
+      {{md-title}}\
+      {{#if md-title-remainder}}\
+       ({{md-title-remainder}})\
+      {{/if}}\
+      {{#if md-title-responsibility}}\
+       <i>{{md-title-responsibility}}</i>\
+      {{/if}}\
+    </td>\
+  </tr>\
+  {{#if md-date}}\
+  <tr>\
+    <th>{{translate "Date"}}</th>\
+    <td>{{md-date}}</td>\
+  </tr>\
+  {{/if}}\
+  {{#if md-author}}\
+  <tr>\
+    <th>{{translate "Author"}}</th>\
+    <td>{{md-author}}</td>\
+  </tr>\
+  {{/if}}\
+  {{#if md-electronic-url}}\
+  <tr>\
+    <th>{{translate "Links"}}</th>\
+    <td>\
+      {{#each md-electronic-url}}\
+       <a href="{{this}}">Link{{index1}}</a>\
+      {{/each}}\
+    </td>\
+  </tr>\
+  {{/if}}\
+  {{#if-any location having="md-subject"}}\
+  <tr>\
+    <th>{{translate "Subject"}}</th>\
+    <td>\
+      {{#first location having="md-subject"}}\
+       {{#if md-subject}}\
+         {{#commaList md-subject}}\
+           {{this}}{{/commaList}}\
+       {{/if}}\
+      {{/first}}\
+    </td>\
+  </tr>\
+  {{/if-any}}\
+  <tr>\
+    <th>{{translate "Locations"}}</th>\
+    <td>\
+      {{#commaList location}}\
+       {{attr "@name"}}{{/commaList}}\
+    </td>\
+  </tr>\
+</table>\
+';
+       } else if (name === "Summary") {
+           return '\
+<a href="#" id="{{_id}}" onclick="{{_onclick}}">\
+  <b>{{md-title}}</b>\
+</a>\
+{{#if md-title-remainder}}\
+  <span>{{md-title-remainder}}</span>\
+{{/if}}\
+{{#if md-title-responsibility}}\
+  <span><i>{{md-title-responsibility}}</i></span>\
+{{/if}}\
+';
+       }
+
+       var s = "There is no default '" + name +"' template!";
+       alert(s);
+       return s;
+    }
+
+
     // main
     (function() {
        try {
-           mkws_html_all()
+           mkwsHtmlAll()
        }
 
        catch (e) {
@@ -1219,7 +1219,7 @@ function team($, teamName) {
     var debug = mkws.debug;
 
 
-    mkws.handle_node_with_team = function(node, callback) {
+    mkws.handleNodeWithTeam = function(node, callback) {
        // First branch for DOM objects; second branch for jQuery objects
        var classes = node.className || node.attr('class');
        if (!classes) {
@@ -1227,7 +1227,7 @@ function team($, teamName) {
            // undefined, we don't get an error message, but this
            // function and its callers, up several stack level,
            // silently return. What a crock.
-           mkws.debug("handle_node_with_team() called on node with no classes");
+           mkws.debug("handleNodeWithTeam() called on node with no classes");
            return;
        }
        var list = classes.split(/\s+/)
@@ -1245,7 +1245,7 @@ function team($, teamName) {
     }
 
 
-    mkws.resize_page = function () {
+    mkws.resizePage = function () {
        var list = ["mkwsSwitch", "mkwsLang"];
 
        var width = mkws_config.responsive_design_width;
@@ -1314,7 +1314,7 @@ function team($, teamName) {
     }
 
 
-    function default_mkws_config() {
+    function defaultMkwsConfig() {
        /* default mkws config */
        var config_default = {
            use_service_proxy: true,
@@ -1364,7 +1364,7 @@ function team($, teamName) {
      * The username/password is configured in the apache config file
      * for the site.
      */
-    function authenticate_session(auth_url, auth_domain, pp2_url) {
+    function authenticateSession(auth_url, auth_domain, pp2_url) {
        debug("Run service proxy auth URL: " + auth_url);
 
        if (!auth_domain) {
@@ -1390,23 +1390,23 @@ function team($, teamName) {
 
            debug("Service proxy auth successfully done");
            mkws.authenticated = true;
-           run_auto_searches();
+           runAutoSearches();
        });
     }
 
 
-    function run_auto_searches() {
+    function runAutoSearches() {
        debug("running auto searches");
 
        for (var teamName in mkws.teams) {
-           mkws.teams[teamName].run_auto_search();
+           mkws.teams[teamName].runAutoSearch();
        }
     }
 
 
     $(document).ready(function() {
        debug("on load ready");
-       default_mkws_config();
+       defaultMkwsConfig();
 
        if (mkws_config.query_width < 5 || mkws_config.query_width > 150) {
            debug("Reset query width: " + mkws_config.query_width);
@@ -1427,9 +1427,9 @@ function team($, teamName) {
        if (mkws_config.responsive_design_width) {
            // Responsive web design - change layout on the fly based on
            // current screen width. Required for mobile devices.
-           $(window).resize(function(e) { mkws.resize_page() });
+           $(window).resize(function(e) { mkws.resizePage() });
            // initial check after page load
-           $(document).ready(function() { mkws.resize_page() });
+           $(document).ready(function() { mkws.resizePage() });
        }
 
        // Backwards compatibility: set new magic class names on any
@@ -1459,7 +1459,7 @@ function team($, teamName) {
        // the mkwsTeam_* class. Make all team objects.
        var then = $.now();
        $('[class^="mkws"],[class*=" mkws"]').each(function () {
-           mkws.handle_node_with_team(this, function(tname, type) {
+           mkws.handleNodeWithTeam(this, function(tname, type) {
                if (!mkws.teams[tname]) {
                    mkws.teams[tname] = team(j, tname);
                    debug("Made MKWS team '" + tname + "'");
@@ -1472,12 +1472,12 @@ function team($, teamName) {
        debug("Walking MKWS nodes took " + (now-then) + " ms");
 
        if (mkws_config.use_service_proxy) {
-           authenticate_session(mkws_config.service_proxy_auth,
-                                mkws_config.service_proxy_auth_domain,
-                                mkws_config.pazpar2_url);
+           authenticateSession(mkws_config.service_proxy_auth,
+                               mkws_config.service_proxy_auth_domain,
+                               mkws_config.pazpar2_url);
        } else {
            // raw pp2
-           run_auto_searches();
+           runAutoSearches();
        }
     });
 })(jQuery);