trotl-table 1.0.90 → 1.0.92

package/dist/Table.cjs.js

@@ -9663,8 +9663,6 @@ function TableInner({
   onCellChange = () => {},
   // callback for any cell click
   cellCallback = () => {},
-  // when true, skip the integrated confirmation modal and delete immediately
-  disableDeleteModal = false,
   deleteLabelKey = "name",
   buttons = ["view", "edit", "delete", "share", "duplicate"],
   enableDragRow = false,
@@ -9717,6 +9715,9 @@ function TableInner({
       return translations[key] || key;
     };
   }
+
+  // normalize delete modal flag early (handles string 'false' etc)
+
   // Use refs for callbacks to prevent infinite loops from dependency changes
   const selectedRowsCallbackRef = React.useRef(selectedRowsCallback);
   const refreshTriggerRef = React.useRef(refreshTrigger);
@@ -9826,28 +9827,71 @@ function TableInner({
       flex: "1 1 0%"
     };
   }, [columnWidths, getColumnKey]);
-  const getAccessorValue = React.useCallback((row, accessor) => {
+  const getNestedValue = React.useCallback((row, accessor) => {
     if (!row || accessor == null) return undefined;
-
-    // Support fallback accessors: ["preverjen", "Preverjen", ...]
     if (Array.isArray(accessor)) {
       for (const key of accessor) {
-        if (row?.[key] !== undefined) return row[key];
+        const value = getNestedValue(row, key);
+        if (value !== undefined) return value;
       }
       return undefined;
     }
-    return row?.[accessor];
+    const path = String(accessor).split(".").filter(Boolean);
+    if (path.length === 0) return undefined;
+    let current = row;
+    for (const key of path) {
+      if (current == null || typeof current !== "object") return undefined;
+      current = current[key];
+    }
+    return current;
   }, []);
+  const getAccessorValue = React.useCallback((row, accessor) => {
+    return getNestedValue(row, accessor);
+  }, [getNestedValue]);
   const getAccessorKey = React.useCallback((row, accessor) => {
     if (accessor == null) return accessor;
     if (Array.isArray(accessor)) {
       for (const key of accessor) {
-        if (row?.[key] !== undefined) return key;
+        if (getNestedValue(row, key) !== undefined) return key;
       }
       return accessor[0];
     }
     return accessor;
-  }, []);
+  }, [getNestedValue]);
+  const updateRowField = React.useCallback((row, accessor, value) => {
+    if (row == null || typeof row !== "object") return row;
+    if (Array.isArray(accessor)) {
+      for (const key of accessor) {
+        if (getNestedValue(row, key) !== undefined) {
+          return updateRowField(row, key, value);
+        }
+      }
+      return {
+        ...row,
+        [accessor[0]]: value
+      };
+    }
+    const path = String(accessor).split(".").filter(Boolean);
+    if (path.length === 0) return row;
+    if (path.length === 1) return {
+      ...row,
+      [path[0]]: value
+    };
+    const nextRow = {
+      ...row
+    };
+    let current = nextRow;
+    for (let i = 0; i < path.length - 1; i += 1) {
+      const key = path[i];
+      const nextValue = current[key];
+      current[key] = nextValue && typeof nextValue === "object" ? {
+        ...nextValue
+      } : {};
+      current = current[key];
+    }
+    current[path[path.length - 1]] = value;
+    return nextRow;
+  }, [getNestedValue]);
   const normalizeHighlightComparable = React.useCallback((value, compareTo) => {
     if (value instanceof Date || compareTo instanceof Date) {
       const leftTime = value == null ? NaN : new Date(value).getTime();
@@ -10141,17 +10185,25 @@ function TableInner({
     return a === b;
   }, []);
   const formatArrayValue = React.useCallback((value, col) => {
-    if (!Array.isArray(value)) return value;
+    if (!Array.isArray(value)) return "";
     const objectKey = col?.objectKey;
     const parts = value.map(item => {
       if (item == null) return "";
+
+      // When objectKey is configured, only use that field. This prevents
+      // objects without a label from displaying their key or JSON value.
+      if (objectKey) {
+        if (typeof item !== "object" || typeof item[objectKey] !== "string") {
+          return "";
+        }
+        return item[objectKey].trim();
+      }
       if (typeof item !== "object") return String(item);
-      if (objectKey && item?.[objectKey] !== undefined) return String(item[objectKey]);
       if (item.label !== undefined) return String(item.label);
       if (item.value !== undefined) return String(item.value);
       return JSON.stringify(item);
-    }).filter(item => item !== "");
-    return parts.join(", ");
+    }).filter(Boolean);
+    return [...new Set(parts)].join(", ");
   }, []);
 
   // console.log(extraSearchTerm)
@@ -10577,16 +10629,10 @@ function TableInner({
             if (isGrouped) {
               return prev.map(g => ({
                 ...g,
-                rows: (g.rows || []).map(r => r.id === row.id ? {
-                  ...r,
-                  [accessor]: next
-                } : r)
+                rows: (g.rows || []).map(r => r.id === row.id ? updateRowField(r, accessor, next) : r)
               }));
             }
-            return (prev || []).map(r => r.id === row.id ? {
-              ...r,
-              [accessor]: next
-            } : r);
+            return (prev || []).map(r => r.id === row.id ? updateRowField(r, accessor, next) : r);
           });
           refreshTriggerRef.current?.();
           try {
@@ -10615,16 +10661,10 @@ function TableInner({
             if (isGrouped) {
               return prev.map(g => ({
                 ...g,
-                rows: (g.rows || []).map(r => r.id === row.id ? {
-                  ...r,
-                  [accessor]: next
-                } : r)
+                rows: (g.rows || []).map(r => r.id === row.id ? updateRowField(r, accessor, next) : r)
               }));
             }
-            return (prev || []).map(r => r.id === row.id ? {
-              ...r,
-              [accessor]: next
-            } : r);
+            return (prev || []).map(r => r.id === row.id ? updateRowField(r, accessor, next) : r);
           });
           refreshTriggerRef.current?.();
           try {
@@ -10679,16 +10719,10 @@ function TableInner({
             if (isGrouped) {
               return prev.map(g => ({
                 ...g,
-                rows: (g.rows || []).map(r => r.id === row.id ? {
-                  ...r,
-                  [accessor]: next
-                } : r)
+                rows: (g.rows || []).map(r => r.id === row.id ? updateRowField(r, accessor, next) : r)
               }));
             }
-            return (prev || []).map(r => r.id === row.id ? {
-              ...r,
-              [accessor]: next
-            } : r);
+            return (prev || []).map(r => r.id === row.id ? updateRowField(r, accessor, next) : r);
           });
           refreshTriggerRef.current?.();
           try {
@@ -10723,16 +10757,10 @@ function TableInner({
             if (isGrouped) {
               return prev.map(g => ({
                 ...g,
-                rows: (g.rows || []).map(r => r.id === row.id ? {
-                  ...r,
-                  [accessor]: next
-                } : r)
+                rows: (g.rows || []).map(r => r.id === row.id ? updateRowField(r, accessor, next) : r)
               }));
             }
-            return (prev || []).map(r => r.id === row.id ? {
-              ...r,
-              [accessor]: next
-            } : r);
+            return (prev || []).map(r => r.id === row.id ? updateRowField(r, accessor, next) : r);
           });
           refreshTriggerRef.current?.();
           try {
@@ -10774,10 +10802,8 @@ function TableInner({
       }, formatted);
     }
     if (col && col.type === "array") {
-      const arr = Array.isArray(v) ? v : [];
-      const list = formatArrayValue(arr, col);
-      const count = arr.length;
-      const display = count > 0 ? `${count}: ${list}` : "...";
+      const list = formatArrayValue(v, col);
+      const display = list || "...";
       return /*#__PURE__*/React.createElement("span", {
         title: list || translate("noData")
       }, display);
@@ -10961,50 +10987,6 @@ function TableInner({
     });
   }, [tableDataFlat, isGrouped, groupKey]);
 
-  // DUPLICATE
-  React.useCallback(row => {
-    if (!row) return;
-    // try user callback first; if it returns an object with newRow use that
-    let newRow = null;
-    if (duplicateCallbackRef.current) {
-      try {
-        const res = duplicateCallbackRef.current(row);
-        if (res && res.newRow) {
-          newRow = res.newRow;
-        }
-      } catch (err) {
-        console.error("duplicateCallback threw", err);
-      }
-    }
-    // fallback simple clone if callback didn't provide newRow
-    if (!newRow) {
-      newRow = {
-        ...row
-      };
-      // give it a distinct id if possible
-      if (newRow.id != null) {
-        newRow.id = `${newRow.id}-dup-${Date.now()}`;
-      }
-    }
-    setLocalData(prev => {
-      if (isGrouped) {
-        return prev.map(g => {
-          const gid = g[groupKey] ?? g.groupId ?? g.groupName;
-          // insert into same group as original row
-          if (row[groupKey] === gid || g.rows.some(r => r.id === row.id)) {
-            return {
-              ...g,
-              rows: [...(g.rows || []), newRow]
-            };
-          }
-          return g;
-        });
-      }
-      return [...(prev || []), newRow];
-    });
-    refreshTriggerRef.current?.();
-  }, [duplicateCallbackRef, isGrouped, groupKey, refreshTriggerRef]);
-
   // ROW RENDERER
   const rowRenderer = React.useCallback(({
     index,