@odoo/o-spreadsheet 17.4.8 → 17.4.10

package/dist/o-spreadsheet.iife.js

@@ -2,14 +2,119 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.4.8
- * @date 2024-09-17T08:10:18.777Z
- * @hash caad8bd
+ * @version 17.4.10
+ * @date 2024-10-24T08:54:56.262Z
+ * @hash c82d9c1
  */
 
 (function (exports, owl) {
   'use strict';
 
+  function createActions(menuItems) {
+      return menuItems.map(createAction).sort((a, b) => a.sequence - b.sequence);
+  }
+  let nextItemId = 1;
+  function createAction(item) {
+      const name = item.name;
+      const children = item.children;
+      const description = item.description;
+      const icon = item.icon;
+      const secondaryIcon = item.secondaryIcon;
+      const itemId = item.id || nextItemId++;
+      return {
+          id: itemId.toString(),
+          name: typeof name === "function" ? name : () => name,
+          isVisible: item.isVisible ? item.isVisible : () => true,
+          isEnabled: item.isEnabled ? item.isEnabled : () => true,
+          isActive: item.isActive,
+          execute: item.execute,
+          children: children
+              ? (env) => {
+                  return children
+                      .map((child) => (typeof child === "function" ? child(env) : child))
+                      .flat()
+                      .map(createAction);
+              }
+              : () => [],
+          isReadonlyAllowed: item.isReadonlyAllowed || false,
+          separator: item.separator || false,
+          icon: typeof icon === "function" ? icon : () => icon || "",
+          secondaryIcon: typeof secondaryIcon === "function" ? secondaryIcon : () => secondaryIcon || "",
+          description: typeof description === "function" ? description : () => description || "",
+          textColor: item.textColor,
+          sequence: item.sequence || 0,
+          onStartHover: item.onStartHover,
+          onStopHover: item.onStopHover,
+      };
+  }
+
+  /**
+   * Registry
+   *
+   * The Registry class is basically just a mapping from a string key to an object.
+   * It is really not much more than an object. It is however useful for the
+   * following reasons:
+   *
+   * 1. it let us react and execute code when someone add something to the registry
+   *   (for example, the FunctionRegistry subclass this for this purpose)
+   * 2. it throws an error when the get operation fails
+   * 3. it provides a chained API to add items to the registry.
+   */
+  class Registry {
+      content = {};
+      /**
+       * Add an item to the registry
+       *
+       * Note that this also returns the registry, so another add method call can
+       * be chained
+       */
+      add(key, value) {
+          this.content[key] = value;
+          return this;
+      }
+      /**
+       * Get an item from the registry
+       */
+      get(key) {
+          /**
+           * Note: key in {} is ~12 times slower than {}[key].
+           * So, we check the absence of key only when the direct access returns
+           * a falsy value. It's done to ensure that the registry can contains falsy values
+           */
+          const content = this.content[key];
+          if (!content) {
+              if (!(key in this.content)) {
+                  throw new Error(`Cannot find ${key} in this registry!`);
+              }
+          }
+          return content;
+      }
+      /**
+       * Check if the key is already in the registry
+       */
+      contains(key) {
+          return key in this.content;
+      }
+      /**
+       * Get a list of all elements in the registry
+       */
+      getAll() {
+          return Object.values(this.content);
+      }
+      /**
+       * Get a list of all keys in the registry
+       */
+      getKeys() {
+          return Object.keys(this.content);
+      }
+      /**
+       * Remove an item from the registry
+       */
+      remove(key) {
+          delete this.content[key];
+      }
+  }
+
   const CANVAS_SHIFT = 0.5;
   // Colors
   const HIGHLIGHT_COLOR = "#37A850";
@@ -166,7 +271,7 @@
       underline: false,
       fontSize: 10,
       fillColor: "",
-      textColor: "#000000",
+      textColor: "",
   };
   const DEFAULT_VERTICAL_ALIGN = DEFAULT_STYLE.verticalAlign;
   const DEFAULT_WRAPPING_MODE = DEFAULT_STYLE.wrapping;
@@ -5398,110 +5503,6 @@
       }
   }
 
