@odoo/o-spreadsheet 17.0.48 → 17.0.50

package/dist/o-spreadsheet.iife.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
  */
 
 (function (exports, owl) {
@@ -687,9 +687,16 @@
       }
       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.
@@ -1902,12 +1909,12 @@
   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",
@@ -1917,6 +1924,7 @@
       "UNDO",
       "REDO",
       "ADD_MERGE",
+      "DUPLICATE_SHEET",
       "UPDATE_LOCALE",
   ]);
   const invalidateDependenciesCommands = new Set([
@@ -1925,7 +1933,6 @@
   ]);
   const invalidateCFEvaluationCommands = new Set([
       ...invalidateEvaluationCommands,
-      "DUPLICATE_SHEET",
       "EVALUATE_CELLS",
       "ADD_CONDITIONAL_FORMAT",
       "REMOVE_CONDITIONAL_FORMAT",
@@ -2085,6 +2092,7 @@
       CommandResult["InvalidRange"] = "InvalidRange";
       CommandResult["InvalidZones"] = "InvalidZones";
       CommandResult["InvalidSheetId"] = "InvalidSheetId";
+      CommandResult["InvalidCellId"] = "InvalidCellId";
       CommandResult["InvalidFigureId"] = "InvalidFigureId";
       CommandResult["InputAlreadyFocused"] = "InputAlreadyFocused";
       CommandResult["MaximumRangesReached"] = "MaximumRangesReached";
@@ -3665,17 +3673,20 @@
    */
   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.
@@ -4708,6 +4719,37 @@
       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++;
@@ -4721,7 +4763,7 @@
           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);
               });
           }
@@ -7934,7 +7976,7 @@
   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() });
@@ -7944,7 +7986,7 @@
       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,
@@ -8494,17 +8536,15 @@
           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: [] };
@@ -8785,9 +8825,7 @@
       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));
@@ -9331,8 +9369,8 @@
       }
       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()) {
@@ -9344,23 +9382,26 @@
       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;
       }
@@ -9434,9 +9475,8 @@
       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));
@@ -9451,7 +9491,7 @@
       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",
@@ -9673,9 +9713,7 @@
       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));
@@ -10196,21 +10234,22 @@
       }
       // 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,
@@ -10810,7 +10849,7 @@
   //------------------------------------------------------------------------------
   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();
@@ -10844,7 +10883,7 @@
   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");
@@ -12565,6 +12604,26 @@
       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 = {
@@ -13085,6 +13144,7 @@
     ISODD: ISODD,
     ISO_CEILING: ISO_CEILING,
     LN: LN,
+    LOG: LOG,
     MOD: MOD,
     MUNIT: MUNIT,
     ODD: ODD,
@@ -15946,7 +16006,7 @@
               }
           }
       },
-      isExported: true,
+      isExported: false,
   };
   // -----------------------------------------------------------------------------
   // UNIQUE
@@ -19847,7 +19907,7 @@
       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 });
       },
@@ -21523,7 +21583,7 @@
    */
   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({
@@ -22573,7 +22633,7 @@
       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;
       }
@@ -23601,7 +23661,7 @@
       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)),
@@ -25550,7 +25610,7 @@
               .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,
           };
@@ -34170,7 +34230,7 @@
       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);
           }
@@ -35846,6 +35906,10 @@
                   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 */;
           }
@@ -35887,6 +35951,9 @@
               case "CLEAR_CELLS":
                   this.clearCells(cmd.sheetId, cmd.target);
                   break;
+              case "DELETE_SHEET": {
+                  this.history.update("cells", cmd.sheetId, undefined);
+              }
           }
       }
       /**
@@ -36965,8 +37032,14 @@
       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 */;
                   }
@@ -37140,6 +37213,7 @@
   class FigurePlugin extends CorePlugin {
       static getters = ["getFigures", "getFigure", "getFigureSheetId"];
       figures = {};
+      insertionOrders = []; // TODO use a list in master
       // ---------------------------------------------------------------------------
       // Command Handling
       // ---------------------------------------------------------------------------
@@ -37242,11 +37316,14 @@
       }
       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) {
@@ -37265,7 +37342,14 @@
       // 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];
@@ -37278,11 +37362,9 @@
       // ---------------------------------------------------------------------------
       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) {
@@ -37306,10 +37388,10 @@
           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 */
@@ -37511,7 +37593,7 @@
               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);
@@ -39000,6 +39082,9 @@
               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);
@@ -39012,7 +39097,7 @@
               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":
@@ -39825,6 +39910,10 @@
       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));
       }
   }
@@ -39844,7 +39933,10 @@
       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 */;
                   }
@@ -39857,7 +39949,10 @@
                   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 */;
                   }
@@ -39868,6 +39963,9 @@
               }
               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 */;
@@ -44137,6 +44235,9 @@
       };
   }
   function createSheetTransformation(toTransform, executed) {
+      if (toTransform.sheetId === executed.sheetId) {
+          toTransform = { ...toTransform, sheetId: `${toTransform.sheetId}~` };
+      }
       if (toTransform.name === executed.name) {
           return {
               ...toTransform,
@@ -44773,21 +44874,14 @@
           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,
@@ -44811,7 +44905,6 @@
           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);
@@ -44819,6 +44912,25 @@
                   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) {
@@ -45523,7 +45635,9 @@
           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);
@@ -45555,7 +45669,7 @@
               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.
@@ -47747,6 +47861,10 @@
        */
       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 */;
@@ -47795,23 +47913,23 @@
   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;
@@ -48006,7 +48124,6 @@
           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 = [];
@@ -48076,10 +48193,6 @@
           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 = [];
@@ -48346,7 +48459,7 @@
               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", {
@@ -50049,23 +50162,7 @@
                           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);
@@ -50075,6 +50172,7 @@
           }
       }
       finalize() {
+          this.fallbackToVisibleSheet();
           /** Any change to the selection has to be  reflected in the selection processor. */
           this.selection.resetDefaultAnchor(this, deepCopy(this.gridSelection.anchor));
       }
@@ -50426,6 +50524,25 @@
           }
           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
       // ------------------------------------------
@@ -51573,7 +51690,9 @@
               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]);
@@ -51581,12 +51700,14 @@
           }
       }
       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
@@ -52270,7 +52391,7 @@
       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 });
@@ -53332,7 +53453,7 @@
   .o-font-size-editor {
     height: calc(100% - 4px);
     input.o-font-size {
-      outline-color: ${SELECTION_BORDER_COLOR};
+      outline: none;
       height: 20px;
       width: 23px;
     }
@@ -54507,7 +54628,7 @@
       }
       /**
        * Drop the operation and all following operations in every
-       * branch
+       * branches
        */
       drop(operationId) {
           for (const branch of this.branches) {
@@ -54812,9 +54933,16 @@
           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 = [];
@@ -57391,6 +57519,11 @@
               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;
@@ -57419,7 +57552,7 @@
       }
       setupConfig(config) {
           const client = config.client || {
-              id: this.uuidGenerator.uuidv4(),
+              id: this.uuidGenerator.smallUuid(),
               name: _t("Anonymous").toString(),
           };
           const transportService = config.transportService || new LocalTransportService();
@@ -57840,9 +57973,9 @@
   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";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);