@syncfusion/ej2-spreadsheet - npm Package Compare versions

Comparing version 24.2.5 to 24.2.7

dist/global/index.d.ts

 /*!
 *  filename: index.d.ts
-*  version : 24.2.5
+*  version : 24.2.7
 *  Copyright Syncfusion Inc. 2001 - 2023. All rights reserved.
@@ -5,0 +5,0 @@ *  Use of this code is subject to the terms of our license.

package.json

 {
   "_from": "@syncfusion/ej2-spreadsheet@*",
-  "_id": "@syncfusion/ej2-spreadsheet@24.2.4",
+  "_id": "@syncfusion/ej2-spreadsheet@24.2.5",
   "_inBundle": false,
-  "_integrity": "sha512-ZrsgHMN0CzpFEjfb+yGudVvUH7SL25uJqqoQNSUEbXbmeY2nlpoxW76FncHKfnqChi0LhvFTq+t0lNeO2lBnag==",
+  "_integrity": "sha512-zmIbuMFVQshmvqHZiGh5moh1Lnfco+nc+Bo/8Nr9FWhoNEvuTu+v5O0B8d6Nq7JWMyUBoZUbSCqL/FQVjPzkqw==",
   "_location": "/@syncfusion/ej2-spreadsheet",
@@ -26,4 +26,4 @@ "_phantomChildren": {},
   ],
-  "_resolved": "https://nexus.syncfusioninternal.com/repository/ej2-hotfix-new/@syncfusion/ej2-spreadsheet/-/ej2-spreadsheet-24.2.4.tgz",
-  "_shasum": "fcb5689095f81ac70cb5496219a5fc90e326bee6",
+  "_resolved": "https://nexus.syncfusioninternal.com/repository/ej2-hotfix-new/@syncfusion/ej2-spreadsheet/-/ej2-spreadsheet-24.2.5.tgz",
+  "_shasum": "e41cf4a4e8fa48b0acacd16f312ddcd362431392",
   "_spec": "@syncfusion/ej2-spreadsheet@*",
@@ -36,6 +36,6 @@ "_where": "/jenkins/workspace/elease-automation_release_24.1.1/packages/included",
   "dependencies": {
-    "@syncfusion/ej2-base": "~24.2.5",
-    "@syncfusion/ej2-charts": "~24.2.5",
-    "@syncfusion/ej2-dropdowns": "~24.2.5",
-    "@syncfusion/ej2-grids": "~24.2.5",
+    "@syncfusion/ej2-base": "~24.2.7",
+    "@syncfusion/ej2-charts": "~24.2.7",
+    "@syncfusion/ej2-dropdowns": "~24.2.7",
+    "@syncfusion/ej2-grids": "~24.2.7",
     "@syncfusion/ej2-navigations": "~24.2.4"
@@ -84,5 +84,5 @@ },
   "typings": "index.d.ts",
-  "version": "24.2.5",
+  "version": "24.2.7",
   "sideEffects": true,
   "homepage": "https://www.syncfusion.com/javascript-ui-controls"
 }

src/spreadsheet/actions/show-hide.d.ts

@@ -23,2 +23,3 @@ import { Spreadsheet } from '../base/index';
     private refreshChart;
+    private refreshChartCellModel;
     private addEventListener;
@@ -25,0 +26,0 @@ private destroy;

src/spreadsheet/actions/show-hide.js

@@ -1,2 +0,2 @@
-import { spreadsheetDestroyed } from '../common/index';
+import { spreadsheetDestroyed, getChartsIndexes, refreshChartCellModel } from '../common/index';
 import { autoFit, virtualContentLoaded, completeAction, focus } from '../common/index';
@@ -132,2 +132,4 @@ import { hiddenMerge, updateTableWidth, updateTranslate } from '../common/index';
             var updateBtmIdx = isFinite && args.endIndex === skipHiddenIdx(sheet, sheet.rowCount - 1, false);
+            var prevChartIndexes = [];
+            var currentChartIndexes = [];
             for (var i = args.startIndex; i <= args.endIndex; i++) {
@@ -153,3 +155,9 @@ if (isHiddenRow(sheet, i)) {
                 }
+                if (!args.isFiltering) {
+                    prevChartIndexes = getChartsIndexes(this.parent);
+                }
                 setRow(sheet, i, model);
+                if (!args.isFiltering) {
+                    currentChartIndexes = getChartsIndexes(this.parent);
+                }
                 if (sheetIndex !== this.parent.activeSheetIndex) {
@@ -162,2 +170,5 @@ continue;
                 this.refreshChart(i, 'rows');
+                if (!args.isFiltering) {
+                    this.refreshChartCellModel(prevChartIndexes, currentChartIndexes);
+                }
                 row = content && content.rows[idx];
@@ -339,2 +350,3 @@ if (row) {
             }
+            var prevChartIndexes = getChartsIndexes(this.parent);
             for (var i = args.startIndex, len = args.endIndex; i <= len; i++) {
@@ -428,2 +440,4 @@ if (!isHiddenRow(sheet, i)) {
             }
+            var currentChartIndexes = getChartsIndexes(this.parent);
+            this.refreshChartCellModel(prevChartIndexes, currentChartIndexes);
             if (idx === undefined) {
@@ -543,4 +557,7 @@ return;
             }
+            var prevChartIndexes = getChartsIndexes(this.parent);
             setColumn(sheet, i, { hidden: args.hide });
+            var currentChartIndexes = getChartsIndexes(this.parent);
             this.refreshChart(i, 'columns');
+            this.refreshChartCellModel(prevChartIndexes, currentChartIndexes);
             if (this.parent.scrollSettings.enableVirtualization && !args.freezePane && (i < viewportLeftIdx ||
@@ -886,2 +903,5 @@ i > this.parent.viewport.rightIndex)) {
     };
+    ShowHide.prototype.refreshChartCellModel = function (prevChartIndexes, currentChartIndexes) {
+        this.parent.notify(refreshChartCellModel, { prevChartIndexes: prevChartIndexes, currentChartIndexes: currentChartIndexes });
+    };
     ShowHide.prototype.addEventListener = function () {
@@ -888,0 +908,0 @@ this.parent.on(hideShow, this.hideShow, this);

src/spreadsheet/common/event.d.ts

@@ -261,2 +261,4 @@ /**
 /** @hidden */
