Merge pull request #509 from adobe/zaggino/domain-progress

abose · abose · commit fadd90071ca9 · 2015-06-12T16:10:09.000+05:30
implement sendCommandProgress and progress callback
diff --git a/appshell/node-core/ConnectionManager.js b/appshell/node-core/ConnectionManager.js
@@ -171,6 +171,16 @@ maxerr: 50, node: true */
         }
     };
     
+    /**
+     * Sends a progress message to command execution (async commands only)
+     * @param {number} id unique ID of the command that was executed. ID is
+     *    generated by the client when the command is issued.
+     * @param {string} message Progress message
+     */
+    Connection.prototype.sendCommandProgress = function (id, message) {
+        this._send("commandProgress", {id: id, message: message});
+    };
+
     /**
      * Sends a response indicating that an error occurred during command 
      * execution
diff --git a/appshell/node-core/DomainManager.js b/appshell/node-core/DomainManager.js
@@ -148,7 +148,8 @@ maxerr: 50, node: true */
      * @param {string} domainName The domain name.
      * @param {string} commandName The command name.
      * @param {Array} parameters The parameters to pass to the command function. If
-     *    the command is asynchronous, will be augmented with a callback function.
+     *    the command is asynchronous, will be augmented with a callback function
+     *    and progressCallback function
      *    (see description in registerCommand documentation)
      */
     function executeCommand(connection, id, domainName,
@@ -164,7 +165,10 @@ maxerr: 50, node: true */
                         connection.sendCommandResponse(id, result);
                     }
                 };
-                parameters.push(callback);
+                var progressCallback = function (msg) {
+                    connection.sendCommandProgress(id, msg);
+                };
+                parameters.push(callback, progressCallback);
                 command.commandFunction.apply(connection, parameters);
             } else { // synchronous command
                 try {
