Merge pull request #2752 from gintsgints/2719

FIX #2719 check provided name and give feedback if not.
typeorm · Oct 27, 2018 · 099c654 · 099c654
2 parents 79c98a0 + e68538a
commit 099c654
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/src/commands/MigrationGenerateCommand.ts b/src/commands/MigrationGenerateCommand.ts
@@ -85,12 +85,15 @@ export class MigrationGenerateCommand {
             }
 
             if (upSqls.length) {
-                const fileContent = MigrationGenerateCommand.getTemplate(argv.name, timestamp, upSqls, downSqls.reverse());
-                const path = process.cwd() + "/" + (directory ? (directory + "/") : "") + filename;
-                await CommandUtils.createFile(path, fileContent);
-
-                console.log(chalk.green(`Migration ${chalk.blue(path)} has been generated successfully.`));
-
+                if (argv.name) {
+                    const fileContent = MigrationGenerateCommand.getTemplate(argv.name, timestamp, upSqls, downSqls.reverse());
+                    const path = process.cwd() + "/" + (directory ? (directory + "/") : "") + filename;
+                    await CommandUtils.createFile(path, fileContent);
+
+                    console.log(chalk.green(`Migration ${chalk.blue(path)} has been generated successfully.`));
+                } else {
+                    console.log(chalk.yellow("Please specify migration name"));
+                }
             } else {
                 console.log(chalk.yellow(`No changes in database schema were found - cannot generate a migration. To create a new empty migration use "typeorm migration:create" command`));
             }