@odoo/o-spreadsheet 17.4.22 → 17.4.23

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.4.22
- * @date 2025-02-10T09:14:48.889Z
- * @hash 667f2b1
+ * @version 17.4.23
+ * @date 2025-02-14T08:37:12.219Z
+ * @hash 8339907
  */
 
 'use strict';
@@ -5500,6 +5500,37 @@ class UuidGenerator {
     setIsFastStrategy(isFast) {
         this.isFastIdStrategy = isFast;
     }
+    /**
+     * Generates a custom UUID using a simple 36^12 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++;
@@ -5513,7 +5544,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);
             });
         }
@@ -6900,7 +6931,7 @@ class ChartClipboardHandler extends AbstractFigureClipboardHandler {
         };
     }
     getPasteTarget(sheetId, target, content, options) {
-        const newId = new UuidGenerator().uuidv4();
+        const newId = new UuidGenerator().smallUuid();
         return { zones: [], figureId: newId, sheetId };
     }
     paste(target, clippedContent, options) {
@@ -7042,7 +7073,9 @@ class ConditionalFormatClipboardHandler extends AbstractCellClipboardHandler {
         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: [] };
     }
 }
 
@@ -7127,7 +7160,7 @@ class DataValidationClipboardHandler extends AbstractCellClipboardHandler {
             originRule.isBlocking === rule.isBlocking);
         return ruleInTargetSheet
             ? ruleInTargetSheet
-            : { ...originRule, id: newId ? this.uuidGenerator.uuidv4() : originRule.id, ranges: [] };
+            : { ...originRule, id: newId ? this.uuidGenerator.smallUuid() : originRule.id, ranges: [] };
     }
     /**
      * Add or remove XCs to a given data validation rule.
@@ -7168,7 +7201,7 @@ class ImageClipboardHandler extends AbstractFigureClipboardHandler {
         };
     }
     getPasteTarget(sheetId, target, content, options) {
-        const newId = new UuidGenerator().uuidv4();
+        const newId = new UuidGenerator().smallUuid();
         return { sheetId, zones: [], figureId: newId };
     }
     paste(target, clippedContent, options) {
@@ -13037,6 +13070,25 @@ 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.")),
+    ],
+    compute: function (value, base = { value: 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
 // -----------------------------------------------------------------------------
 function mod(dividend, divisor) {
@@ -13550,6 +13602,7 @@ var math = /*#__PURE__*/Object.freeze({
   ISODD: ISODD,
   ISO_CEILING: ISO_CEILING,
   LN: LN,
+  LOG: LOG,
   MOD: MOD,
   MUNIT: MUNIT,
   ODD: ODD,
@@ -16258,7 +16311,7 @@ const SORTN = {
             }
         }
     },
-    isExported: true,
+    isExported: false,
 };
 // -----------------------------------------------------------------------------
 // UNIQUE
@@ -23524,17 +23577,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: [] };
@@ -23872,9 +23923,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));
@@ -24051,8 +24100,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()) {
@@ -24064,23 +24113,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;
     }
