@odoo/o-spreadsheet 17.0.49 → 17.0.51

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.49
- * @date 2025-02-10T08:55:47.771Z
- * @hash 1049319
+ * @version 17.0.51
+ * @date 2025-02-25T05:56:10.869Z
+ * @hash b433912
  */
 
 'use strict';
@@ -1910,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",
@@ -1925,6 +1925,7 @@ const invalidateEvaluationCommands = new Set([
     "UNDO",
     "REDO",
     "ADD_MERGE",
+    "DUPLICATE_SHEET",
     "UPDATE_LOCALE",
 ]);
 const invalidateDependenciesCommands = new Set([
@@ -1933,7 +1934,6 @@ const invalidateDependenciesCommands = new Set([
 ]);
 const invalidateCFEvaluationCommands = new Set([
     ...invalidateEvaluationCommands,
-    "DUPLICATE_SHEET",
     "EVALUATE_CELLS",
     "ADD_CONDITIONAL_FORMAT",
     "REMOVE_CONDITIONAL_FORMAT",
@@ -2093,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";
@@ -3673,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.
@@ -4716,20 +4720,53 @@ 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);
+        }
+        else if (window.crypto) {
+            return "10000000-1000".replace(/[01]/g, (c) => {
+                const n = Number(c);
+                return (n ^ (crypto.getRandomValues(new Uint8Array(1))[0] & (15 >> (n / 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++;
             return String(this.fastIdStart);
-            //@ts-ignore
         }
-        else if (window.crypto && window.crypto.getRandomValues) {
-            //@ts-ignore
-            return ([1e7] + -1e3 + -4e3 + -8e3 + -1e11).replace(/[018]/g, (c) => (c ^ (crypto.getRandomValues(new Uint8Array(1))[0] & (15 >> (c / 4)))).toString(16));
+        else if (window.crypto) {
+            return "10000000-1000-4000-8000-100000000000".replace(/[018]/g, (c) => {
+                const n = Number(c);
+                return (n ^ (crypto.getRandomValues(new Uint8Array(1))[0] & (15 >> (n / 4)))).toString(16);
+            });
         }
         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);
             });
         }
@@ -6950,10 +6987,6 @@ class Popover extends owl.Component {
             this.currentDisplayValue = newDisplay;
             if (!anchor)
                 return;
-            el.style.top = "";
-            el.style.left = "";
-            el.style["max-height"] = "";
-            el.style["max-width"] = "";
             const propsMaxSize = { width: this.props.maxWidth, height: this.props.maxHeight };
             let elDims = {
                 width: el.getBoundingClientRect().width,
@@ -7942,7 +7975,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() });
@@ -7952,7 +7985,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,
@@ -8502,17 +8535,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: [] };
@@ -8793,9 +8824,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));
@@ -9339,8 +9368,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()) {
@@ -9352,23 +9381,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;
     }
@@ -9442,9 +9474,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));
@@ -9459,7 +9490,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",
@@ -9681,9 +9712,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));
@@ -9699,7 +9728,7 @@ function createPieChartRuntime(chart, getters) {
         const dataset = {
             label,
             data,
-            borderColor: "#FFFFFF",
+            borderColor: chart.background || "#FFFFFF",
             backgroundColor,
         };
         config.data.datasets.push(dataset);
@@ -10204,21 +10233,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,
@@ -10818,7 +10848,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();
@@ -10852,7 +10882,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");
@@ -12573,6 +12603,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 = {
@@ -13093,6 +13143,7 @@ var math = /*#__PURE__*/Object.freeze({
   ISODD: ISODD,
   ISO_CEILING: ISO_CEILING,
   LN: LN,
+  LOG: LOG,
   MOD: MOD,
   MUNIT: MUNIT,
   ODD: ODD,
@@ -15954,7 +16005,7 @@ const SORTN = {
             }
         }
     },
-    isExported: true,
+    isExported: false,
 };
 // -----------------------------------------------------------------------------
 // UNIQUE
@@ -19855,7 +19906,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 });
     },
