From: Mike Taylor <mike@indexdata.com>
Date: Fri, 6 Dec 2013 15:30:00 +0000 (+0000)
Subject: Rationalise the confusion between mkws_config.sort_default and mkws.sort.
X-Git-Tag: 0.9.1~128
X-Git-Url: http://git.indexdata.com/?a=commitdiff_plain;ds=sidebyside;h=859e28a7c5879af9c23b75e219ac7ce0ed317eb7;hp=1164a77a52550a7a69d432c3a8b27af43aff44a3;p=mkws-moved-to-github.git

Rationalise the confusion between mkws_config.sort_default and mkws.sort.
---

diff --git a/tools/htdocs/mkws.js b/tools/htdocs/mkws.js
index 1a80b10..9ffe3e1 100644
--- a/tools/htdocs/mkws.js
+++ b/tools/htdocs/mkws.js
@@ -193,6 +193,9 @@ Handlebars.registerHelper('commaList', function(items, options) {
     }
 }
 
+mkws.sort = mkws_config.sort_default;
+debug("copied mkws_config.sort_default '" + mkws_config.sort_default + "' to mkws.sort");
+
 if (mkws_config.query_width < 5 || mkws_config.query_width > 150) {
     debug("Reset query width: " + mkws_config.query_width);
     mkws_config.query_width = 50;
@@ -952,7 +955,7 @@ function mkws_html_switch() {
 }
 
 function mkws_html_sort() {
-    debug("HTML sort");
+    debug("HTML sort, mkws.sort = '" + mkws.sort + "'");
     var sort_html = '<select name="mkwsSort" id="mkwsSort">';
 
     for(var i = 0; i < mkws_config.sort_options.length; i++) {
@@ -961,7 +964,7 @@ function mkws_html_sort() {
 	var val = opt.length == 1 ? opt[0] : opt[1];
 
 	sort_html += '<option value="' + key + '"';
-	if (key == mkws_config.sort_default) {
+	if (mkws.sort == key || mkws.sort == val) {
 	    sort_html += ' selected="selected"';
 	}
 	sort_html += '>' + M(val) + '</option>';