@@ -24183,9 +24235,8 @@ function createLineOrScatterChartRuntime(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));
@@ -24200,7 +24251,7 @@ function createLineOrScatterChartRuntime(chart, getters) {
     const dataSetFormat = getChartDatasetFormat(getters, chart.dataSets);
     const options = { format: dataSetFormat, locale, truncateLabels };
     const config = getLineOrScatterConfiguration(chart, labels, options);
-    const labelFormat = getChartLabelFormat(getters, chart.labelRange);
+    const labelFormat = getChartLabelFormat(getters, chart.labelRange, removeFirstLabel);
     if (axisType === "time") {
         const axis = {
             type: "time",
@@ -24414,9 +24465,7 @@ function createComboChartRuntime(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));
@@ -25069,9 +25118,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));
@@ -25619,9 +25666,7 @@ function createWaterfallChartRuntime(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));
@@ -27139,7 +27184,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() });
@@ -27149,7 +27194,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,
@@ -27764,20 +27809,21 @@ 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,
-            legendPosition: newLegendPos,
-        };
+    const lineChartDefinition = {
+        title: {},
+        dataSets,
+        labelsAsText: false,
+        stacked: false,
+        aggregated: false,
+        cumulative: false,
+        labelRange: labelRangeXc,
+        type: "line",
+        dataSetsHaveTitle,
+        legendPosition: newLegendPos,
+    };
+    const chart = new LineChart(lineChartDefinition, sheetId, getters);
+    if (canChartParseLabels(chart, getters)) {
+        return lineChartDefinition;
     }
     const _dataSets = createDataSets(getters, dataSets, sheetId, dataSetsHaveTitle);
     if (singleColumn &&
@@ -28471,7 +28517,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();
@@ -28494,8 +28540,8 @@ const CREATE_CHART = (env) => {
 // Pivots
 //------------------------------------------------------------------------------
 const CREATE_PIVOT = (env) => {
-    const pivotId = env.model.uuidGenerator.uuidv4();
-    const newSheetId = env.model.uuidGenerator.uuidv4();
+    const pivotId = env.model.uuidGenerator.smallUuid();
+    const newSheetId = env.model.uuidGenerator.smallUuid();
     const result = env.model.dispatch("INSERT_NEW_PIVOT", { pivotId, newSheetId });
     if (result.isSuccessful) {
         env.openSidePanel("PivotSidePanel", { pivotId });
@@ -28554,7 +28600,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");
@@ -29107,7 +29153,7 @@ const insertCheckbox = {
             ranges,
             sheetId,
             rule: {
-                id: env.model.uuidGenerator.uuidv4(),
+                id: env.model.uuidGenerator.smallUuid(),
                 criterion: {
                     type: "isBoolean",
                     values: [],
@@ -29123,7 +29169,7 @@ const insertDropdown = {
         const zones = env.model.getters.getSelectedZones();
         const sheetId = env.model.getters.getActiveSheetId();
         const ranges = zones.map((zone) => env.model.getters.getRangeDataFromZone(sheetId, zone));
-        const ruleID = env.model.uuidGenerator.uuidv4();
+        const ruleID = env.model.uuidGenerator.smallUuid();
         env.model.dispatch("ADD_DATA_VALIDATION_RULE", {
             ranges,
             sheetId,
@@ -29154,7 +29200,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 });
     },
@@ -32853,7 +32899,7 @@ class LineConfigPanel extends GenericChartConfigPanel {
     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;
     }
@@ -32925,7 +32971,7 @@ class ScatterConfigPanel extends GenericChartConfigPanel {
     get canTreatLabelsAsText() {
         const chart = this.env.model.getters.getChart(this.props.figureId);
         if (chart && chart instanceof ScatterChart) {
-            return canChartParseLabels(chart.labelRange, this.env.model.getters);
+            return canChartParseLabels(chart, this.env.model.getters);
         }
         return false;
     }
@@ -34224,7 +34270,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)),
@@ -35196,7 +35242,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,
         };
@@ -36315,8 +36361,8 @@ class PivotTitleSection extends owl.Component {
         return this.env.model.getters.getPivotDisplayName(this.props.pivotId);
     }
     duplicatePivot() {
-        const newPivotId = this.env.model.uuidGenerator.uuidv4();
-        const newSheetId = this.env.model.uuidGenerator.uuidv4();
+        const newPivotId = this.env.model.uuidGenerator.smallUuid();
+        const newSheetId = this.env.model.uuidGenerator.smallUuid();
         const result = this.env.model.dispatch("DUPLICATE_PIVOT_IN_NEW_SHEET", {
             pivotId: this.props.pivotId,
             newPivotId,
@@ -38645,7 +38691,7 @@ class TableStyleEditorPanel extends owl.Component {
         this.state.selectedTemplateName = templateName;
     }
     onConfirm() {
-        const tableStyleId = this.props.styleId || this.env.model.uuidGenerator.uuidv4();
+        const tableStyleId = this.props.styleId || this.env.model.uuidGenerator.smallUuid();
         this.env.model.dispatch("CREATE_TABLE_STYLE", {
             tableStyleId,
             tableStyleName: this.state.styleName,
@@ -50926,7 +50972,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":
@@ -51783,7 +51829,7 @@ class TablePlugin extends CorePlugin {
                 const union = this.getters.getRangesUnion(ranges);
                 const mergesInTarget = this.getters.getMergesInZone(cmd.sheetId, union.zone);
                 this.dispatch("REMOVE_MERGE", { sheetId: cmd.sheetId, target: mergesInTarget });
-                const id = this.uuidGenerator.uuidv4();
+                const id = this.uuidGenerator.smallUuid();
                 const config = cmd.config || DEFAULT_TABLE_CONFIG;
                 const newTable = cmd.tableType === "dynamic"
                     ? this.createDynamicTable(id, union, config)
@@ -51936,7 +51982,7 @@ class TablePlugin extends CorePlugin {
             filters = [];
             for (const i of range(zone.left, zone.right + 1)) {
                 const filterZone = { ...zone, left: i, right: i };
-                const uid = this.uuidGenerator.uuidv4();
+                const uid = this.uuidGenerator.smallUuid();
                 filters.push(this.createFilterFromZone(uid, tableRange.sheetId, filterZone, config));
             }
         }
@@ -52001,7 +52047,7 @@ class TablePlugin extends CorePlugin {
                     ? table.filters.find((f) => f.col === i)
                     : undefined;
                 const filterZone = { ...tableZone, left: i, right: i };
-                const filterId = oldFilter?.id || this.uuidGenerator.uuidv4();
+                const filterId = oldFilter?.id || this.uuidGenerator.smallUuid();
                 filters.push(this.createFilterFromZone(filterId, tableRange.sheetId, filterZone, config));
             }
         }
@@ -52102,7 +52148,7 @@ class TablePlugin extends CorePlugin {
         if (filters.length < zoneToDimension(tableZone).numberOfCols) {
             for (let col = tableZone.left; col <= tableZone.right; col++) {
                 if (!filters.find((filter) => filter.col === col)) {
-                    const uid = this.uuidGenerator.uuidv4();
+                    const uid = this.uuidGenerator.smallUuid();
                     const filterZone = { ...tableZone, left: col, right: col };
                     filters.push(this.createFilterFromZone(uid, sheetId, filterZone, table.config));
                 }
@@ -59051,23 +59097,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;
@@ -60537,23 +60583,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);
@@ -60563,6 +60593,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));
     }
@@ -60866,6 +60897,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
     // ------------------------------------------
@@ -62845,7 +62895,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 });
@@ -68403,7 +68453,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();
@@ -68922,6 +68972,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "17.4.22";
-__info__.date = "2025-02-10T09:14:48.889Z";
-__info__.hash = "667f2b1";
+__info__.version = "17.4.23";
+__info__.date = "2025-02-14T08:37:12.219Z";
+__info__.hash = "8339907";

package/dist/o-spreadsheet.d.ts

@@ -4884,6 +4884,7 @@ declare class GridSelectionPlugin extends UIPlugin {
     private onAddElements;
     private onMoveElements;
     private isMoveElementAllowed;
+    private fallbackToVisibleSheet;
     /**
      * Clip the selection if it spans outside the sheet
      */
@@ -5642,6 +5643,20 @@ declare class UuidGenerator {
     private isFastIdStrategy;
     private fastIdStart;
     setIsFastStrategy(isFast: boolean): void;
+    /**
+     * Generates a custom UUID using a simple 36^12 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(): string;
+    /**
+     * 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(): string;
 }