-  function createActions(menuItems) {
-      return menuItems.map(createAction).sort((a, b) => a.sequence - b.sequence);
-  }
-  const uuidGenerator$1 = new UuidGenerator();
-  function createAction(item) {
-      const name = item.name;
-      const children = item.children;
-      const description = item.description;
-      const icon = item.icon;
-      const secondaryIcon = item.secondaryIcon;
-      return {
-          id: item.id || uuidGenerator$1.uuidv4(),
-          name: typeof name === "function" ? name : () => name,
-          isVisible: item.isVisible ? item.isVisible : () => true,
-          isEnabled: item.isEnabled ? item.isEnabled : () => true,
-          isActive: item.isActive,
-          execute: item.execute,
-          children: children
-              ? (env) => {
-                  return children
-                      .map((child) => (typeof child === "function" ? child(env) : child))
-                      .flat()
-                      .map(createAction);
-              }
-              : () => [],
-          isReadonlyAllowed: item.isReadonlyAllowed || false,
-          separator: item.separator || false,
-          icon: typeof icon === "function" ? icon : () => icon || "",
-          secondaryIcon: typeof secondaryIcon === "function" ? secondaryIcon : () => secondaryIcon || "",
-          description: typeof description === "function" ? description : () => description || "",
-          textColor: item.textColor,
-          sequence: item.sequence || 0,
-          onStartHover: item.onStartHover,
-          onStopHover: item.onStopHover,
-      };
-  }
-
-  /**
-   * Registry
-   *
-   * The Registry class is basically just a mapping from a string key to an object.
-   * It is really not much more than an object. It is however useful for the
-   * following reasons:
-   *
-   * 1. it let us react and execute code when someone add something to the registry
-   *   (for example, the FunctionRegistry subclass this for this purpose)
-   * 2. it throws an error when the get operation fails
-   * 3. it provides a chained API to add items to the registry.
-   */
-  class Registry {
-      content = {};
-      /**
-       * Add an item to the registry
-       *
-       * Note that this also returns the registry, so another add method call can
-       * be chained
-       */
-      add(key, value) {
-          this.content[key] = value;
-          return this;
-      }
-      /**
-       * Get an item from the registry
-       */
-      get(key) {
-          /**
-           * Note: key in {} is ~12 times slower than {}[key].
-           * So, we check the absence of key only when the direct access returns
-           * a falsy value. It's done to ensure that the registry can contains falsy values
-           */
-          const content = this.content[key];
-          if (!content) {
-              if (!(key in this.content)) {
-                  throw new Error(`Cannot find ${key} in this registry!`);
-              }
-          }
-          return content;
-      }
-      /**
-       * Check if the key is already in the registry
-       */
-      contains(key) {
-          return key in this.content;
-      }
-      /**
-       * Get a list of all elements in the registry
-       */
-      getAll() {
-          return Object.values(this.content);
-      }
-      /**
-       * Get a list of all keys in the registry
-       */
-      getKeys() {
-          return Object.keys(this.content);
-      }
-      /**
-       * Remove an item from the registry
-       */
-      remove(key) {
-          delete this.content[key];
-      }
-  }
-
   function getClipboardDataPositions(sheetId, zones) {
       const lefts = new Set(zones.map((z) => z.left));
       const rights = new Set(zones.map((z) => z.right));
@@ -7280,31 +7281,34 @@
               for (let col of columnsIndexes) {
                   const position = { col, row, sheetId };
                   const table = this.getters.getTable(position);
-                  if (!table || copiedTablesIds.has(table.id)) {
+                  if (!table) {
                       tableCellsInRow.push({});
                       continue;
                   }
                   const coreTable = this.getters.getCoreTable(position);
                   const tableZone = coreTable?.range.zone;
+                  let copiedTable = undefined;
                   // Copy whole table
-                  if (coreTable && tableZone && zones.some((z) => isZoneInside(tableZone, z))) {
-                      copiedTablesIds.add(coreTable.id);
+                  if (!copiedTablesIds.has(table.id) &&
+                      coreTable &&
+                      tableZone &&
+                      zones.some((z) => isZoneInside(tableZone, z))) {
+                      copiedTablesIds.add(table.id);
                       const values = [];
                       for (const col of range(tableZone.left, tableZone.right + 1)) {
                           values.push(this.getters.getFilterHiddenValues({ sheetId, col, row: tableZone.top }));
                       }
-                      tableCellsInRow.push({
-                          table: {
-                              range: coreTable.range,
-                              config: coreTable.config,
-                              type: coreTable.type,
-                          },
-                      });
-                  }
-                  // Copy only style of cell
-                  else if (table) {
-                      tableCellsInRow.push({ style: this.getTableStyleToCopy(position) });
+                      copiedTable = {
+                          range: coreTable.range,
+                          config: coreTable.config,
+                          type: coreTable.type,
+                      };
                   }
+                  tableCellsInRow.push({
+                      table: copiedTable,
+                      style: this.getTableStyleToCopy(position),
+                      isWholeTableCopied: copiedTablesIds.has(table.id),
+                  });
               }
           }
           return {
@@ -7385,11 +7389,14 @@
                   tableType: tableCell.table.type,
               });
           }
-          // Do not paste table style if we're inside another table
           // We cannot check for dynamic tables, because at this point the paste can have changed the evaluation, and the
           // dynamic tables are not yet computed
