adding some tracability into the plugins code

[unfold.git] / plugins / senslabmap / static / js / senslabmap.js

diff --git a/plugins/senslabmap/static/js/senslabmap.js b/plugins/senslabmap/static/js/senslabmap.js
index 2945e0d..e6a64b5 100644 (file)
--- a/plugins/senslabmap/static/js/senslabmap.js
+++ b/plugins/senslabmap/static/js/senslabmap.js
@@ -1,6 +1,7 @@
 (function($){
   var SensLabMap = Plugin.extend({
     init: function(options, element) {
+       this.classname="senslabmap";
       this._super(options, element);
       
       this.elmt().on('show', this, this.on_show);
@@ -22,7 +23,7 @@
     },
     
     on_all_new_record: function(node) {
-      Senslab.normalize(node, this.key);
+      Senslab.normalize(node);
       if (node.normalized) {
         var site = node.site;
         if ($.inArray(site, this.sites) == -1) {
@@ -31,20 +32,20 @@
         }
         this.nodes[site].push(node);
       } else {
-        console.warn("node has no site:");
-        console.warn(node);
+// xxx would need a smarter way to report these only on nodes that deserve it
+// these messages are all over the place in a PL setup and tend to hide helpful signal
+//        console.warn("node has no site:");
+//        console.warn(node);
       }
     },
     
     on_all_query_done: function() {
-      var self = this;
+      var uuid = this.options.query_uuid;
+      var key = this.key
       
-      Senslab.createMaps($('#maps-container'), this.sites, this.nodes);
+      Senslab.createMaps($("#maps-container"), this.sites, this.nodes);
       Senslab.notify = function(node) {
-        manifold.raise_event(self.options.query_uuid,
-                             node.boot_state == "Alive" ? SET_REMOVED : SET_ADD,
-                             node[self.key]
-                            );
+        manifold.raise_event(uuid, node.selected ? SET_ADD : SET_REMOVED, node[key]);
       }
     }
   });