From: Thierry Parmentelat <thierry.parmentelat@inria.fr>
Date: Mon, 16 Sep 2013 07:04:04 +0000 (+0200)
Subject: new publish_result_debug flag
X-Git-Tag: myslice-0.2-3~21
X-Git-Url: http://git.onelab.eu/?p=myslice.git;a=commitdiff_plain;h=5d411f201209ba1c90b936eb9c301915000695d2

new publish_result_debug flag
---

diff --git a/manifold/js/manifold.js b/manifold/js/manifold.js
index fe16d0af..bf76fd82 100644
--- a/manifold/js/manifold.js
+++ b/manifold/js/manifold.js
@@ -314,7 +314,10 @@ var manifold = {
     // trigger a query asynchroneously
     proxy_url : '/manifold/proxy/json/',
 
+    // reasonably low-noise, shows manifold requests coming in and out
     asynchroneous_debug : true,
+    // print our more details on result publication and related callbacks
+    publish_result_debug : false,
 
     /**
      * \brief We use js function closure to be able to pass the query (array)
@@ -371,7 +374,8 @@ var manifold = {
 
             // not quite sure what happens if we send a string directly, as POST data is named..
             // this gets reconstructed on the proxy side with ManifoldQuery.fill_from_POST
-            jQuery.post(manifold.proxy_url, {'json':query_json} , manifold.success_closure(query, publish_uuid, tuple.callback /*domid*/));
+            jQuery.post(manifold.proxy_url, {'json':query_json}, 
+			manifold.success_closure(query, publish_uuid, tuple.callback /*domid*/));
         })
     },
 
@@ -384,7 +388,8 @@ var manifold = {
      */
     forward: function(query, callback /*domid*/) {
         var query_json = JSON.stringify(query);
-        $.post(manifold.proxy_url, {'json': query_json} , manifold.success_closure(query, query.query_uuid, callback/*domid*/));
+        $.post(manifold.proxy_url, {'json': query_json} , 
+	       manifold.success_closure(query, query.query_uuid, callback/*domid*/));
     },
 
     /*!
@@ -439,6 +444,7 @@ var manifold = {
          * otherwise, publish the main object as well as subqueries
          * XXX how much recursive are we ?
          */
+	if (manifold.publish_result_debug) messages.debug (">>>>> publish_result_rec " + query.object);
         if (manifold.query_expects_unique_result(query)) {
             /* Also publish subqueries */
             jQuery.each(query.subqueries, function(object, subquery) {
@@ -446,7 +452,9 @@ var manifold = {
                 /* TODO remove object from result */
             });
         }
+	if (manifold.publish_result_debug) messages.debug ("===== publish_result_rec " + query.object);
         manifold.publish_result(query, result);
+	if (manifold.publish_result_debug) messages.debug ("<<<<< publish_result_rec " + query.object);
     },
 
     setup_update_query: function(query, records) {
@@ -688,14 +696,17 @@ var manifold = {
     asynchroneous_success : function (data, query, publish_uuid, callback /*domid*/) {
         // xxx should have a nicer declaration of that enum in sync with the python code somehow
 	
+	var start = new Date();
 	if (manifold.asynchroneous_debug)
-	    messages.debug(">>> asynchroneous_success (json returned) for " + query.object + " " + publish_uuid);
+	    messages.debug(">>>>>>>>>> asynchroneous_success query.object=" + query.object);
 
         /* If a callback has been specified, we redirect results to it */
         if (!!callback) { 
 	    callback(data); 
-	    if (manifold.asynchroneous_debug)
-		messages.debug ("<<< asynchroneous_success " + query.object + " " + publish_uuid + " -- callback ended");
+	    if (manifold.asynchroneous_debug) {
+		duration=new Date()-start;
+		messages.debug ("<<<<<<<<<< asynchroneous_success " + query.object + " -- callback ended " + duration + " ms");
+	    }
 	    return; 
 	}
 
@@ -703,8 +714,10 @@ var manifold = {
             // We need to make sense of error codes here
             alert("Your session has expired, please log in again");
             window.location="/logout/";
-	    if (manifold.asynchroneous_debug)
-		messages.debug ("<<< asynchroneous_success " + query.object + " " + publish_uuid + " -- error returned - logging out");
+	    if (manifold.asynchroneous_debug) {
+		duration=new Date()-start;
+		messages.debug ("<<<<<<<<<< asynchroneous_success " + query.object + " -- error returned - logging out " + duration + " ms");
+	    }
             return;
         }
         if (data.code == 1) { // WARNING
@@ -722,8 +735,8 @@ var manifold = {
             });
             
         }
-	if (manifold.asynchroneous_debug)
-	    messages.debug ("=== asynchroneous_success " + query.object + " " + publish_uuid + " -- before process_query_records");
+	if (manifold.asynchroneous_debug) 
+	    messages.debug ("========== asynchroneous_success " + query.object + " -- before process_query_records");
 
         // once everything is checked we can use the 'value' part of the manifoldresult
         var result=data.value;
@@ -735,8 +748,11 @@ var manifold = {
             //tmp_query = manifold.find_query(query.query_uuid);
             //manifold.publish_result_rec(tmp_query.analyzed_query, result);
         }
-	if (manifold.asynchroneous_debug)
-	    messages.debug ("<<< asynchroneous_success " + query.object + " " + publish_uuid + " -- done");
+	if (manifold.asynchroneous_debug) {
+	    duration=new Date()-start;
+	    messages.debug ("<<<<<<<<<< asynchroneous_success " + query.object + " -- done " + duration + " ms");
+	}
+
     },
 
     /************************************************************************** 
@@ -750,10 +766,11 @@ var manifold = {
         var channels = [ manifold.get_channel(type, query_uuid), manifold.get_channel(type, '*') ];
 
         $.each(channels, function(i, channel) {
-            if (value === undefined)
+            if (value === undefined) {
                 $('.plugin').trigger(channel, [event_type]);
-            else
+	    } else {
                 $('.plugin').trigger(channel, [event_type, value]);
+	    }
         });
     },
 
@@ -870,7 +887,8 @@ var manifold = {
                 // NOTE : we have to modify all child queries
                 // NOTE : parts of a query might not be started (eg slice.measurements, how to handle ?)
 
-                // if everything is done right, update_query should not be null. It is updated when we received results from the get query
+                // if everything is done right, update_query should not be null. 
+	        // It is updated when we received results from the get query
                 // object = the same as get
                 // filter = key : update a single object for now
                 // fields = the same as get
@@ -1015,4 +1033,3 @@ $.ajaxSetup({
          }
      } 
 });
-