Extract value-expansion code from widget.autosearch into its own function widget...

[mkws-moved-to-github.git] / src / mkws-widget.js

diff --git a/src/mkws-widget.js b/src/mkws-widget.js
index ca8b12d..d7ce3ba 100644 (file)
--- a/src/mkws-widget.js
+++ b/src/mkws-widget.js
@@ -66,50 +66,63 @@ function widget($, team, type, node) {
     return s.join('');
   };
 
+  that.expandValue = function(val) {
+    if (val.match(/^!param!/)) {
+      var param = val.replace(/^!param!/, '');
+      val = mkws.getParameterByName(param);
+      this.log("obtained val '" + val + "' from param '" + param + "'");
+      if (!val) {
+        alert("This page has a MasterKey widget that needs a val specified by the '" + param + "' parameter");
+      }
+    } else if (val.match(/^!path!/)) {
+      var index = val.replace(/^!path!/, '');
+      var path = window.location.pathname.split('/');
+      val = path[path.length - index];
+      this.log("obtained val '" + val + "' from path-component '" + index + "'");
+      if (!val) {
+        alert("This page has a MasterKey widget that needs a val specified by the path-component " + index);
+      }
+    } else if (val.match(/^!var!/)) {
+      var name = val.replace(/^!var!/, '');
+      val = window[name]; // It's ridiculous that this works
+      this.log("obtained val '" + val + "' from variable '" + name + "'");
+      if (!val) {
+        alert("This page has a MasterKey widget that needs a val specified by the '" + name + "' variable");
+      }
+    }
+    return val;
+  };
+
   // Utility function for use by all widgets that can invoke autosearch.
   that.autosearch = function() {
-    var widget = this;
-    var query = widget.config.autosearch;
+    var that = this;
+    var query = this.config.autosearch;
     if (query) {
-      if (query.match(/^!param!/)) {
-        var param = query.replace(/^!param!/, '');
-        query = mkws.getParameterByName(param);
-        widget.log("obtained query '" + query + "' from param '" + param + "'");
-        if (!query) {
-          alert("This page has a MasterKey widget that needs a query specified by the '" + param + "' parameter");
-        }
-      } else if (query.match(/^!path!/)) {
-        var index = query.replace(/^!path!/, '');
-        var path = window.location.pathname.split('/');
-        query = path[path.length - index];
-        widget.log("obtained query '" + query + "' from path-component '" + index + "'");
-        if (!query) {
-          alert("This page has a MasterKey widget that needs a query specified by the path-component " + index);
-        }
-      } else if (query.match(/^!var!/)) {
-        var name = query.replace(/^!var!/, '');
-        query = window[name]; // It's ridiculous that this works
-        widget.log("obtained query '" + query + "' from variable '" + name + "'");
-        if (!query) {
-          alert("This page has a MasterKey widget that needs a query specified by the '" + name + "' variable");
-        }
+      query = this.expandValue(query);
+      var old = this.team.config().query;
+      if (!old) {
+        // Stash this for subsequent inspection
+        this.team.config().query = query;
+      } else if (old === query) {
+        this.log("duplicate autosearch: '" + query + "': ignoring");
+        return;
+      } else {
+        this.log("conflicting autosearch: '" + query + "' vs '" + old + "': ignoring");
+        return;
       }
 
-      // Stash this for subsequent inspection
-      widget.team.config().query = query;
-
-      widget.team.queue("ready").subscribe(function() {
+      this.team.queue("ready").subscribe(function() {
         // Postpone testing for the configuration items: these are not
         // yet set for Record subclass widgets that fill them in in the
         // subclass, as widget.autosearch is called in the superclass,
         // before the subclass fiddles with the configuration.
-        var sortOrder = widget.config.sort;
-        var maxrecs = widget.config.maxrecs;
-        var perpage = widget.config.perpage;
-        var limit = widget.config.limit;
-        var targets = widget.config.targets;
-        var targetfilter = widget.config.targetfilter;
-        var target = widget.config.target;
+        var sortOrder = that.config.sort;
+        var maxrecs = that.config.maxrecs;
+        var perpage = that.config.perpage;
+        var limit = that.config.limit;
+        var targets = that.config.targets;
+        var targetfilter = that.config.targetfilter;
+        var target = that.config.target;
         if (target) targetfilter = 'udb=="' + target + '"';
 
         var s = "running auto search: '" + query + "'";
@@ -119,21 +132,21 @@ function widget($, team, type, node) {
         if (limit) s += " limited by '" + limit + "'";
         if (targets) s += " in targets '" + targets + "'";
         if (targetfilter) s += " constrained by targetfilter '" + targetfilter + "'";
-        widget.log(s);
+        that.log(s);
 
-        widget.team.newSearch(query, sortOrder, maxrecs, perpage, limit, targets, targetfilter);
+        that.team.newSearch(query, sortOrder, maxrecs, perpage, limit, targets, targetfilter);
       });
     }
   };
 
   // Utility function for all widgets that want to hide in narrow windows
   that.hideWhenNarrow = function() {
-    var widget = this;
-    widget.team.queue("resize-narrow").subscribe(function(n) {
-      widget.node.hide();
+    var that = this;
+    this.team.queue("resize-narrow").subscribe(function(n) {
+      that.node.hide();
     });
-    widget.team.queue("resize-wide").subscribe(function(n) {
-      widget.node.show();
+    this.team.queue("resize-wide").subscribe(function(n) {
+      that.node.show();
     });
   };