Skip to content

Commit 1f2f759

Browse files
committed
Merge pull request #60 from NativeScript/totev/appid
Validate Project Name on project creation
2 parents 48a8393 + 391ee93 commit 1f2f759

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

lib/services/project-service.ts

+4-1
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@ export class ProjectService implements IProjectService {
4949
private $errors: IErrors,
5050
private $fs: IFileSystem,
5151
private $projectTemplatesService: IProjectTemplatesService,
52+
private $projectNameValidator: IProjectNameValidator,
5253
private $projectHelper: IProjectHelper,
5354
private $staticConfig: IStaticConfig) { }
5455

@@ -57,7 +58,9 @@ export class ProjectService implements IProjectService {
5758
if (!projectName) {
5859
this.$errors.fail("You must specify <App name> when creating a new project.");
5960
}
60-
projectId = options.appid || this.$projectHelper.generateDefaultAppId(projectName);
61+
this.$projectNameValidator.validate(projectName);
62+
63+
projectId = options.appid || this.$projectHelper.generateDefaultAppId(projectName);
6164

6265
var projectDir = path.join(path.resolve(options.path || "."), projectName);
6366
this.$fs.createDirectory(projectDir).wait();

0 commit comments

Comments
 (0)