@@ -21531,7 +21582,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({
@@ -22581,7 +22632,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;
     }
@@ -23609,7 +23660,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)),
@@ -25558,7 +25609,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,
         };
@@ -26709,25 +26760,21 @@ css /* scss */ `
         }
       }
     }
+  }
+  .o-spreadsheet .o-composer-assistant {
+    pointer-events: none;
 
-    .o-composer-assistant {
-      position: absolute;
-      margin: 1px 4px;
-      pointer-events: none;
-      overflow: auto;
-
-      .o-semi-bold {
-        /** FIXME: to remove in favor of Bootstrap
-        * 'fw-semibold' when we upgrade to Bootstrap 5.2
-        */
-        font-weight: 600 !important;
-      }
+    .o-semi-bold {
+      /** FIXME: to remove in favor of Bootstrap
+      * 'fw-semibold' when we upgrade to Bootstrap 5.2
+      */
+      font-weight: 600 !important;
     }
   }
 `;
 class Composer extends owl.Component {
     static template = "o-spreadsheet-Composer";
-    static components = { TextValueProvider, FunctionDescriptionProvider };
+    static components = { TextValueProvider, FunctionDescriptionProvider, Popover };
     static defaultProps = {
         inputStyle: "",
         isDefaultFocus: false,
@@ -26752,35 +26799,13 @@ class Composer extends owl.Component {
         argToFocus: 0,
     });
     compositionActive = false;
-    spreadsheetRect = useSpreadsheetRect();
     get assistantStyle() {
-        const composerRect = this.composerRef.el.getBoundingClientRect();
         const assistantStyle = {};
-        assistantStyle["min-width"] = `${this.props.rect?.width || ASSISTANT_WIDTH}px`;
+        const minWidth = Math.min(getBoundingRectAsPOJO(this.composerRef.el).width || Infinity, ASSISTANT_WIDTH);
+        assistantStyle["min-width"] = `${minWidth}px`;
         if (this.autoCompleteState.type === "function") {
             assistantStyle.width = `${ASSISTANT_WIDTH}px`;
         }
-        if (this.props.delimitation && this.props.rect) {
-            const { x: cellX, y: cellY, height: cellHeight } = this.props.rect;
-            const remainingHeight = this.props.delimitation.height - (cellY + cellHeight);
-            assistantStyle["max-height"] = `${remainingHeight}px`;
-            if (cellY > remainingHeight) {
-                const availableSpaceAbove = cellY;
-                assistantStyle["max-height"] = `${availableSpaceAbove}px`;
-                // render top
-                // We compensate 2 px of margin on the assistant style + 1px for design reasons
-                assistantStyle.top = `-3px`;
-                assistantStyle.transform = `translate(0, -100%)`;
-            }
-            if (cellX + ASSISTANT_WIDTH > this.props.delimitation.width) {
-                // render left
-                assistantStyle.right = `0px`;
-            }
-        }
-        if (this.props.focus === "contentFocus" &&
-            composerRect.left + ASSISTANT_WIDTH > this.spreadsheetRect.width) {
-            assistantStyle.right = "0px";
-        }
         return cssPropertiesToCss(assistantStyle);
     }
     // we can't allow input events to be triggered while we remove and add back the content of the composer in processContent
@@ -27029,7 +27054,7 @@ class Composer extends owl.Component {
         this.autoCompleteState.selectedIndex = 0;
     }
     updateAutoCompleteIndex(index) {
-        this.autoCompleteState.selectedIndex = clip(0, index, 10);
+        this.autoCompleteState.selectedIndex = index;
     }
     /**
      * This is required to ensure the content helper selection is
@@ -27317,11 +27342,16 @@ class Composer extends owl.Component {
         this.autoCompleteState.values = values.map((value) => ({ text: value, description: "" }));
         this.autoCompleteState.getHtmlContent = (value) => [{ value }];
     }
+    get popoverProps() {
+        return {
+            anchorRect: getBoundingRectAsPOJO(this.composerRef.el),
+            positioning: "BottomLeft",
+            verticalOffset: 0,
+        };
+    }
 }
 Composer.props = {
     inputStyle: { type: String, optional: true },
-    rect: { type: Object, optional: true },
-    delimitation: { type: Object, optional: true },
     focus: { validate: (value) => ["inactive", "cellFocus", "contentFocus"].includes(value) },
     onComposerCellFocused: { type: Function, optional: true },
     onComposerContentFocused: Function,
@@ -27391,13 +27421,7 @@ class GridComposer extends owl.Component {
         });
     }
     get composerProps() {
-        const { width, height } = this.env.model.getters.getSheetViewDimensionWithHeaders();
         return {
-            rect: { ...this.rect },
-            delimitation: {
-                width,
-                height,
-            },
             focus: this.props.focus,
             isDefaultFocus: true,
             onComposerContentFocused: this.props.onComposerContentFocused,
@@ -33844,7 +33868,7 @@ function normalizeV9(formula) {
  * a breaking change is made in the way the state is handled, and an upgrade
  * function should be defined
  */
-const CURRENT_VERSION = 14;
+const CURRENT_VERSION = 14.5;
 const INITIAL_SHEET_ID = "Sheet1";
 /**
  * This function tries to load anything that could look like a valid
@@ -34178,7 +34202,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);
         }
@@ -35854,6 +35878,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 */;
         }
