You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by ag...@apache.org on 2014/03/03 20:30:17 UTC

git commit: CB-6151 Fix exception when adding a new platform to a CLI project

Repository: cordova-plugman
Updated Branches:
  refs/heads/master 635a921e9 -> 2d2c79154


CB-6151 Fix exception when adding a new platform to a CLI project

Options param was being treated as required instead of optional.


Project: http://git-wip-us.apache.org/repos/asf/cordova-plugman/repo
Commit: http://git-wip-us.apache.org/repos/asf/cordova-plugman/commit/2d2c7915
Tree: http://git-wip-us.apache.org/repos/asf/cordova-plugman/tree/2d2c7915
Diff: http://git-wip-us.apache.org/repos/asf/cordova-plugman/diff/2d2c7915

Branch: refs/heads/master
Commit: 2d2c7915421d124bb3a574465af44bba887e6707
Parents: 635a921
Author: Andrew Grieve <ag...@chromium.org>
Authored: Mon Mar 3 14:29:10 2014 -0500
Committer: Andrew Grieve <ag...@chromium.org>
Committed: Mon Mar 3 14:29:10 2014 -0500

----------------------------------------------------------------------
 src/install.js | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cordova-plugman/blob/2d2c7915/src/install.js
----------------------------------------------------------------------
diff --git a/src/install.js b/src/install.js
index 3878e61..0949068 100644
--- a/src/install.js
+++ b/src/install.js
@@ -35,6 +35,7 @@ var path = require('path'),
 // possible options: subdir, cli_variables, www_dir
 // Returns a promise.
 module.exports = function installPlugin(platform, project_dir, id, plugins_dir, options) {
+    options = options || {};
     if (!platform_modules[platform]) {
         return Q.reject(new Error(platform + " not supported."));
     }