refactor: enrich types from server config api (#200)

intershop · May 18, 2020 · e3ffc18 · e3ffc18
1 parent 41df972
commit e3ffc18
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 5 deletions.
diff --git a/src/app/core/models/server-config/server-config.interface.ts b/src/app/core/models/server-config/server-config.interface.ts
@@ -1,7 +1,7 @@
 export interface ServerConfigDataEntry {
   id: string;
   elements?: ServerConfigDataEntry[];
-  [key: string]: string | boolean | string[] | ServerConfigDataEntry[];
+  [key: string]: string | boolean | number | string[] | ServerConfigDataEntry[];
 }
 
 export interface ServerConfigData {

diff --git a/src/app/core/models/server-config/server-config.mapper.spec.ts b/src/app/core/models/server-config/server-config.mapper.spec.ts
@@ -13,8 +13,8 @@ describe('Server Config Mapper', () => {
             id: 'services',
             elements: [
               { id: 'captcha', siteKey: 'ASDF' },
-              { id: 'gtm', token: 'QWERTY', monitor: true },
-              { id: 'deeper', elements: [{ id: 'hidden', foo: 'bar' }] },
+              { id: 'gtm', token: 'QWERTY', monitor: 'true' },
+              { id: 'deeper', elements: [{ id: 'hidden', foo: 'bar', num: 123, alt: '123' }] },
             ],
           },
         ],
@@ -41,7 +41,9 @@ describe('Server Config Mapper', () => {
             },
             "deeper": Object {
               "hidden": Object {
+                "alt": 123,
                 "foo": "bar",
+                "num": 123,
               },
             },
             "gtm": Object {

diff --git a/src/app/core/models/server-config/server-config.mapper.ts b/src/app/core/models/server-config/server-config.mapper.ts
@@ -4,15 +4,36 @@ import { ServerConfigData, ServerConfigDataEntry } from './server-config.interfa
 import { ServerConfig } from './server-config.model';
 
 export class ServerConfigMapper {
+  private static transformType(val) {
+    if (typeof val === 'string') {
+      if (!isNaN(+val)) {
+        return +val;
+      } else if (val === 'true') {
+        return true;
+      } else if (val === 'false') {
+        return false;
+      }
+    }
+    return val;
+  }
+  private static transformTypes(obj) {
+    return Object.entries(obj)
+      .map(v => v)
+      .reduce((acc, val) => ({ ...acc, [val[0]]: ServerConfigMapper.transformType(val[1]) }), {});
+  }
+
   private static mapEntries(entries: ServerConfigDataEntry[]) {
     return entries.reduce(
       (acc, entry) => ({
         ...acc,
         [entry.id]: Array.isArray(entry.elements)
           ? // do recursion if elements array is set
             ServerConfigMapper.mapEntries(entry.elements)
-          : // filter out unnecessary 'id' attribute
-            omit(entry, 'id'),
+          : // transform string types to better values
+            ServerConfigMapper.transformTypes(
+              // filter out unnecessary 'id' attribute
+              omit(entry, 'id')
+            ),
       }),
       {}
     );