fix(formula): fix formula update operation

dream-num · Nov 9, 2024 · 9169219 · 9169219
1 parent 0afcc5d
commit 9169219
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/packages/sheets-formula/src/controllers/utils/ref-range-formula.ts b/packages/sheets-formula/src/controllers/utils/ref-range-formula.ts
@@ -16,7 +16,7 @@
 
 import type { ICellData, IMutationInfo, IObjectMatrixPrimitiveType, IRange, Nullable } from '@univerjs/core';
 import type { ISetRangeValuesMutationParams } from '@univerjs/sheets';
-import { cellToRange, Direction, isFormulaId, isFormulaString, ObjectMatrix, Rectangle } from '@univerjs/core';
+import { cellToRange, Direction, isFormulaId, isFormulaString, ObjectMatrix, Rectangle, Tools } from '@univerjs/core';
 import { deserializeRangeWithSheetWithCache, type IFormulaData, type IFormulaDataItem, type IRangeChange, type ISequenceNode, sequenceNodeType, serializeRangeToRefString } from '@univerjs/engine-formula';
 import { EffectRefRangId, handleDeleteRangeMoveLeft, handleDeleteRangeMoveUp, handleInsertCol, handleInsertRangeMoveDown, handleInsertRangeMoveRight, handleInsertRow, handleIRemoveCol, handleIRemoveRow, handleMoveCols, handleMoveRange, handleMoveRows, runRefRangeMutations, SetRangeValuesMutation } from '@univerjs/sheets';
 import { checkFormulaDataNull } from './offset-formula-data';
@@ -151,6 +151,14 @@ export function getFormulaReferenceRange(oldFormulaData: IFormulaData,
     const { sheetId: subUnitId, unitId } = formulaReferenceMoveParam;
     const { redoFormulaData, undoFormulaData } = refRangeFormula(oldFormulaData, newFormulaData, formulaReferenceMoveParam);
 
+    // If the formula data is the same, no operation is required
+    if (Tools.diffValue(redoFormulaData, undoFormulaData)) {
+        return {
+            undos: [],
+            redos: [],
+        };
+    }
+
     const redos: IMutationInfo[] = [];
     const undos: IMutationInfo[] = [];