@leankylin-sheet/core 5.2.45 → 5.2.47

package/dist/api/range.d.ts

@@ -18,6 +18,15 @@ export declare function getSelectionCoordinates(ctx: Context): string[];
 export declare function getCellsByRange(ctx: Context, range: Selection, options?: CommonOptions): (Cell | null)[][];
 export declare function getHtmlByRange(ctx: Context, range: Range, options?: CommonOptions): string | null;
 export declare function setSelection(ctx: Context, range: Range, options: CommonOptions): void;
+export declare function registerExecFunctionExist(ctx: Context, sheet: {
+    id?: string;
+}, formulaCells: Array<{
+    r: number;
+    c: number;
+}>, modified: Array<{
+    r: number;
+    c: number;
+}>): void;
 export declare function setCellValuesByRange(ctx: Context, data: any[][], range: SingleRange, cellInput: HTMLDivElement | null, options?: CommonOptions, setMerge?: boolean): void;
 export declare function setCellValues(ctx: Context, data: any[], cellInput: HTMLDivElement | null): void;
 export declare function setCellFormatByRange(ctx: Context, attr: keyof Cell, value: any, range: Range | SingleRange, options?: CommonOptions): void;

package/dist/index.esm.js

@@ -42689,7 +42689,6 @@ function deepClear(ctx) {
       }
     }
   }
-  updateFunctionGroup(ctx);
   return "success";
 }
 function deleteSelectedCellText(ctx, deep) {
@@ -42746,9 +42745,6 @@ function deleteSelectedCellText(ctx, deep) {
         }
       }
     }
-    console.time('deleteSelectedCellText');
-    updateFunctionGroup(ctx);
-    console.timeEnd('deleteSelectedCellText');
   }
   return "success";
 }
@@ -56201,6 +56197,7 @@ function setCellValueInBatch(ctx, sheet, row, column, value, cellInput, formulaC
       data[row][column] = {};
     }
     var cell = data[row][column];