+export declare const refreshChartCellModel: string;
+/** @hidden */
 export declare const refreshImagePosition: string;
@@ -263,0 +265,0 @@ /** @hidden */

src/spreadsheet/common/event.js

@@ -261,2 +261,4 @@ /**
 /** @hidden */
+export var refreshChartCellModel = 'refreshChartCellModel';
+/** @hidden */
 export var refreshImagePosition = 'refreshImagePosition';
@@ -263,0 +265,0 @@ /** @hidden */

src/spreadsheet/common/util.d.ts

 import { StyleType, CollaborativeEditArgs, IAriaOptions } from './index';
 import { Spreadsheet } from '../index';
 import { SheetModel, CellStyleModel } from '../../workbook/index';
-import { Workbook } from '../../workbook/index';
+import { Workbook, ChartModel } from '../../workbook/index';
 import { ActionEventArgs } from '../../workbook/index';
@@ -441,1 +441,13 @@ /**
 export declare function getSheetProperties(context: Spreadsheet, keys?: string[]): string;
+/**
+ * Returns the row indexes and column indexes of the charts in the active sheet
+ *
+ * @param {Spreadsheet} context - Specifies the Spreadsheet instance.
+ * @returns { {chart: ChartModel, chartRowIdx: number, chartColIdx: number}[] } - Returns the row indexes and column indexes of the charts in the active sheet
+ * @hidden
+ */
+export declare function getChartsIndexes(context?: Spreadsheet): {
+    chart: ChartModel;
+    chartRowIdx: number;
+    chartColIdx: number;
+}[];

