X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=plugins%2Fgooglemap%2Fstatic%2Fjs%2Fgooglemap.js;h=0c511a08df1529241868cebc3ad154f1378174c2;hb=8e47d1e76551223a1997d9383ac0bb987e04dfad;hp=34b2b8de563b927c0af4d205a5ec6ae47a8e4c6d;hpb=a387c58445b7cde5ecf05a022c5a875645eeb463;p=myslice.git

diff --git a/plugins/googlemap/static/js/googlemap.js b/plugins/googlemap/static/js/googlemap.js
index 34b2b8de..0c511a08 100644
--- a/plugins/googlemap/static/js/googlemap.js
+++ b/plugins/googlemap/static/js/googlemap.js
@@ -27,29 +27,29 @@ googlemap_debug_detailed=false;
             this.in_set_backlog = [];
 
             // we keep a couple of global hashes
-	    // lat_lon --> { marker, <ul> }
-	    // hrn --> { <li>, <input> }
-	    this.by_lat_lon = {};
-	    this.by_hrn = {};
+            // lat_lon --> { marker, <ul> }
+            // hrn --> { <li>, <input> }
+            this.by_lat_lon = {};
+            this.by_hrn = {};
 
             /* XXX Events */
             this.elmt().on('show', this, this.on_show);
             // TODO in destructor
-            // $(window).unbind('Hazelnut');
+            // $(window).unbind('QueryTable');
 
             var query = manifold.query_store.find_analyzed_query(this.options.query_uuid);
             this.object = query.object;
 
             var keys = manifold.metadata.get_key(this.object);
-	    // 
+            // 
             this.key = (keys && keys.length == 1) ? keys[0] : null;
 
             //// Setup query and record handlers 
-	    // this query is the one about the slice itself 
-	    // event related to this query will trigger callbacks like on_new_record
+            // this query is the one about the slice itself 
+            // event related to this query will trigger callbacks like on_new_record
             this.listen_query(options.query_uuid);
-	    // this one is the complete list of resources
-	    // and will be bound to callbacks like on_all_new_record
+            // this one is the complete list of resources
+            // and will be bound to callbacks like on_all_new_record
             this.listen_query(options.query_all_uuid, 'all');
 
             /* GUI setup and event binding */
@@ -59,8 +59,7 @@ googlemap_debug_detailed=false;
         /* PLUGIN EVENTS */
 
         on_show: function(e) {
-//	    if (googlemap_debug) messages.debug("on_show");
-	    messages.debug("on_show");
+            if (googlemap_debug) messages.debug("googlemap.on_show");
             var googlemap = e.data;
             google.maps.event.trigger(googlemap.map, 'resize');
         }, // on_show
@@ -76,119 +75,132 @@ googlemap_debug_detailed=false;
             var myOptions = {
                 zoom: this.options.zoom,
                 center: center,
+                scrollwheel: false,
                 mapTypeId: google.maps.MapTypeId.ROADMAP,
             }
-	    
+            
             var domid = this.options.plugin_uuid + '--' + 'googlemap';
-	    var elmt = document.getElementById(domid);
-	    if (googlemap_debug) messages.debug("gmap.initialize_map based on  domid=" + domid + " elmt=" + elmt);
+            var elmt = document.getElementById(domid);
+            if (googlemap_debug) messages.debug("gmap.initialize_map based on  domid=" + domid + " elmt=" + elmt);
             this.map = new google.maps.Map(elmt, myOptions);
             this.infowindow = new google.maps.InfoWindow();
         }, // initialize_map
 
-	// xxx probably not the right place
+        // xxx probably not the right place
         // The function accepts both records and their key 
