fix(bodyValue): account for null or undefined previous body values

app/components/ComponentList.tsx

@@ -126,7 +126,6 @@ function removeHiddenComponents (status: DatasetStatus) {
   }
 
   return (component): boolean => {
-    console.log(status[component.name])
     return status[component.name] || !hideWhenMissing[component.name]
   }
 }

app/utils/bodyValue.ts

@@ -1,16 +1,12 @@
 import { PageInfo } from '../../app/models/store'
 
 export default function bodyValue (prev: Object | any[] | undefined, curr: Object | any[], pageInfo: PageInfo): Object | any[] {
-  // if there is no previous body value, just return the current body value
-  if (!prev) return curr
   // work through cases where the current body value is an array
   if (Array.isArray(curr)) {
-    // if the previous body value is not an array
-    // just return the current body value
-    if (!Array.isArray(prev)) {
-      return curr
+    if (curr.length === 0) {
+      if (!prev) return curr
+      return prev
     }
-    // otherwise, it is an array and we should concat the lists
 
     if (Array.isArray(curr[0])) {
       // for tabular/2D arrays only:
@@ -23,6 +19,9 @@ export default function bodyValue (prev: Object | any[] | undefined, curr: Objec
         d.unshift(index)
       })
 
+      // if there is no previous body value, just return the current body value
+      if (!prev || !Array.isArray(prev)) return curr
+
       if (prev.length !== 0) {
         const currLastIndex = curr[curr.length - 1][0]
         const prevLastIndex = prev[prev.length - 1][0] || 0
@@ -51,14 +50,12 @@ export default function bodyValue (prev: Object | any[] | undefined, curr: Objec
 
       return [...prev, ...curr]
     }
-
-    return [...prev, ...curr]
   }
   // work through cases where the current body value is an object (but not an array)
   if (typeof curr === 'object') {
     // if the previous body value is not an object (or is an object but is also an array)
     // return the current body value
-    if (!(typeof prev === 'object') && Array.isArray(prev)) {
+    if (!prev || (!(typeof prev === 'object') && Array.isArray(prev))) {
       return curr
     }
     // otherwise return the spread!

test/app/utils/bodyValue.test.ts

@@ -30,6 +30,13 @@ describe('bodyValue', () => {
       pageInfo: pageInfo2,
       expected: []
     },
+    {
+      describe: 'prev is undefined',
+      prev: undefined,
+      curr: [['a','b','c'],['d','e','f']],
+      pageInfo: initPageInfo,
+      expected: [[0, 'a','b','c'],[1, 'd','e','f']]
+    },
     {
       describe: 'prev is empty',
       prev: [],