refactor: metadata op api

frontend/src/metadata/hooks/metadata-view.js

@@ -4,9 +4,12 @@ import toaster from '../../components/toast';
 import Context from '../context';
 import Store from '../store';
 import { EVENT_BUS_TYPE, PER_LOAD_NUMBER } from '../constants';
-import { Utils } from '../../utils/utils';
+import { Utils, validateName } from '../../utils/utils';
 import { useMetadata } from './metadata';
 import { useCollaborators } from './collaborators';
+import { getRowById } from '../utils/table';
+import { getFileNameFromRecord, getParentDirFromRecord } from '../utils/cell';
+import { gettext } from '../../utils/constants';
 
 const MetadataViewContext = React.createContext(null);
 
@@ -14,6 +17,8 @@ export const MetadataViewProvider = ({
   children,
   repoID,
   viewID,
+  renameFileCallback,
+  deleteFilesCallback,
   ...params
 }) => {
   const [isLoading, setLoading] = useState(true);
@@ -48,32 +53,142 @@ export const MetadataViewProvider = ({
   }, []);
 
   const modifyFilters = useCallback((filters, filterConjunction, basicFilters) => {
-    window.sfMetadataStore.modifyFilters(filterConjunction, filters, basicFilters);
-  }, []);
+    storeRef.current.modifyFilters(filterConjunction, filters, basicFilters);
+  }, [storeRef]);
 
   const modifySorts = useCallback((sorts, displaySorts = false) => {
-    window.sfMetadataStore.modifySorts(sorts, displaySorts);
-  }, []);
+    storeRef.current.modifySorts(sorts, displaySorts);
+  }, [storeRef]);
 
   const modifyGroupbys = useCallback((groupbys) => {
-    window.sfMetadataStore.modifyGroupbys(groupbys);
-  }, []);
+    storeRef.current.modifyGroupbys(groupbys);
+  }, [storeRef]);
 
   const modifyHiddenColumns = useCallback((hiddenColumns) => {
-    window.sfMetadataStore.modifyHiddenColumns(hiddenColumns);
-  }, []);
-
-  const modifyColumnOrder = useCallback((sourceColumnKey, targetColumnKey) => {
-    window.sfMetadataStore.modifyColumnOrder(sourceColumnKey, targetColumnKey);
-  }, []);
+    storeRef.current.modifyHiddenColumns(hiddenColumns);
+  }, [storeRef]);
 
   const modifySettings = useCallback((settings) => {
-    window.sfMetadataStore.modifySettings(settings);
-  }, []);
+    storeRef.current.modifySettings(settings);
+  }, [storeRef]);
 
   const updateLocalRecord = useCallback((recordId, update) => {
-    window.sfMetadataStore.modifyLocalRecord(recordId, update);
-  }, []);
+    storeRef.current.modifyLocalRecord(recordId, update);
+  }, [storeRef]);
+
+  const modifyRecords = (rowIds, idRowUpdates, idOriginalRowUpdates, idOldRowData, idOriginalOldRowData, isCopyPaste = false, { success_callback, fail_callback } = {}) => {
+    const isRename = storeRef.current.checkIsRenameFileOperator(rowIds, idOriginalRowUpdates);
+    let newName = null;
+    if (isRename) {
+      const rowId = rowIds[0];
+      const row = getRowById(metadata, rowId);
+      const rowUpdates = idOriginalRowUpdates[rowId];
+      const { _parent_dir, _name } = row;
+      newName = getFileNameFromRecord(rowUpdates);
+      const { isValid, errMessage } = validateName(newName);
+      if (!isValid) {
+        toaster.danger(errMessage);
+        return;
+      }
+      if (newName === _name) {
+        return;
+      }
+      if (storeRef.current.checkDuplicatedName(newName, _parent_dir)) {
+        let errMessage = gettext('The name "{name}" is already taken. Please choose a different name.');
+        errMessage = errMessage.replace('{name}', Utils.HTMLescape(newName));
+        toaster.danger(errMessage);
+        return;
+      }
+    }
+    storeRef.current.modifyRecords(rowIds, idRowUpdates, idOriginalRowUpdates, idOldRowData, idOriginalOldRowData, isCopyPaste, isRename, {
+      fail_callback: (error) => {
+        fail_callback && fail_callback(error);
+        error && toaster.danger(error);
+      },
+      success_callback: (operation) => {
+        if (operation.is_rename) {
+          const rowId = operation.row_ids[0];
+          const row = getRowById(metadata, rowId);
+          const rowUpdates = operation.id_original_row_updates[rowId];
+          const oldRow = operation.id_original_old_row_data[rowId];
+          const parentDir = getParentDirFromRecord(row);
+          const oldName = getFileNameFromRecord(oldRow);
+          const path = Utils.joinPath(parentDir, oldName);
+          const newName = getFileNameFromRecord(rowUpdates);
+          renameFileCallback(path, newName);
+        }
+        success_callback && success_callback();
+      },
+    });
+  };
+
+  const deleteRecords = (recordsIds, { success_callback, fail_callback } = {}) => {
+    if (!Array.isArray(recordsIds) || recordsIds.length === 0) return;
+    let paths = [];
+    let fileNames = [];
+    recordsIds.forEach((recordId) => {
+      const record = getRowById(metadata, recordId);
+      const { _parent_dir, _name } = record || {};
+      if (_parent_dir && _name) {
+        const path = Utils.joinPath(_parent_dir, _name);
+        paths.push(path);
+        fileNames.push(_name);
+      }
+    });
+    storeRef.current.deleteRecords(recordsIds, {
+      fail_callback: (error) => {
+        fail_callback && fail_callback(error);
+        error && toaster.danger(error);
+      },
+      success_callback: () => {
+        deleteFilesCallback(paths, fileNames);
+        let msg = fileNames.length > 1
+          ? gettext('Successfully deleted {name} and {n} other items')
+          : gettext('Successfully deleted {name}');
+        msg = msg.replace('{name}', fileNames[0])
+          .replace('{n}', fileNames.length - 1);
+        toaster.success(msg);
+        success_callback && success_callback();
+      },
+    });
+  };
+
+  const modifyRecord = (rowId, updates, oldRowData, originalUpdates, originalOldRowData, isCopyPaste, { success_callback, fail_callback } = {}) => {
+    const rowIds = [rowId];
+    const idRowUpdates = { [rowId]: updates };
+    const idOriginalRowUpdates = { [rowId]: originalUpdates };
+    const idOldRowData = { [rowId]: oldRowData };
+    const idOriginalOldRowData = { [rowId]: originalOldRowData };
+    modifyRecords(rowIds, idRowUpdates, idOriginalRowUpdates, idOldRowData, idOriginalOldRowData, isCopyPaste, { success_callback, fail_callback });
+  };
+
+  const renameColumn = useCallback((columnKey, newName, oldName) => {
+    storeRef.current.renameColumn(columnKey, newName, oldName);
+  }, [storeRef]);
+
+  const deleteColumn = useCallback((columnKey, oldColumn) => {
+    storeRef.current.deleteColumn(columnKey, oldColumn);
+  }, [storeRef]);
+
+  const modifyColumnData = useCallback((columnKey, newData, oldData, { optionModifyType } = {}) => {
+    storeRef.current.modifyColumnData(columnKey, newData, oldData, { optionModifyType });
+  }, [storeRef]);
+
+  const modifyColumnWidth = useCallback((columnKey, newWidth) => {
+    storeRef.current.modifyColumnWidth(columnKey, newWidth);
+  }, [storeRef]);
+
+  const modifyColumnOrder = useCallback((sourceColumnKey, targetColumnKey) => {
+    storeRef.current.modifyColumnOrder(sourceColumnKey, targetColumnKey);
+  }, [storeRef]);
+
+  const insertColumn = useCallback((name, type, { key, data }) => {
+    storeRef.current.insertColumn(name, type, { key, data });
+  }, [storeRef]);
+
+  const updateFileTags = useCallback((data) => {
+    storeRef.current.updateFileTags(data);
+  }, [storeRef]);
 
   // init
   useEffect(() => {
@@ -137,11 +252,22 @@ export const MetadataViewProvider = ({
         metadata,
         store: storeRef.current,
         isDirentDetailShow: params.isDirentDetailShow,
-        deleteFilesCallback: params.deleteFilesCallback,
-        renameFileCallback: params.renameFileCallback,
         updateCurrentDirent: params.updateCurrentDirent,
         closeDirentDetail: params.closeDirentDetail,
         showDirentDetail: params.showDirentDetail,
+        deleteFilesCallback: deleteFilesCallback,
+        renameFileCallback: renameFileCallback,
+        modifySettings,
+        modifyRecords,
+        deleteRecords,
+        modifyRecord,
+        renameColumn,
+        deleteColumn,
+        modifyColumnOrder,
+        modifyColumnData,
+        modifyColumnWidth,
+        insertColumn,
+        updateFileTags,
       }}
     >
       {children}

frontend/src/metadata/views/face-recognition/person-photos/index.js

@@ -73,7 +73,7 @@ const PeoplePhotos = ({ view, people, onClose, onDeletePeoplePhotos }) => {
     onDeletePeoplePhotos && onDeletePeoplePhotos(people._id, ids);
   }, [metadata, onClose, people, onDeletePeoplePhotos]);
 
-  const handelDelete = useCallback((deletedImages, callback) => {
+  const handelDelete = useCallback((deletedImages, { success_callback } = {}) => {
     if (!deletedImages.length) return;
     let recordIds = [];
     let paths = [];
@@ -88,7 +88,6 @@ const PeoplePhotos = ({ view, people, onClose, onDeletePeoplePhotos }) => {
       }
     });
     window.sfMetadataContext.batchDeleteFiles(repoID, paths).then(res => {
-      callback && callback();
       deletedByIds(recordIds);
       deleteFilesCallback(paths, fileNames);
       let msg = fileNames.length > 1
@@ -97,6 +96,7 @@ const PeoplePhotos = ({ view, people, onClose, onDeletePeoplePhotos }) => {
       msg = msg.replace('{name}', fileNames[0])
         .replace('{n}', fileNames.length - 1);
       toaster.success(msg);
+      success_callback && success_callback();
     }).catch(error => {
       toaster.danger(gettext('Failed to delete records'));
     });

frontend/src/metadata/views/gallery/index.js

@@ -3,7 +3,6 @@ import toaster from '../../../components/toast';
 import Main from './main';
 import { useMetadataView } from '../../hooks/metadata-view';
 import { Utils } from '../../../utils/utils';
-import { gettext } from '../../../utils/constants';
 import { PER_LOAD_NUMBER } from '../../constants';
 
 import './index.css';
@@ -11,7 +10,7 @@ import './index.css';
 const Gallery = () => {
   const [isLoadingMore, setLoadingMore] = useState(false);
 
-  const { metadata, store, deleteFilesCallback } = useMetadataView();
+  const { metadata, store, deleteRecords } = useMetadataView();
 
   const onLoadMore = useCallback(async () => {
     if (isLoadingMore) return;
@@ -30,36 +29,17 @@ const Gallery = () => {
 
   }, [isLoadingMore, metadata, store]);
 
-  const handleDelete = useCallback((deletedImages, callback) => {
+  const handleDelete = useCallback((deletedImages, { success_callback } = {}) => {
     if (!deletedImages.length) return;
     let recordsIds = [];
-    let paths = [];
-    let fileNames = [];
     deletedImages.forEach((record) => {
       const { path: parentDir, name } = record || {};
       if (parentDir && name) {
-        const path = Utils.joinPath(parentDir, name);
         recordsIds.push(record.id);
-        paths.push(path);
-        fileNames.push(name);
       }
     });
-    store.deleteRecords(recordsIds, {
-      fail_callback: (error) => {
-        toaster.danger(error);
-      },
-      success_callback: () => {
-        callback && callback();
-        deleteFilesCallback(paths, fileNames);
-        let msg = fileNames.length > 1
-          ? gettext('Successfully deleted {name} and {n} other items')
-          : gettext('Successfully deleted {name}');
-        msg = msg.replace('{name}', fileNames[0])
-          .replace('{n}', fileNames.length - 1);
-        toaster.success(msg);
-      },
-    });
-  }, [store, deleteFilesCallback]);
+    deleteRecords(recordsIds, { success_callback });
+  }, [deleteRecords]);
 
   return (
     <div className="sf-metadata-container">

frontend/src/metadata/views/gallery/main.js

@@ -292,9 +292,11 @@ const Main = ({ isLoadingMore, metadata, onDelete, onLoadMore }) => {
 
   const handleDeleteSelectedImages = useCallback((selectedImages) => {
     if (!selectedImages.length) return;
-    onDelete(selectedImages, () => {
+    onDelete(selectedImages, {
+      success_callback: () => {
         updateCurrentDirent();
         setSelectedImages([]);
+      }
     });
   }, [onDelete, updateCurrentDirent]);
 

frontend/src/metadata/views/kanban/boards/index.js

@@ -17,6 +17,7 @@ import EmptyTip from '../../../../components/empty-tip';
 import Board from './board';
 import ImagePreviewer from '../../../components/cell-formatter/image-previewer';
 import ContextMenu from '../context-menu';
+import { getRowById } from '../../../utils/table';
 
 import './index.css';
 
@@ -192,9 +193,9 @@ const Boards = ({ modifyRecord, deleteRecords, modifyColumnData, onCloseSettings
     setImagePreviewerVisible(false);
   }, []);
 
-  const onSelectCard = useCallback((record) => {
+  const handelUpdateCurrentDirent = useCallback((record) => {
+    if (!record) return;
     const recordId = getRecordIdFromRecord(record);
-    if (selectedCard === recordId) return;
     const name = getFileNameFromRecord(record);
     const path = getParentDirFromRecord(record);
     const isDir = checkIsDir(record);
@@ -206,9 +207,15 @@ const Boards = ({ modifyRecord, deleteRecords, modifyColumnData, onCloseSettings
       file_tags: []
     });
     setSelectedCard(recordId);
+  }, [updateCurrentDirent]);
+
+  const onSelectCard = useCallback((record) => {
+    const recordId = getRecordIdFromRecord(record);
+    if (selectedCard === recordId) return;
+    handelUpdateCurrentDirent(record);
     onCloseSettings();
     showDirentDetail();
-  }, [selectedCard, onCloseSettings, showDirentDetail, updateCurrentDirent]);
+  }, [selectedCard, onCloseSettings, showDirentDetail, handelUpdateCurrentDirent]);
 
   const handleClickOutside = useCallback((event) => {
     if (isDragging) return;
@@ -222,8 +229,10 @@ const Boards = ({ modifyRecord, deleteRecords, modifyColumnData, onCloseSettings
 
   const onContextMenu = useCallback((event, recordId) => {
     event.preventDefault();
-    setSelectedCard(recordId);
-  }, []);
+    if (selectedCard === recordId) return;
+    const record = getRowById(metadata, recordId);
+    handelUpdateCurrentDirent(record);
+  }, [metadata, selectedCard, handelUpdateCurrentDirent]);
 
   const onDeleteRecords = useCallback((recordIds) => {
     deleteRecords(recordIds, {

frontend/src/metadata/views/kanban/index.js

@@ -1,112 +1,31 @@
 import React, { useState, useMemo, useEffect, useCallback } from 'react';
 import { useMetadataView } from '../../hooks/metadata-view';
 import { EVENT_BUS_TYPE } from '../../constants';
-import toaster from '../../../components/toast';
 import Boards from './boards';
 import Settings from './settings';
-import { getRowById } from '../../utils/table';
-import { getFileNameFromRecord, getParentDirFromRecord } from '../../utils/cell';
-import { Utils, validateName } from '../../../utils/utils';
-import { gettext } from '../../../utils/constants';
 
 import './index.css';
 
 const Kanban = () => {
   const [isShowSettings, setShowSettings] = useState(false);
 
-  const { metadata, store, renameFileCallback, deleteFilesCallback } = useMetadataView();
+  const {
+    metadata,
+    modifySettings,
+    modifyRecord: modifyRecordAPI,
+    deleteRecords,
+    modifyColumnData,
+  } = useMetadataView();
 
   const columns = useMemo(() => metadata.view.columns, [metadata.view.columns]);
 
-  const modifyRecord = useCallback((rowId, updates, oldRowData, originalUpdates, originalOldRowData, { success_callback }) => {
-    const rowIds = [rowId];
-    const idRowUpdates = { [rowId]: updates };
-    const idOriginalRowUpdates = { [rowId]: originalUpdates };
-    const idOldRowData = { [rowId]: oldRowData };
-    const idOriginalOldRowData = { [rowId]: originalOldRowData };
-    const isRename = store.checkIsRenameFileOperator(rowIds, idOriginalRowUpdates);
-    let newName = null;
-    if (isRename) {
-      const rowId = rowIds[0];
-      const row = getRowById(metadata, rowId);
-      const rowUpdates = idOriginalRowUpdates[rowId];
-      const { _parent_dir, _name } = row;
-      newName = getFileNameFromRecord(rowUpdates);
-      const { isValid, errMessage } = validateName(newName);
-      if (!isValid) {
-        toaster.danger(errMessage);
-        return;
-      }
-      if (newName === _name) {
-        return;
-      }
-      if (store.checkDuplicatedName(newName, _parent_dir)) {
-        let errMessage = gettext('The name "{name}" is already taken. Please choose a different name.');
-        errMessage = errMessage.replace('{name}', Utils.HTMLescape(newName));
-        toaster.danger(errMessage);
-        return;
-      }
-    }
-    store.modifyRecords(rowIds, idRowUpdates, idOriginalRowUpdates, idOldRowData, idOriginalOldRowData, false, isRename, {
-      fail_callback: (error) => {
-        error && toaster.danger(error);
-      },
-      success_callback: (operation) => {
-        if (operation.is_rename) {
-          const rowId = operation.row_ids[0];
-          const row = getRowById(metadata, rowId);
-          const rowUpdates = operation.id_original_row_updates[rowId];
-          const oldRow = operation.id_original_old_row_data[rowId];
-          const parentDir = getParentDirFromRecord(row);
-          const oldName = getFileNameFromRecord(oldRow);
-          const path = Utils.joinPath(parentDir, oldName);
-          const newName = getFileNameFromRecord(rowUpdates);
-          renameFileCallback(path, newName);
+  const modifyRecord = useCallback((rowId, updates, oldRowData, originalUpdates, originalOldRowData, { success_callback } = {}) => {
+    modifyRecordAPI(rowId, updates, oldRowData, originalUpdates, originalOldRowData, false, { success_callback: () => {
       success_callback && success_callback();
-        }
       const eventBus = window.sfMetadataContext.eventBus;
       eventBus.dispatch(EVENT_BUS_TYPE.LOCAL_RECORD_DETAIL_CHANGED, rowId, updates);
-      },
-    });
-  }, [store, metadata, renameFileCallback]);
-
-  const deleteRecords = useCallback((recordsIds, { success_callback }) => {
-    if (!Array.isArray(recordsIds) || recordsIds.length === 0) return;
-    let paths = [];
-    let fileNames = [];
-    recordsIds.forEach((recordId) => {
-      const record = getRowById(metadata, recordId);
-      const { _parent_dir, _name } = record || {};
-      if (_parent_dir && _name) {
-        const path = Utils.joinPath(_parent_dir, _name);
-        paths.push(path);
-        fileNames.push(_name);
-      }
-    });
-    store.deleteRecords(recordsIds, {
-      fail_callback: (error) => {
-        toaster.danger(error);
-      },
-      success_callback: () => {
-        deleteFilesCallback(paths, fileNames);
-        let msg = fileNames.length > 1
-          ? gettext('Successfully deleted {name} and {n} other items')
-          : gettext('Successfully deleted {name}');
-        msg = msg.replace('{name}', fileNames[0])
-          .replace('{n}', fileNames.length - 1);
-        toaster.success(msg);
-        success_callback && success_callback();
-      },
-    });
-  }, [metadata, store, deleteFilesCallback]);
-
-  const modifySettings = useCallback((newSettings) => {
-    store.modifySettings(newSettings);
-  }, [store]);
-
-  const modifyColumnData = useCallback((columnKey, newData, oldData, { optionModifyType } = {}) => {
-    store.modifyColumnData(columnKey, newData, oldData, { optionModifyType });
-  }, [store]);
+    } });
+  }, [modifyRecordAPI]);
 
   const closeSettings = useCallback(() => {
     setShowSettings(false);

frontend/src/metadata/views/table/index.js

@@ -3,10 +3,8 @@ import { toKeyCode } from 'is-hotkey';
 import toaster from '../../../components/toast';
 import TableMain from './table-main';
 import { useMetadataView } from '../../hooks/metadata-view';
-import { Utils, validateName } from '../../../utils/utils';
+import { Utils } from '../../../utils/utils';
 import { isModZ, isModShiftZ } from '../../utils/hotkey';
-import { gettext } from '../../../utils/constants';
-import { getFileNameFromRecord, getParentDirFromRecord } from '../../utils/cell';
 import { getValidGroupbys } from '../../utils/group';
 import { EVENT_BUS_TYPE, PER_LOAD_NUMBER, MAX_LOAD_NUMBER } from '../../constants';
 
@@ -14,7 +12,21 @@ import './index.css';
 
 const Table = () => {
   const [isLoadingMore, setLoadingMore] = useState(false);
-  const { isLoading, metadata, store, renameFileCallback, deleteFilesCallback } = useMetadataView();
+  const {
+    isLoading,
+    metadata,
+    store,
+    modifyRecords,
+    deleteRecords,
+    modifyRecord,
+    renameColumn,
+    deleteColumn,
+    modifyColumnData,
+    modifyColumnOrder,
+    modifyColumnWidth,
+    insertColumn,
+    updateFileTags
+  } = useMetadataView();
   const containerRef = useRef(null);
 
   const canModify = useMemo(() => window.sfMetadataContext.canModify(), []);
@@ -91,87 +103,6 @@ const Table = () => {
     }
   }, [metadata, store]);
 
-  const modifyRecords = (rowIds, idRowUpdates, idOriginalRowUpdates, idOldRowData, idOriginalOldRowData, isCopyPaste = false) => {
-    const isRename = store.checkIsRenameFileOperator(rowIds, idOriginalRowUpdates);
-    let newName = null;
-    if (isRename) {
-      const rowId = rowIds[0];
-      const row = recordGetterById(rowId);
-      const rowUpdates = idOriginalRowUpdates[rowId];
-      const { _parent_dir, _name } = row;
-      newName = getFileNameFromRecord(rowUpdates);
-      const { isValid, errMessage } = validateName(newName);
-      if (!isValid) {
-        toaster.danger(errMessage);
-        return;
-      }
-      if (newName === _name) {
-        return;
-      }
-      if (store.checkDuplicatedName(newName, _parent_dir)) {
-        let errMessage = gettext('The name "{name}" is already taken. Please choose a different name.');
-        errMessage = errMessage.replace('{name}', Utils.HTMLescape(newName));
-        toaster.danger(errMessage);
-        return;
-      }
-    }
-    store.modifyRecords(rowIds, idRowUpdates, idOriginalRowUpdates, idOldRowData, idOriginalOldRowData, isCopyPaste, isRename, {
-      fail_callback: (error) => {
-        error && toaster.danger(error);
-      },
-      success_callback: (operation) => {
-        if (operation.is_rename) {
-          const rowId = operation.row_ids[0];
-          const row = recordGetterById(rowId);
-          const rowUpdates = operation.id_original_row_updates[rowId];
-          const oldRow = operation.id_original_old_row_data[rowId];
-          const parentDir = getParentDirFromRecord(row);
-          const oldName = getFileNameFromRecord(oldRow);
-          const path = Utils.joinPath(parentDir, oldName);
-          const newName = getFileNameFromRecord(rowUpdates);
-          renameFileCallback(path, newName);
-        }
-      },
-    });
-  };
-
-  const deleteRecords = (recordsIds) => {
-    let paths = [];
-    let fileNames = [];
-    recordsIds.forEach((recordId) => {
-      const record = recordGetterById(recordId);
-      const { _parent_dir, _name } = record || {};
-      if (_parent_dir && _name) {
-        const path = Utils.joinPath(_parent_dir, _name);
-        paths.push(path);
-        fileNames.push(_name);
-      }
-    });
-    store.deleteRecords(recordsIds, {
-      fail_callback: (error) => {
-        toaster.danger(error);
-      },
-      success_callback: () => {
-        deleteFilesCallback(paths, fileNames);
-        let msg = fileNames.length > 1
-          ? gettext('Successfully deleted {name} and {n} other items')
-          : gettext('Successfully deleted {name}');
-        msg = msg.replace('{name}', fileNames[0])
-          .replace('{n}', fileNames.length - 1);
-        toaster.success(msg);
-      },
-    });
-  };
-
-  const modifyRecord = (rowId, updates, oldRowData, originalUpdates, originalOldRowData) => {
-    const rowIds = [rowId];
-    const idRowUpdates = { [rowId]: updates };
-    const idOriginalRowUpdates = { [rowId]: originalUpdates };
-    const idOldRowData = { [rowId]: oldRowData };
-    const idOriginalOldRowData = { [rowId]: originalOldRowData };
-    modifyRecords(rowIds, idRowUpdates, idOriginalRowUpdates, idOldRowData, idOriginalOldRowData);
-  };
-
   const getAdjacentRowsIds = useCallback((rowIds) => {
     const rowIdsLen = metadata.row_ids.length;
     let rowIdsInOrder = [];
@@ -192,34 +123,6 @@ const Table = () => {
     return { rowIdsInOrder, upperRowIds, belowRowIds };
   }, [metadata]);
 
-  const renameColumn = useCallback((columnKey, newName, oldName) => {
-    store.renameColumn(columnKey, newName, oldName);
-  }, [store]);
-
-  const deleteColumn = useCallback((columnKey, oldColumn) => {
-    store.deleteColumn(columnKey, oldColumn);
-  }, [store]);
-
-  const modifyColumnData = useCallback((columnKey, newData, oldData, { optionModifyType } = {}) => {
-    store.modifyColumnData(columnKey, newData, oldData, { optionModifyType });
-  }, [store]);
-
-  const modifyColumnWidth = useCallback((columnKey, newWidth) => {
-    store.modifyColumnWidth(columnKey, newWidth);
-  }, [store]);
-
-  const modifyColumnOrder = useCallback((sourceColumnKey, targetColumnKey) => {
-    store.modifyColumnOrder(sourceColumnKey, targetColumnKey);
-  }, [store]);
-
-  const updateFileTags = useCallback((data) => {
-    store.updateFileTags(data);
-  }, [store]);
-
-  const insertColumn = useCallback((name, type, { key, data }) => {
-    store.insertColumn(name, type, { key, data });
-  }, [store]);
-
   const recordGetterById = useCallback((recordId) => {
     return metadata.id_row_map[recordId];
   }, [metadata]);