Merge pull request #38 from codeBelt/bugfix_stringReplacers_optional

Bugfix make stringReplacers optional

Author: codeBelt

package.json

@@ -1,6 +1,6 @@
 {
   "name": "generate-template-files",
-  "version": "2.2.3",
+  "version": "2.3.0",
   "description": "A generator to create custom template files for your applications",
   "keywords": [
     "generate",
@@ -51,7 +51,7 @@
     "dist/"
   ],
   "dependencies": {
-    "enquirer": "2.3.4",
+    "enquirer": "2.3.5",
     "lodash.get": "4.4.2",
     "path-exists": "4.0.0",
     "recursive-copy": "2.0.10",
@@ -62,30 +62,30 @@
   "devDependencies": {
     "@babel/core": "7.9.0",
     "@babel/plugin-proposal-class-properties": "7.8.3",
-    "@babel/plugin-proposal-object-rest-spread": "7.9.0",
+    "@babel/plugin-proposal-object-rest-spread": "7.9.5",
     "@babel/plugin-transform-runtime": "7.9.0",
-    "@babel/preset-env": "7.9.0",
+    "@babel/preset-env": "7.9.5",
     "@babel/preset-typescript": "7.9.0",
     "@babel/runtime": "7.9.2",
-    "@types/jest": "25.1.4",
+    "@types/jest": "25.2.1",
     "@types/lodash.get": "4.4.6",
     "@types/path-exists": "4.0.2",
     "@types/replace-string": "3.0.0",
     "@types/through2": "2.0.34",
-    "husky": "4.2.3",
-    "jest": "25.2.3",
-    "prettier": "2.0.2",
+    "husky": "4.2.5",
+    "jest": "25.4.0",
+    "prettier": "2.0.4",
     "pretty-quick": "2.0.1",
     "rimraf": "3.0.2",
-    "rollup": "2.2.0",
+    "rollup": "2.6.1",
     "rollup-plugin-babel": "4.4.0",
     "rollup-plugin-commonjs": "10.1.0",
     "rollup-plugin-node-resolve": "5.2.0",
-    "ts-jest": "25.2.1",
+    "ts-jest": "25.4.0",
     "tslib": "1.11.1",
-    "tslint": "6.1.0",
+    "tslint": "6.1.1",
     "tslint-config-prettier": "1.18.0",
-    "typedoc": "0.17.3",
+    "typedoc": "0.17.4",
     "typescript": "3.8.3"
   },
   "author": {

src/GenerateTemplateFiles.ts

@@ -83,7 +83,8 @@ export default class GenerateTemplateFiles {
      * New question asking what should text should be used to replace the template text.
      */
     private async _getReplacerSlotValues(selectedConfigItem: IConfigItem): Promise<IReplacer[]> {
-        const replacerQuestions: any[] = selectedConfigItem.stringReplacers.map((item: string | IReplacerSlotQuestion) => {
+        const stringReplacers: (string | IReplacerSlotQuestion)[] = selectedConfigItem.stringReplacers ?? [];
+        const replacerQuestions: any[] = stringReplacers.map((item: string | IReplacerSlotQuestion) => {
             return {
                 type: 'input',
                 name: StringUtility.isString(item) ? item : item.slot,
@@ -98,8 +99,6 @@ export default class GenerateTemplateFiles {
 
         const answer: {[replacer: string]: string} = await enquirer.prompt(replacerQuestions);
 
-        CheckUtility.check(Object.keys(answer).length > 0, '"stringReplacers" needs at least one item.');
-
         const replacers: IReplacer[] = Object.entries(answer).map(
             ([key, value]: [string, string]): IReplacer => {
                 return {
@@ -110,6 +109,10 @@ export default class GenerateTemplateFiles {
         );
         const dynamicReplacers: IReplacer[] = selectedConfigItem.dynamicReplacers || [];
 
+        const hasStringOrDynamicReplacers: boolean = stringReplacers.length > 0 || dynamicReplacers.length > 0;
+
+        CheckUtility.check(hasStringOrDynamicReplacers, '"stringReplacers" or "dynamicReplacers" needs at least one item.');
+
         return [...replacers, ...dynamicReplacers];
     }
 

src/models/IConfigItem.ts

@@ -54,7 +54,7 @@ export default interface IConfigItem {
      * stringReplacers: ['__store__', '__model__'],
      * ```
      */
-    stringReplacers: (string | IReplacerSlotQuestion)[];
+    stringReplacers?: (string | IReplacerSlotQuestion)[];
     /**
      * ```
      * dynamicReplacers: [{slot:'__description__', slotValue: config.description}],