@eeacms/volto-tabs-block 10.0.1 → 10.0.3

package/CHANGELOG.md

@@ -4,7 +4,18 @@ All notable changes to this project will be documented in this file. Dates are d
 
 Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
-### [10.0.1](https://github.com/eea/volto-tabs-block/compare/10.0.0...10.0.1) - 8 April 2026
+### [10.0.3](https://github.com/eea/volto-tabs-block/compare/10.0.2...10.0.3) - 4 June 2026
+
+#### :bug: Bug Fixes
+
+- fix: .items on undefined - refs sentry #609609 [Alin Voinea - [`b2aa46b`](https://github.com/eea/volto-tabs-block/commit/b2aa46be528028d53a0c199d6c7fbd1cb831aa63)]
+
+### [10.0.2](https://github.com/eea/volto-tabs-block/compare/10.0.1...10.0.2) - 21 April 2026
+
+#### :hammer_and_wrench: Others
+
+- add tests for cloneData [Miu Razvan - [`8db3bac`](https://github.com/eea/volto-tabs-block/commit/8db3bac901d0a98e72bcd401e811f5b67c6c5bff)]
+### [10.0.1](https://github.com/eea/volto-tabs-block/compare/10.0.0...10.0.1) - 9 April 2026
 
 ## [10.0.0](https://github.com/eea/volto-tabs-block/compare/9.1.0...10.0.0) - 27 March 2026
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eeacms/volto-tabs-block",
-  "version": "10.0.1",
+  "version": "10.0.3",
   "description": "volto-tabs-block: Volto add-on",
   "main": "src/index.js",
   "author": "European Environment Agency: IDM2 A-Team",

package/src/components/variations/default/Edit.jsx

@@ -268,7 +268,7 @@ const Edit = (props) => {
                     blocks: {
                       ...tabsData.blocks,
                       [activeTab]: {
-                        ...(newFormData.blocks_layout.items.length > 0
+                        ...(newFormData.blocks_layout?.items.length > 0
                           ? newFormData
                           : emptyTab({
                               schema: schema?.properties?.data?.schema || {},

package/src/index.js

@@ -21,6 +21,7 @@ import { TabsWidget } from './widgets';
 import tabsSVG from '@eeacms/volto-tabs-block//icons/tabs.svg';
 import rightSVG from '@plone/volto/icons/right-key.svg';
 import downSVG from '@plone/volto/icons/down-key.svg';
+import { cloneData } from './utils';
 
 defineMessages({
   default: {
@@ -55,6 +56,7 @@ const applyConfig = (config) => {
     view: TabsView,
     restricted: false,
     mostUsed: false,
+    cloneData,
     sidebarTab: 1,
     security: {
       addPermission: [],

package/src/utils/cloneData.js

@@ -0,0 +1,51 @@
+import { v4 as uuid } from 'uuid';
+import {
+  getBlocksFieldname,
+  getBlocksLayoutFieldname,
+  getBlocks,
+  hasBlocksData,
+} from '@plone/volto/helpers/Blocks/Blocks';
+import config from '@plone/volto/registry';
+
+/* Inline deep‑clone helpers – used only by the Tabs block */
+function deepCloneFormData(formData) {
+  const blocksField = getBlocksFieldname(formData);
+  const layoutField = getBlocksLayoutFieldname(formData);
+  const childBlocks = getBlocks(formData);
+
+  const cloned = childBlocks
+    .map(([_, child]) => {
+      const childCfg = config.blocks.blocksConfig[child['@type']] || {};
+      if (childCfg.cloneData) {
+        return childCfg.cloneData(child);
+      }
+      return hasBlocksData(child) ? deepCloneFormData(child) : [uuid(), child];
+    })
+    .filter(Boolean);
+
+  const newData = {
+    ...formData,
+    [blocksField]: {
+      ...Object.fromEntries(cloned.map(([newId, data]) => [newId, data])),
+    },
+    [layoutField]: {
+      items: cloned.map(([newId]) => newId),
+    },
+  };
+
+  // Return a fresh UUID for the *parent* block and its cloned content
+  return [uuid(), newData];
+}
+
+export function cloneData(blockData) {
+  const tabsData = blockData.data;
+  const cloneWithIds = deepCloneFormData(tabsData);
+  const [id, newBlockData] = cloneWithIds;
+  return [
+    id,
+    {
+      ...blockData,
+      data: newBlockData,
+    },
+  ];
+}

package/src/utils/cloneData.test.js

@@ -0,0 +1,113 @@
+import { v4 as uuid } from 'uuid';
+import { getBlocks } from '@plone/volto/helpers/Blocks/Blocks';
+import config from '@plone/volto/registry';
+import { cloneData } from './cloneData';
+
+jest.mock('@plone/volto/helpers/Blocks/Blocks', () => ({
+  getBlocks: jest.fn(),
+  getBlocksFieldname: jest.fn(() => 'blocks'),
+  getBlocksLayoutFieldname: jest.fn(() => 'blocks_layout'),
+  hasBlocksData: jest.fn((block) => !!block.blocks),
+}));
+
+describe('cloneData', () => {
+  it('should clone the blockData with calling cloneData directly', () => {
+    const blockId = uuid();
+    const mockBlockData = {
+      data: {
+        '@type': 'tabs',
+        blocks: {
+          [blockId]: {
+            '@type': 'test',
+          },
+        },
+        blocks_layout: {
+          items: [blockId],
+        },
+      },
+    };
+
+    getBlocks.mockReturnValue([[blockId, mockBlockData.data.blocks[blockId]]]);
+
+    const [id, clonedBlockData] = cloneData(mockBlockData);
+    const clonedBlockId = clonedBlockData.data.blocks_layout.items[0];
+    const clonedBlock = clonedBlockData.data.blocks[clonedBlockId];
+    expect(clonedBlock).toBeDefined();
+    expect(clonedBlockId).not.toEqual(blockId);
+    expect(clonedBlock).toEqual(mockBlockData.data.blocks[blockId]);
+    expect(id).not.toBeNull();
+  });
+
+  it('should clone the blockData with calling cloneData from block config', () => {
+    const blockId = uuid();
+    const mockBlockData = {
+      data: {
+        '@type': 'tabs',
+        blocks: {
+          [blockId]: {
+            '@type': 'test',
+            blocks: {},
+          },
+        },
+        blocks_layout: {
+          items: [blockId],
+        },
+      },
+    };
+
+    getBlocks.mockReturnValue([[blockId, mockBlockData.data.blocks[blockId]]]);
+
+    config.blocks.blocksConfig = {
+      test: {
+        cloneData: jest.fn(() => [
+          'test_uuid',
+          mockBlockData.data.blocks[blockId],
+        ]),
+      },
+    };
+    const [id, clonedBlockData] = cloneData(mockBlockData);
+    expect(id).not.toEqual('test_uuid');
+    expect(clonedBlockData.data.blocks['test_uuid']).toEqual(
+      mockBlockData.data.blocks[blockId],
+    );
+  });
+
+  it('should recursively clone nested blocks', () => {
+    const outerBlockId = uuid();
+    const innerBlockId = uuid();
+    const innerBlock = {
+      '@type': 'inner',
+      blocks: {
+        [innerBlockId]: { '@type': 'leaf' },
+      },
+      blocks_layout: { items: [innerBlockId] },
+    };
+    const mockBlockData = {
+      data: {
+        '@type': 'tabs',
+        blocks: {
+          [outerBlockId]: innerBlock,
+        },
+        blocks_layout: {
+          items: [outerBlockId],
+        },
+      },
+    };
+
+    getBlocks
+      .mockReturnValueOnce([[outerBlockId, innerBlock]])
+      .mockReturnValueOnce([[innerBlockId, innerBlock.blocks[innerBlockId]]]);
+
+    const [id, clonedBlockData] = cloneData(mockBlockData);
+    const newOuterId = clonedBlockData.data.blocks_layout.items[0];
+    expect(newOuterId).not.toEqual(outerBlockId);
+    expect(id).not.toBeNull();
+    const newOuterBlock = clonedBlockData.data.blocks[newOuterId];
+    expect(newOuterBlock).toBeDefined();
+    const newInnerId = newOuterBlock.blocks_layout.items[0];
+    expect(newInnerId).not.toEqual(innerBlockId);
+    expect(newOuterBlock.blocks[newInnerId]).toEqual(
+      innerBlock.blocks[innerBlockId],
+    );
+  });
+});

package/src/utils/index.js

@@ -44,3 +44,4 @@ const toggleItem = (container, item, hidden) => {
 
 export { SimpleMarkdown, getMenuPosition, toggleItem };
 export * from './dimensions';
+export { cloneData } from './cloneData';