@odoo/o-spreadsheet 17.0.2 → 17.0.4

package/dist/o-spreadsheet.esm.js

@@ -2,9 +2,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.0.2
- * @date 2023-11-03T12:22:44.945Z
- * @hash 3184728
+ * @version 17.0.4
+ * @date 2023-11-24T13:13:42.575Z
+ * @hash 7346d0a
  */
 
 import { Component, useRef, onMounted, useEffect, onWillPatch, useState, onWillUpdateProps, onPatched, useComponent, useExternalListener, onWillUnmount, onWillStart, xml, useChildSubEnv, useSubEnv, markRaw } from '@odoo/owl';
@@ -691,8 +691,7 @@ function removeFalsyAttributes(obj) {
  *
  * https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions/Character_Classes
  */
-const whiteSpaceCharacters = [
-    " ",
+const whiteSpaceSpecialCharacters = [
     "\t",
     "\f",
     "\v",
@@ -707,8 +706,7 @@ const whiteSpaceCharacters = [
     String.fromCharCode(parseInt("3000", 16)),
     String.fromCharCode(parseInt("feff", 16)),
 ];
-const whiteSpaceRegexp = new RegExp(whiteSpaceCharacters.join("|"), "g");
-const newLineRegex = /\r\n|\r|\n/g;
+const whiteSpaceRegexp = new RegExp(whiteSpaceSpecialCharacters.join("|") + "|(\r\n|\r|\n)", "g");
 /**
  * Replace all the special spaces in a string (non-breaking, tabs, ...) by normal spaces, and all the
  * different newlines types by \n.
@@ -716,9 +714,9 @@ const newLineRegex = /\r\n|\r|\n/g;
 function replaceSpecialSpaces(text) {
     if (!text)
         return "";
-    text = text.replace(whiteSpaceRegexp, " ");
-    text = text.replace(newLineRegex, NEWLINE);
-    return text;
+    if (!whiteSpaceRegexp.test(text))
+        return text;
+    return text.replace(whiteSpaceRegexp, (match, newLine) => (newLine ? NEWLINE : " "));
 }
 /**
  * Determine if the numbers are consecutive.
@@ -2060,6 +2058,7 @@ var CommandResult;
     CommandResult["InvalidNumberOfCriterionValues"] = "InvalidNumberOfCriterionValues";
     CommandResult["BlockingValidationRule"] = "BlockingValidationRule";
     CommandResult["InvalidCopyPasteSelection"] = "InvalidCopyPasteSelection";
+    CommandResult["NoChanges"] = "NoChanges";
 })(CommandResult || (CommandResult = {}));
 
 const DEFAULT_LOCALES = [
@@ -2189,12 +2188,12 @@ function toString(value) {
     }
 }
 /** Normalize string by setting it to lowercase and replacing accent letters with plain letters */
-function normalizeString(str) {
+const normalizeString = memoize(function normalizeString(str) {
     return str
         .toLowerCase()
         .normalize("NFD")
         .replace(/[\u0300-\u036f]/g, "");
-}
+});
 const expectBooleanValueError = (value) => _t("The function [[FUNCTION_NAME]] expects a boolean value, but '%s' is a text, and cannot be coerced to a number.", value);
 function toBoolean(value) {
     switch (typeof value) {
@@ -4540,14 +4539,14 @@ function drawDecoratedText(context, text, position, underline = false, strikethr
     switch (context.textBaseline) {
         case "top":
             underlineY += boxHeight - 2 * strokeWidth;
-            strikeY += boxHeight - textHeight;
+            strikeY += boxHeight / 2 - strokeWidth;
             break;
         case "middle":
             underlineY += boxHeight / 2 - strokeWidth;
             break;
         case "alphabetic":
             underlineY += 2 * strokeWidth;
-            strikeY -= textHeight / 2 - strokeWidth / 2;
+            strikeY -= 3 * strokeWidth;
             break;
         case "bottom":
             underlineY = y;
@@ -5966,7 +5965,7 @@ autofillModifiersRegistry
             return { cellData: {} };
         }
         const sheetId = data.sheetId;
-        const content = getters.getTranslatedCellFormula(sheetId, x, y, cell.compiledFormula, cell.dependencies);
+        const content = getters.getTranslatedCellFormula(sheetId, x, y, cell.compiledFormula);
         return {
             cellData: {
                 border: data.border,
@@ -7329,7 +7328,7 @@ function tokenize(str, locale = DEFAULT_LOCALE) {
     return result;
 }
 function tokenizeDebugger(chars) {
-    if (chars.current() === "?") {
+    if (chars.current === "?") {
         chars.shift();
         return { type: "DEBUGGER", value: "?" };
     }
@@ -7340,7 +7339,7 @@ const misc$1 = {
     ")": "RIGHT_PAREN",
 };
 function tokenizeMisc(chars) {
-    if (chars.current() in misc$1) {
+    if (chars.current in misc$1) {
         const value = chars.shift();
         const type = misc$1[value];
         return { type, value };
@@ -7348,7 +7347,7 @@ function tokenizeMisc(chars) {
     return null;
 }
 function tokenizeArgsSeparator(chars, locale) {
-    if (chars.current() === locale.formulaArgSeparator) {
+    if (chars.current === locale.formulaArgSeparator) {
         const value = chars.shift();
         const type = "ARG_SEPARATOR";
         return { type, value };
@@ -7373,14 +7372,13 @@ function tokenizeNumber(chars, locale) {
     return null;
 }
 function tokenizeString(chars) {
-    if (chars.current() === '"') {
+    if (chars.current === '"') {
         const startChar = chars.shift();
         let letters = startChar;
-        while (chars.current() &&
-            (chars.current() !== startChar || letters[letters.length - 1] === "\\")) {
+        while (chars.current && (chars.current !== startChar || letters[letters.length - 1] === "\\")) {
             letters += chars.shift();
         }
-        if (chars.current() === '"') {
+        if (chars.current === '"') {
             letters += chars.shift();
         }
         return {
@@ -7407,14 +7405,14 @@ function tokenizeSymbol(chars) {
     let result = "";
     // there are two main cases to manage: either something which starts with
     // a ', like 'Sheet 2'A2, or a word-like element.
-    if (chars.current() === "'") {
+    if (chars.current === "'") {
         let lastChar = chars.shift();
         result += lastChar;
-        while (chars.current()) {
+        while (chars.current) {
             lastChar = chars.shift();
             result += lastChar;
             if (lastChar === "'") {
-                if (chars.current() && chars.current() === "'") {
+                if (chars.current && chars.current === "'") {
                     lastChar = chars.shift();
                     result += lastChar;
                 }
@@ -7430,7 +7428,7 @@ function tokenizeSymbol(chars) {
             };
         }
     }
-    while (chars.current() && separatorRegexp.test(chars.current())) {
+    while (chars.current && separatorRegexp.test(chars.current)) {
         result += chars.shift();
     }
     if (result.length) {
@@ -7445,14 +7443,14 @@ function tokenizeSymbol(chars) {
 }
 function tokenizeSpace(chars) {
     let length = 0;
-    while (chars.current() === NEWLINE) {
+    while (chars.current === NEWLINE) {
         length++;
         chars.shift();
     }
     if (length) {
         return { type: "SPACE", value: NEWLINE.repeat(length) };
     }
-    while (chars.current() === " ") {
+    while (chars.current === " ") {
         length++;
         chars.shift();
     }
@@ -7471,17 +7469,20 @@ function tokenizeInvalidRange(chars) {
 class TokenizingChars {
     text;
     currentIndex = 0;
+    current;
     constructor(text) {
         this.text = text;
-    }
-    current() {
-        return this.text[this.currentIndex];
+        this.current = text[0];
     }
     shift() {
-        return this.text[this.currentIndex++];
+        const current = this.current;
+        const next = this.text[++this.currentIndex];
+        this.current = next;
+        return current;
     }
     advanceBy(length) {
         this.currentIndex += length;
+        this.current = this.text[this.currentIndex];
     }
     isOver() {
         return this.currentIndex >= this.text.length;
@@ -10693,6 +10694,17 @@ const SELECTION_CONTAINS_FILTER = (env) => {
 const IS_ONLY_ONE_RANGE = (env) => {
     return env.model.getters.getSelectedZones().length === 1;
 };
+const CAN_INSERT_HEADER = (env, dimension) => {
+    if (!IS_ONLY_ONE_RANGE(env)) {
+        return false;
+    }
+    const activeHeaders = dimension === "COL" ? env.model.getters.getActiveCols() : env.model.getters.getActiveRows();
+    const ortogonalActiveHeaders = dimension === "COL" ? env.model.getters.getActiveRows() : env.model.getters.getActiveCols();
+    const sheetId = env.model.getters.getActiveSheetId();
+    const zone = env.model.getters.getSelectedZone();
+    const allSheetSelected = isEqual(zone, env.model.getters.getSheetZone(sheetId));
+    return isConsecutive(activeHeaders) && (ortogonalActiveHeaders.size === 0 || allSheetSelected);
+};
 
 const undo = {
     name: _t("Undo"),
@@ -19370,9 +19382,7 @@ const insertRow = {
         const number = getRowsNumber(env);
         return number === 1 ? _t("Insert row") : _t("Insert %s rows", number.toString());
     },
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveRows()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveCols().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "ROW"),
     icon: "o-spreadsheet-Icon.INSERT_ROW",
 };
 const rowInsertRowBefore = {
@@ -19381,9 +19391,7 @@ const rowInsertRowBefore = {
         return number === 1 ? _t("Insert row above") : _t("Insert %s rows above", number.toString());
     },
     execute: INSERT_ROWS_BEFORE_ACTION,
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveRows()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveCols().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "ROW"),
     icon: "o-spreadsheet-Icon.INSERT_ROW_BEFORE",
 };
 const topBarInsertRowsBefore = {
@@ -19414,9 +19422,7 @@ const rowInsertRowsAfter = {
         const number = getRowsNumber(env);
         return number === 1 ? _t("Insert row below") : _t("Insert %s rows below", number.toString());
     },
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveRows()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveCols().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "ROW"),
     icon: "o-spreadsheet-Icon.INSERT_ROW_AFTER",
 };
 const topBarInsertRowsAfter = {
@@ -19434,9 +19440,7 @@ const insertCol = {
         const number = getColumnsNumber(env);
         return number === 1 ? _t("Insert column") : _t("Insert %s columns", number.toString());
     },
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveCols()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveRows().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "COL"),
     icon: "o-spreadsheet-Icon.INSERT_COL",
 };
 const colInsertColsBefore = {
@@ -19447,9 +19451,7 @@ const colInsertColsBefore = {
             : _t("Insert %s columns left", number.toString());
     },
     execute: INSERT_COLUMNS_BEFORE_ACTION,
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveCols()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveRows().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "COL"),
     icon: "o-spreadsheet-Icon.INSERT_COL_BEFORE",
 };
 const topBarInsertColsBefore = {
@@ -19482,9 +19484,7 @@ const colInsertColsAfter = {
             : _t("Insert %s columns right", number.toString());
     },
     execute: INSERT_COLUMNS_AFTER_ACTION,
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveCols()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveRows().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "COL"),
     icon: "o-spreadsheet-Icon.INSERT_COL_AFTER",
 };
 const topBarInsertColsAfter = {
@@ -23825,7 +23825,7 @@ css /* scss */ `
       }
       .o-input-count {
         width: fit-content;
-        padding: 4 0 4 4;
+        padding: 4px 0 4px 4px;
       }
     }
   }
@@ -29758,6 +29758,7 @@ class Grid extends Component {
             const content = clipboardData.getData(ClipboardMIMEType.PlainText);
             const target = this.env.model.getters.getSelectedZones();
             const clipboardString = this.env.model.getters.getClipboardTextContent();
+            const isCutOperation = this.env.model.getters.isCutOperation();
             if (clipboardString === content) {
                 // the paste actually comes from o-spreadsheet itself
                 interactivePaste(this.env, target);
@@ -29765,7 +29766,7 @@ class Grid extends Component {
             else {
                 interactivePasteFromOS(this.env, target, content);
             }
-            if (this.env.model.getters.isCutOperation()) {
+            if (isCutOperation) {
                 await this.env.clipboard.write({ [ClipboardMIMEType.PlainText]: "" });
             }
         }
@@ -34059,6 +34060,14 @@ class BordersPlugin extends CorePlugin {
     // ---------------------------------------------------------------------------
     // Command Handling
     // ---------------------------------------------------------------------------
+    allowDispatch(cmd) {
+        switch (cmd.type) {
+            case "SET_BORDER":
+                return this.checkBordersUnchanged(cmd);
+            default:
+                return "Success" /* CommandResult.Success */;
+        }
+    }
     handle(cmd) {
         switch (cmd.type) {
             case "ADD_MERGE":
@@ -34478,6 +34487,14 @@ class BordersPlugin extends CorePlugin {
             this.setBorders(sheetId, [{ ...zone, left: right }], "right", bordersTopLeft.right);
         }
     }
+    checkBordersUnchanged(cmd) {
+        const currentBorder = this.getCellBorder(cmd);
+        const areAllNewBordersUndefined = !cmd.border?.bottom && !cmd.border?.left && !cmd.border?.right && !cmd.border?.top;
+        if ((!currentBorder && areAllNewBordersUndefined) || deepEquals(currentBorder, cmd.border)) {
+            return "NoChanges" /* CommandResult.NoChanges */;
+        }
+        return "Success" /* CommandResult.Success */;
+    }
     // ---------------------------------------------------------------------------
     // Import/Export
     // ---------------------------------------------------------------------------
@@ -34946,6 +34963,9 @@ const functionCache = {};
 // -----------------------------------------------------------------------------
 function compile(formula) {
     const tokens = rangeTokenize(formula);
+    return compileTokens(tokens);
+}
+function compileTokens(tokens) {
     const { dependencies, constantValues } = formulaArguments(tokens);
     const cacheKey = compilationCacheKey(tokens, dependencies, constantValues);
     if (!functionCache[cacheKey]) {
@@ -35312,11 +35332,11 @@ class CellPlugin extends CorePlugin {
         for (const sheet of Object.keys(this.cells)) {
             for (const cell of Object.values(this.cells[sheet] || {})) {
                 if (cell.isFormula) {
-                    for (const range of cell.dependencies) {
+                    for (const range of cell.compiledFormula.dependencies) {
                         if (!sheetId || range.sheetId === sheetId) {
                             const change = applyChange(range);
                             if (change.changeType !== "NONE") {
-                                this.history.update("cells", sheet, cell.id, "dependencies", cell.dependencies.indexOf(range), change.range);
+                                this.history.update("cells", sheet, cell.id, "compiledFormula", "dependencies", cell.compiledFormula.dependencies.indexOf(range), change.range);
                             }
                         }
                     }
@@ -35330,8 +35350,9 @@ class CellPlugin extends CorePlugin {
     allowDispatch(cmd) {
         switch (cmd.type) {
             case "UPDATE_CELL":
+                return this.checkCellOutOfSheet(cmd);
             case "CLEAR_CELL":
-                return this.checkCellOutOfSheet(cmd.sheetId, cmd.col, cmd.row);
+                return this.checkValidations(cmd, this.chainValidations(this.checkCellOutOfSheet, this.checkUselessClearCell));
             default:
                 return "Success" /* CommandResult.Success */;
         }
@@ -35500,22 +35521,17 @@ class CellPlugin extends CorePlugin {
      */
     getCellById(cellId) {
         // this must be as fast as possible
-        for (const sheetId in this.cells) {
-            const sheet = this.cells[sheetId];
-            const cell = sheet[cellId];
-            if (cell) {
-                return cell;
-            }
-        }
-        return undefined;
+        const position = this.getters.getCellPosition(cellId);
+        const sheet = this.cells[position.sheetId];
+        return sheet[cellId];
     }
     /*
      * Reconstructs the original formula string based on a normalized form and its dependencies
      */
-    getFormulaCellContent(sheetId, cell, dependencies) {
-        const ranges = dependencies || cell.dependencies;
+    getFormulaCellContent(sheetId, compiledFormula, dependencies) {
+        const ranges = dependencies || compiledFormula.dependencies;
         let rangeIndex = 0;
-        return concat(cell.compiledFormula.tokens.map((token) => {
+        return concat(compiledFormula.tokens.map((token) => {
             if (token.type === "REFERENCE") {
                 const range = ranges[rangeIndex++];
                 return this.getters.getRangeString(range, sheetId);
@@ -35526,12 +35542,9 @@ class CellPlugin extends CorePlugin {
     /*
      * Constructs a formula string based on an initial formula and a translation vector
      */
-    getTranslatedCellFormula(sheetId, offsetX, offsetY, compiledFormula, dependencies) {
-        const adaptedDependencies = this.getters.createAdaptedRanges(dependencies, offsetX, offsetY, sheetId);
-        return this.getFormulaCellContent(sheetId, {
-            compiledFormula,
-            dependencies: adaptedDependencies,
-        });
+    getTranslatedCellFormula(sheetId, offsetX, offsetY, compiledFormula) {
+        const adaptedDependencies = this.getters.createAdaptedRanges(compiledFormula.dependencies, offsetX, offsetY, sheetId);
+        return this.getFormulaCellContent(sheetId, compiledFormula, adaptedDependencies);
     }
     getCellStyle(position) {
         return this.getters.getCell(position)?.style || {};
@@ -35711,8 +35724,10 @@ class CellPlugin extends CorePlugin {
             style,
             format,
             isFormula: true,
-            compiledFormula,
-            dependencies: [],
+            compiledFormula: {
+                ...compiledFormula,
+                dependencies: [],
+            },
         };
     }
     /**
@@ -35721,7 +35736,7 @@ class CellPlugin extends CorePlugin {
      */
     createFormulaCellWithDependencies(id, compiledFormula, format, style, sheetId) {
         const dependencies = compiledFormula.dependencies.map((xc) => this.getters.getRangeFromSheetXC(sheetId, xc));
-        return new FormulaCellWithDependencies(id, compiledFormula, format, style, dependencies, sheetId, this.getFormulaCellContent.bind(this));
+        return new FormulaCellWithDependencies(id, compiledFormula, format, style, dependencies, sheetId, this.getters.getRangeString.bind(this));
     }
     createErrorFormula(id, content, format, style, error) {
         return {
@@ -35731,46 +35746,72 @@ class CellPlugin extends CorePlugin {
             format,
             isFormula: true,
             compiledFormula: {
-                dependencies: [],
                 tokens: tokenize(content),
+                dependencies: [],
                 execute: function () {
                     throw error;
                 },
             },
-            dependencies: [],
         };
     }
-    checkCellOutOfSheet(sheetId, col, row) {
+    checkCellOutOfSheet(cmd) {
+        const { sheetId, col, row } = cmd;
         const sheet = this.getters.tryGetSheet(sheetId);
         if (!sheet)
             return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
         const sheetZone = this.getters.getSheetZone(sheetId);
         return isInside(col, row, sheetZone) ? "Success" /* CommandResult.Success */ : "TargetOutOfSheet" /* CommandResult.TargetOutOfSheet */;
     }
+    checkUselessClearCell(cmd) {
+        const cell = this.getters.getCell(cmd);
+        if (!cell)
+            return "NoChanges" /* CommandResult.NoChanges */;
+        if (!cell.content && !cell.style && !cell.format) {
+            return "NoChanges" /* CommandResult.NoChanges */;
+        }
+        return "Success" /* CommandResult.Success */;
+    }
 }
 class FormulaCellWithDependencies {
     id;
-    compiledFormula;
     format;
     style;
-    dependencies;
     sheetId;
-    getFormulaCellContent;
+    getRangeString;
     isFormula = true;
-    constructor(id, compiledFormula, format, style, dependencies, sheetId, getFormulaCellContent) {
+    compiledFormula;
+    constructor(id, compiledFormula, format, style, dependencies, sheetId, getRangeString) {
         this.id = id;
-        this.compiledFormula = compiledFormula;
         this.format = format;
         this.style = style;
-        this.dependencies = dependencies;
         this.sheetId = sheetId;
-        this.getFormulaCellContent = getFormulaCellContent;
+        this.getRangeString = getRangeString;
+        let rangeIndex = 0;
+        const tokens = compiledFormula.tokens.map((token) => {
+            if (token.type === "REFERENCE") {
+                const index = rangeIndex++;
+                return new RangeReferenceToken(() => this.getRangeString(dependencies[index], this.sheetId));
+            }
+            return token;
+        });
+        this.compiledFormula = {
+            ...compiledFormula,
+            dependencies,
+            tokens,
+        };
     }
     get content() {
-        return this.getFormulaCellContent(this.sheetId, {
-            dependencies: this.dependencies,
-            compiledFormula: this.compiledFormula,
-        });
+        return concat(this.compiledFormula.tokens.map((token) => token.value));
+    }
+}
+class RangeReferenceToken {
+    getValue;
+    type = "REFERENCE";
+    constructor(getValue) {
+        this.getValue = getValue;
+    }
+    get value() {
+        return this.getValue();
     }
 }
 
@@ -36419,6 +36460,9 @@ class DataValidationPlugin extends CorePlugin {
         return this.rules[sheetId].find((rule) => rule.id === id);
     }
     getValidationRuleForCell({ sheetId, col, row }) {
+        if (!this.rules[sheetId]) {
+            return undefined;
+        }
         for (const rule of this.rules[sheetId]) {
             for (const range of rule.ranges) {
                 if (isInside(col, row, range.zone)) {
@@ -36915,6 +36959,13 @@ class FiltersPlugin extends CorePlugin {
     }
     onDeleteColumnsRows(cmd) {
         for (const table of this.getFilterTables(cmd.sheetId)) {
+            // Remove the filter tables whose data filter headers are in the removed rows.
+            if (cmd.dimension === "ROW" && cmd.elements.includes(table.zone.top)) {
+                const tables = { ...this.tables[cmd.sheetId] };
+                delete tables[table.id];
+                this.history.update("tables", cmd.sheetId, tables);
+                continue;
+            }
             const zone = reduceZoneOnDeletion(table.zone, cmd.dimension === "COL" ? "left" : "top", cmd.elements);
             if (!zone) {
                 const tables = { ...this.tables[cmd.sheetId] };
@@ -39009,10 +39060,13 @@ class SheetPlugin extends CorePlugin {
         // begin with the end.
         rows.sort((a, b) => b - a);
         for (let group of groupConsecutive(rows)) {
+            // indexes are sorted in the descending order
+            const from = group[group.length - 1];
+            const to = group[0];
             // Move the cells.
-            this.moveCellOnRowsDeletion(sheet, group[group.length - 1], group[0]);
-            // Effectively delete the element and recompute the left-right/top-bottom.
-            group.map((row) => this.updateRowsStructureOnDeletion(row, sheet));
+            this.moveCellOnRowsDeletion(sheet, from, to);
+            // Effectively delete the rows
+            this.updateRowsStructureOnDeletion(sheet, from, to);
         }
         const count = rows.filter((row) => row < sheet.panes.ySplit).length;
         if (count) {
@@ -39034,8 +39088,6 @@ class SheetPlugin extends CorePlugin {
         this.addEmptyRows(sheet, quantity);
         // Move the cells.
         this.moveCellsOnAddition(sheet, index, quantity, "rows");
-        // Recompute the left-right/top-bottom.
-        this.updateRowsStructureOnAddition(sheet, row, quantity);
         if (index < sheet.panes.ySplit) {
             this.setPaneDivisions(sheet.id, sheet.panes.ySplit + quantity, "ROW");
         }
@@ -39136,35 +39188,20 @@ class SheetPlugin extends CorePlugin {
             }
         }
     }
-    updateRowsStructureOnDeletion(index, sheet) {
+    updateRowsStructureOnDeletion(sheet, deleteFromRow, deleteToRow) {
         const rows = [];
-        const cellsQueue = sheet.rows.map((row) => row.cells);
+        const cellsQueue = sheet.rows.map((row) => row.cells).reverse();
         for (let i in sheet.rows) {
-            if (Number(i) === index) {
+            const row = Number(i);
+            if (row >= deleteFromRow && row <= deleteToRow) {
                 continue;
             }
             rows.push({
-                cells: cellsQueue.shift(),
+                cells: cellsQueue.pop(),
             });
         }
         this.history.update("sheets", sheet.id, "rows", rows);
     }
-    /**
-     * Update the rows of the sheet after an addition:
-     * - Rename the rows
-     *
-     * @param sheet Sheet on which the deletion occurs
-     * @param addedRow Index of the added row
-     * @param rowsToAdd Number of the rows to add
-     */
-    updateRowsStructureOnAddition(sheet, addedRow, rowsToAdd) {
-        const rows = [];
-        const cellsQueue = sheet.rows.map((row) => row.cells);
-        sheet.rows.forEach(() => rows.push({
-            cells: cellsQueue.shift(),
-        }));
-        this.history.update("sheets", sheet.id, "rows", rows);
-    }
     /**
      * Add empty rows at the end of the rows
      *
@@ -39691,6 +39728,7 @@ class CompilationParametersBuilder {
     getters;
     computeCell;
     evalContext;
+    rangeCache = {};
     constructor(context, getters, computeCell) {
         this.getters = getters;
         this.computeCell = computeCell;
@@ -39776,17 +39814,24 @@ class CompilationParametersBuilder {
         if (!_zone) {
             return [[]];
         }
+        const { top, left, bottom, right } = zone;
+        const cacheKey = `${sheetId}-${top}-${left}-${bottom}-${right}`;
+        if (cacheKey in this.rangeCache) {
+            return this.rangeCache[cacheKey];
+        }
         const height = _zone.bottom - _zone.top + 1;
         const width = _zone.right - _zone.left + 1;
-        const matrix = Array.from({ length: width }, () => Array.from({ length: height }));
+        const matrix = new Array(width);
         // Performance issue: nested loop is faster than a map here
         for (let col = _zone.left; col <= _zone.right; col++) {
+            const colIndex = col - _zone.left;
+            matrix[colIndex] = new Array(height);
             for (let row = _zone.top; row <= _zone.bottom; row++) {
-                const colIndex = col - _zone.left;
                 const rowIndex = row - _zone.top;
                 matrix[colIndex][rowIndex] = this.readCell({ sheetId, col, row });
             }
         }
+        this.rangeCache[cacheKey] = matrix;
         return matrix;
     }
 }
@@ -39963,6 +40008,13 @@ class Evaluator {
         return (this.evaluatedCells.get(this.encodePosition(position)) ||
             createEvaluatedCell("", { locale: this.getters.getLocale() }));
     }
+    getSpreadPositionsOf(position) {
+        const positionId = this.encodePosition(position);
+        if (!this.spreadingRelations.isArrayFormula(positionId)) {
+            return [];
+        }
+        return Array.from(this.spreadingRelations.getArrayResultPositionIds(positionId)).map(this.decodePosition.bind(this));
+    }
     getArrayFormulaSpreadingOn(position) {
         const positionId = this.encodePosition(position);
         const formulaPosition = this.getArrayFormulaSpreadingOnId(positionId);
@@ -39985,6 +40037,7 @@ class Evaluator {
         this.formulaDependencies().addDependencies(positionId, dependencies);
     }
     updateCompilationParameters() {
+        // rebuild the compilation parameters (with a clean cache)
         this.compilationParams = buildCompilationParameters(this.context, this.getters, this.computeAndSave.bind(this));
     }
     evaluateCells(positions) {
@@ -40028,6 +40081,16 @@ class Evaluator {
         this.evaluatedCells = new Map();
         this.evaluate(this.getAllCells());
     }
+    evaluateFormula(sheetId, formulaString) {
+        const compiledFormula = compile(formulaString);
+        const ranges = compiledFormula.dependencies.map((xc) => this.getters.getRangeFromSheetXC(sheetId, xc));
+        this.updateCompilationParameters();
+        const array = compiledFormula.execute(ranges, ...this.compilationParams);
+        if (isMatrix(array)) {
+            return matrixMap(array, (cell) => cell.value);
+        }
+        return array.value;
+    }
     getAllCells() {
         const positionIds = new JetSet();
         for (const sheetId of this.getters.getSheetIds()) {
@@ -40054,6 +40117,7 @@ class Evaluator {
         this.nextPositionsToUpdate = cells;
         let currentIteration = 0;
         while (this.nextPositionsToUpdate.size && currentIteration++ < MAX_ITERATION) {
+            this.updateCompilationParameters();
             const positionIds = Array.from(this.nextPositionsToUpdate);
             this.nextPositionsToUpdate.clear();
             for (let i = 0; i < positionIds.length; ++i) {
@@ -40121,7 +40185,7 @@ class Evaluator {
             const position = this.compilationParams[2].getters.getCellPosition(cellId);
             return toXC(position.col, position.row);
         };
-        const formulaReturn = cellData.compiledFormula.execute(cellData.dependencies, ...this.compilationParams);
+        const formulaReturn = cellData.compiledFormula.execute(cellData.compiledFormula.dependencies, ...this.compilationParams);
         if (!isMatrix(formulaReturn)) {
             return createEvaluatedCell(formulaReturn.value, {
                 format: cellData.format || formulaReturn.format,
@@ -40221,7 +40285,7 @@ class Evaluator {
             return [];
         }
         const dependencies = [];
-        for (const range of cell.dependencies) {
+        for (const range of cell.compiledFormula.dependencies) {
             if (range.invalidSheetName || range.invalidXc) {
                 continue;
             }
@@ -40417,7 +40481,6 @@ class PositionBitsEncoder {
 // of other cells depending on it, at the next iteration.
 //#endregion
 class EvaluationPlugin extends UIPlugin {
-    config;
     static getters = [
         "evaluateFormula",
         "getCorrespondingFormulaCell",
@@ -40427,15 +40490,14 @@ class EvaluationPlugin extends UIPlugin {
         "getEvaluatedCell",
         "getEvaluatedCells",
         "getEvaluatedCellsInZone",
+        "getSpreadPositionsOf",
+        "getArrayFormulaSpreadingOn",
     ];
     shouldRebuildDependenciesGraph = true;
     evaluator;
-    compilationParams;
     positionsToUpdate = [];
     constructor(config) {
         super(config);
-        this.config = config;
-        this.compilationParams = this.getCompilationParameters();
         this.evaluator = new Evaluator(config.custom, this.getters);
     }
     // ---------------------------------------------------------------------------
@@ -40460,10 +40522,6 @@ class EvaluationPlugin extends UIPlugin {
             case "EVALUATE_CELLS":
                 this.evaluator.evaluateAllCells();
                 break;
-            case "UPDATE_LOCALE":
-                this.compilationParams = this.getCompilationParameters();
-                this.evaluator.updateCompilationParameters();
-                break;
         }
     }
     finalize() {
@@ -40481,16 +40539,7 @@ class EvaluationPlugin extends UIPlugin {
     // Getters
     // ---------------------------------------------------------------------------
     evaluateFormula(sheetId, formulaString) {
-        const compiledFormula = compile(formulaString);
-        const ranges = [];
-        for (let xc of compiledFormula.dependencies) {
-            ranges.push(this.getters.getRangeFromSheetXC(sheetId, xc));
-        }
-        const array = compiledFormula.execute(ranges, ...this.compilationParams);
-        if (isMatrix(array)) {
-            return array.map((col) => col.map((row) => row.value));
-        }
-        return array.value;
+        return this.evaluator.evaluateFormula(sheetId, formulaString);
     }
     /**
      * Return the value of each cell in the range as they are displayed in the grid.
@@ -40536,6 +40585,12 @@ class EvaluationPlugin extends UIPlugin {
     getEvaluatedCellsInZone(sheetId, zone) {
         return positions(zone).map(({ col, row }) => this.getters.getEvaluatedCell({ sheetId, col, row }));
     }
+    getSpreadPositionsOf(position) {
+        return this.evaluator.getSpreadPositionsOf(position);
+    }
+    getArrayFormulaSpreadingOn(position) {
+        return this.evaluator.getArrayFormulaSpreadingOn(position);
+    }
     // ---------------------------------------------------------------------------
     // Export
     // ---------------------------------------------------------------------------
@@ -40573,13 +40628,13 @@ class EvaluationPlugin extends UIPlugin {
      */
     getCorrespondingFormulaCell(position) {
         const cell = this.getters.getCell(position);
-        if (cell && cell.content) {
-            if (cell.isFormula && !isBadExpression(cell.content)) {
-                return cell;
-            }
+        if (cell && cell.isFormula) {
+            return isBadExpression(cell.compiledFormula.tokens) ? undefined : cell;
+        }
+        else if (cell && cell.content) {
             return undefined;
         }
-        const spreadingFormulaPosition = this.evaluator.getArrayFormulaSpreadingOn(position);
+        const spreadingFormulaPosition = this.getArrayFormulaSpreadingOn(position);
         if (spreadingFormulaPosition === undefined) {
             return undefined;
         }
@@ -40589,13 +40644,10 @@ class EvaluationPlugin extends UIPlugin {
         }
         return undefined;
     }
-    getCompilationParameters() {
-        return buildCompilationParameters(this.config.custom, this.getters, (position) => this.evaluator.getEvaluatedCell(position));
-    }
 }
-function isBadExpression(formula) {
+function isBadExpression(tokens) {
     try {
-        compile(formula);
+        compileTokens(tokens);
         return false;
     }
     catch (error) {
@@ -40764,22 +40816,20 @@ class EvaluationChartPlugin extends UIPlugin {
             invalidateCFEvaluationCommands.has(cmd.type) ||
             cmd.type === "EVALUATE_CELLS" ||
             cmd.type === "UPDATE_CELL") {
-            if (cmd.type !== "UNDO" && cmd.type !== "REDO") {
-                for (const chartId in this.charts) {
-                    this.history.update("charts", chartId, undefined);
-                }
+            for (const chartId in this.charts) {
+                this.charts[chartId] = undefined;
             }
         }
         switch (cmd.type) {
             case "UPDATE_CHART":
             case "CREATE_CHART":
             case "DELETE_FIGURE":
-                this.history.update("charts", cmd.id, undefined);
+                this.charts[cmd.id] = undefined;
                 break;
             case "DELETE_SHEET":
                 for (let chartId in this.charts) {
                     if (!this.getters.isChartDefined(chartId)) {
-                        this.history.update("charts", chartId, undefined);
+                        this.charts[chartId] = undefined;
                     }
                 }
                 break;
@@ -40791,8 +40841,7 @@ class EvaluationChartPlugin extends UIPlugin {
             if (!chart) {
                 throw new Error(`No chart for the given id: ${figureId}`);
             }
-            const runtime = this.createRuntimeChart(chart);
-            this.history.update("charts", figureId, runtime);
+            this.charts[figureId] = this.createRuntimeChart(chart);
         }
         return this.charts[figureId];
     }
@@ -40944,7 +40993,10 @@ class EvaluationConditionalFormatPlugin extends UIPlugin {
                                     const values = cf.rule.values.map((value, i) => {
                                         const compiledFormula = formulas[i];
                                         if (compiledFormula) {
-                                            return this.getters.getTranslatedCellFormula(sheetId, col - zone.left, row - zone.top, compiledFormula, compiledFormula.dependencies.map((d) => this.getters.getRangeFromSheetXC(sheetId, d)));
+                                            return this.getters.getTranslatedCellFormula(sheetId, col - zone.left, row - zone.top, {
+                                                ...compiledFormula,
+                                                dependencies: compiledFormula.dependencies.map((d) => this.getters.getRangeFromSheetXC(sheetId, d)),
+                                            });
                                         }
                                         return value;
                                     });
@@ -41316,7 +41368,10 @@ class EvaluationDataValidationPlugin extends UIPlugin {
             }
             try {
                 const formula = compile(value);
-                const translatedFormula = this.getters.getTranslatedCellFormula(sheetId, offset.col, offset.row, formula, formula.dependencies.map((d) => this.getters.getRangeFromSheetXC(sheetId, d)));
+                const translatedFormula = this.getters.getTranslatedCellFormula(sheetId, offset.col, offset.row, {
+                    ...formula,
+                    dependencies: formula.dependencies.map((d) => this.getters.getRangeFromSheetXC(sheetId, d)),
+                });
                 const evaluated = this.getters.evaluateFormula(sheetId, translatedFormula);
                 return evaluated && !isMatrix(evaluated) ? evaluated.toString() : "";
             }
@@ -43287,11 +43342,26 @@ class ClipboardCellsState extends ClipboardCellsAbstractState {
             let cellsInRow = [];
             for (let col of columnsIndex) {
                 const position = { col, row, sheetId };
+                const spreader = getters.getArrayFormulaSpreadingOn(position);
+                let cell = getters.getCell(position);
+                const evaluatedCell = getters.getEvaluatedCell(position);
+                if (spreader) {
+                    const isSpreaderCopied = rowsIndex.includes(spreader.row) && columnsIndex.includes(spreader.col);
+                    const content = isSpreaderCopied
+                        ? ""
+                        : formatValue(evaluatedCell.value, { locale: getters.getLocale() });
+                    cell = {
+                        id: cell?.id || "",
+                        style: cell?.style,
+                        format: evaluatedCell.format,
+                        content,
+                        isFormula: false,
+                    };
+                }
                 cellsInRow.push({
-                    cell: getters.getCell(position),
-                    style: getters.getCellComputedStyle(position),
-                    evaluatedCell: getters.getEvaluatedCell(position),
+                    cell,
                     border: getters.getCellBorder(position) || undefined,
+                    evaluatedCell,
                     position,
                 });
             }
@@ -43413,13 +43483,6 @@ class ClipboardCellsState extends ClipboardCellsAbstractState {
             });
         }
         this.pasteCopiedTables(target);
-        this.cells.forEach((row) => {
-            row.forEach((c) => {
-                if (c.cell) {
-                    c.cell = undefined;
-                }
-            });
-        });
     }
     /**
      * The clipped zone is copied as many times as it fits in the target.
@@ -43491,7 +43554,7 @@ class ClipboardCellsState extends ClipboardCellsAbstractState {
     clearClippedZones() {
         for (const row of this.cells) {
             for (const cell of row) {
-                if (cell.cell) {
+                if (cell?.cell) {
                     this.dispatch("CLEAR_CELL", cell.position);
                 }
             }
@@ -43517,6 +43580,9 @@ class ClipboardCellsState extends ClipboardCellsAbstractState {
             const rowCells = this.cells[r];
             for (let c = 0; c < width; c++) {
                 const origin = rowCells[c];
+                if (!origin) {
+                    continue;
+                }
                 const position = { col: col + c, row: row + r, sheetId: sheetId };
                 // TODO: refactor this part. the "Paste merge" action is also executed with
                 // MOVE_RANGES in pasteFromCut. Adding a condition on the operation type here
@@ -43540,53 +43606,42 @@ class ClipboardCellsState extends ClipboardCellsAbstractState {
     pasteCell(origin, target, operation, clipboardOption) {
         const { sheetId, col, row } = target;
         const targetCell = this.getters.getEvaluatedCell(target);
-        if (clipboardOption?.pasteOption !== "onlyValue") {
-            const targetBorders = this.getters.getCellBorder(target);
-            const originBorders = origin.border;
-            const border = {
-                top: targetBorders?.top || originBorders?.top,
-                bottom: targetBorders?.bottom || originBorders?.bottom,
-                left: targetBorders?.left || originBorders?.left,
-                right: targetBorders?.right || originBorders?.right,
-            };
-            this.dispatch("SET_BORDER", { sheetId, col, row, border });
+        if (clipboardOption?.pasteOption === "onlyValue") {
+            const locale = this.getters.getLocale();
+            const content = formatValue(origin.evaluatedCell.value, { locale });
+            this.dispatch("UPDATE_CELL", { ...target, content });
+            return;
         }
-        if (origin.cell) {
-            if (clipboardOption?.pasteOption === "onlyFormat") {
-                this.dispatch("UPDATE_CELL", {
-                    ...target,
-                    style: origin.cell.style,
-                    format: origin.evaluatedCell.format,
-                });
-                return;
-            }
-            if (clipboardOption?.pasteOption === "onlyValue") {
-                const locale = this.getters.getLocale();
-                const content = formatValue(origin.evaluatedCell.value, { locale });
-                this.dispatch("UPDATE_CELL", { ...target, content });
-                return;
-            }
-            let content = origin.cell.content;
-            if (origin.cell.isFormula && operation === "COPY") {
-                content = this.getters.getTranslatedCellFormula(sheetId, col - origin.position.col, row - origin.position.row, origin.cell.compiledFormula, origin.cell.dependencies);
-            }
+        const targetBorders = this.getters.getCellBorder(target);
+        const originBorders = origin.border;
+        const border = {
+            top: targetBorders?.top || originBorders?.top,
+            bottom: targetBorders?.bottom || originBorders?.bottom,
+            left: targetBorders?.left || originBorders?.left,
+            right: targetBorders?.right || originBorders?.right,
+        };
+        this.dispatch("SET_BORDER", { sheetId, col, row, border });
+        if (clipboardOption?.pasteOption === "onlyFormat") {
+            this.dispatch("UPDATE_CELL", {
+                ...target,
+                style: origin.cell?.style ?? null,
+                format: origin.cell?.format ?? origin.evaluatedCell.format ?? targetCell.format,
+            });
+            return;
+        }
+        const content = origin.cell && origin.cell.isFormula && operation === "COPY"
+            ? this.getters.getTranslatedCellFormula(sheetId, col - origin.position.col, row - origin.position.row, origin.cell.compiledFormula)
+            : origin.cell?.content;
+        if (content !== "" || origin.cell?.format || origin.cell?.style) {
             this.dispatch("UPDATE_CELL", {
                 ...target,
                 content,
-                style: origin.cell.style || null,
-                format: origin.cell.format,
+                style: origin.cell?.style || null,
+                format: origin.cell?.format,
             });
         }
         else if (targetCell) {
-            if (clipboardOption?.pasteOption === "onlyValue") {
-                this.dispatch("UPDATE_CELL", { ...target, content: "" });
-            }
-            else if (clipboardOption?.pasteOption === "onlyFormat") {
-                this.dispatch("UPDATE_CELL", { ...target, style: null, format: "" });
-            }
-            else {
-                this.dispatch("CLEAR_CELL", target);
-            }
+            this.dispatch("CLEAR_CELL", target);
         }
     }
     /**
@@ -43648,22 +43703,28 @@ class ClipboardCellsState extends ClipboardCellsAbstractState {
         return (this.cells
             .map((cells) => {
             return cells
-                .map((c) => this.getters.shouldShowFormulas() && c.cell?.isFormula
+                .map((c) => this.getters.shouldShowFormulas() && c?.cell?.isFormula
                 ? c.cell?.content || ""
-                : c.evaluatedCell?.formattedValue || "")
+                : c?.evaluatedCell?.formattedValue || "")
                 .join("\t");
         })
             .join("\n") || "\t");
     }
     getHTMLContent() {
         if (this.cells.length === 1 && this.cells[0].length === 1) {
+            if (!this.cells[0][0]) {
+                return "";
+            }
             return this.getters.getCellText(this.cells[0][0].position);
         }
         let htmlTable = '<table border="1" style="border-collapse:collapse">';
         for (const row of this.cells) {
             htmlTable += "<tr>";
             for (const cell of row) {
-                const cssStyle = cssPropertiesToCss(cellStyleToCss(cell.style));
+                if (!cell) {
+                    continue;
+                }
+                const cssStyle = cssPropertiesToCss(cellStyleToCss(this.getters.getCellComputedStyle(cell.position)));
                 const cellText = this.getters.getCellText(cell.position);
                 htmlTable += `<td style="${cssStyle}">` + xmlEscape(cellText) + "</td>";
             }
@@ -44100,20 +44161,29 @@ class FindAndReplacePlugin extends UIPlugin {
         const sheetId = this.getters.getActiveSheetId();
         const cells = this.getters.getCells(sheetId);
         const matches = [];
-        if (this.toSearch) {
+        if (this.toSearch && this.currentSearchRegex) {
             for (const cell of Object.values(cells)) {
                 const { col, row } = this.getters.getCellPosition(cell.id);
+                const cellPosition = { sheetId, col, row };
                 const isColHidden = this.getters.isColHidden(sheetId, col);
                 const isRowHidden = this.getters.isRowHidden(sheetId, row);
                 if (isColHidden || isRowHidden) {
                     continue;
                 }
-                if (cell &&
-                    this.currentSearchRegex &&
-                    this.currentSearchRegex.test(this.getSearchableString({ sheetId, col, row }))) {
+                if (this.currentSearchRegex.test(this.getSearchableString(cellPosition))) {
                     const match = { col, row, selected: false };
                     matches.push(match);
                 }
+                for (const spreadPosition of this.getters.getSpreadPositionsOf(cellPosition)) {
+                    if (this.currentSearchRegex.test(this.getSearchableString(spreadPosition))) {
+                        const match = {
+                            col: spreadPosition.col,
+                            row: spreadPosition.row,
+                            selected: false,
+                        };
+                        matches.push(match);
+                    }
+                }
             }
         }
         return matches.sort(this.sortByRowThenColumn);
@@ -44172,16 +44242,19 @@ class FindAndReplacePlugin extends UIPlugin {
             return;
         }
         const sheetId = this.getters.getActiveSheetId();
-        const cell = this.getters.getCell({ sheetId, ...selectedMatch });
-        const { col, row } = selectedMatch;
+        const position = { sheetId, ...selectedMatch };
+        const cell = this.getters.getCell(position);
+        if (!cell?.content) {
+            return;
+        }
         if (cell?.isFormula && !this.searchOptions.searchFormulas) {
             return;
         }
         const replaceRegex = new RegExp(this.currentSearchRegex.source, this.currentSearchRegex.flags + "g");
-        const toReplace = this.getSearchableString({ sheetId, col, row });
+        const toReplace = this.getSearchableString(position);
         const content = toReplace.replace(replaceRegex, replaceWith);
         const canonicalContent = canonicalizeNumberContent(content, this.getters.getLocale());
-        this.dispatch("UPDATE_CELL", { sheetId, col, row, content: canonicalContent });
+        this.dispatch("UPDATE_CELL", { ...position, content: canonicalContent });
     }
     /**
      * Replace the value of the currently selected match
@@ -45756,7 +45829,7 @@ class SortPlugin extends UIPlugin {
                     if (cell.isFormula) {
                         const position = this.getters.getCellPosition(cell.id);
                         // we only have a vertical offset
-                        content = this.getters.getTranslatedCellFormula(sheetId, 0, newRow - position.row, cell.compiledFormula, cell.dependencies);
+                        content = this.getters.getTranslatedCellFormula(sheetId, 0, newRow - position.row, cell.compiledFormula);
                     }
                     newCellValues.style = cell.style;
                     newCellValues.content = content;
@@ -46781,11 +46854,14 @@ class ClipboardPlugin extends UIPlugin {
                 }
                 const pasteOption = cmd.pasteOption || (this.paintFormatStatus !== "inactive" ? "onlyFormat" : undefined);
                 this.state.paste(cmd.target, { pasteOption, shouldPasteCF: true, selectTarget: true });
+                if (this.state.operation === "CUT") {
+                    this.state = undefined;
+                }
                 this.lastPasteState = this.state;
                 if (this.paintFormatStatus === "oneOff") {
                     this.paintFormatStatus = "inactive";
-                    this.status = "invisible";
                 }
+                this.status = "invisible";
                 break;
             case "COPY_PASTE_CELLS_ABOVE":
                 {
@@ -47819,6 +47895,8 @@ class FilterEvaluationPlugin extends UIPlugin {
             case "EVALUATE_CELLS":
             case "ACTIVATE_SHEET":
             case "REMOVE_FILTER_TABLE":
+            case "ADD_COLUMNS_ROWS":
+            case "REMOVE_COLUMNS_ROWS":
                 this.isEvaluationDirty = true;
                 break;
             case "START":
@@ -55507,19 +55585,22 @@ class Model extends EventBus {
      * Check if the given command is allowed by all the plugins and the history.
      */
     checkDispatchAllowed(command) {
-        if (isCoreCommand(command)) {
-            return this.checkDispatchAllowedCoreCommand(command);
+        const results = isCoreCommand(command)
+            ? this.checkDispatchAllowedCoreCommand(command)
+            : this.checkDispatchAllowedLocalCommand(command);
+        if (results.some((r) => r !== "Success" /* CommandResult.Success */)) {
+            return new DispatchResult(results.flat());
         }
-        return this.checkDispatchAllowedLocalCommand(command);
+        return DispatchResult.Success;
     }
     checkDispatchAllowedCoreCommand(command) {
         const results = this.corePlugins.map((handler) => handler.allowDispatch(command));
         results.push(this.range.allowDispatch(command));
-        return new DispatchResult(results.flat());
+        return results;
     }
     checkDispatchAllowedLocalCommand(command) {
         const results = this.uiHandlers.map((handler) => handler.allowDispatch(command));
-        return new DispatchResult(results.flat());
+        return results;
     }
     finalize() {
         this.status = 3 /* Status.Finalizing */;
@@ -55828,9 +55909,9 @@ const constants = {
     DEFAULT_LOCALE,
 };
 
-export { AbstractChart, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, __info__, addFunction, astToFormula, compile, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, hooks, invalidateCFEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, tokenize };
+export { AbstractChart, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, __info__, addFunction, astToFormula, compile, compileTokens, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, hooks, invalidateCFEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, parseTokens, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, tokenize };
 
 
-__info__.version = "17.0.2";
-__info__.date = "2023-11-03T12:22:44.945Z";
-__info__.hash = "3184728";
+__info__.version = "17.0.4";
+__info__.date = "2023-11-24T13:13:42.575Z";
+__info__.hash = "7346d0a";