refactor: treat empty expandable arrays as non expandable (fixes SFKUI-7728) (#1094)

Author: jensviklander

packages/vue-labs/src/components/FTable/get-meta-rows.spec.ts

@@ -0,0 +1,33 @@
+import { setItemIdentifiers } from "@fkui/vue";
+import { getMetaRows } from "./get-meta-rows";
+
+const rows = [
+    {
+        name: "A",
+    },
+    {
+        name: "B",
+        expandable: [],
+    },
+    {
+        name: "C",
+        expandable: [
+            {
+                name: "C1",
+            },
+        ],
+    },
+];
+
+it("should only mark rows with non-empty children as expandable", () => {
+    const keyedRows = setItemIdentifiers(rows, "name", "expandable");
+    const result = getMetaRows(keyedRows, new Set(), "expandable");
+
+    expect(result).toHaveLength(3);
+
+    expect(result[0].isExpandable).toBe(false);
+
+    expect(result[1].isExpandable).toBe(false);
+
+    expect(result[2].isExpandable).toBe(true);
+});

packages/vue-labs/src/components/FTable/get-meta-rows.ts

@@ -30,7 +30,9 @@ export function getMetaRows<T>(
     walk(keyedRows, expandableAttribute, (row, level) => {
         const key = getItemIdentifier(row);
         const isExpandable = Boolean(
-            expandableAttribute && row[expandableAttribute],
+            expandableAttribute &&
+            Array.isArray(row[expandableAttribute]) &&
+            row[expandableAttribute].length > 0,
         );
         const isExpanded = isExpandable && expandedKeys.has(key);