-          if (!this.getters.getCoreTable(position)) {
-              if (tableCell.style?.style && options?.pasteOption !== "asValue") {
+          if (this.getters.getCoreTable(position) || options?.pasteOption === "asValue") {
+              return;
+          }
+          if ((!options?.pasteOption && !tableCell.isWholeTableCopied) ||
+              options?.pasteOption === "onlyFormat") {
+              if (tableCell.style?.style) {
                   this.dispatch("UPDATE_CELL", { ...position, style: tableCell.style.style });
               }
               if (tableCell.style?.border) {
@@ -10617,67 +10624,108 @@ stores.inject(MyMetaStore, storeInstance);
           ctx.save();
           ctx.textAlign = "center";
           ctx.textBaseline = "middle";
-          ctx.fillStyle = chartFontColor(options.background);
-          ctx.strokeStyle = chartFontColor(ctx.fillStyle);
-          chart._metasets.forEach(function (dataset) {
-              switch (dataset.type) {
-                  case "doughnut":
-                  case "pie": {
-                      for (let i = 0; i < dataset._parsed.length; i++) {
-                          const bar = dataset.data[i];
-                          const { startAngle, endAngle, innerRadius, outerRadius } = bar;
-                          const midAngle = (startAngle + endAngle) / 2;
-                          const midRadius = (innerRadius + outerRadius) / 2;
-                          const x = bar.x + midRadius * Math.cos(midAngle);
-                          const y = bar.y + midRadius * Math.sin(midAngle) + 7;
-                          ctx.fillStyle = chartFontColor(bar.options.backgroundColor);
-                          ctx.strokeStyle = chartFontColor(ctx.fillStyle);
-                          const value = options.callback(dataset._parsed[i]);
-                          ctx.strokeText(value, x, y);
-                          ctx.fillText(value, x, y);
-                      }
-                      break;
-                  }
-                  case "bar":
-                  case "line": {
-                      const yOffset = dataset.type === "bar" && !options.horizontal ? 0 : 3;
-                      for (let i = 0; i < dataset._parsed.length; i++) {
-                          const point = dataset.data[i];
-                          const value = options.horizontal ? dataset._parsed[i].x : dataset._parsed[i].y;
-                          const displayedValue = options.callback(value - 0);
-                          let xPosition = 0, yPosition = 0;
-                          if (options.horizontal) {
-                              yPosition = point.y;
-                              if (value < 0) {
-                                  ctx.textAlign = "right";
-                                  xPosition = point.x - yOffset;
-                              }
-                              else {
-                                  ctx.textAlign = "left";
-                                  xPosition = point.x + yOffset;
-                              }
-                          }
-                          else {
-                              xPosition = point.x;
-                              if (value < 0) {
-                                  ctx.textBaseline = "top";
-                                  yPosition = point.y + yOffset;
-                              }
-                              else {
-                                  ctx.textBaseline = "bottom";
-                                  yPosition = point.y - yOffset;
-                              }
-                          }
-                          ctx.strokeText(displayedValue, xPosition, yPosition);
-                          ctx.fillText(displayedValue, xPosition, yPosition);
-                      }
-                      break;
-                  }
-              }
-          });
+          ctx.miterLimit = 1; // Avoid sharp artifacts on strokeText
+          switch (chart.config.type) {
+              case "pie":
+              case "doughnut":
+                  drawPieChartValues(chart, options, ctx);
+                  break;
+              case "bar":
+              case "line":
+                  options.horizontal
+                      ? drawHorizontalBarChartValues(chart, options, ctx)
+                      : drawLineOrBarChartValues(chart, options, ctx);
+                  break;
+          }
           ctx.restore();
       },
   };
+  function drawTextWithBackground(text, x, y, ctx) {
+      ctx.lineWidth = 3; // Stroke the text with a big lineWidth width to have some kind of background
+      ctx.strokeText(text, x, y);
+      ctx.lineWidth = 1;
+      ctx.fillText(text, x, y);
+  }
+  function drawLineOrBarChartValues(chart, options, ctx) {
+      const yMax = chart.chartArea.bottom;
+      const yMin = chart.chartArea.top;
+      const textsPositions = {};
+      for (const dataset of chart._metasets) {
+          for (let i = 0; i < dataset._parsed.length; i++) {
+              const value = dataset._parsed[i].y;
+              const point = dataset.data[i];
+              const xPosition = point.x;
+              let yPosition = 0;
+              if (chart.config.type === "line") {
+                  yPosition = point.y - 10;
+              }
+              else {
+                  yPosition = value < 0 ? point.y - point.height / 2 : point.y + point.height / 2;
+              }
+              yPosition = Math.min(yPosition, yMax);
+              yPosition = Math.max(yPosition, yMin);
+              // Avoid overlapping texts with same X
+              if (!textsPositions[xPosition]) {
+                  textsPositions[xPosition] = [];
+              }
+              for (const otherPosition of textsPositions[xPosition] || []) {
+                  if (Math.abs(otherPosition - yPosition) < 13) {
+                      yPosition = otherPosition - 13;
+                  }
+              }
+              textsPositions[xPosition].push(yPosition);
+              ctx.fillStyle = point.options.backgroundColor;
+              ctx.strokeStyle = options.background || "#ffffff";
+              drawTextWithBackground(options.callback(value - 0), xPosition, yPosition, ctx);
+          }
+      }
+  }
+  function drawHorizontalBarChartValues(chart, options, ctx) {
+      const xMax = chart.chartArea.right;
+      const xMin = chart.chartArea.left;
+      const textsPositions = {};
+      for (const dataset of chart._metasets) {
+          for (let i = 0; i < dataset._parsed.length; i++) {
+              const value = dataset._parsed[i].x;
+              const displayValue = options.callback(value - 0);
+              const point = dataset.data[i];
+              const yPosition = point.y;
+              let xPosition = value < 0 ? point.x + point.width / 2 : point.x - point.width / 2;
+              xPosition = Math.min(xPosition, xMax);
+              xPosition = Math.max(xPosition, xMin);
+              // Avoid overlapping texts with same Y
+              if (!textsPositions[yPosition]) {
+                  textsPositions[yPosition] = [];
+              }
+              const textWidth = computeTextWidth(ctx, displayValue, { fontSize: 12 }, "px");
+              for (const otherPosition of textsPositions[yPosition]) {
+                  if (Math.abs(otherPosition - xPosition) < textWidth) {
+                      xPosition = otherPosition + textWidth + 3;
+                  }
+              }
+              textsPositions[yPosition].push(xPosition);
+              ctx.fillStyle = point.options.backgroundColor;
+              ctx.strokeStyle = options.background || "#ffffff";
+              drawTextWithBackground(displayValue, xPosition, yPosition, ctx);
+          }
+      }
+  }
+  function drawPieChartValues(chart, options, ctx) {
+      for (const dataset of chart._metasets) {
+          for (let i = 0; i < dataset._parsed.length; i++) {
+              const bar = dataset.data[i];
+              const { startAngle, endAngle, innerRadius, outerRadius } = bar;
+              const midAngle = (startAngle + endAngle) / 2;
+              const midRadius = (innerRadius + outerRadius) / 2;
+              const x = bar.x + midRadius * Math.cos(midAngle);
+              const y = bar.y + midRadius * Math.sin(midAngle) + 7;
+              ctx.fillStyle = chartFontColor(options.background);
+              ctx.strokeStyle = options.background || "#ffffff";
+              const value = options.callback(dataset._parsed[i]);
+              drawTextWithBackground(value, x, y, ctx);
+          }
+      }
+  }
 
   /** This is a chartJS plugin that will draw connector lines between the bars of a Waterfall chart */
   const waterfallLinesPlugin = {
@@ -19155,9 +19203,10 @@ stores.inject(MyMetaStore, storeInstance);
               return error;
           }
           if (!pivot.areDomainArgsFieldsValid(domainArgs)) {
+              const suggestion = _t("Consider using a dynamic pivot formula: %s. Or re-insert the static pivot from the Data menu.", `=PIVOT(${_pivotFormulaId})`);
               return {
                   value: CellErrorType.GenericError,
-                  message: _t("Dimensions don't match the pivot definition"),
+                  message: _t("Dimensions don't match the pivot definition") + ". " + suggestion,
               };
           }
           const domain = pivot.parseArgsToPivotDomain(domainArgs);
@@ -19184,9 +19233,10 @@ stores.inject(MyMetaStore, storeInstance);
               return error;
           }
           if (!pivot.areDomainArgsFieldsValid(domainArgs)) {
+              const suggestion = _t("Consider using a dynamic pivot formula: %s. Or re-insert the static pivot from the Data menu.", `=PIVOT(${_pivotFormulaId})`);
               return {
                   value: CellErrorType.GenericError,
-                  message: _t("Dimensions don't match the pivot definition"),
+                  message: _t("Dimensions don't match the pivot definition") + ". " + suggestion,
               };
           }
           const domain = pivot.parseArgsToPivotDomain(domainArgs);
@@ -22362,6 +22412,7 @@ stores.inject(MyMetaStore, storeInstance);
           // replace the whole token
           start = tokenAtCursor.start;
       }
