From: Mike Taylor <mike@indexdata.com>
Date: Fri, 21 Mar 2014 12:34:56 +0000 (+0000)
Subject: onSortChange and onPerpageChange set their member variables directly.
X-Git-Tag: 1.0.0~1288
X-Git-Url: http://git.indexdata.com/?p=mkws-moved-to-github.git;a=commitdiff_plain;h=d6d518d60b8cf23880727046412b177c12406a85;ds=sidebyside

onSortChange and onPerpageChange set their member variables directly.
loadSelect() is now just an assertion that things are set correctly.
---

diff --git a/tools/htdocs/mkws.js b/tools/htdocs/mkws.js
index 32be2c7..5ea64a0 100644
--- a/tools/htdocs/mkws.js
+++ b/tools/htdocs/mkws.js
@@ -561,6 +561,7 @@ function team($, teamName) {
 
     function onSortChange()
     {
+	m_sortOrder = findnode('.mkwsSort').val();
 	if (!m_submitted) return false;
 	resetPage();
 	loadSelect();
@@ -571,6 +572,7 @@ function team($, teamName) {
 
     function onPerpageChange()
     {
+	m_perpage = findnode('.mkwsPerpage').val();
 	if (!m_submitted) return false;
 	resetPage();
 	loadSelect();
@@ -596,13 +598,11 @@ function team($, teamName) {
     {
 	var node = findnode('.mkwsSort');
 	if (node.length && node.val() != m_sortOrder) {
-	    debug("changing m_sortOrder from " + m_sortOrder + " to " + node.val());
-	    m_sortOrder = node.val();
+	    alert("m_sortOrder '" + m_sortOrder + "' != node.val() '" + node.val() + "'");
 	}
 	node = findnode('.mkwsPerpage');
 	if (node.length && node.val() != m_perpage) {
-	    debug("changing m_perpage from " + m_perpage + " to " + node.val());
-	    m_perpage = node.val();
+	    alert("m_perpage '" + m_perpage + "' != node.val() '" + node.val() + "'");
 	}
     }