src/spreadsheet/integrations/chart.d.ts

@@ -26,2 +26,3 @@ /**
     private updateChartHandler;
+    private refreshChartCellModel;
     private refreshChartCellObj;
@@ -28,0 +29,0 @@ private processChartRange;

src/workbook/actions/find-and-replace.js

@@ -536,2 +536,5 @@ var __assign = (this && this.__assign) || function () {
                     startSheet++;
+                    if (startSheet > sheetLength - 1) {
+                        startSheet = 0;
+                    }
                     if (initialSheet === startSheet) {
@@ -543,5 +546,2 @@ if (count === 0) {
                     }
-                    if (startSheet > sheetLength - 1) {
-                        startSheet = 0;
-                    }
                     sheet = this.parent.sheets[startSheet];
@@ -548,0 +548,0 @@ if (sheet) {

src/workbook/integrations/formula.js

@@ -353,13 +353,37 @@ import { getSheetName, getSheet, getSheetIndexFromId } from '../index';
         var regx = new regExp(pName.replace(escapeRegx, '\\$&') + exp, 'gi');
+        var renameValidationSheetRef = function (validation) {
+            if (checkIsFormula(validation.value1) && validation.value1.toUpperCase().includes(uPName) && validation.value1.match(regx)) {
+                validation.value1 = validation.value1.replace(regx, name);
+            }
+            if (checkIsFormula(validation.value2) && validation.value2.toUpperCase().includes(uPName) && validation.value2.match(regx)) {
+                validation.value2 = validation.value2.replace(regx, name);
+            }
+        };
         this.sheetInfo.forEach(function (info, index) {
             sheet = getSheet(_this.parent, index);
-            for (var i = 0, rowLen = sheet.usedRange.rowIndex; i <= rowLen; i++) {
-                for (var j = 0, colLen = sheet.usedRange.colIndex; j <= colLen; j++) {
-                    cell = getCell(i, j, sheet, false, true);
-                    if (cell.formula && checkIsFormula(cell.formula) && cell.formula.toUpperCase().includes(uPName) &&
-                        cell.formula.match(regx)) {
-                        cell.formula = cell.formula.replace(regx, name);
+            if (sheet && sheet.rows && sheet.rows.length) {
+                for (var i = 0, rowLen = sheet.rows.length; i < rowLen; i++) {
+                    if (sheet.rows[i] && sheet.rows[i].cells) {
+                        for (var j = 0, cellsLen = sheet.rows[i].cells.length; j < cellsLen; j++) {
+                            cell = getCell(i, j, sheet, false, true);
+                            if (cell.formula && checkIsFormula(cell.formula) && cell.formula.toUpperCase().includes(uPName) &&
+                                cell.formula.match(regx)) {
+                                cell.formula = cell.formula.replace(regx, name);
+                            }
+                            if (cell.validation) {
+                                renameValidationSheetRef(cell.validation);
+                            }
+                        }
                     }
                 }
             }
+            if (sheet && sheet.columns && sheet.columns.length) {
+                var column = void 0;
+                for (var i = 0, colsLen = sheet.columns.length; i < colsLen; i++) {
+                    column = sheet.columns[i];
+                    if (column && column.validation) {
+                        renameValidationSheetRef(column.validation);
+                    }
+                }
+            }
             if (info.visibleName === pName) {
@@ -366,0 +390,0 @@ info.visibleName = name;

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

45312110

increased by0.15%

Lines of code

376878

increased by0.18%

Dependency changes

• Updated@syncfusion/ej2-base@~24.2.7

• Updated@syncfusion/ej2-charts@~24.2.7

• Updated@syncfusion/ej2-dropdowns@~24.2.7

• Updated@syncfusion/ej2-grids@~24.2.7