From: Mike Taylor <mike@indexdata.com>
Date: Fri, 11 Oct 2013 16:06:48 +0000 (+0100)
Subject: Eliminate confusing and redundant local aliasing of mkws_config to
X-Git-Tag: 0.9.1~165
X-Git-Url: http://git.indexdata.com/?a=commitdiff_plain;h=980b18b55feef84bd8a83fb9d02b5b6e45472353;p=mkws-moved-to-github.git

Eliminate confusing and redundant local aliasing of mkws_config to
config in mkws_html_sort()
---

diff --git a/tools/htdocs/mkws.js b/tools/htdocs/mkws.js
index feb8723..0cad519 100644
--- a/tools/htdocs/mkws.js
+++ b/tools/htdocs/mkws.js
@@ -645,7 +645,7 @@ function mkws_html_all() {
 	var ranking_data = '';
 	ranking_data += '<form name="mkwsSelect" id="mkwsSelect" action="" >';
 	if (mkws_config.show_sort) {
-	    ranking_data +=  M('Sort by') + ' ' + mkws_html_sort(mkws_config) + ' ';
+	    ranking_data +=  M('Sort by') + ' ' + mkws_html_sort() + ' ';
 	}
 	if (mkws_config.show_perpage) {
 	    ranking_data += M('and show') + ' ' + mkws_html_perpage(mkws_config) + ' ' + M('per page') + '.';
@@ -708,16 +708,16 @@ function mkws_html_switch(config) {
     $("#mkwsTargets").css("display", "none");
 }
 
-function mkws_html_sort(config) {
+function mkws_html_sort() {
     debug("HTML sort");
     var sort_html = '<select name="mkwsSort" id="mkwsSort">';
 
-    for(var i = 0; i < config.sort_options.length; i++) {
-	var key = config.sort_options[i][0];
-	var val = config.sort_options[i].length == 1 ? config.sort_options[i][0] : config.sort_options[i][1];
+    for(var i = 0; i < mkws_config.sort_options.length; i++) {
+	var key = mkws_config.sort_options[i][0];
+	var val = mkws_config.sort_options[i].length == 1 ? mkws_config.sort_options[i][0] : mkws_config.sort_options[i][1];
 
 	sort_html += '<option value="' + key + '"';
-	if (key == config.sort_default) {
+	if (key == mkws_config.sort_default) {
 	    sort_html += ' selected="selected"';
 	}
 	sort_html += '>' + val + '</option>';