pretty printed
authorThierry Parmentelat <thierry.parmentelat@inria.fr>
Fri, 5 Apr 2013 11:23:11 +0000 (13:23 +0200)
committerThierry Parmentelat <thierry.parmentelat@inria.fr>
Fri, 5 Apr 2013 11:23:11 +0000 (13:23 +0200)
plugins/senslabmap/senslabmap.js

index d3dc01d..c5aef27 100644 (file)
@@ -8,89 +8,76 @@
  */
 
 // xxx TODO -- this plugin has never been tested 
+// update_map looks suspiciously empty...
 // in addition it could use a bit of cleaning like what was done for the first plugins
 // especially wrt using 'instance' and 'data' in such a confusing way
 
-(function( jQuery ){
-
-  var methods = {
-     init : function( options ) {
-
-       return this.each(function(){
-         
-         var $this = jQuery(this),
-             data = $this.data('SensLabMap'), SensLabMap = jQuery('<div />', { text : $this.attr('title') });
-         
-         // If the plugin hasn't been initialized yet
-         if ( ! data ) {
-         
-           /* Plugin initialization */
-
-            //google.load('maps', '3', { other_params: 'sensor=false' });
-            //google.setOnLoadCallback(initialize);
-
-            init();
-            /* End of plugin initialization */
-
-            jQuery(this).data('SensLabMap', {
-                               plugin_uuid: options.plugin_uuid,
-                               query_uuid: options.query_uuid,
-                target : $this,
-                SensLabMap : SensLabMap
-            });
-
-                       /* Subscribe to query updates */
-                       jQuery.subscribe('/results/' + options.query_uuid + '/changed', {instance: $this}, update_map);
-                       
-
-         }
-       });
-     },
-    destroy : function( ) {
-
-        return this.each(function(){
-            var $this = jQuery(this), data = $this.data('SensLabMap');
-                       jQuery(window).unbind('SensLabMap');
-                       data.SensLabMap.remove();
-                       $this.removeData('SensLabMap');
-               })
+(function( $ ){
+
+    $.fn.SensLabMap = function( method ) {
+       /* Method calling logic */
+       if ( methods[method] ) {
+           return methods[ method ].apply( this, Array.prototype.slice.call( arguments, 1 ));
+       } else if ( typeof method === 'object' || ! method ) {
+           return methods.init.apply( this, arguments );
+       } else {
+           $.error( 'Method ' +  method + ' does not exist on $.SensLabMap' );
+       }    
+    };
 
-    },
+    var methods = {
+       init : function( options ) {
+           return this.each(function(){
+               var $this = $(this),
+               data = $this.data('SensLabMap'), SensLabMap = $('<div />', { text : $this.attr('title') });
+               // If the plugin hasn't been initialized yet
+               if ( ! data ) {
+                   /* Plugin initialization */
+                   //google.load('maps', '3', { other_params: 'sensor=false' });
+                   //google.setOnLoadCallback(initialize);
+                   init();
+                   /* End of plugin initialization */
+                   $(this).data('SensLabMap', {
+                       plugin_uuid: options.plugin_uuid,
+                       query_uuid: options.query_uuid,
+                       target : $this,
+                       SensLabMap : SensLabMap
+                   });
+                   /* Subscribe to query updates */
+                   $.subscribe('/results/' + options.query_uuid + '/changed', {instance: $this}, update_map);
+               }
+           });
+       },
+       destroy : function( ) {
+            return this.each(function(){
+               var $this = $(this), data = $this.data('SensLabMap');
+               $(window).unbind('SensLabMap');
+               data.SensLabMap.remove();
+               $this.removeData('SensLabMap');
+           });
+       },
 /*
     reposition : function( ) { // ... },
     show : function( ) { // ... },
     hide : function( ) { // ... },
 */
-    update : function( content ) { 
-        // should be made a private function
-        set3dsize()
-    }
+       update : function( content ) { 
+            // should be made a private function
+            set3dsize()
+       },
   };
 
-    jQuery.fn.SensLabMap = function( method ) {
-               /* Method calling logic */
-               if ( methods[method] ) {
-                       return methods[ method ].apply( this, Array.prototype.slice.call( arguments, 1 ));
-               } else if ( typeof method === 'object' || ! method ) {
-                       return methods.init.apply( this, arguments );
-               } else {
-                       jQuery.error( 'Method ' +  method + ' does not exist on jQuery.SensLabMap' );
-               }    
-
-    };
-
     /* Private methods */
-
-       function update_map(e, rows) {
-            var $plugindiv = e.data.instance;
-           $plugindiv.closest('.need-spin').spin(false);
-           
-            if(rows) {
-               /* TODO rendering */   
-            } else {
-               alert('error');
-            }
-           
-       }
+    function update_map(e, rows) {
+        var $plugindiv = e.data.instance;
+       $plugindiv.closest('.need-spin').spin(false);
+       
+        if(rows) {
+           /* TODO rendering */   
+        } else {
+           alert('error');
+        }
+       
+    }
 
 })( jQuery );