+      this.composer.stopComposerRangeSelection();
       this.composer.changeComposerCursorSelection(start, end);
       this.composer.replaceComposerCursorSelection(value);
   }
@@ -22379,6 +22430,7 @@ stores.inject(MyMetaStore, storeInstance);
           // replace the whole token
           start = tokenAtCursor.start;
       }
+      this.composer.stopComposerRangeSelection();
       this.composer.changeComposerCursorSelection(start, end);
       this.composer.replaceComposerCursorSelection(value);
   }
@@ -23403,7 +23455,7 @@ stores.inject(MyMetaStore, storeInstance);
                           const xLabel = tooltipItem.dataset?.label || tooltipItem.label;
                           // tooltipItem.parsed can be an object or a number for pie charts
                           let yLabel = horizontalChart ? tooltipItem.parsed.x : tooltipItem.parsed.y;
-                          if (!yLabel) {
+                          if (yLabel === undefined || yLabel === null) {
                               yLabel = tooltipItem.parsed;
                           }
                           const toolTipFormat = !format && Math.abs(yLabel) >= 1000 ? "#,##" : format;
@@ -37194,6 +37246,8 @@ stores.inject(MyMetaStore, storeInstance);
       mutators = ["reset", "deferUpdates", "applyUpdate", "discardPendingUpdate", "update"];
       updatesAreDeferred = false;
       draft = null;
+      notification = this.get(NotificationStore);
+      alreadyNotified = false;
       constructor(get, pivotId) {
           super(get);
           this.pivotId = pivotId;
@@ -37300,6 +37354,16 @@ stores.inject(MyMetaStore, storeInstance);
                   pivot: this.draft,
               });
               this.draft = null;
