Support empty string parentIds as root objects

Author: philipstanislaus

build/arrayToTree.js

@@ -193,6 +193,29 @@ describe('arrayToTree', () => {
     ])
   })
 
+  it('should treat objects with empty string as parentId as root objects', () => {
+    expect(arrayToTree([
+      { id: '4', parentId: '', custom: 'abc' },
+      { id: '31', parentId: '4', custom: '12' },
+      { id: '1941', parentId: '418', custom: 'de' },
+      { id: '1', parentId: '418', custom: 'ZZZz' },
+      { id: '418', parentId: '', custom: 'ü' },
+      { id: '1313', parentId: '13', custom: 'Not existing' },
+    ])).to.deep.equal([
+      {
+        data: { id: '4', parentId: '', custom: 'abc' }, children: [
+          { data: { id: '31', parentId: '4', custom: '12' }, children: [] },
+        ],
+      },
+      {
+        data: { id: '418', parentId: '', custom: 'ü' }, children: [
+          { data: { id: '1941', parentId: '418', custom: 'de' }, children: [] },
+          { data: { id: '1', parentId: '418', custom: 'ZZZz' }, children: [] },
+        ],
+      },
+    ])
+  })
+
   it('should work with empty inputs', () => {
     expect(arrayToTree([])).to.deep.equal([])
   })

build/arrayToTree.js.map

@@ -59,7 +59,7 @@ export function arrayToTree (items: Item[], config: Partial<Config> = {}): TreeI
 
     const TreeItem = lookup[itemId]
 
-    if (parentId === null || parentId === undefined) {
+    if (parentId === null || parentId === undefined || parentId === '') {
       // is a root item
       rootItems.push(TreeItem)
     } else {