Edit machineDetails

1 files changed, 30 insertions, 25 deletions

diff --git a/client/js/services/Compute.js b/client/js/services/Compute.js
index f4e8fa8..e2fb3df 100644
--- a/client/js/services/Compute.js
+++ b/client/js/services/Compute.js
@@ -2,19 +2,27 @@
 mainApp.factory('Compute',[ '$http', 'Identity', function($http, Identity){
 
 
-	// Create data
+	// Init data
 	var data={};
-	data.machines={};
+	data.machines=null;
 	
 
 	
-	// Parser
-	var parseGetMachinesAnswer=function(response, failedToSendRequest){
+
+	/**
+	 * Retrieve machine list 
+	 * @param {response} the server response
+	 * @param {boolean} false if the request as been send true else
+	 * @return {requestParserResult} the result of parsing
+	 */
+	var parsePullMachinesAnswer=function(response, failedToSendRequest){
+
 		// Defined return object
 		var requestParserResult={};
 		requestParserResult.status=1;
 		requestParserResult.failReason=null;
-		console.log(response.data.Servers[Object.keys(response.data.Servers)[0]])
+
+		
 		if (typeof response.data.Servers !== 'undefined') {
 			// Set status code
 			requestParserResult.status=0;
@@ -27,44 +35,41 @@ mainApp.factory('Compute',[ '$http', 'Identity', function($http, Identity){
 		else{
 			requestParserResult.failReason="Error";
 		}
-		
 		return requestParserResult;
-
-		
 	};
 
 
-	// Get Machine
+	/**
+	 * Retrieve machine list 
+	 * @param {callback} function to call after request complete
+	 */
 	var pullMachines=function(callback){
-
-		var params={
-			"token" : Identity.getToken(),
-			"task" : "compute",
-			"action":"listServers"
-		};
-		
+		// Send listServers request
 		var result=$http.post('../server/index.php',
-							  $.param(params));		
+							  $.param({"token" : Identity.getToken(), "task" : "compute", "action":"listServers"}));		
 
 		// Wait and handle the response
         result.then(function (response){
-
-			callback(parseGetMachinesAnswer(response, false));
+			callback(parsePullMachinesAnswer(response, false));
         },function(response){
-			alert(response.status);
-			callback(parseGetMachinesAnswer(response, true));
-        });		
-
-		
+			callback(parsePullMachinesAnswer(response, true));
+        });
 	};
 	
 
-
+	/**
+	 * Retrieve all data 
+	 * @param {callback} function to call after request complete
+	 */
 	var pullData=function(callback){
 		// TODO call getMachines etc...
 	}
 
 
+	/**
+	 * Get Data
+	 * @return {data} return the data object
+	 */
 	var getData=function(){
 		return data;
 	}