@odoo/o-spreadsheet 17.4.8 → 17.4.9

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.9
+ * @date 2024-10-14T07:53:51.633Z
+ * @hash b7015b7
  */
 
 (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) {
@@ -19155,9 +19162,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 +19192,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 +22371,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 +22389,7 @@ stores.inject(MyMetaStore, storeInstance);
           // replace the whole token
           start = tokenAtCursor.start;
       }
+      this.composer.stopComposerRangeSelection();
       this.composer.changeComposerCursorSelection(start, end);
       this.composer.replaceComposerCursorSelection(value);
   }
@@ -37194,6 +37205,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 +37313,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 +37353,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;
@@ -52429,7 +52459,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;
               }
@@ -57518,6 +57548,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 +58064,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 +58078,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 +58675,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 +59996,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 +65944,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 +66262,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 +66284,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 +67319,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 +67422,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 +67458,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;
@@ -67884,7 +67946,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);
@@ -68548,9 +68609,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.9";
+  __info__.date = "2024-10-14T07:53:51.633Z";
+  __info__.hash = "b7015b7";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);