@abgov/jsonforms-components 2.6.8 → 2.6.9

package/index.esm.js

@@ -6617,7 +6617,7 @@ const CheckboxGroup = props => {
           } else {
             newData = newData.filter(item => item !== enumValue);
           }
-          handleChange(path, newData);
+          handleChange(path, newData.length === 0 ? undefined : newData);
         }
       }));
     })
@@ -9975,8 +9975,8 @@ const ObjectArrayList = ({
   enabled,
   cells,
   translations,
-  currentTab,
-  setCurrentTab
+  currentIndex,
+  setCurrentIndex
 }) => {
   const isEmptyList = data === 0;
   const rightRef = useRef(null);
@@ -10005,7 +10005,7 @@ const ObjectArrayList = ({
   }
   const appliedUiSchemaOptions = merge({}, config, uischema.options);
   const selectCurrentTab = index => {
-    setCurrentTab(index);
+    setCurrentIndex(index);
   };
   const paddedHeight = rightHeight && rightHeight + 48;
   return jsx(ListContainer, {
@@ -10020,7 +10020,7 @@ const ObjectArrayList = ({
           return jsx(LeftTab, {
             childPath: childPath,
             rowIndex: index,
-            currentTab: currentTab,
+            currentTab: currentIndex,
             name: name,
             openDeleteDialog: openDeleteDialog,
             selectCurrentTab: selectCurrentTab,
@@ -10031,8 +10031,8 @@ const ObjectArrayList = ({
       }), jsx(FlexForm, {
         ref: rightRef,
         children: jsx(NonEmptyList, {
-          childPath: Paths.compose(path, `${currentTab}`),
-          rowIndex: currentTab,
+          childPath: Paths.compose(path, `${currentIndex}`),
+          rowIndex: currentIndex,
           schema: schema,
           openDeleteDialog: openDeleteDialog,
           showSortButtons: appliedUiSchemaOptions.showSortButtons || appliedUiSchemaOptions.showArrayTableSortButtons,
@@ -10041,7 +10041,7 @@ const ObjectArrayList = ({
           path: path,
           uischema: uischema,
           translations: translations
-        }, Paths.compose(path, `${currentTab}`))
+        }, Paths.compose(path, `${currentIndex}`))
       })]
     })
   });
@@ -10052,16 +10052,19 @@ class ListWithDetailControl extends React.Component {
     super(...arguments);
     // eslint-disable-next-line
     this.addItem = (path, value) => {
-      const pathIdValue = (path === null || path === void 0 ? void 0 : path.split('.')) || '';
-      if (pathIdValue.length > 1 && +this.props.data >= 0 || pathIdValue.length === 1) {
-        this.props.addItem(path, value)();
-        this.setState({
-          currentTab: this.props.data + 1
-        });
+      const {
+        data,
+        addItem,
+        setCurrentTab
+      } = this.props;
+      const isNonEmpty = data !== undefined && data !== null;
+      const newIndex = isNonEmpty ? data !== null && data !== void 0 ? data : 0 : 0;
+      if (addItem) {
+        addItem(path, value)();
+      }
+      if (typeof setCurrentTab === 'function') {
+        setCurrentTab(newIndex);
       }
-    };
-    this.state = {
-      currentTab: 0
     };
   }
   render() {
@@ -10119,10 +10122,8 @@ class ListWithDetailControl extends React.Component {
           data: data,
           cells: cells,
           config: config,
-          currentTab: this.state.currentTab,
-          setCurrentTab: i => this.setState({
-            currentTab: i
-          })
+          currentIndex: this.props.currentTab,
+          setCurrentIndex: this.props.setCurrentTab
         }, additionalProps))
       })]
     });
@@ -10139,6 +10140,7 @@ const ListWithDetailsControl = props => {
     visible,
     handleChange
   } = props;
+  const [currentTab, setCurrentTab] = useState(0);
   const openDeleteDialog = useCallback((p, rowIndex, name) => {
     setOpen(true);
     setPath(p);
@@ -10152,18 +10154,22 @@ const ListWithDetailsControl = props => {
     if (removeItems && p) {
       if (props.data === 1) {
         handleChange(p, null);
+        setCurrentTab(0);
       } else {
         removeItems(p, [rowData])();
+        setCurrentTab(prev => Math.max(0, rowData - 1)); // Safe fallback
       }
     }
     setOpen(false);
     // eslint-disable-next-line
-  }, [setOpen, path, rowData]);
+  }, [setOpen, path, rowData, rowData]);
   return jsxs(Visible, {
     visible: visible,
     children: [jsx(ListWithDetailControl, Object.assign({}, props, {
       openDeleteDialog: openDeleteDialog,
-      enabled: true
+      enabled: true,
+      currentTab: currentTab,
+      setCurrentTab: setCurrentTab
     })), jsx(DeleteDialog, {
       open: open,
       onCancel: deleteCancel,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abgov/jsonforms-components",
-  "version": "2.6.8",
+  "version": "2.6.9",
   "license": "Apache-2.0",
   "description": "Government of Alberta - React renderers for JSON Forms based on the design system.",
   "repository": "https://github.com/GovAlta/adsp-monorepo",

package/src/lib/Controls/ObjectArray/ListWithDetailControl.d.ts

@@ -32,11 +32,12 @@ interface NonEmptyRowProps {
     uischema: ControlElement;
 }
 export declare const NonEmptyList: React.MemoExoticComponent<({ childPath, schema, enabled, cells, uischema, }: NonEmptyRowProps & WithDeleteDialogSupport) => import("react/jsx-runtime").JSX.Element>;
-export declare class ListWithDetailControl extends React.Component<ObjectArrayControlProps, any> {
+interface ListWithDetailControlProps extends ObjectArrayControlProps {
+    currentTab: number;
+    setCurrentTab: (index: number) => void;
+}
+export declare class ListWithDetailControl extends React.Component<ListWithDetailControlProps, any> {
     addItem: (path: string, value: any) => void;
-    state: {
-        currentTab: number;
-    };
     render(): import("react/jsx-runtime").JSX.Element;
 }
 export {};