More code review changes

Author: Fatme

lib/declarations.ts

@@ -1,8 +1,7 @@
 interface INodePackageManager {
 	cache: string;
 	load(config?: any): IFuture<void>;
-	install(where: string, what: string): IFuture<any>;
-	installSafe(packageName: string, pathToSave?: string): IFuture<string>;
+	install(packageName: string, pathToSave?: string): IFuture<string>;
 }
 
 interface IPropertiesParser {

lib/node-package-manager.ts

@@ -27,7 +27,7 @@ export class NodePackageManager implements INodePackageManager {
 		return future;
 	}
 
-	public install(where: string, what: string): IFuture<any> {
+	private installCore(where: string, what: string): IFuture<any> {
 		var future = new Future<any>();
 		npm.commands["install"](where, what, (err, data) => {
 			if(err) {
@@ -51,10 +51,10 @@ export class NodePackageManager implements INodePackageManager {
 		}).future<void>()();
 	}
 
-	public installSafe(packageName: string, pathToSave?: string): IFuture<string> {
+	public install(packageName: string, pathToSave?: string): IFuture<string> {
 		return (() => {
 			var action = (packageName: string) => {
-				this.install(pathToSave || npm.cache, packageName).wait();
+				this.installCore(pathToSave || npm.cache, packageName).wait();
 			};
 
 			this.tryExecuteAction(action, packageName).wait();

lib/services/platform-project-service.ts

@@ -32,7 +32,7 @@ class PlatformProjectService implements IPlatformProjectService {
 			this.$logger.out("Copying template files...");
 
 			// get path to downloaded framework package
-			var frameworkDir = this.$npm.installSafe(this.$platformsData.getPlatformData(platform).frameworkPackageName,
+			var frameworkDir = this.$npm.install(this.$platformsData.getPlatformData(platform).frameworkPackageName,
 				path.join(this.$projectData.platformsDir, platform)).wait();
 			frameworkDir = path.join(frameworkDir, PlatformProjectService.PROJECT_FRAMEWORK_DIR);
 

lib/services/project-templates-service.ts

@@ -14,7 +14,7 @@ export class ProjectTemplatesService implements IProjectTemplatesService {
 	public constructor(private $npm: INodePackageManager) { }
 
 	public get defaultTemplatePath(): IFuture<string> {
-		return this.$npm.installSafe(ProjectTemplatesService.NPM_DEFAULT_TEMPLATE_NAME);
+		return this.$npm.install(ProjectTemplatesService.NPM_DEFAULT_TEMPLATE_NAME);
 	}
 }
 $injector.register("projectTemplatesService", ProjectTemplatesService);