Remove extraneous debugging.

[mkws-moved-to-github.git] / tools / htdocs / mkws.js

diff --git a/tools/htdocs/mkws.js b/tools/htdocs/mkws.js
index aced7e2..809fc18 100644 (file)
--- a/tools/htdocs/mkws.js
+++ b/tools/htdocs/mkws.js
@@ -224,7 +224,6 @@ function widget($, team, type, node) {
 
     function promoteTermlists() {
        team.queue("termlists").subscribe(function(data) {
-           mkws.debug("in termlist consumer");
            if (!node) {
                alert("termlists event when there are no termlists");
                return;
@@ -399,32 +398,29 @@ function widget($, team, type, node) {
 
 
     function promoteSort() {
-       team.queue("navi").subscribe(function() {})
-       $(node).change(onSortChange);
-
-       function onSortChange()
-       {
+       // It seems this and the Perpage widget doen't need to
+       // subscribe to anything, since they produce events rather
+       // than consuming them.
+       $(node).change(function () {
            team.set_sortOrder($(node).val());
-           if (!team.submitted()) return false;
-           team.resetPage();
-           team.reShow();
+           if (team.submitted()) {
+               team.resetPage();
+               team.reShow();
+           }
            return false;
-       }
+       });
     }
 
 
     function promotePerpage() {
-       team.queue("navi").subscribe(function() {})
-       $(node).change(onPerpageChange);
-
-       function onPerpageChange()
-       {
+       $(node).change(function() {
            team.set_perpage($(node).val());
-           if (!team.submitted()) return false;
-           team.resetPage();
-           team.reShow();
+           if (team.submitted()) {
+               team.resetPage();
+               team.reShow();
+           }
            return false;
-       }
+       });
     }
 }