+    var oldF = cell.f;
     if (value.f != null && value.v == null) {
       curv.f = value.f;
       if (value.ct != null) {
@@ -56208,19 +56205,20 @@ function setCellValueInBatch(ctx, sheet, row, column, value, cellInput, formulaC
       }
       Object.assign(cell, curv);
       if (_.isString(value.f)) {
-        formulaCells.push({
-          r: row,
-          c: column,
-          f: value.f,
-          ct: value.ct
-        });
+        if (value.f !== oldF) {
+          formulaCells.push({
+            r: row,
+            c: column,
+            f: value.f,
+            ct: value.ct
+          });
+        }
       } else {
         updateCell(ctx, row, column, cellInput, curv);
       }
     } else {
       var needupdateFdev = false;
       var needUpdateM = false;
-      var oldF = cell.f;
       var nf = value.f;
       var oldV = cell.v;
       var nv = value.v;
@@ -56323,12 +56321,15 @@ function setCellValueInBatch(ctx, sheet, row, column, value, cellInput, formulaC
         data[row][column] = {};
       }
       var _cell = data[row][column];
+      var _oldF = _cell.f;
       _cell.f = text;
-      formulaCells.push({
-        r: row,
-        c: column,
-        f: text
-      });
+      if (_oldF !== text) {
+        formulaCells.push({
+          r: row,
+          c: column,
+          f: text
+        });
+      }
     } else if (text.substr(0, 5) === "<span") {
       updateCell(ctx, row, column, cellInput, value);
     } else {
@@ -56654,6 +56655,54 @@ function setSelection(ctx, range, options) {
     ctx.luckysheet_select_save = sheet.luckysheet_select_save;
   }
 }
+function registerExecFunctionExist(ctx, sheet, formulaCells, modified) {
+  var seen = new Set();
+  var exist = [];
+  var pushUnique = function pushUnique(r, c) {
+    var key = "".concat(r, "_").concat(c);
+    if (seen.has(key)) return;
+    seen.add(key);
+    exist.push({
+      r: r,
+      c: c,
+      i: sheet.id
+    });
+  };
+  for (var i = 0; i < formulaCells.length; i += 1) {
+    pushUnique(formulaCells[i].r, formulaCells[i].c);
+  }
+  for (var _i = 0; _i < modified.length; _i += 1) {
+    pushUnique(modified[_i].r, modified[_i].c);
+  }
+  ctx.formulaCache.execFunctionExist = exist;
+}
+function isCellValueUnchanged(sheet, r, c, v) {
+  var _sheet$data, _sheet$data$r;
+  if (_typeof(v) === 'object' && !('f' in v) && !('v' in v)) {
+    return true;
+  }
+  var oldCell = (_sheet$data = sheet.data) === null || _sheet$data === void 0 ? void 0 : (_sheet$data$r = _sheet$data[r]) === null || _sheet$data$r === void 0 ? void 0 : _sheet$data$r[c];
+  if (oldCell == null) return false;
+  if (v == null || v.toString().length === 0) return false;
+  var newF;
+  var newV;
+  if (v instanceof Object) {
+    newF = v.f;
+    newV = v.v;
+  } else if (typeof v === "string" && v[0] === "=") {
+    newF = v;
+  } else {
+    newV = v;
+  }
+  var oldF = oldCell.f;
+  if (newF != null || oldF != null) {
+    if (newF !== oldF) return false;
+    if (newV == null) return true;
+    return String(newV) === String(oldCell.v);
+  }
+  if (newV == null) return false;
+  return String(newV) === String(oldCell.v);
+}
 function setCellValuesByRange(ctx, data, range, cellInput) {
   var options = arguments.length > 4 && arguments[4] !== undefined ? arguments[4] : {};
   var setMerge = arguments.length > 5 && arguments[5] !== undefined ? arguments[5] : false;
@@ -56726,18 +56775,9 @@ function setCellValuesByRange(ctx, data, range, cellInput) {
     sheet.config = cfg;
   }
   if (modifiedCells.length > 0) {
-    ctx.formulaCache.execFunctionExist = formulaCells.map(function (fc) {
-      return {
-        r: fc.r,
-        c: fc.c,
-        i: sheet.id
-      };
-    });
-    for (var k = 0; k < modifiedCells.length; k += 1) {
-      ctx.formulaCache.execFunctionExist.push(modifiedCells[k]);
-    }
-    for (var _k = 0; _k < formulaCells.length; _k += 1) {
-      var fc = formulaCells[_k];
+    registerExecFunctionExist(ctx, sheet, formulaCells, modifiedCells);
+    for (var k = 0; k < formulaCells.length; k += 1) {
+      var fc = formulaCells[k];
       var v = execfunction(ctx, fc.f, fc.r, fc.c, sheet.id, false, true);
       setCellValue(ctx, fc.r, fc.c, flowdata, {
         v: v[1],
@@ -56785,27 +56825,21 @@ function setCellValues(ctx, data, cellInput) {
     throw SHEET_NOT_FOUND;
   }
   var formulaCells = [];
+  var changedData = [];
   for (var _j = 0; _j < data.length; _j += 1) {
     var _data$_j = data[_j],
       _r = _data$_j.r,
       _c = _data$_j.c,
       _v = _data$_j.v;
+    if (!isCellValueUnchanged(sheet, _r, _c, _v)) {
+      changedData.push(data[_j]);
+    }
     setCellValueInBatch(ctx, sheet, _r, _c, _v, cellInput, formulaCells);
   }
-  ctx.formulaCache.execFunctionExist = formulaCells.map(function (fc) {
-    return {
-      r: fc.r,
-      c: fc.c,
-      i: sheet.id
-    };
-  });
-  for (var _j2 = 0; _j2 < data.length; _j2 += 1) {
-    ctx.formulaCache.execFunctionExist.push({
-      r: data[_j2].r,
-      c: data[_j2].c,
-      i: sheet.id
-    });
+  if (formulaCells.length === 0 && changedData.length === 0) {
+    return;
   }
+  registerExecFunctionExist(ctx, sheet, formulaCells, changedData);
   for (var k = 0; k < formulaCells.length; k += 1) {
     var fc = formulaCells[k];
     var val = execfunction(ctx, fc.f, fc.r, fc.c, sheet.id, false, true);
@@ -56944,6 +56978,7 @@ var index = /*#__PURE__*/Object.freeze({
   getCellsByRange: getCellsByRange,
   getHtmlByRange: getHtmlByRange,
   setSelection: setSelection,
+  registerExecFunctionExist: registerExecFunctionExist,
   setCellValuesByRange: setCellValuesByRange,
   setCellValues: setCellValues,
   setCellFormatByRange: setCellFormatByRange,

package/dist/index.js

@@ -42699,7 +42699,6 @@ function deepClear(ctx) {
       }
     }
   }
-  updateFunctionGroup(ctx);
   return "success";
 }
 function deleteSelectedCellText(ctx, deep) {
@@ -42756,9 +42755,6 @@ function deleteSelectedCellText(ctx, deep) {
         }
       }
     }
-    console.time('deleteSelectedCellText');
-    updateFunctionGroup(ctx);
-    console.timeEnd('deleteSelectedCellText');
   }
   return "success";
 }
@@ -56211,6 +56207,7 @@ function setCellValueInBatch(ctx, sheet, row, column, value, cellInput, formulaC
       data[row][column] = {};
     }
     var cell = data[row][column];
+    var oldF = cell.f;
     if (value.f != null && value.v == null) {
       curv.f = value.f;
       if (value.ct != null) {
@@ -56218,19 +56215,20 @@ function setCellValueInBatch(ctx, sheet, row, column, value, cellInput, formulaC
       }
       Object.assign(cell, curv);
       if (___default['default'].isString(value.f)) {
-        formulaCells.push({
-          r: row,
-          c: column,
-          f: value.f,
-          ct: value.ct
-        });
+        if (value.f !== oldF) {
+          formulaCells.push({
+            r: row,
+            c: column,
+            f: value.f,
+            ct: value.ct
+          });
+        }
       } else {
         updateCell(ctx, row, column, cellInput, curv);
       }
     } else {
       var needupdateFdev = false;
       var needUpdateM = false;
-      var oldF = cell.f;
       var nf = value.f;
       var oldV = cell.v;
       var nv = value.v;
@@ -56333,12 +56331,15 @@ function setCellValueInBatch(ctx, sheet, row, column, value, cellInput, formulaC
         data[row][column] = {};
       }
       var _cell = data[row][column];
+      var _oldF = _cell.f;
       _cell.f = text;
-      formulaCells.push({
-        r: row,
-        c: column,
-        f: text
-      });
+      if (_oldF !== text) {
+        formulaCells.push({
+          r: row,
+          c: column,
+          f: text
+        });
+      }
     } else if (text.substr(0, 5) === "<span") {
       updateCell(ctx, row, column, cellInput, value);
     } else {
@@ -56664,6 +56665,54 @@ function setSelection(ctx, range, options) {
     ctx.luckysheet_select_save = sheet.luckysheet_select_save;
   }
 }
+function registerExecFunctionExist(ctx, sheet, formulaCells, modified) {
+  var seen = new Set();
+  var exist = [];
+  var pushUnique = function pushUnique(r, c) {
+    var key = "".concat(r, "_").concat(c);
+    if (seen.has(key)) return;
+    seen.add(key);
+    exist.push({
+      r: r,
+      c: c,
+      i: sheet.id
+    });
+  };
+  for (var i = 0; i < formulaCells.length; i += 1) {
+    pushUnique(formulaCells[i].r, formulaCells[i].c);
+  }
+  for (var _i = 0; _i < modified.length; _i += 1) {
+    pushUnique(modified[_i].r, modified[_i].c);
+  }
+  ctx.formulaCache.execFunctionExist = exist;
+}
+function isCellValueUnchanged(sheet, r, c, v) {
+  var _sheet$data, _sheet$data$r;
+  if (_typeof(v) === 'object' && !('f' in v) && !('v' in v)) {
+    return true;
+  }
+  var oldCell = (_sheet$data = sheet.data) === null || _sheet$data === void 0 ? void 0 : (_sheet$data$r = _sheet$data[r]) === null || _sheet$data$r === void 0 ? void 0 : _sheet$data$r[c];
+  if (oldCell == null) return false;
+  if (v == null || v.toString().length === 0) return false;
+  var newF;
+  var newV;
+  if (v instanceof Object) {
+    newF = v.f;
+    newV = v.v;
+  } else if (typeof v === "string" && v[0] === "=") {
+    newF = v;
+  } else {
+    newV = v;
+  }
+  var oldF = oldCell.f;
+  if (newF != null || oldF != null) {
+    if (newF !== oldF) return false;
+    if (newV == null) return true;
+    return String(newV) === String(oldCell.v);
+  }
+  if (newV == null) return false;
+  return String(newV) === String(oldCell.v);
+}
 function setCellValuesByRange(ctx, data, range, cellInput) {
   var options = arguments.length > 4 && arguments[4] !== undefined ? arguments[4] : {};
   var setMerge = arguments.length > 5 && arguments[5] !== undefined ? arguments[5] : false;
@@ -56736,18 +56785,9 @@ function setCellValuesByRange(ctx, data, range, cellInput) {
     sheet.config = cfg;
   }
   if (modifiedCells.length > 0) {
-    ctx.formulaCache.execFunctionExist = formulaCells.map(function (fc) {
-      return {
-        r: fc.r,
-        c: fc.c,
-        i: sheet.id
-      };
-    });
-    for (var k = 0; k < modifiedCells.length; k += 1) {
-      ctx.formulaCache.execFunctionExist.push(modifiedCells[k]);
-    }
-    for (var _k = 0; _k < formulaCells.length; _k += 1) {
-      var fc = formulaCells[_k];
+    registerExecFunctionExist(ctx, sheet, formulaCells, modifiedCells);
+    for (var k = 0; k < formulaCells.length; k += 1) {
+      var fc = formulaCells[k];
       var v = execfunction(ctx, fc.f, fc.r, fc.c, sheet.id, false, true);
       setCellValue(ctx, fc.r, fc.c, flowdata, {
         v: v[1],
@@ -56795,27 +56835,21 @@ function setCellValues(ctx, data, cellInput) {
     throw SHEET_NOT_FOUND;
   }
   var formulaCells = [];
+  var changedData = [];
   for (var _j = 0; _j < data.length; _j += 1) {
     var _data$_j = data[_j],
       _r = _data$_j.r,
       _c = _data$_j.c,
       _v = _data$_j.v;
+    if (!isCellValueUnchanged(sheet, _r, _c, _v)) {
+      changedData.push(data[_j]);
+    }
     setCellValueInBatch(ctx, sheet, _r, _c, _v, cellInput, formulaCells);
   }
-  ctx.formulaCache.execFunctionExist = formulaCells.map(function (fc) {
-    return {
-      r: fc.r,
-      c: fc.c,
-      i: sheet.id
-    };
-  });
-  for (var _j2 = 0; _j2 < data.length; _j2 += 1) {
-    ctx.formulaCache.execFunctionExist.push({
-      r: data[_j2].r,
-      c: data[_j2].c,
-      i: sheet.id
-    });
+  if (formulaCells.length === 0 && changedData.length === 0) {
+    return;
   }
+  registerExecFunctionExist(ctx, sheet, formulaCells, changedData);
   for (var k = 0; k < formulaCells.length; k += 1) {
     var fc = formulaCells[k];
     var val = execfunction(ctx, fc.f, fc.r, fc.c, sheet.id, false, true);
@@ -56954,6 +56988,7 @@ var index = /*#__PURE__*/Object.freeze({
   getCellsByRange: getCellsByRange,
   getHtmlByRange: getHtmlByRange,
   setSelection: setSelection,
+  registerExecFunctionExist: registerExecFunctionExist,
   setCellValuesByRange: setCellValuesByRange,
   setCellValues: setCellValues,
   setCellFormatByRange: setCellFormatByRange,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@leankylin-sheet/core",
-  "version": "5.2.45",
+  "version": "5.2.47",
   "main": "dist/index.js",
   "module": "dist/index.esm.js",
   "typings": "dist/index.d.ts",
@@ -13,7 +13,7 @@
     "build": "father-build"
   },
   "dependencies": {
-    "@leankylin-sheet/formula-parser": "5.2.45",
+    "@leankylin-sheet/formula-parser": "5.2.47",
     "dayjs": "^1.11.0",
     "immer": "^9.0.12",
     "lodash": "^4.17.21",