Clean code, add comment etc..

client/js/controllers/home/machineDetails.js

@@ -11,14 +11,14 @@ mainApp.controller('machineDetailsCtrl', [ '$scope', 'Compute', '$rootScope', '$
         $scope.machine = {};
         $scope.machineIsStarting = false; // For loading icon
 
-
+        // When we need to show details of machine
         $scope.$on('showMachineDetailsEvent', function (eventName, machine, axioms) {
             $scope.machine = machine;
             $scope.axioms = axioms;
             $('#machineDetailsModal').modal({backdrop: false, keyboard: true});
         });
 
-
+        // Try to stop or start a machine
         $scope.toggleMachineState = function () {
             // Display gif
             $scope.machineIsStarting = true;
@@ -35,9 +35,10 @@ mainApp.controller('machineDetailsCtrl', [ '$scope', 'Compute', '$rootScope', '$
 
         };
 
+        // Apply modifications
         $scope.applyModifications = function () {
             //Todo
-	}
+        };
 
 
 

client/js/controllers/image/image.js

@@ -6,21 +6,19 @@
 mainApp.controller('imageCtrl', ['$scope', 'Image', 'Loading', 'Identity', function ($scope, Image, Loading, Identity)
     {
 
-	var callbackTest=function(){
+        // Update view
+        var callMeAfterGetImage = function () {
             $scope.images = Image.getData().images;
             Loading.stop();
         };
 
-	
+        // If user is login try to retrieve data
         if (Identity.isAlreadyLogin()) {
-
             if (Image.getData().images == null) {
                 Loading.start();
-			Image.getImages(callbackTest);
+                Image.getImages(callMeAfterGetImage);
+            } else {
+                callMeAfterGetImage();
             }
-		else{
-			callbackTest();
-		}
-
         }
     }]);

client/js/controllers/image/upload.js

@@ -23,7 +23,9 @@ mainApp.controller('uploadImageCtrl', ['$scope', 'Image', 'Loading', 'Identity',
 
         $scope.doUpload = function () {
             console.log($('#imageToUpload').prop('files')[0]);
-		Image.uploadImage($('#imageToUpload').prop('files')[0], function(){alert("done")})
+            Image.uploadImage($('#imageToUpload').prop('files')[0], function () {
+                alert("done")
+            })
             /*$("#drop-area-div").dmUploader({
              extraData: 	{
              "token" : Identity.getToken(),

client/js/controllers/network/network.js

@@ -5,4 +5,5 @@
  */
 mainApp.controller('networkCtrl', function ($scope)
 {
+
 });

client/js/services/Image.js

@@ -1,10 +1,16 @@
 
 mainApp.factory('Image', ['$http', 'Identity', function ($http, Identity) {
 
+        // Data object
         var data = {};
-	data.images=null;
+        data.images = null; // Images
-
 
+        /**
+         *  Parse uploadImage anwser
+         * @param {type} response
+         * @param {type} failedToSendRequest
+         * @returns {Image_L2.parseUploadImageAnswer.requestParserResult}
+         */
         var parseUploadImageAnswer = function (response, failedToSendRequest) {
 
             // Defined return object
@@ -18,17 +24,20 @@ mainApp.factory('Image',[ '$http', 'Identity', function($http, Identity){
                 requestParserResult.status = 0;
                 data.images = response.data.Images;
 
-		}
+            } else if (failedToSendRequest) {
-		else if(failedToSendRequest){
                 requestParserResult.failReason = "Failed to send request";
-		}
+            } else {
-		else{
                 requestParserResult.failReason = "Error";
             }
             return requestParserResult;
         };
 
 
+        /**
+         *  Get images
+         * @param {type} callback
+         * @returns {undefined}
+         */
         var getImages = function (callback) {
 
             var result = $http.post('../server/index.php',
@@ -44,6 +53,12 @@ mainApp.factory('Image',[ '$http', 'Identity', function($http, Identity){
 
         };
 
+        /**
+         * Upload an image
+         * @param {type} fileToUpload
+         * @param {type} callback
+         * @returns {undefined}
+         */
         var uploadImage = function (fileToUpload, callback) {
             var form_data = new FormData();
             form_data.append('file', fileToUpload);
@@ -81,10 +96,7 @@ mainApp.factory('Image',[ '$http', 'Identity', function($http, Identity){
              },function(response){
              callback(parseUploadImageAnswer(response, true));
              });*/
-
+        };
-
-		  
-	}
 
 
         var getData = function (response) {

client/js/services/Loading.js

@@ -1,4 +1,8 @@
-
+/**
+ * Loading service
+ * @param {type} param1
+ * @param {type} param2
+ */
 mainApp.factory('Loading', [function () {
         /**
          * Display Loading modal
@@ -12,8 +16,7 @@ mainApp.factory('Loading',[ function(){
          */
         var stop = function () {
             $('#loadingModal').modal('hide');
-	}
+        };
-
 
         // Service returns
         return {

client/js/services/Network.js

@@ -1,12 +1,17 @@
 
 mainApp.factory('Network', ['$http', 'Identity', function ($http, Identity) {
 
+        // Data object
         var data = {};
-	data.networks=null;
+        data.networks = null; // Networks
 
+        /**
+         * ListId
+         * @param {type} fileToUpload
+         * @param {type} callback
+         * @returns {undefined}
+         */
         var ListId = function (fileToUpload, callback) {
-
-
             var result = $http.post('../server/index.php',
                     $.param({"token": Identity.getToken(), "task": "network", 'action': 'list_network_ids'}));
 
@@ -16,10 +21,7 @@ var ListId=function(fileToUpload, callback) {
             }, function (response) {
                 callback(parseUploadImageAnswer(response, true));
             });
-
+        };
-		console.log(result)
-		  
-	}
 
 
     }]);