@odoo/o-spreadsheet 17.0.49 → 17.0.51

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.49
- * @date 2025-02-10T08:55:47.771Z
- * @hash 1049319
+ * @version 17.0.51
+ * @date 2025-02-25T05:56:10.869Z
+ * @hash b433912
  */
 
 (function (exports, owl) {
@@ -1909,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",
@@ -1924,6 +1924,7 @@
       "UNDO",
       "REDO",
       "ADD_MERGE",
+      "DUPLICATE_SHEET",
       "UPDATE_LOCALE",
   ]);
   const invalidateDependenciesCommands = new Set([
@@ -1932,7 +1933,6 @@
   ]);
   const invalidateCFEvaluationCommands = new Set([
       ...invalidateEvaluationCommands,
-      "DUPLICATE_SHEET",
       "EVALUATE_CELLS",
       "ADD_CONDITIONAL_FORMAT",
       "REMOVE_CONDITIONAL_FORMAT",
@@ -2092,6 +2092,7 @@
       CommandResult["InvalidRange"] = "InvalidRange";
       CommandResult["InvalidZones"] = "InvalidZones";
       CommandResult["InvalidSheetId"] = "InvalidSheetId";
+      CommandResult["InvalidCellId"] = "InvalidCellId";
       CommandResult["InvalidFigureId"] = "InvalidFigureId";
       CommandResult["InputAlreadyFocused"] = "InputAlreadyFocused";
       CommandResult["MaximumRangesReached"] = "MaximumRangesReached";
@@ -3672,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.
@@ -4715,20 +4719,53 @@
       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);
               });
           }
@@ -6949,10 +6986,6 @@
               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,
@@ -7941,7 +7974,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() });
@@ -7951,7 +7984,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,
@@ -8501,17 +8534,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: [] };
@@ -8792,9 +8823,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));
@@ -9338,8 +9367,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()) {
@@ -9351,23 +9380,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;
       }
@@ -9441,9 +9473,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));
@@ -9458,7 +9489,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",
@@ -9680,9 +9711,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));
@@ -9698,7 +9727,7 @@
           const dataset = {
               label,
               data,
-              borderColor: "#FFFFFF",
+              borderColor: chart.background || "#FFFFFF",
               backgroundColor,
           };
           config.data.datasets.push(dataset);
@@ -10203,21 +10232,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,
@@ -10817,7 +10847,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();
@@ -10851,7 +10881,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");
@@ -12572,6 +12602,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 = {
@@ -13092,6 +13142,7 @@
     ISODD: ISODD,
     ISO_CEILING: ISO_CEILING,
     LN: LN,
+    LOG: LOG,
     MOD: MOD,
     MUNIT: MUNIT,
     ODD: ODD,
@@ -15953,7 +16004,7 @@
               }
           }
       },
-      isExported: true,
+      isExported: false,
   };
   // -----------------------------------------------------------------------------
   // UNIQUE
@@ -19854,7 +19905,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 });
       },
@@ -21530,7 +21581,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({
@@ -22580,7 +22631,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;
       }
@@ -23608,7 +23659,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)),
@@ -25557,7 +25608,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,
           };
@@ -26708,25 +26759,21 @@
         }
       }
     }
+  }
+  .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,
@@ -26751,35 +26798,13 @@
           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
@@ -27028,7 +27053,7 @@
           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
@@ -27316,11 +27341,16 @@
           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,
@@ -27390,13 +27420,7 @@
           });
       }
       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,
@@ -33843,7 +33867,7 @@
    * 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
@@ -34177,7 +34201,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);
           }
@@ -35853,6 +35877,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 */;
           }
@@ -35894,6 +35922,9 @@
               case "CLEAR_CELLS":
                   this.clearCells(cmd.sheetId, cmd.target);
                   break;
+              case "DELETE_SHEET": {
+                  this.history.update("cells", cmd.sheetId, undefined);
+              }
           }
       }
       /**
@@ -36972,8 +37003,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 */;
                   }
@@ -37147,6 +37184,7 @@
   class FigurePlugin extends CorePlugin {
       static getters = ["getFigures", "getFigure", "getFigureSheetId"];
       figures = {};
+      insertionOrders = []; // TODO use a list in master
       // ---------------------------------------------------------------------------
       // Command Handling
       // ---------------------------------------------------------------------------
@@ -37249,11 +37287,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) {
@@ -37272,7 +37313,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];
@@ -37285,11 +37333,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) {
@@ -37313,10 +37359,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 */
@@ -37518,7 +37564,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);
@@ -39007,6 +39053,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);
@@ -39019,7 +39068,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":
@@ -39832,6 +39881,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));
       }
   }
@@ -39851,7 +39904,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 */;
                   }
@@ -39864,7 +39920,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 */;
                   }
@@ -39875,6 +39934,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 */;
@@ -44144,6 +44206,9 @@
       };
   }
   function createSheetTransformation(toTransform, executed) {
+      if (toTransform.sheetId === executed.sheetId) {
+          toTransform = { ...toTransform, sheetId: `${toTransform.sheetId}~` };
+      }
       if (toTransform.name === executed.name) {
           return {
               ...toTransform,
@@ -44780,21 +44845,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,
@@ -44818,7 +44876,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);
@@ -44826,6 +44883,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) {
@@ -45530,7 +45606,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);
@@ -45562,7 +45640,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.
@@ -47754,6 +47832,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 */;
@@ -47802,23 +47884,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;
@@ -48013,7 +48095,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 = [];
@@ -48083,10 +48164,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 = [];
@@ -48353,7 +48430,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", {
@@ -50056,23 +50133,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);
@@ -50082,6 +50143,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));
       }
@@ -50433,6 +50495,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
       // ------------------------------------------
@@ -51580,7 +51661,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]);
@@ -51588,12 +51671,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
@@ -52277,7 +52362,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 });
@@ -53339,7 +53424,7 @@
   .o-font-size-editor {
     height: calc(100% - 4px);
     input.o-font-size {
-      outline-color: ${SELECTION_BORDER_COLOR};
+      outline: none;
       height: 20px;
       width: 23px;
     }
@@ -54514,7 +54599,7 @@
       }
       /**
        * Drop the operation and all following operations in every
-       * branch
+       * branches
        */
       drop(operationId) {
           for (const branch of this.branches) {
@@ -54819,9 +54904,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 = [];
@@ -57398,6 +57490,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;
@@ -57426,7 +57523,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();
@@ -57847,9 +57944,9 @@
   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";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);