@odoo/o-spreadsheet 17.0.48 → 17.0.50

package/dist/o-spreadsheet.cjs.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.48
- * @date 2025-02-05T07:18:39.331Z
- * @hash aea9101
+ * @version 17.0.50
+ * @date 2025-02-14T08:37:34.289Z
+ * @hash cc1037d
  */
 
 'use strict';
@@ -688,9 +688,16 @@ function deepEquals(o1, o2) {
     }
     return true;
 }
-/** Check if the given array contains all the values of the other array. */
+/**
+ * Check if the given array contains all the values of the other array.
+ * It makes the assumption that both array do not contain duplicates.
+ */
 function includesAll(arr, values) {
-    return values.every((value) => arr.includes(value));
+    if (arr.length < values.length) {
+        return false;
+    }
+    const set = new Set(arr);
+    return values.every((value) => set.has(value));
 }
 /**
  * Return an object with all the keys in the object that have a falsy value removed.
@@ -1903,12 +1910,12 @@ function isTargetDependent(cmd) {
 function isRangeDependant(cmd) {
     return "ranges" in cmd;
 }
-function isZoneDependent(cmd) {
-    return "zone" in cmd;
-}
 function isPositionDependent(cmd) {
     return "col" in cmd && "row" in cmd && "sheetId" in cmd;
 }
+function isZoneDependent(cmd) {
+    return "sheetId" in cmd && "zone" in cmd;
+}
 const invalidateEvaluationCommands = new Set([
     "RENAME_SHEET",
     "DELETE_SHEET",
@@ -1918,6 +1925,7 @@ const invalidateEvaluationCommands = new Set([
     "UNDO",
     "REDO",
     "ADD_MERGE",
+    "DUPLICATE_SHEET",
     "UPDATE_LOCALE",
 ]);
 const invalidateDependenciesCommands = new Set([
@@ -1926,7 +1934,6 @@ const invalidateDependenciesCommands = new Set([
 ]);
 const invalidateCFEvaluationCommands = new Set([
     ...invalidateEvaluationCommands,
-    "DUPLICATE_SHEET",
     "EVALUATE_CELLS",
     "ADD_CONDITIONAL_FORMAT",
     "REMOVE_CONDITIONAL_FORMAT",
@@ -2086,6 +2093,7 @@ exports.CommandResult = void 0;
     CommandResult["InvalidRange"] = "InvalidRange";
     CommandResult["InvalidZones"] = "InvalidZones";
     CommandResult["InvalidSheetId"] = "InvalidSheetId";
+    CommandResult["InvalidCellId"] = "InvalidCellId";
     CommandResult["InvalidFigureId"] = "InvalidFigureId";
     CommandResult["InputAlreadyFocused"] = "InputAlreadyFocused";
     CommandResult["MaximumRangesReached"] = "MaximumRangesReached";
@@ -3666,17 +3674,20 @@ function toZoneWithoutBoundaryChanges(xc) {
  */
 function toUnboundedZone(xc) {
     const zone = toZoneWithoutBoundaryChanges(xc);
+    reorderZone(zone);
+    return zone;
+}
+function reorderZone(zone) {
     if (zone.right !== undefined && zone.right < zone.left) {
-        const tmp = zone.left;
+        const right = zone.left;
         zone.left = zone.right;
-        zone.right = tmp;
+        zone.right = right;
     }
     if (zone.bottom !== undefined && zone.bottom < zone.top) {
-        const tmp = zone.top;
+        const bottom = zone.top;
         zone.top = zone.bottom;
-        zone.bottom = tmp;
+        zone.bottom = bottom;
     }
-    return zone;
 }
 /**
  * Convert from a cartesian reference to a Zone.
@@ -4709,6 +4720,37 @@ class UuidGenerator {
     setIsFastStrategy(isFast) {
         this.isFastIdStrategy = isFast;
     }
+    /**
+     * Generates a custom UUID using a simple 26^8 method (8-character alphanumeric string with lowercase letters)
+     * This has a higher chance of collision than a UUIDv4, but not only faster to generate than an UUIDV4,
+     * it also has a smaller size, which is preferable to alleviate the overall data size.
+     *
+     * This method is preferable when generating uuids for the core data (sheetId, figureId, etc)
+     * as they will appear several times in the revisions and local history.
+     *
+     */
+    smallUuid() {
+        if (this.isFastIdStrategy) {
+            this.fastIdStart++;
+            return String(this.fastIdStart);
+            //@ts-ignore
+        }
+        else if (window.crypto && window.crypto.getRandomValues) {
+            //@ts-ignore
+            return ([1e7] + -1e3).replace(/[018]/g, (c) => (c ^ (crypto.getRandomValues(new Uint8Array(1))[0] & (15 >> (c / 4)))).toString(16));
+        }
+        else {
+            // mainly for jest and other browsers that do not have the crypto functionality
+            return "xxxxxxxx-xxxx".replace(/[xy]/g, function (c) {
+                var r = (Math.random() * 16) | 0, v = c == "x" ? r : (r & 0x3) | 0x8;
+                return v.toString(16);
+            });
+        }
+    }
+    /**
+     * Generates an UUIDV4, has astronomically low chance of collision, but is larger in size than the smallUuid.
+     * This method should be used when you need to avoid collisions at all costs, like the id of a revision.
+     */
     uuidv4() {
         if (this.isFastIdStrategy) {
             this.fastIdStart++;
@@ -4722,7 +4764,7 @@ class UuidGenerator {
         else {
             // mainly for jest and other browsers that do not have the crypto functionality
             return "xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx".replace(/[xy]/g, function (c) {
-                var r = (Math.random() * 16) | 0, v = c === "x" ? r : (r & 0x3) | 0x8;
+                var r = (Math.random() * 16) | 0, v = c == "x" ? r : (r & 0x3) | 0x8;
                 return v.toString(16);
             });
         }
@@ -7935,7 +7977,7 @@ const linkSheet = {
 const deleteSheet = {
     name: _t("Delete"),
     isVisible: (env) => {
-        return env.model.getters.getSheetIds().length > 1;
+        return env.model.getters.getVisibleSheetIds().length > 1;
     },
     execute: (env) => env.askConfirmation(_t("Are you sure you want to delete this sheet?"), () => {
         env.model.dispatch("DELETE_SHEET", { sheetId: env.model.getters.getActiveSheetId() });
@@ -7945,7 +7987,7 @@ const duplicateSheet = {
     name: _t("Duplicate"),
     execute: (env) => {
         const sheetIdFrom = env.model.getters.getActiveSheetId();
-        const sheetIdTo = env.model.uuidGenerator.uuidv4();
+        const sheetIdTo = env.model.uuidGenerator.smallUuid();
         env.model.dispatch("DUPLICATE_SHEET", {
             sheetId: sheetIdFrom,
             sheetIdTo,
@@ -8495,17 +8537,15 @@ function getDefaultChartJsRuntime(chart, labels, fontColor, { format, locale, tr
         plugins: [],
     };
 }
-function getChartLabelFormat(getters, range) {
+function getChartLabelFormat(getters, range, shouldRemoveFirstLabel) {
     if (!range)
         return undefined;
-    const { sheetId, zone: { left, top, bottom }, } = range;
-    for (let row = top; row <= bottom; row++) {
-        const format = getters.getEvaluatedCell({ sheetId, col: left, row }).format;
-        if (format) {
-            return format;
-        }
+    const { sheetId, zone } = range;
+    const formats = positions(zone).map((position) => getters.getEvaluatedCell({ sheetId, ...position }).format);
+    if (shouldRemoveFirstLabel) {
+        formats.shift();
     }
-    return undefined;
+    return formats.find((format) => format !== undefined);
 }
 function getChartLabelValues(getters, dataSets, labelRange) {
     let labels = { values: [], formattedValues: [] };
@@ -8786,9 +8826,7 @@ function createBarChartRuntime(chart, getters) {
     const labelValues = getChartLabelValues(getters, chart.dataSets, chart.labelRange);
     let labels = labelValues.formattedValues;
     let dataSetsValues = getChartDatasetValues(getters, chart.dataSets);
-    if (chart.dataSetsHaveTitle &&
-        dataSetsValues[0] &&
-        labels.length > dataSetsValues[0].data.length) {
+    if (shouldRemoveFirstLabel(chart.labelRange, chart.dataSets[0], chart.dataSetsHaveTitle)) {
         labels.shift();
     }
     ({ labels, dataSetsValues } = filterEmptyDataPoints(labels, dataSetsValues));
@@ -9332,8 +9370,8 @@ function fixEmptyLabelsForDateCharts(labels, dataSetsValues) {
     }
     return { labels: newLabels, dataSetsValues: newDatasets };
 }
-function canChartParseLabels(labelRange, getters) {
-    return canBeDateChart(labelRange, getters) || canBeLinearChart(labelRange, getters);
+function canChartParseLabels(chart, getters) {
+    return canBeDateChart(chart, getters) || canBeLinearChart(chart, getters);
 }
 function getChartAxisType(chart, getters) {
     if (isDateChart(chart, getters) && isLuxonTimeAdapterInstalled()) {
@@ -9345,23 +9383,26 @@ function getChartAxisType(chart, getters) {
     return "category";
 }
 function isDateChart(chart, getters) {
-    return !chart.labelsAsText && canBeDateChart(chart.labelRange, getters);
+    return !chart.labelsAsText && canBeDateChart(chart, getters);
 }
 function isLinearChart(chart, getters) {
-    return !chart.labelsAsText && canBeLinearChart(chart.labelRange, getters);
+    return !chart.labelsAsText && canBeLinearChart(chart, getters);
 }
-function canBeDateChart(labelRange, getters) {
-    if (!labelRange || !canBeLinearChart(labelRange, getters)) {
+function canBeDateChart(chart, getters) {
+    if (!chart.labelRange || !canBeLinearChart(chart, getters)) {
         return false;
     }
-    const labelFormat = getChartLabelFormat(getters, labelRange);
+    const labelFormat = getChartLabelFormat(getters, chart.labelRange, shouldRemoveFirstLabel(chart.labelRange, chart.dataSets[0], chart.dataSetsHaveTitle));
     return Boolean(labelFormat && timeFormatLuxonCompatible.test(labelFormat));
 }
-function canBeLinearChart(labelRange, getters) {
-    if (!labelRange) {
+function canBeLinearChart(chart, getters) {
+    if (!chart.labelRange) {
         return false;
     }
-    const labels = getters.getRangeValues(labelRange);
+    const labels = getters.getRangeValues(chart.labelRange);
+    if (shouldRemoveFirstLabel(chart.labelRange, chart.dataSets[0], chart.dataSetsHaveTitle)) {
+        labels.shift();
+    }
     if (labels.some((label) => isNaN(Number(label)) && label)) {
         return false;
     }
@@ -9435,9 +9476,8 @@ function createLineChartRuntime(chart, getters) {
     const labelValues = getChartLabelValues(getters, chart.dataSets, chart.labelRange);
     let labels = axisType === "linear" ? labelValues.values : labelValues.formattedValues;
     let dataSetsValues = getChartDatasetValues(getters, chart.dataSets);
-    if (chart.dataSetsHaveTitle &&
-        dataSetsValues[0] &&
-        labels.length > dataSetsValues[0].data.length) {
+    const removeFirstLabel = shouldRemoveFirstLabel(chart.labelRange, chart.dataSets[0], chart.dataSetsHaveTitle);
+    if (removeFirstLabel) {
         labels.shift();
     }
     ({ labels, dataSetsValues } = filterEmptyDataPoints(labels, dataSetsValues));
@@ -9452,7 +9492,7 @@ function createLineChartRuntime(chart, getters) {
     const dataSetFormat = getChartDatasetFormat(getters, chart.dataSets);
     const options = { format: dataSetFormat, locale, truncateLabels };
     const config = getLineConfiguration(chart, labels, options);
-    const labelFormat = getChartLabelFormat(getters, chart.labelRange);
+    const labelFormat = getChartLabelFormat(getters, chart.labelRange, removeFirstLabel);
     if (axisType === "time") {
         const axis = {
             type: "time",
@@ -9674,9 +9714,7 @@ function createPieChartRuntime(chart, getters) {
     const labelValues = getChartLabelValues(getters, chart.dataSets, chart.labelRange);
     let labels = labelValues.formattedValues;
     let dataSetsValues = getChartDatasetValues(getters, chart.dataSets);
-    if (chart.dataSetsHaveTitle &&
-        dataSetsValues[0] &&
-        labels.length > dataSetsValues[0].data.length) {
+    if (shouldRemoveFirstLabel(chart.labelRange, chart.dataSets[0], chart.dataSetsHaveTitle)) {
         labels.shift();
     }
     ({ labels, dataSetsValues } = filterEmptyDataPoints(labels, dataSetsValues));
@@ -10197,21 +10235,22 @@ function getSmartChartDefinition(zone, getters) {
     }
     // Only display legend for several datasets.
     const newLegendPos = dataSetZone.right === dataSetZone.left ? "none" : "top";
-    const labelRange = labelRangeXc ? getters.getRangeFromSheetXC(sheetId, labelRangeXc) : undefined;
-    if (canChartParseLabels(labelRange, getters)) {
-        return {
-            title,
-            dataSets,
-            labelsAsText: false,
-            stacked: false,
-            aggregated: false,
-            cumulative: false,
-            labelRange: labelRangeXc,
-            type: "line",
-            dataSetsHaveTitle,
-            verticalAxisPosition: "left",
-            legendPosition: newLegendPos,
-        };
+    const lineChartDefinition = {
+        title,
+        dataSets,
+        labelsAsText: false,
+        stacked: false,
+        aggregated: false,
+        cumulative: false,
+        labelRange: labelRangeXc,
+        type: "line",
+        dataSetsHaveTitle,
+        verticalAxisPosition: "left",
+        legendPosition: newLegendPos,
+    };
+    const chart = new LineChart(lineChartDefinition, sheetId, getters);
+    if (canChartParseLabels(chart, getters)) {
+        return lineChartDefinition;
     }
     return {
         title,
@@ -10811,7 +10850,7 @@ const HIDE_ROWS_NAME = (env) => {
 //------------------------------------------------------------------------------
 const CREATE_CHART = (env) => {
     const getters = env.model.getters;
-    const id = env.model.uuidGenerator.uuidv4();
+    const id = env.model.uuidGenerator.smallUuid();
     const sheetId = getters.getActiveSheetId();
     if (getZoneArea(env.model.getters.getSelectedZone()) === 1) {
         env.model.selection.selectTableAroundSelection();
@@ -10845,7 +10884,7 @@ async function requestImage(env) {
 const CREATE_IMAGE = async (env) => {
     if (env.imageProvider) {
         const sheetId = env.model.getters.getActiveSheetId();
-        const figureId = env.model.uuidGenerator.uuidv4();
+        const figureId = env.model.uuidGenerator.smallUuid();
         const image = await requestImage(env);
         if (!image) {
             throw new Error("No image provider was given to the environment");
@@ -12566,6 +12605,26 @@ const LN = {
     isExported: true,
 };
 // -----------------------------------------------------------------------------
+// LOG
+// -----------------------------------------------------------------------------
+const LOG = {
+    description: _t("The logarithm of a number, for a given base."),
+    args: [
+        arg("value (number)", _t("The value for which to calculate the logarithm.")),
+        arg("base (number, default=10)", _t("The base of the logarithm.")),
+    ],
+    returns: ["NUMBER"],
+    compute: function (value, base = 10) {
+        const _value = toNumber(value, this.locale);
+        const _base = toNumber(base, this.locale);
+        assert(() => _value > 0, _t("The value (%s) must be strictly positive.", _value.toString()));
+        assert(() => _base > 0, _t("The base (%s) must be strictly positive.", _base.toString()));
+        assert(() => _base !== 1, _t("The base must be different from 1."));
+        return Math.log10(_value) / Math.log10(_base);
+    },
+    isExported: true,
+};
+// -----------------------------------------------------------------------------
 // MOD
 // -----------------------------------------------------------------------------
 const MOD = {
@@ -13086,6 +13145,7 @@ var math = /*#__PURE__*/Object.freeze({
   ISODD: ISODD,
   ISO_CEILING: ISO_CEILING,
   LN: LN,
+  LOG: LOG,
   MOD: MOD,
   MUNIT: MUNIT,
   ODD: ODD,
@@ -15947,7 +16007,7 @@ const SORTN = {
             }
         }
     },
-    isExported: true,
+    isExported: false,
 };
 // -----------------------------------------------------------------------------
 // UNIQUE
@@ -19848,7 +19908,7 @@ const insertSheet = {
     execute: (env) => {
         const activeSheetId = env.model.getters.getActiveSheetId();
         const position = env.model.getters.getSheetIds().indexOf(activeSheetId) + 1;
-        const sheetId = env.model.uuidGenerator.uuidv4();
+        const sheetId = env.model.uuidGenerator.smallUuid();
         env.model.dispatch("CREATE_SHEET", { sheetId, position });
         env.model.dispatch("ACTIVATE_SHEET", { sheetIdFrom: activeSheetId, sheetIdTo: sheetId });
     },
@@ -21524,7 +21584,7 @@ css /* scss */ `
  */
 class SelectionInput extends owl.Component {
     static template = "o-spreadsheet-SelectionInput";
-    id = uuidGenerator$1.uuidv4();
+    id = uuidGenerator$1.smallUuid();
     previousRanges = this.props.ranges() || [];
     originSheet = this.env.model.getters.getActiveSheetId();
     state = owl.useState({
@@ -22574,7 +22634,7 @@ class LineConfigPanel extends LineBarPieConfigPanel {
     get canTreatLabelsAsText() {
         const chart = this.env.model.getters.getChart(this.props.figureId);
         if (chart && chart instanceof LineChart) {
-            return canChartParseLabels(chart.labelRange, this.env.model.getters);
+            return canChartParseLabels(chart, this.env.model.getters);
         }
         return false;
     }
@@ -23602,7 +23662,7 @@ class ConditionalFormattingEditor extends owl.Component {
     state;
     setup() {
         const cf = this.props.editedCf || {
-            id: this.env.model.uuidGenerator.uuidv4(),
+            id: this.env.model.uuidGenerator.smallUuid(),
             ranges: this.env.model.getters
                 .getSelectedZones()
                 .map((zone) => this.env.model.getters.zoneToXC(this.env.model.getters.getActiveSheetId(), zone)),
@@ -25551,7 +25611,7 @@ class DataValidationEditor extends owl.Component {
             .getSelectedZones()
             .map((zone) => zoneToXc(this.env.model.getters.getUnboundedZone(sheetId, zone)));
         return {
-            id: this.env.model.uuidGenerator.uuidv4(),
+            id: this.env.model.uuidGenerator.smallUuid(),
             criterion: { type: "textContains", values: [""] },
             ranges,
         };
@@ -34171,7 +34231,7 @@ function forceUnicityOfFigure(data) {
     for (const sheet of data.sheets || []) {
         for (const figure of sheet.figures || []) {
             if (figureIds.has(figure.id)) {
-                figure.id += uuidGenerator.uuidv4();
+                figure.id += uuidGenerator.smallUuid();
             }
             figureIds.add(figure.id);
         }
@@ -35847,6 +35907,10 @@ class CellPlugin extends CorePlugin {
                 return this.checkValidations(cmd, this.checkCellOutOfSheet, this.checkUselessUpdateCell);
             case "CLEAR_CELL":
                 return this.checkValidations(cmd, this.checkCellOutOfSheet, this.checkUselessClearCell);
+            case "UPDATE_CELL_POSITION":
+                return !cmd.cellId || this.cells[cmd.sheetId]?.[cmd.cellId]
+                    ? "Success" /* CommandResult.Success */
+                    : "InvalidCellId" /* CommandResult.InvalidCellId */;
             default:
                 return "Success" /* CommandResult.Success */;
         }
@@ -35888,6 +35952,9 @@ class CellPlugin extends CorePlugin {
             case "CLEAR_CELLS":
                 this.clearCells(cmd.sheetId, cmd.target);
                 break;
+            case "DELETE_SHEET": {
+                this.history.update("cells", cmd.sheetId, undefined);
+            }
         }
     }
     /**
@@ -36966,8 +37033,14 @@ class DataValidationPlugin extends CorePlugin {
     allowDispatch(cmd) {
         switch (cmd.type) {
             case "ADD_DATA_VALIDATION_RULE":
+                if (!this.getters.tryGetSheet(cmd.sheetId)) {
+                    return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+                }
                 return this.checkValidations(cmd, this.chainValidations(this.checkCriterionTypeIsValid, this.checkCriterionHasValidNumberOfValues, this.checkCriterionValuesAreValid));
             case "REMOVE_DATA_VALIDATION_RULE":
+                if (!this.getters.tryGetSheet(cmd.sheetId)) {
+                    return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+                }
                 if (!this.rules[cmd.sheetId].find((rule) => rule.id === cmd.id)) {
                     return "UnknownDataValidationRule" /* CommandResult.UnknownDataValidationRule */;
                 }
@@ -37141,6 +37214,7 @@ class DataValidationPlugin extends CorePlugin {
 class FigurePlugin extends CorePlugin {
     static getters = ["getFigures", "getFigure", "getFigureSheetId"];
     figures = {};
+    insertionOrders = []; // TODO use a list in master
     // ---------------------------------------------------------------------------
     // Command Handling
     // ---------------------------------------------------------------------------
@@ -37243,11 +37317,14 @@ class FigurePlugin extends CorePlugin {
     }
     addFigure(figure, sheetId) {
         this.history.update("figures", sheetId, figure.id, figure);
+        this.history.update("insertionOrders", this.insertionOrders.length, figure.id);
     }
     deleteSheet(sheetId) {
+        this.history.update("insertionOrders", this.insertionOrders.filter((id) => !this.figures[sheetId]?.[id]));
         this.history.update("figures", sheetId, undefined);
     }
     removeFigure(id, sheetId) {
+        this.history.update("insertionOrders", this.insertionOrders.filter((figureId) => figureId !== id));
         this.history.update("figures", sheetId, id, undefined);
     }
     checkFigureExists(sheetId, figureId) {
@@ -37266,7 +37343,14 @@ class FigurePlugin extends CorePlugin {
     // Getters
     // ---------------------------------------------------------------------------
     getFigures(sheetId) {
-        return Object.values(this.figures[sheetId] || {}).filter(isDefined$1);
+        const figures = [];
+        for (const figureId of this.insertionOrders) {
+            const figure = this.figures[sheetId]?.[figureId];
+            if (figure) {
+                figures.push(figure);
+            }
+        }
+        return figures;
     }
     getFigure(sheetId, figureId) {
         return this.figures[sheetId]?.[figureId];
@@ -37279,11 +37363,9 @@ class FigurePlugin extends CorePlugin {
     // ---------------------------------------------------------------------------
     import(data) {
         for (let sheet of data.sheets) {
-            const figures = {};
-            sheet.figures.forEach((figure) => {
-                figures[figure.id] = figure;
-            });
-            this.figures[sheet.id] = figures;
+            for (const figure of sheet.figures) {
+                this.addFigure(figure, sheet.id);
+            }
         }
     }
     export(data) {
@@ -37307,10 +37389,10 @@ class FilterTable {
         this.filters = [];
         this.zone = zone;
         const uuid = new UuidGenerator();
-        this.id = uuid.uuidv4();
+        this.id = uuid.smallUuid();
         for (const i of range(zone.left, zone.right + 1)) {
             const filterZone = { ...this.zone, left: i, right: i };
-            this.filters.push(new Filter(uuid.uuidv4(), filterZone));
+            this.filters.push(new Filter(uuid.smallUuid(), filterZone));
         }
     }
     /** Get zone of the table without the headers */
@@ -37512,7 +37594,7 @@ class FiltersPlugin extends CorePlugin {
             if (filters.length < zoneToDimension(zone).numberOfCols) {
                 for (let col = zone.left; col <= zone.right; col++) {
                     if (!filters.find((filter) => filter.col === col)) {
-                        filters.push(new Filter(this.uuidGenerator.uuidv4(), { ...zone, left: col, right: col }));
+                        filters.push(new Filter(this.uuidGenerator.smallUuid(), { ...zone, left: col, right: col }));
                     }
                 }
                 filters.sort((f1, f2) => f1.col - f2.col);
@@ -39001,6 +39083,9 @@ class SheetPlugin extends CorePlugin {
             case "CREATE_SHEET": {
                 return this.checkValidations(cmd, this.checkSheetName, this.checkSheetPosition);
             }
+            case "DUPLICATE_SHEET": {
+                return this.sheets[cmd.sheetIdTo] ? "DuplicatedSheetId" /* CommandResult.DuplicatedSheetId */ : "Success" /* CommandResult.Success */;
+            }
             case "MOVE_SHEET":
                 try {
                     const currentIndex = this.orderedSheetIds.findIndex((id) => id === cmd.sheetId);
@@ -39013,7 +39098,7 @@ class SheetPlugin extends CorePlugin {
             case "RENAME_SHEET":
                 return this.isRenameAllowed(cmd);
             case "DELETE_SHEET":
-                return this.orderedSheetIds.length > 1
+                return this.getVisibleSheetIds().length > 1
                     ? "Success" /* CommandResult.Success */
                     : "NotEnoughSheets" /* CommandResult.NotEnoughSheets */;
             case "ADD_COLUMNS_ROWS":
@@ -39826,6 +39911,10 @@ class SheetPlugin extends CorePlugin {
     checkZonesAreInSheet(cmd) {
         if (!("sheetId" in cmd))
             return "Success" /* CommandResult.Success */;
+        if ("ranges" in cmd &&
+            cmd.ranges.some((rangeData) => !this.getters.tryGetSheet(rangeData._sheetId))) {
+            return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+        }
         return this.checkZonesExistInSheet(cmd.sheetId, this.getCommandZones(cmd));
     }
 }
@@ -39845,7 +39934,10 @@ class HeaderGroupingPlugin extends CorePlugin {
     allowDispatch(cmd) {
         switch (cmd.type) {
             case "GROUP_HEADERS": {
-                const { start, end } = cmd;
+                const { start, end, sheetId } = cmd;
+                if (!this.getters.tryGetSheet(sheetId)) {
+                    return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+                }
                 if (!this.getters.doesHeadersExist(cmd.sheetId, cmd.dimension, [start, end])) {
                     return "InvalidHeaderGroupStartEnd" /* CommandResult.InvalidHeaderGroupStartEnd */;
                 }
@@ -39858,7 +39950,10 @@ class HeaderGroupingPlugin extends CorePlugin {
                 break;
             }
             case "UNGROUP_HEADERS": {
-                const { start, end } = cmd;
+                const { start, end, sheetId } = cmd;
+                if (!this.getters.tryGetSheet(sheetId)) {
+                    return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+                }
                 if (!this.getters.doesHeadersExist(cmd.sheetId, cmd.dimension, [start, end])) {
                     return "InvalidHeaderGroupStartEnd" /* CommandResult.InvalidHeaderGroupStartEnd */;
                 }
@@ -39869,6 +39964,9 @@ class HeaderGroupingPlugin extends CorePlugin {
             }
             case "UNFOLD_HEADER_GROUP":
             case "FOLD_HEADER_GROUP":
+                if (!this.getters.tryGetSheet(cmd.sheetId)) {
+                    return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+                }
                 const group = this.findGroupWithStartEnd(cmd.sheetId, cmd.dimension, cmd.start, cmd.end);
                 if (!group) {
                     return "UnknownHeaderGroup" /* CommandResult.UnknownHeaderGroup */;
@@ -44138,6 +44236,9 @@ function updateChartRangesTransformation(toTransform, executed) {
     };
 }
 function createSheetTransformation(toTransform, executed) {
+    if (toTransform.sheetId === executed.sheetId) {
+        toTransform = { ...toTransform, sheetId: `${toTransform.sheetId}~` };
+    }
     if (toTransform.name === executed.name) {
         return {
             ...toTransform,
@@ -44774,21 +44875,14 @@ class Session extends EventBus {
         if (!message)
             return;
         if (message.type === "REMOTE_REVISION") {
-            const revision = this.revisions.get(message.nextRevisionId);
+            let revision = this.revisions.get(message.nextRevisionId);
             if (revision.commands.length === 0) {
                 /**
-                 * The command is empty, we have to drop all the next local revisions
+                 * The command is empty, we have to rebase all the next local revisions
                  * to avoid issues with undo/redo
                  */
-                this.revisions.drop(revision.id);
-                const revisionIds = this.pendingMessages
-                    .filter((message) => message.type === "REMOTE_REVISION")
-                    .map((message) => message.nextRevisionId);
-                this.trigger("pending-revisions-dropped", { revisionIds });
-                this.waitingAck = false;
-                this.waitingUndoRedoAck = false;
-                this.pendingMessages = [];
-                return;
+                this.revisions.rebase(revision.id);
+                revision = this.revisions.get(message.nextRevisionId);
             }
             message = {
                 ...message,
@@ -44812,7 +44906,6 @@ class Session extends EventBus {
         switch (message.type) {
             case "REMOTE_REVISION":
             case "REVISION_REDONE":
-            case "REVISION_UNDONE":
             case "SNAPSHOT_CREATED":
                 this.waitingAck = false;
                 this.pendingMessages = this.pendingMessages.filter((msg) => msg.nextRevisionId !== message.nextRevisionId);
@@ -44820,6 +44913,25 @@ class Session extends EventBus {
                 this.processedRevisions.add(message.nextRevisionId);
                 this.sendPendingMessage();
                 break;
+            case "REVISION_UNDONE": {
+                this.waitingAck = false;
+                this.pendingMessages = this.pendingMessages.filter((msg) => msg.nextRevisionId !== message.nextRevisionId);
+                const firstPendingRevisionId = this.pendingMessages.findIndex((message) => message.type === "REMOTE_REVISION");
+                if (firstPendingRevisionId !== -1) {
+                    /**
+                     * Some revisions undergo transformations that may cause issues with
+                     * undo/redo if the transformation is destructive (we don't get back
+                     * the original command by transforming it with the inverse).
+                     * To prevent these problems, we must rebase all subsequent local
+                     * revisions.
+                     */
+                    this.revisions.rebase(this.pendingMessages[firstPendingRevisionId].nextRevisionId);
+                }
+                this.serverRevisionId = message.nextRevisionId;
+                this.processedRevisions.add(message.nextRevisionId);
+                this.sendPendingMessage();
+                break;
+            }
         }
     }
     isAlreadyProcessed(message) {
@@ -45524,7 +45636,9 @@ class ClipboardCellsState extends ClipboardCellsAbstractState {
         const cfInTarget = this.getters
             .getConditionalFormats(targetSheetId)
             .find((cf) => cf.stopIfTrue === originCF.stopIfTrue && deepEquals(cf.rule, originCF.rule));
-        return cfInTarget ? cfInTarget : { ...originCF, id: this.uuidGenerator.uuidv4(), ranges: [] };
+        return cfInTarget
+            ? cfInTarget
+            : { ...originCF, id: this.uuidGenerator.smallUuid(), ranges: [] };
     }
     pasteDataValidation(origin, target) {
         const rule = this.getters.getValidationRuleForCell(origin);
@@ -45556,7 +45670,7 @@ class ClipboardCellsState extends ClipboardCellsAbstractState {
             originRule.isBlocking === rule.isBlocking);
         return ruleInTargetSheet
             ? ruleInTargetSheet
-            : { ...originRule, id: this.uuidGenerator.uuidv4(), ranges: [] };
+            : { ...originRule, id: this.uuidGenerator.smallUuid(), ranges: [] };
     }
     /**
      * Add or remove XCs to a given data validation rule.
@@ -47748,6 +47862,10 @@ class SheetUIPlugin extends UIPlugin {
      */
     checkZonesAreInSheet(cmd) {
         const sheetId = "sheetId" in cmd ? cmd.sheetId : this.getters.tryGetActiveSheetId();
+        if ("ranges" in cmd &&
+            cmd.ranges.some((rangeData) => !this.getters.tryGetSheet(rangeData._sheetId))) {
+            return "InvalidSheetId" /* CommandResult.InvalidSheetId */;
+        }
         const zones = this.getters.getCommandZones(cmd);
         if (!sheetId && zones.length > 0) {
             return "NoActiveSheet" /* CommandResult.NoActiveSheet */;
@@ -47796,23 +47914,23 @@ const uuidGenerator = new UuidGenerator();
 function repeatCreateChartCommand(getters, cmd) {
     return {
         ...repeatSheetDependantCommand(getters, cmd),
-        id: uuidGenerator.uuidv4(),
+        id: uuidGenerator.smallUuid(),
     };
 }
 function repeatCreateImageCommand(getters, cmd) {
     return {
         ...repeatSheetDependantCommand(getters, cmd),
-        figureId: uuidGenerator.uuidv4(),
+        figureId: uuidGenerator.smallUuid(),
     };
 }
 function repeatCreateFigureCommand(getters, cmd) {
     const newCmd = repeatSheetDependantCommand(getters, cmd);
-    newCmd.figure.id = uuidGenerator.uuidv4();
+    newCmd.figure.id = uuidGenerator.smallUuid();
     return newCmd;
 }
 function repeatCreateSheetCommand(getters, cmd) {
     const newCmd = deepCopy(cmd);
-    newCmd.sheetId = uuidGenerator.uuidv4();
+    newCmd.sheetId = uuidGenerator.smallUuid();
     const sheetName = cmd.name || getters.getSheet(getters.getActiveSheetId()).name;
     // Extract the prefix of the sheet name (everything before the number at the end of the name)
     const namePrefix = sheetName.match(/(.+?)\d*$/)?.[1] || sheetName;
@@ -48007,7 +48125,6 @@ class HistoryPlugin extends UIPlugin {
         super(config);
         this.session = config.session;
         this.session.on("new-local-state-update", this, this.onNewLocalStateUpdate);
-        this.session.on("pending-revisions-dropped", this, ({ revisionIds }) => this.drop(revisionIds));
         this.session.on("snapshot", this, () => {
             this.undoStack = [];
             this.redoStack = [];
@@ -48077,10 +48194,6 @@ class HistoryPlugin extends UIPlugin {
         const lastNonRedoRevision = this.getPossibleRevisionToRepeat();
         return canRepeatRevision(lastNonRedoRevision);
     }
-    drop(revisionIds) {
-        this.undoStack = this.undoStack.filter((id) => !revisionIds.includes(id));
-        this.redoStack = [];
-    }
     onNewLocalStateUpdate({ id }) {
         this.undoStack.push(id);
         this.redoStack = [];
@@ -48347,7 +48460,7 @@ class ClipboardFigureState {
             x: maxX < width ? 0 : Math.min(targetX, maxX - width),
             y: maxY < height ? 0 : Math.min(targetY, maxY - height),
         };
-        const newId = new UuidGenerator().uuidv4();
+        const newId = new UuidGenerator().smallUuid();
         this.copiedFigureContent.paste(sheetId, newId, position, { height, width });
         if (this.operation === "CUT") {
             this.dispatch("DELETE_FIGURE", {
@@ -50050,23 +50163,7 @@ class GridSelectionPlugin extends UIPlugin {
                         gridSelection: deepCopy(gridSelection),
                     };
                 }
-                if (!this.getters.tryGetSheet(this.getters.getActiveSheetId())) {
-                    const currentSheetIds = this.getters.getVisibleSheetIds();
-                    this.activeSheet = this.getters.getSheet(currentSheetIds[0]);
-                    if (this.activeSheet.id in this.sheetsData) {
-                        const { anchor } = this.clipSelection(this.activeSheet.id, this.sheetsData[this.activeSheet.id].gridSelection);
-                        this.selectCell(anchor.cell.col, anchor.cell.row);
-                    }
-                    else {
-                        this.selectCell(0, 0);
-                    }
-                    const { col, row } = this.gridSelection.anchor.cell;
-                    this.moveClient({
-                        sheetId: this.getters.getActiveSheetId(),
-                        col,
-                        row,
-                    });
-                }
+                this.fallbackToVisibleSheet();
                 const sheetId = this.getters.getActiveSheetId();
                 this.gridSelection.zones = this.gridSelection.zones.map((z) => this.getters.expandZone(sheetId, z));
                 this.gridSelection.anchor.zone = this.getters.expandZone(sheetId, this.gridSelection.anchor.zone);
@@ -50076,6 +50173,7 @@ class GridSelectionPlugin extends UIPlugin {
         }
     }
     finalize() {
+        this.fallbackToVisibleSheet();
         /** Any change to the selection has to be  reflected in the selection processor. */
         this.selection.resetDefaultAnchor(this, deepCopy(this.gridSelection.anchor));
     }
@@ -50427,6 +50525,25 @@ class GridSelectionPlugin extends UIPlugin {
         }
         return "Success" /* CommandResult.Success */;
     }
+    fallbackToVisibleSheet() {
+        if (!this.getters.tryGetSheet(this.getters.getActiveSheetId())) {
+            const currentSheetIds = this.getters.getVisibleSheetIds();
+            this.activeSheet = this.getters.getSheet(currentSheetIds[0]);
+            if (this.activeSheet.id in this.sheetsData) {
+                const { anchor } = this.clipSelection(this.activeSheet.id, this.sheetsData[this.activeSheet.id].gridSelection);
+                this.selectCell(anchor.cell.col, anchor.cell.row);
+            }
+            else {
+                this.selectCell(0, 0);
+            }
+            const { col, row } = this.gridSelection.anchor.cell;
+            this.moveClient({
+                sheetId: this.getters.getActiveSheetId(),
+                col,
+                row,
+            });
+        }
+    }
     //-------------------------------------------
     // Helpers for extensions
     // ------------------------------------------
@@ -51574,7 +51691,9 @@ class HeaderPositionsUIPlugin extends UIPlugin {
             case "UNGROUP_HEADERS":
             case "GROUP_HEADERS":
             case "CREATE_SHEET":
-                this.headerPositions[cmd.sheetId] = this.computeHeaderPositionsOfSheet(cmd.sheetId);
+                if (this.getters.tryGetSheet(cmd.sheetId)) {
+                    this.headerPositions[cmd.sheetId] = this.computeHeaderPositionsOfSheet(cmd.sheetId);
+                }
                 break;
             case "DUPLICATE_SHEET":
                 this.headerPositions[cmd.sheetIdTo] = deepCopy(this.headerPositions[cmd.sheetId]);
@@ -51582,12 +51701,14 @@ class HeaderPositionsUIPlugin extends UIPlugin {
         }
     }
     finalize() {
-        if (this.isDirty) {
-            for (const sheetId of this.getters.getSheetIds()) {
+        for (const sheetId of this.getters.getSheetIds()) {
+            // sheets can be created without this plugin being aware of it
+            // in concurrent situations.
+            if (this.isDirty || !this.headerPositions[sheetId]) {
                 this.headerPositions[sheetId] = this.computeHeaderPositionsOfSheet(sheetId);
             }
-            this.isDirty = false;
         }
+        this.isDirty = false;
     }
     /**
      * Returns the size, start and end coordinates of a column on an unfolded sheet
@@ -52271,7 +52392,7 @@ class BottomBar extends owl.Component {
     clickAddSheet(ev) {
         const activeSheetId = this.env.model.getters.getActiveSheetId();
         const position = this.env.model.getters.getSheetIds().findIndex((sheetId) => sheetId === activeSheetId) + 1;
-        const sheetId = this.env.model.uuidGenerator.uuidv4();
+        const sheetId = this.env.model.uuidGenerator.smallUuid();
         const name = this.env.model.getters.getNextSheetName(_t("Sheet"));
         this.env.model.dispatch("CREATE_SHEET", { sheetId, position, name });
         this.env.model.dispatch("ACTIVATE_SHEET", { sheetIdFrom: activeSheetId, sheetIdTo: sheetId });
@@ -53333,7 +53454,7 @@ css /* scss */ `
   .o-font-size-editor {
     height: calc(100% - 4px);
     input.o-font-size {
-      outline-color: ${SELECTION_BORDER_COLOR};
+      outline: none;
       height: 20px;
       width: 23px;
     }
@@ -54508,7 +54629,7 @@ class Tree {
     }
     /**
      * Drop the operation and all following operations in every
-     * branch
+     * branches
      */
     drop(operationId) {
         for (const branch of this.branches) {
@@ -54813,9 +54934,16 @@ class SelectiveHistory {
         this.fastForward();
         this.insert(redoId, this.buildEmpty(redoId), insertAfter);
     }
-    drop(operationId) {
+    rebase(operationId) {
+        const operation = this.get(operationId);
+        const execution = [...this.tree.execution(this.HEAD_BRANCH).startAfter(operationId)];
         this.revertBefore(operationId);
+        const baseId = this.HEAD_OPERATION.id;
         this.tree.drop(operationId);
+        this.insert(operationId, operation, baseId);
+        for (const { operation } of execution) {
+            this.insert(operation.id, operation.data, this.HEAD_OPERATION.id);
+        }
     }
     getRevertedExecution() {
         const data = [];
@@ -57392,6 +57520,11 @@ class Model extends EventBus {
             dispatch: (command) => {
                 const result = this.checkDispatchAllowed(command);
                 if (!result.isSuccessful) {
+                    // core views plugins need to be invalidated
+                    this.dispatchToHandlers(this.coreHandlers, {
+                        type: "UNDO",
+                        commands: [command],
+                    });
                     return;
                 }
                 this.isReplayingCommand = true;
@@ -57420,7 +57553,7 @@ class Model extends EventBus {
     }
     setupConfig(config) {
         const client = config.client || {
-            id: this.uuidGenerator.uuidv4(),
+            id: this.uuidGenerator.smallUuid(),
             name: _t("Anonymous").toString(),
         };
         const transportService = config.transportService || new LocalTransportService();
@@ -57841,6 +57974,6 @@ exports.setTranslationMethod = setTranslationMethod;
 exports.tokenize = tokenize;
 
 
-__info__.version = "17.0.48";
-__info__.date = "2025-02-05T07:18:39.331Z";
-__info__.hash = "aea9101";
+__info__.version = "17.0.50";
+__info__.date = "2025-02-14T08:37:34.289Z";
+__info__.hash = "cc1037d";