+              if (!this.alreadyNotified && !this.isDynamicPivotInViewport()) {
+                  const formulaId = this.getters.getPivotFormulaId(this.pivotId);
+                  const pivotExample = `=PIVOT(${formulaId})`;
+                  this.alreadyNotified = true;
+                  this.notification.notifyUser({
+                      type: "info",
+                      text: _t("Pivot updates only work with dynamic pivot tables. Use %s or re-insert the static pivot from the Data menu.", pivotExample),
+                      sticky: false,
+                  });
+              }
           }
       }
       discardPendingUpdate() {
@@ -37330,15 +37394,22 @@ stores.inject(MyMetaStore, storeInstance);
               return;
           }
           const cleanedWithGranularity = this.addDefaultDateTimeGranularity(this.fields, cleanedDefinition);
-          if (this.updatesAreDeferred) {
-              this.draft = cleanedWithGranularity;
+          this.draft = cleanedWithGranularity;
+          if (!this.updatesAreDeferred) {
+              this.applyUpdate();
           }
-          else {
-              this.model.dispatch("UPDATE_PIVOT", {
-                  pivotId: this.pivotId,
-                  pivot: cleanedWithGranularity,
-              });
+      }
+      isDynamicPivotInViewport() {
+          const sheetId = this.getters.getActiveSheetId();
+          for (const col of this.getters.getSheetViewVisibleCols()) {
+              for (const row of this.getters.getSheetViewVisibleRows()) {
+                  const isDynamicPivot = this.getters.isSpillPivotFormula({ sheetId, col, row });
+                  if (isDynamicPivot) {
+                      return true;
+                  }
+              }
           }
+          return false;
       }
       addDefaultDateTimeGranularity(fields, definition) {
           const { columns, rows } = definition;
@@ -46501,7 +46572,7 @@ stores.inject(MyMetaStore, storeInstance);
       if (!data) {
           return createEmptyWorkbookData();
       }
-      console.group("Loading data");
+      console.debug("### Loading data ###");
       const start = performance.now();
       if (data["[Content_Types].xml"]) {
           const reader = new XlsxReader(data);
@@ -46515,13 +46586,13 @@ stores.inject(MyMetaStore, storeInstance);
       // apply migrations, if needed
       if ("version" in data) {
           if (data.version < CURRENT_VERSION) {
-              console.info("Migrating data from version", data.version);
+              console.debug("Migrating data from version", data.version);
               data = migrate(data);
           }
       }
       data = repairData(data);
-      console.info("Data loaded in", performance.now() - start, "ms");
-      console.groupEnd();
+      console.debug("Data loaded in", performance.now() - start, "ms");
+      console.debug("###");
       return data;
   }
   function migrate(data) {
@@ -46530,7 +46601,7 @@ stores.inject(MyMetaStore, storeInstance);
       for (let i = index; i < MIGRATIONS.length; i++) {
           data = MIGRATIONS[i].applyMigration(data);
       }
-      console.info("Data migrated in", performance.now() - start, "ms");
+      console.debug("Data migrated in", performance.now() - start, "ms");
       return data;
   }
   const MIGRATIONS = [
@@ -52429,7 +52500,7 @@ stores.inject(MyMetaStore, storeInstance);
               case "DUPLICATE_PIVOT": {
                   const { pivotId, newPivotId } = cmd;
                   const pivot = deepCopy(this.getPivotCore(pivotId).definition);
-                  pivot.name = _t("%s (copy)", pivot.name);
+                  pivot.name = cmd.duplicatedPivotName ?? pivot.name + " (copy)";
                   this.addPivot(newPivotId, pivot);
                   break;
               }
@@ -54090,7 +54161,7 @@ stores.inject(MyMetaStore, storeInstance);
           cellsToCompute.addMany(arrayFormulasPositions);
           cellsToCompute.addMany(this.getCellsDependingOn(arrayFormulasPositions));
           this.evaluate(cellsToCompute);
-          console.info("evaluate Cells", performance.now() - start, "ms");
+          console.debug("evaluate Cells", performance.now() - start, "ms");
       }
       getArrayFormulasImpactedByChangesOf(positions) {
           const impactedPositions = this.createEmptyPositionSet();
@@ -54129,7 +54200,7 @@ stores.inject(MyMetaStore, storeInstance);
           const start = performance.now();
           this.evaluatedCells = new PositionMap();
           this.evaluate(this.getAllCells());
-          console.info("evaluate all cells", performance.now() - start, "ms");
+          console.debug("evaluate all cells", performance.now() - start, "ms");
       }
       evaluateFormulaResult(sheetId, formulaString) {
           try {
@@ -57279,7 +57350,7 @@ stores.inject(MyMetaStore, storeInstance);
               this.onMessageReceived(message);
           }
           this.isReplayingInitialRevisions = false;
-          console.info("Replayed", numberOfCommands, "commands in", performance.now() - start, "ms");
+          console.debug("Replayed", numberOfCommands, "commands in", performance.now() - start, "ms");
       }
       /**
        * Notify the server that the user client left the collaborative session
@@ -57518,6 +57589,7 @@ stores.inject(MyMetaStore, storeInstance);
               case "REMOTE_REVISION":
               case "REVISION_REDONE":
               case "REVISION_UNDONE":
+              case "SNAPSHOT_CREATED":
                   return this.processedRevisions.has(message.nextRevisionId);
               default:
                   return false;
@@ -58033,12 +58105,13 @@ stores.inject(MyMetaStore, storeInstance);
           this.dispatch("DUPLICATE_PIVOT", {
               pivotId,
               newPivotId,
+              duplicatedPivotName: _t("%s (copy)", this.getters.getPivotCoreDefinition(pivotId).name),
           });
           const activeSheetId = this.getters.getActiveSheetId();
           const position = this.getters.getSheetIds().indexOf(activeSheetId) + 1;
           const formulaId = this.getters.getPivotFormulaId(newPivotId);
           const newPivotName = this.getters.getPivotName(newPivotId);
-          this.dispatch("CREATE_SHEET", {
+          const result = this.dispatch("CREATE_SHEET", {
               sheetId: newSheetId,
               name: this.getPivotDuplicateSheetName(_t("%(newPivotName)s (Pivot #%(formulaId)s)", {
                   newPivotName,
@@ -58046,20 +58119,23 @@ stores.inject(MyMetaStore, storeInstance);
               })),
               position,
           });
-          this.dispatch("ACTIVATE_SHEET", { sheetIdFrom: activeSheetId, sheetIdTo: newSheetId });
-          this.dispatch("UPDATE_CELL", {
-              sheetId: newSheetId,
-              col: 0,
-              row: 0,
-              content: `=PIVOT(${formulaId})`,
-          });
+          if (result.isSuccessful) {
+              this.dispatch("ACTIVATE_SHEET", { sheetIdFrom: activeSheetId, sheetIdTo: newSheetId });
+              this.dispatch("UPDATE_CELL", {
+                  sheetId: newSheetId,
+                  col: 0,
+                  row: 0,
+                  content: `=PIVOT(${formulaId})`,
+              });
+          }
       }
       getPivotDuplicateSheetName(pivotName) {
           let i = 1;
           const names = this.getters.getSheetIds().map((id) => this.getters.getSheetName(id));
-          let name = pivotName;
+          const sanitizedName = pivotName.replace(new RegExp(FORBIDDEN_IN_EXCEL_REGEX, "g"), " ");
+          let name = sanitizedName;
           while (names.includes(name)) {
-              name = `${pivotName} (${i})`;
+              name = `${sanitizedName} (${i})`;
               i++;
           }
           return name;
@@ -58640,9 +58716,12 @@ stores.inject(MyMetaStore, storeInstance);
       "HIDE_COLUMNS_ROWS",
       "UNHIDE_COLUMNS_ROWS",
       "UNFOLD_HEADER_GROUP",
+      "UNGROUP_HEADERS",
       "FOLD_HEADER_GROUP",
       "FOLD_ALL_HEADER_GROUPS",
       "UNFOLD_ALL_HEADER_GROUPS",
+      "FOLD_HEADER_GROUPS_IN_ZONE",
+      "UNFOLD_HEADER_GROUPS_IN_ZONE",
       "CREATE_TABLE",
       "UPDATE_TABLE",
       "UPDATE_FILTER",
@@ -59958,6 +60037,8 @@ stores.inject(MyMetaStore, storeInstance);
               case "UNFOLD_HEADER_GROUP":
               case "FOLD_ALL_HEADER_GROUPS":
               case "UNFOLD_ALL_HEADER_GROUPS":
+              case "FOLD_HEADER_GROUPS_IN_ZONE":
+              case "UNFOLD_HEADER_GROUPS_IN_ZONE":
                   this.updateHiddenRows(cmd.sheetId);
                   break;
               case "UPDATE_FILTER":
@@ -65904,7 +65985,9 @@ stores.inject(MyMetaStore, storeInstance);
    * The value does not matter, it can be hardcoded.
    */
   const catAxId = 17781237;
+  const secondaryCatAxId = 17781238;
   const valAxId = 88853993;
+  const secondaryValAxId = 88853994;
   function createChart(chart, chartSheetIndex, data) {
       const namespaces = [
           ["xmlns:r", RELATIONSHIP_NSR],
@@ -66220,8 +66303,8 @@ stores.inject(MyMetaStore, storeInstance);
       <!-- each data marker in the series does not have a different color -->
       <c:varyColors val="0"/>
       ${barDataSetNode}
-      <c:axId val="${catAxId + (useRightAxisForBarSerie ? 1 : 0)}" />
-      <c:axId val="${valAxId + (useRightAxisForBarSerie ? 1 : 0)}" />
+      <c:axId val="${useRightAxisForBarSerie ? secondaryCatAxId : catAxId}" />
+      <c:axId val="${useRightAxisForBarSerie ? secondaryValAxId : valAxId}" />
     </c:barChart>
     ${leftDataSetsNodes.length
         ? escapeXml /*xml*/ `
@@ -66242,21 +66325,21 @@ stores.inject(MyMetaStore, storeInstance);
           <!-- each data marker in the series does not have a different color -->
           <c:varyColors val="0"/>
           ${joinXmlNodes(rightDataSetsNodes)}
-          <c:axId val="${catAxId + 1}" />
-          <c:axId val="${valAxId + 1}" />
+          <c:axId val="${secondaryCatAxId}" />
+          <c:axId val="${secondaryValAxId}" />
         </c:lineChart>
       `
         : ""}
     ${!useRightAxisForBarSerie || leftDataSetsNodes.length
         ? escapeXml /*xml*/ `
-        ${addAx("b", "c:catAx", catAxId + 1, valAxId + 1, chart.axesDesign?.x?.title, chart.fontColor, leftDataSetsNodes.length ? 1 : 0)}
-        ${addAx("r", "c:valAx", valAxId + 1, catAxId + 1, chart.axesDesign?.y1?.title, chart.fontColor)}
+        ${addAx("b", "c:catAx", catAxId, valAxId, chart.axesDesign?.x?.title, chart.fontColor, leftDataSetsNodes.length ? 1 : 0)}
+        ${addAx("l", "c:valAx", valAxId, catAxId, chart.axesDesign?.y?.title, chart.fontColor)}
       `
         : ""}
     ${useRightAxisForBarSerie || rightDataSetsNodes.length
         ? escapeXml /*xml*/ `
-        ${addAx("b", "c:catAx", catAxId, valAxId, chart.axesDesign?.x?.title, chart.fontColor, leftDataSetsNodes.length || !useRightAxisForBarSerie ? 1 : 0)}
-        ${addAx("l", "c:valAx", valAxId, catAxId, chart.axesDesign?.y?.title, chart.fontColor)}
+        ${addAx("b", "c:catAx", secondaryCatAxId, secondaryValAxId, chart.axesDesign?.x?.title, chart.fontColor, leftDataSetsNodes.length || !useRightAxisForBarSerie ? 1 : 0)}
+        ${addAx("r", "c:valAx", secondaryValAxId, secondaryCatAxId, chart.axesDesign?.y1?.title, chart.fontColor)}
       `
         : ""}
   `;
@@ -67277,6 +67360,15 @@ stores.inject(MyMetaStore, storeInstance);
               ["id", i + 1], // id cannot be 0
               ["name", colName],
           ];
+          if (table.config.totalRow) {
+              // Note: To be 100% complete, we could also add a `totalsRowLabel` attribute for total strings, and a tag
+              // `<totalsRowFormula>` for the formula of the total. But those doesn't seem to be mandatory for Excel.
+              const colTotalXc = toXC(tableZone.left + i, tableZone.bottom);
+              const colTotalContent = sheetData.cells[colTotalXc]?.content;
+              if (colTotalContent?.startsWith("=")) {
+                  colAttributes.push(["totalsRowFunction", "custom"]);
+              }
+          }
           columns.push(escapeXml /*xml*/ `<tableColumn ${formatAttributes(colAttributes)}/>`);
       }
       return escapeXml /*xml*/ `
@@ -67371,8 +67463,9 @@ stores.inject(MyMetaStore, storeInstance);
                   }
                   else if (cell.content && cell.content !== "") {
                       const isTableHeader = isCellTableHeader(c, r, sheet);
+                      const isTableTotal = isCellTableTotal(c, r, sheet);
                       const isPlainText = !!(cell.format && data.formats[cell.format] === PLAIN_TEXT_FORMAT);
-                      ({ attrs: additionalAttrs, node: cellNode } = addContent(cell.content, construct.sharedStrings, isTableHeader || isPlainText));
+                      ({ attrs: additionalAttrs, node: cellNode } = addContent(cell.content, construct.sharedStrings, isTableHeader || isTableTotal || isPlainText));
                   }
                   attributes.push(...additionalAttrs);
                   // prettier-ignore
@@ -67406,6 +67499,16 @@ stores.inject(MyMetaStore, storeInstance);
           return isInside(col, row, headerZone);
       });
   }
