Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Commit

Permalink
consistent function style
Browse files Browse the repository at this point in the history
  • Loading branch information
zaggino committed Feb 13, 2017
1 parent 74bec9e commit 3f1ed5d
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions src/extensibility/node/ExtensionManagerDomain.js
Original file line number Diff line number Diff line change
Expand Up @@ -90,19 +90,19 @@ function _removeFailedInstallation(installDirectory) {
function _performInstall(packagePath, installDirectory, validationResult, callback) {
validationResult.installedTo = installDirectory;

var fail = function (err) {
function fail(err) {
_removeFailedInstallation(installDirectory);
callback(err, null);
};
}

var finish = function () {
function finish() {
// The status may have already been set previously (as in the
// DISABLED case.
if (!validationResult.installationStatus) {
validationResult.installationStatus = Statuses.INSTALLED;
}
callback(null, validationResult);
};
}

fs.mkdirs(installDirectory, function (err) {
if (err) {
Expand Down Expand Up @@ -220,7 +220,7 @@ function _cmdInstall(packagePath, destinationDirectory, options, callback, pCall
return;
}

var validateCallback = function (err, validationResult) {
function validateCallback(err, validationResult) {
validationResult.localPath = packagePath;

// This is a wrapper for the callback that will delete the temporary
Expand Down Expand Up @@ -307,7 +307,7 @@ function _cmdInstall(packagePath, destinationDirectory, options, callback, pCall
validationResult.disabledReason = null;
_performInstall(packagePath, installDirectory, validationResult, deleteTempAndCallback);
}
};
}

validate(packagePath, {}, validateCallback);
}
Expand Down

0 comments on commit 3f1ed5d

Please sign in to comment.