Merge pull request #305 from Element84/ba/bug/metadata-config-excepti…

…on-error

metadata config exception error

src/utils/dataHelper.js

@@ -71,21 +71,37 @@ export function setScenesForCartLayer() {
 }
 
 export function processDisplayFieldValues(value) {
-  if (typeof value === 'boolean') {
-    return value.toString()
-  } else if (Array.isArray(value)) {
-    return value.map(processDisplayFieldValues).join(', ')
-  } else if (typeof value === 'number') {
-    return value.toString()
-  } else if (typeof value === 'string') {
-    return value
-  } else if (typeof value === 'object' && value !== null) {
-    const processedObject = {}
-    for (const key in value) {
-      processedObject[key] = processDisplayFieldValues(value[key])
+  switch (true) {
+    case value === null:
+    case value === undefined:
+      return 'No Data'
+    case typeof value === 'boolean':
+    case typeof value === 'number':
+    case typeof value === 'string':
+      return value.toString()
+    case Array.isArray(value):
+      return value.map(processDisplayFieldValues).join(', ')
+    case typeof value === 'object':
+      return formatNestedObjectDisplayFieldValues(value)
+    default:
+      return 'Unsupported Type'
+  }
+}
+
+function formatNestedObjectDisplayFieldValues(inputObject) {
+  function formatValue(value) {
+    if (Array.isArray(value)) {
+      return `[${value.map(formatValue).join(', ')}]`
+    } else if (typeof value === 'object') {
+      return `{${formatObject(value)}}`
+    } else {
+      return value
     }
-    return processedObject
-  } else {
-    return 'Unsupported Type'
   }
+  function formatObject(obj) {
+    return Object.entries(obj)
+      .map(([key, value]) => `${key}: ${formatValue(value)}`)
+      .join(', ')
+  }
+  return `${formatObject(inputObject)}`
 }

src/utils/dataHelper.test.js

@@ -149,29 +149,22 @@ describe('dataHelper', () => {
       expect(actual).toEqual(expected)
     })
 
-    it('should recursively process object values', () => {
+    it('should process nested object values into string', () => {
       const input = {
-        bool: true,
-        arr: [1, 2, 3],
-        num: 42,
-        str: 'Hello, world!',
         nestedObj: {
           nestedBool: false,
-          nestedArr: [4, 5, 6]
+          nestedArr: [4, 5, 6],
+          platform: {
+            eq: 'landsat-8'
+          },
+          collections: ['landsat-c2-l2']
         }
       }
 
       const result = processDisplayFieldValues(input)
-      expect(result).toEqual({
-        bool: 'true',
-        arr: '1, 2, 3',
-        num: '42',
-        str: 'Hello, world!',
-        nestedObj: {
-          nestedBool: 'false',
-          nestedArr: '4, 5, 6'
-        }
-      })
+      expect(result).toEqual(
+        'nestedObj: {nestedBool: false, nestedArr: [4, 5, 6], platform: {eq: landsat-8}, collections: [landsat-c2-l2]}'
+      )
     })
 
     it('returns Unsupported Type for other types', () => {