Merge pull request #450 from mobxjs/fieldOverrides-fix

Field overrides fix
mobxjs · Sep 3, 2024 · 1d0e6e7 · 1d0e6e7
2 parents 7d473fc + 07eb474
commit 1d0e6e7
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## 0.17.3
+
+...
+
 ## 0.17.2
 
 - Fix tests, upgrade minor versions of mobx, mobx-state-tree

diff --git a/generator/config.js b/generator/config.js
@@ -72,16 +72,19 @@ exports.mergeConfigs = function mergeConfigs(args, config) {
       !!args["--useIdentifierNumber"] || config.useIdentifierNumber,
     fieldOverrides: args["--fieldOverrides"]
       ? parseFieldOverrides(args["--fieldOverrides"])
-      : parseFieldOverrides(config.fieldOverrides),
+      : config.fieldOverrides
+      ? parseFieldOverrides(config.fieldOverrides)
+      : [],
     dynamicArgs: !!args["--dynamicArgs"] || config.dynamicArgs,
     debug: !!args["--debug"] || config.debug,
     disableLogColors: !!args["--disableLogColors"] || config.disableLogColors
   }
 }
 
 const parseFieldOverrides = (fieldOverrides) => {
-  const _fieldOverrides = 
-    Array.isArray(fieldOverrides) ? fieldOverrides : fieldOverrides.split(",");
+  const _fieldOverrides = Array.isArray(fieldOverrides)
+    ? fieldOverrides
+    : fieldOverrides.split(",")
 
   return _fieldOverrides
     .map((s) => s.trim())