@@ -35895,6 +35923,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);
+            }
         }
     }
     /**
@@ -36973,8 +37004,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 */;
                 }
@@ -37148,6 +37185,7 @@ class DataValidationPlugin extends CorePlugin {
 class FigurePlugin extends CorePlugin {
     static getters = ["getFigures", "getFigure", "getFigureSheetId"];
     figures = {};
+    insertionOrders = []; // TODO use a list in master
     // ---------------------------------------------------------------------------
     // Command Handling
     // ---------------------------------------------------------------------------
@@ -37250,11 +37288,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) {
@@ -37273,7 +37314,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];
@@ -37286,11 +37334,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) {
@@ -37314,10 +37360,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 */
@@ -37519,7 +37565,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);
@@ -39008,6 +39054,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);
@@ -39020,7 +39069,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":
@@ -39833,6 +39882,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));
     }
 }
@@ -39852,7 +39905,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 */;
                 }
@@ -39865,7 +39921,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 */;
                 }
@@ -39876,6 +39935,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 */;
@@ -44145,6 +44207,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,
@@ -44781,21 +44846,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,
@@ -44819,7 +44877,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);
@@ -44827,6 +44884,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) {
@@ -45531,7 +45607,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);
@@ -45563,7 +45641,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.
@@ -47755,6 +47833,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 */;
@@ -47803,23 +47885,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;
@@ -48014,7 +48096,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 = [];
@@ -48084,10 +48165,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 = [];
@@ -48354,7 +48431,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", {
@@ -50057,23 +50134,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);
@@ -50083,6 +50144,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));
     }
@@ -50434,6 +50496,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
     // ------------------------------------------
@@ -51581,7 +51662,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]);
@@ -51589,12 +51672,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
@@ -52278,7 +52363,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 });
@@ -53340,7 +53425,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;
     }
@@ -54515,7 +54600,7 @@ class Tree {
     }
     /**
      * Drop the operation and all following operations in every
-     * branch
+     * branches
      */
     drop(operationId) {
         for (const branch of this.branches) {
@@ -54820,9 +54905,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 = [];
@@ -57399,6 +57491,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;
@@ -57427,7 +57524,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();
@@ -57848,6 +57945,6 @@ exports.setTranslationMethod = setTranslationMethod;
 exports.tokenize = tokenize;
 
 
-__info__.version = "17.0.49";
-__info__.date = "2025-02-10T08:55:47.771Z";
-__info__.hash = "1049319";
+__info__.version = "17.0.51";
+__info__.date = "2025-02-25T05:56:10.869Z";
+__info__.hash = "b433912";