-	record_hrn : function (record) {
+        record_hrn : function (record) {
             var key_value;
             switch (manifold.get_type(record)) {
             case TYPE_VALUE:
-		key_value = record;
+                key_value = record;
                 break;
             case TYPE_RECORD:
-		if ( ! this.key in record ) return;
+                if ( ! this.key in record ) return;
                 key_value = record[this.key];
                 break;
             default:
                 throw "Not implemented";
                 break;
             }
-	    return unfold.escape_id(key_value).replace(/\\/g, '');
-	},	    
-
-	// return { marker: gmap_marker, ul : <ul DOM> }
-	create_marker_struct: function (object,lat,lon) {
-	    // the DOM fragment
-	    var dom = $("<p>").addClass("geo").append(object+"(s)");
-	    var ul = $("<ul>").addClass("geo");
-	    dom.append(ul);
-	    // add a gmap marker to the mix
-	    var marker = new google.maps.Marker({
-		position: new google.maps.LatLng(lat, lon),
+            // XXX BACKSLASHES original code was reading like this
+            //return this.escape_id(key_value).replace(/\\/g, '');
+            //  however this sequence removes backslashes from hrn's and as a result
+            // queryupdater was getting all mixed up
+            // querytable does publish hrn's with backslashes and that seems like the thing to do
+            return key_value;
+        },            
+
+        // return { marker: gmap_marker, ul : <ul DOM> }
+        create_marker_struct: function (object,lat,lon) {
+            // the DOM fragment
+            var dom = $("<p>").addClass("geo").append(object+"(s)");
+            var ul = $("<ul>").addClass("geo");
+            dom.append(ul);
+            // add a gmap marker to the mix
+            var marker = new google.maps.Marker({
+                position: new google.maps.LatLng(lat, lon),
                 title: object,
-		// gmap can deal with a DOM element but not a jquery object
+                // gmap can deal with a DOM element but not a jquery object
                 content: dom.get(0),
             }); 
-	    return {marker:marker, ul:ul};
-	},
-
-	// add an entry in the marker <ul> tag for that record
-	// returns { checkbox : <input DOM> }
-	create_record_checkbox: function (record,ul,checked) {
-	    var checkbox = $("<input>", {type:'checkbox', checked:checked, class:'geo'});
-	    var hrn=this.record_hrn(record);
-	    ul.append($("<li>").addClass("geo").append(checkbox).
-		      append($("<span>").addClass("geo").append(hrn)));
-	    var googlemap=this;
-	    // the callback for when a user clicks
-	    // NOTE: this will *not* be called for changes done by program
-	    checkbox.change( function (e) {
-		manifold.raise_event (googlemap.options.query_uuid, 
-				      this.checked ? SET_ADD : SET_REMOVED, hrn);
-	    });
-	    return checkbox;
-	},
-	    
-	// retrieve DOM checkbox and make sure it is checked/unchecked
+            return {marker:marker, ul:ul};
+        },
+
+        // add an entry in the marker <ul> tag for that record
+        // returns { checkbox : <input DOM> }
+        create_record_checkbox: function (record,ul,checked) {
+            var checkbox = $("<input>", {type:'checkbox', checked:checked, class:'geo'});
+            var hrn=this.record_hrn(record);
+            ul.append($("<li>").addClass("geo").append(checkbox).
+                      append($("<span>").addClass("geo").append(hrn)));
+            var googlemap=this;
+            // the callback for when a user clicks
+            // NOTE: this will *not* be called for changes done by program
+            checkbox.change( function (e) {
+                if (googlemap_debug) messages.debug("googlemap click handler checked= " + this.checked + " hrn=" + hrn);
+                manifold.raise_event (googlemap.options.query_uuid, 
+                                      this.checked ? SET_ADD : SET_REMOVED, hrn);
+            });
+            return checkbox;
+        },
+            
+        // retrieve DOM checkbox and make sure it is checked/unchecked
         set_checkbox: function(record, checked) {
-	    var hrn=this.record_hrn (record);
-	    if (! hrn) { messages.warning ("googlemap.set_checkbox: record has no hrn"); return; }
-	    var checkbox_s = this.by_hrn [ hrn ];
-	    if (! checkbox_s ) { messages.warning ("googlemap.set_checkbox: could not spot checkbox for hrn "+hrn); return; }
-	    checkbox_s.checkbox.prop('checked',checked);
+            var hrn=this.record_hrn (record);
+            if (! hrn) { 
+                try {messages.warning ("googlemap.set_checkbox: record has no hrn -- hostname="+record.hostname); }
+                catch (err) {messages.warning ("googlemap.set_checkbox: record has no hrn"); }
+                return; 
+            }
+            var checkbox_s = this.by_hrn [ hrn ];
+            if (! checkbox_s ) { messages.warning ("googlemap.set_checkbox: could not spot checkbox for hrn "+hrn); return; }
+            checkbox_s.checkbox.prop('checked',checked);
         }, // set_checkbox
 
-	// this record is *in* the slice
+        // this record is *in* the slice
         new_record: function(record) {
-	    if (googlemap_debug_detailed) messages.debug ("new_record");
+                if (googlemap_debug_detailed) messages.debug ("new_record");
             if (!(record['latitude'])) return false;
-	    
+            
             // get the coordinates
-            var latitude=get_value(record['latitude']);
-            var longitude=get_value(record['longitude']);
+            var latitude=unfold.get_value(record['latitude']);
+            var longitude=unfold.get_value(record['longitude']);
             var lat_lon = latitude + longitude;
 
-	    // check if we've seen anything at that place already
-	    // xxx might make sense to allow for some fuzziness, 
-	    // i.e. consider 2 places equal if not further away than 300m or so...
-	    var marker_s = this.by_lat_lon [lat_lon];
-	    if ( marker_s == null ) {
-		marker_s = this.create_marker_struct (this.object, latitude, longitude);
-		this.by_lat_lon [ lat_lon ] = marker_s;
-		this.arm_marker(marker_s.marker, this.map);
-	    }
-	    
-	    // now add a line for this resource in the marker
-	    // xxx should compute checked here ?
-	    // this is where the checkbox will be appended
-	    var ul=marker_s.ul;
-	    var checkbox = this.create_record_checkbox (record, ul, false);
-	    if ( ! this.key in record ) return;
+                // check if we've seen anything at that place already
+                // xxx might make sense to allow for some fuzziness, 
+                // i.e. consider 2 places equal if not further away than 300m or so...
+                var marker_s = this.by_lat_lon [lat_lon];
+                if ( marker_s == null ) {
+                        marker_s = this.create_marker_struct (this.object, latitude, longitude);
+                        this.by_lat_lon [ lat_lon ] = marker_s;
+                        this.arm_marker(marker_s.marker, this.map);
+                }
+            
+                // now add a line for this resource in the marker
+                // xxx should compute checked here ?
+                // this is where the checkbox will be appended
+                var ul=marker_s.ul;
+                var checkbox = this.create_record_checkbox (record, ul, false);
+                if ( ! this.key in record ) return;
             var key_value = record[this.key];
-	    var hrn = unfold.escape_id(key_value).replace(/\\/g, '');
+                // see XXX BACKSLASHES 
+                //var hrn = this.escape_id(key_value).replace(/\\/g, '');
+                var hrn = key_value;
             this.by_hrn[hrn] = {
-		checkbox: checkbox,
-		// xxx Thierry sept 2013
-		// xxx actually we might have just used a domid-based scheme instead of the hash
-		// since at this point we only need to retrieve the checkbox from an hrn
-		// but I was not sure enough that extra needs would not show up so I kept this in place
-		// xxx not sure these are actually useful :
+                        checkbox: checkbox,
+                        // xxx Thierry sept 2013
+                        // xxx actually we might have just used a domid-based scheme instead of the hash
+                        // since at this point we only need to retrieve the checkbox from an hrn
+                        // but I was not sure enough that extra needs would not show up so I kept this in place
+                        // xxx not sure these are actually useful :
                 value: key_value,
                 record: record,
             }
         }, // new_record
 
         arm_marker: function(marker, map) {
-	    if (googlemap_debug_detailed) messages.debug ("arm_marker content="+marker.content);
+            if (googlemap_debug_detailed) messages.debug ("arm_marker content="+marker.content);
             var googlemap = this;
             google.maps.event.addListener(marker, 'click', function () {
                 googlemap.infowindow.close();
@@ -201,7 +213,7 @@ googlemap_debug_detailed=false;
 
         /*************************** RECORD HANDLER ***************************/
         on_new_record: function(record) {
-	    if (googlemap_debug_detailed) messages.debug("on_new_record");
+            if (googlemap_debug_detailed) messages.debug("on_new_record");
             if (this.received_all)
                 // update checkbox for record
                 this.set_checkbox(record, true);
@@ -211,25 +223,25 @@ googlemap_debug_detailed=false;
         },
 
         on_clear_records: function(record) {
-	    if (googlemap_debug_detailed) messages.debug("on_clear_records");
+            if (googlemap_debug_detailed) messages.debug("on_clear_records");
         },
 
         // Could be the default in parent
         on_query_in_progress: function() {
-	    if (googlemap_debug) messages.debug("on_query_in_progress (spinning)");
+            if (googlemap_debug) messages.debug("on_query_in_progress (spinning)");
             this.spin();
         },
 
         on_query_done: function() {
-	    if (googlemap_debug) messages.debug("on_query_done");	    
+            if (googlemap_debug) messages.debug("on_query_done");            
             if (this.received_all) {
                 this.unspin();
-	    }
+            }
             this.received_set = true;
         },
 
         on_field_state_changed: function(data) {
-	    if (googlemap_debug_detailed) messages.debug("on_field_state_changed");	    
+            if (googlemap_debug_detailed) messages.debug("on_field_state_changed");            
             switch(data.request) {
             case FIELD_REQUEST_ADD:
             case FIELD_REQUEST_ADD_RESET:
@@ -248,22 +260,22 @@ googlemap_debug_detailed=false;
         // all : this 
 
         on_all_new_record: function(record) {
-	    if (googlemap_debug_detailed) messages.debug("on_all_new_record");
+            if (googlemap_debug_detailed) messages.debug("on_all_new_record");
             this.new_record(record);
         },
 
         on_all_clear_records: function() {
-	    if (googlemap_debug) messages.debug("on_all_clear_records");	    
+            if (googlemap_debug) messages.debug("on_all_clear_records");            
         },
 
         on_all_query_in_progress: function() {
-	    if (googlemap_debug) messages.debug("on_all_query_in_progress (spinning)");
+            if (googlemap_debug) messages.debug("on_all_query_in_progress (spinning)");
             // XXX parent
             this.spin();
         },
 
         on_all_query_done: function() {
-	    if (googlemap_debug) messages.debug("on_all_query_done");
+            if (googlemap_debug) messages.debug("on_all_query_done");
 
             // MarkerClusterer
             var markers = [];
@@ -285,10 +297,10 @@ googlemap_debug_detailed=false;
                 $.each(this.in_set_backlog, function(i, record) {
                     googlemap.set_checkbox(record, true);
                 });
-		// reset 
-		googlemap.in_set_backlog = [];
+                // reset 
+                googlemap.in_set_backlog = [];
 
-		if (googlemap_debug) messages.debug("unspinning");
+                if (googlemap_debug) messages.debug("unspinning");
                 this.unspin();
             }
             this.received_all = true;