+  function isCellTableTotal(col, row, sheet) {
+      return sheet.tables.some((table) => {
+          if (!table.config.totalRow) {
+              return false;
+          }
+          const zone = toZone(table.range);
+          const totalZone = { ...zone, top: zone.bottom };
+          return isInside(col, row, totalZone);
+      });
+  }
   function addHyperlinks(construct, data, sheetIndex) {
       const sheet = data.sheets[sheetIndex];
       const cells = sheet.cells;
@@ -67857,7 +67960,7 @@ stores.inject(MyMetaStore, storeInstance);
       coreHandlers = [];
       constructor(data = {}, config = {}, stateUpdateMessages = [], uuidGenerator = new UuidGenerator(), verboseImport = true) {
           const start = performance.now();
-          console.group("Model creation");
+          console.debug("##### Model creation #####");
           super();
           setDefaultTranslationMethod();
           stateUpdateMessages = repairInitialMessages(data, stateUpdateMessages);
@@ -67884,7 +67987,6 @@ stores.inject(MyMetaStore, storeInstance);
               isReadonly: () => this.config.mode === "readonly" || this.config.mode === "dashboard",
               isDashboard: () => this.config.mode === "dashboard",
           };
-          this.uuidGenerator.setIsFastStrategy(true);
           // Initiate stream processor
           this.selection = new SelectionStreamProcessorImpl(this.getters);
           this.coreHandlers.push(this.range);
@@ -67930,16 +68032,16 @@ stores.inject(MyMetaStore, storeInstance);
           this.joinSession();
           if (config.snapshotRequested) {
               const startSnapshot = performance.now();
-              console.info("Snapshot requested");
+              console.debug("Snapshot requested");
               this.session.snapshot(this.exportData());
               this.garbageCollectExternalResources();
-              console.info("Snapshot taken in", performance.now() - startSnapshot, "ms");
+              console.debug("Snapshot taken in", performance.now() - startSnapshot, "ms");
           }
           // mark all models as "raw", so they will not be turned into reactive objects
           // by owl, since we do not rely on reactivity
           owl.markRaw(this);
-          console.info("Model created in", performance.now() - start, "ms");
-          console.groupEnd();
+          console.debug("Model created in", performance.now() - start, "ms");
+          console.debug("######");
       }
       joinSession() {
           this.session.join(this.config.client);
@@ -68162,7 +68264,7 @@ stores.inject(MyMetaStore, storeInstance);
                       this.finalize();
                       const time = performance.now() - start;
                       if (time > 5) {
-                          console.info(type, time, "ms");
+                          console.debug(type, time, "ms");
                       }
                   });
                   this.session.save(command, commands, changes);
@@ -68548,9 +68650,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.4.8";
-  __info__.date = "2024-09-17T08:10:18.777Z";
-  __info__.hash = "caad8bd";
+  __info__.version = "17.4.10";
+  __info__.date = "2024-10-24T08:54:56.262Z";
+  __info__.hash = "c82d9c1";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);