Merge pull request #9 from AckeeCZ/grissius-patch-1

Remove redundant array check
AckeeCZ · Jan 11, 2019 · ce0a0c2 · ce0a0c2
2 parents 72b16dd + 09aab61
commit ce0a0c2
Showing 1 changed file with 4 additions and 10 deletions.
diff --git a/src/serializers.ts b/src/serializers.ts
@@ -54,13 +54,10 @@ const serializers: Dictionary<SerializerFn> = {
     },
 };
 
-const sliceByPrefix = (prefix: string, paths: string[]) =>
-    paths.filter(field => field.startsWith(prefix)).map(field => field.slice(prefix.length));
+const sliceByPrefix = (prefix: string, paths?: string[]) =>
+    (paths || []).filter(field => field.startsWith(prefix)).map(field => field.slice(prefix.length));
 
-const disablePaths = (paths: string[] | undefined) => {
-    if (!paths || paths.length <= 0) {
-        return;
-    }
+const disablePaths = (paths?: string[]) => {
     forEach(serializers, (value, key) => {
         const affectedFields = sliceByPrefix(`${key}.`, paths);
 
@@ -73,10 +70,7 @@ const disablePaths = (paths: string[] | undefined) => {
     });
 };
 
-const enablePaths = (paths: string[] | undefined) => {
-    if (!paths || paths.length <= 0) {
-        return;
-    }
+const enablePaths = (paths?: string[]) => {
     forEach(serializers, (value, key) => {
         const affectedFields = sliceByPrefix(`${key}.`, paths);