always use messages.* stuff

[myslice.git] / plugins / lists / simplelist.js

diff --git a/plugins/lists/simplelist.js b/plugins/lists/simplelist.js
index 6fdb4c3..11f812b 100644 (file)
--- a/plugins/lists/simplelist.js
+++ b/plugins/lists/simplelist.js
@@ -1,8 +1,5 @@
 /**
- * MySlice SimpleList plugin
- * URL: http://trac.myslice.info
  * Description: display simple lists like slices or testbeds
- * Author: The MySlice Team
  * Copyright (c) 2012 UPMC Sorbonne Universite - INRIA
  * License: GPLv3
  */
@@ -10,7 +7,7 @@
 (function($){
 
     var debug=false;
-    //debug=true
+    // debug=true
 
     $.fn.SimpleList = function( method ) {
         /* Method calling logic */
@@ -27,19 +24,16 @@
        init : function( options ) {
            return this.each(function(){
                var $this = $(this), data = $this.data('SimpleList');
-               // If the plugin hasn't been initialized yet
-               if ( ! data ) {
-                   /* Subscribe to query updates */
-                   var channel='/results/' + options.query_uuid + '/changed';
-                   /* passing $this as 2nd arg: callbacks will retrieve $this as e.data */
-                   $.subscribe(channel, $this, update_plugin);
-                   if (debug) window.console.log('subscribing to ' + channel);
-                   $this.data('SimpleList', {options: options});
-               }
+               /* Subscribe to query updates */
+               var channel='/results/' + options.query_uuid + '/changed';
+               /* passing $this as 2nd arg: callbacks will retrieve $this as e.data */
+               $.subscribe(channel, $this, update_plugin);
+               if (debug) window.messages.debug('subscribing to ' + channel);
+               $this.data('SimpleList', options);
            });
        },
        destroy : function( ) {
-           if (debug) console.log("SimpleList.destroy...");
+           if (debug) messages.debug("SimpleList.destroy...");
             return this.each(function(){
                var $this = $(this), data = $this.data('SimpleList');
                // xxx not too sure what this is about
@@ -48,7 +42,7 @@
             });
        },
        update : function( content ) { 
-           if (debug) console.log("SimpleList.update...");
+           if (debug) messages.debug("SimpleList.update...");
        },
     }; // methods
 
@@ -65,24 +59,24 @@
        // also we may or may not have a header
        var $tbody=$table.find("tbody.simplelist").first();
        var use_datatables = $table.hasClass("with-datatables");
-       if (debug) console.log($plugindiv.attr('id') + " udt= " + use_datatables);
+       if (debug) messages.debug($plugindiv.attr('id') + " udt= " + use_datatables);
        
        // clear the spinning wheel: look up an ancestor that has the need-spin class
        // do this before we might return
        $plugindiv.closest('.need-spin').spin(false);
 
         if (rows.length == 0) {
-           if (use_datatables) datatables_set_message ("No result");
-           else                regular_set_message ("No result");
+           if (use_datatables) datatables_set_message ($table, $tbody, unfold.warning("No result"));
+           else                regular_set_message ($table, $tbody, unfold.warning("No result"));
             return;
         }
         if (typeof rows[0].error != 'undefined') {
            var error="ERROR: " + rows[0].error;
-           if (use_datatables) datatables_set_message (error);
-           else                regular_set_message (error);
+           if (use_datatables) datatables_set_message ($table, $tbody, unfold.error(error));
+           else                regular_set_message ($table, $tbody, unfold.error(error));
             return;
         }
-        var options = $plugindiv.data().SimpleList.options;
+        var options = $plugindiv.data().SimpleList;
        if (use_datatables)     datatables_update_table ($table,$tbody,rows,options.key);
        else                    regular_update_table ($table,$tbody,rows,options.key);
 
@@ -104,7 +98,7 @@
     }
 
     function regular_update_table ($table, $tbody, rows, key) {
-       if (debug) console.log('regular_update_table ' + rows.length + " rows");
+       if (debug) messages.debug('regular_update_table ' + rows.length + " rows");
        var html=$.map(rows, function (row) { return html_row ( cell (key, row[key])); }).join();
        $tbody.html(html);
     }
@@ -116,7 +110,7 @@
     }
 
     function datatables_update_table ($table, $tbody, rows, key) {
-       if (debug) console.log('datatables_update_table ' + rows.length + " rows");
+       if (debug) messages.debug('datatables_update_table ' + rows.length + " rows");
        $table.dataTable().fnClearTable();
        // the lambda here returns a [[]] because $.map is kind of broken; as per the doc:
        // The function can return any value to add to the array. A returned array will be flattened into the resulting array.