From d6d518d60b8cf23880727046412b177c12406a85 Mon Sep 17 00:00:00 2001 From: Mike Taylor Date: Fri, 21 Mar 2014 12:34:56 +0000 Subject: [PATCH] onSortChange and onPerpageChange set their member variables directly. loadSelect() is now just an assertion that things are set correctly. --- tools/htdocs/mkws.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) 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() + "'"); } } -- 1.7.10.4