fix: added async to before hook

Author: ethanneff

hooks.es6/beforePluginInstallHook.js

@@ -1,154 +1,77 @@
-/**
-Hook is executed when plugin is added to the project.
-It will check all necessary module dependencies and install the missing ones locally.
-*/
+'use strict';
 
-var exec = require('child_process').exec,
-  path = require('path'),
-  fs = require('fs'),
-  INSTALLATION_FLAG_FILE_NAME = '.installed';
-
-// region NPM specific
-
-/**
- * Check if node package is installed.
- *
- * @param {String} moduleName
- * @return {Boolean} true if package already installed
- */
-function isNodeModuleInstalled(moduleName) {
-  var installed = true;
-  try {
-    var module = require(moduleName);
-  } catch (err) {
-    installed = false;
+function installNodeModule(exec, modules, callback) {
+  if (modules.length <= 0) {
+    return callback(false);
   }
 
-  return installed;
-}
-
-/**
- * Install node module locally.
- * Basically, it runs 'npm install module_name'.
- *
- * @param {String} moduleName
- * @param {Callback(error)} callback
- */
-function installNodeModule(moduleName, callback) {
-  if (isNodeModuleInstalled(moduleName)) {
-    printLog('Node module ' + moduleName + ' is found');
-    callback(null);
-    return;
-  }
-  printLog('Can\'t find module ' + moduleName + ', running npm install');
-
-  var cmd = 'cd plugins/io.branch.sdk && npm install -D ' + moduleName;
-  exec(cmd, function(err, stdout, stderr) {
-    callback(err);
-  });
-}
-
-/**
- * Install all required node packages.
- */
-function installRequiredNodeModules(modulesToInstall) {
-  if (!modulesToInstall.length) {
-    return;
-  }
+  var module = modules.pop();
+  var install = 'npm install --prefix ./plugins/io.branch.sdk -D ' + module;
 
-  var moduleName = modulesToInstall.shift();
-  installNodeModule(moduleName, function(err) {
+  console.log('Installing "' + module + '"');
+  exec(install, function(err, stdout, stderr) {
     if (err) {
-      printLog('Failed to install module ' + moduleName + ':' + err);
-      return;
+      console.error('Failed to install Branch Dependency: "' + module + '"');
+      return callback(true);
+    }
+    else {
+      installNodeModule(exec, modules, callback);
     }
-
-    printLog('Module ' + moduleName + ' is installed');
-    installRequiredNodeModules(modulesToInstall);
   });
 }
 
-// endregion
-
-// region Logging
-
-function logStart() {
-  console.log('Checking dependencies:');
-}
-
-function printLog(msg) {
-  var formattedMsg = '    ' + msg;
-  console.log(formattedMsg);
+function getUninstalledNodeModules(dependencies) {
+  var modules = [];
+  for (var module in dependencies) {
+    if (dependencies.hasOwnProperty(module)) {
+      try {
+        var exists = require(module);
+      } catch (err) {
+        modules.push(module);
+      }
+    }
+  }
+  return modules;
 }
 
-// endregion
-
-// region Private API
-
-/**
- * Check if we already executed this hook.
- *
- * @param {Object} ctx - cordova context
- * @return {Boolean} true if already executed; otherwise - false
- */
-function isInstallationAlreadyPerformed(ctx) {
-  var pathToInstallFlag = path.join(ctx.opts.projectRoot, 'plugins', ctx.opts.plugin.id, INSTALLATION_FLAG_FILE_NAME),
-    isInstalled = false;
+function getPackageInstalled(filesave, installFlagLocation) {
   try {
-    var content = fs.readFileSync(pathToInstallFlag);
-    isInstalled = true;
+    var exists = filesave.readFileSync(installFlagLocation);
+    return true;
   } catch (err) {
+    return false;
   }
-
-  return isInstalled;
 }
 
-/**
- * Create empty file - indicator, that we tried to install dependency modules after installation.
- * We have to do that, or this hook is gonna be called on any plugin installation.
- */
-function createPluginInstalledFlag(ctx) {
-  var pathToInstallFlag = path.join(ctx.opts.projectRoot, 'plugins', ctx.opts.plugin.id, INSTALLATION_FLAG_FILE_NAME);
-
-  fs.closeSync(fs.openSync(pathToInstallFlag, 'w'));
-}
-
-// endregion
-
-/**
- * Read dependencies from the package.json.
- * We will install them on the next step.
- *
- * @param {Object} ctx - cordova context
- * @return {Array} list of modules to install
- */
-function readDependenciesFromPackageJson(ctx) {
-  var data = require(path.join(ctx.opts.projectRoot, 'plugins', ctx.opts.plugin.id, 'package.json')),
-    dependencies = data['dependencies'],
-    modules = [];
-
-  if (!dependencies) {
-    return modules;
-  }
-
-  for (var module in dependencies) {
-    modules.push(module);
-  }
-
-  return modules;
+function setPackageInstalled(filesave, installFlagLocation) {
+  filesave.closeSync(filesave.openSync(installFlagLocation, 'w'));
 }
 
 // hook's entry point
-module.exports = function(ctx) {
-  // exit if we already executed this hook once
-  if (isInstallationAlreadyPerformed(ctx)) {
+module.exports = function(context) {
+  var q = context.requireCordovaModule('q');
+  var async = new q.defer();
+  var filesave = require('fs');
+  var path = require('path');
+  var exec = require('child_process').exec;
+  var installFlagName = '.installed';
+  var installFlagLocation = path.join(context.opts.projectRoot, 'plugins', context.opts.plugin.id, installFlagName);
+  var dependencies = require(path.join(context.opts.projectRoot, 'plugins', context.opts.plugin.id, 'package.json')).dependencies;
+
+  if (getPackageInstalled(filesave, installFlagLocation)) {
     return;
   }
 
-  logStart();
-
-  var modules = readDependenciesFromPackageJson(ctx);
-  installRequiredNodeModules(modules);
+  var modules = getUninstalledNodeModules(dependencies);
+  installNodeModule(exec, modules, function(err) {
+    if (err) {
+      console.error('Failed to install the Branch SDK');
+    }
+    else {
+      setPackageInstalled(filesave, installFlagLocation);
+    }
+    async.resolve();
+  });
 
-  createPluginInstalledFlag(ctx);
+  return async.promise;
 };