@odoo/o-spreadsheet 17.3.7 → 17.3.9

package/dist/o-spreadsheet.iife.js

@@ -3,9 +3,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.3.7
- * @date 2024-07-02T10:38:52.530Z
- * @hash c570df3
+ * @version 17.3.9
+ * @date 2024-07-11T06:37:05.603Z
+ * @hash 2346a16
  */
 
 (function (exports, owl) {
@@ -592,7 +592,7 @@
   /**
    * Compares two objects.
    */
-  function deepEquals(o1, o2, ignoreFunctions) {
+  function deepEquals(o1, o2) {
       if (o1 === o2)
           return true;
       if ((o1 && !o2) || (o2 && !o1))
@@ -608,17 +608,13 @@
           }
       }
       for (const key in o1) {
-          const typeOfO1Key = typeof o1[key];
-          if (typeOfO1Key !== typeof o2[key])
+          if (typeof o1[key] !== typeof o2[key])
               return false;
-          if (typeOfO1Key === "object") {
-              if (!deepEquals(o1[key], o2[key], ignoreFunctions))
+          if (typeof o1[key] === "object") {
+              if (!deepEquals(o1[key], o2[key]))
                   return false;
           }
           else {
-              if (ignoreFunctions && typeOfO1Key === "function") {
-                  continue;
-              }
               if (o1[key] !== o2[key])
                   return false;
           }
@@ -1831,7 +1827,7 @@
   });
   const getNumberRegex = memoize(function getNumberRegex(locale) {
       const decimalSeparator = escapeRegExp(locale.decimalSeparator);
-      const thousandsSeparator = escapeRegExp(locale.thousandsSeparator);
+      const thousandsSeparator = escapeRegExp(locale.thousandsSeparator || "");
       const pIntegerAndDecimals = `(\\d+(${thousandsSeparator}\\d{3,})*(${decimalSeparator}\\d*)?)`; // pattern that match integer number with or without decimal digits
       const pOnlyDecimals = `(${decimalSeparator}\\d+)`; // pattern that match only expression with decimal digits
       const pScientificFormat = "(e(\\+|-)?\\d+)?"; // pattern that match scientific format between zero and one time (should be placed before pPercentFormat)
@@ -1858,7 +1854,7 @@
       return getNumberRegex(locale).test(value.trim());
   }
   const getInvaluableSymbolsRegexp = memoize(function getInvaluableSymbolsRegexp(locale) {
-      return new RegExp(`[\$€${escapeRegExp(locale.thousandsSeparator)}]`, "g");
+      return new RegExp(`[\$€${escapeRegExp(locale.thousandsSeparator || "")}]`, "g");
   });
   /**
    * Convert a string into a number. It assumes that the string actually represents
@@ -2765,6 +2761,9 @@
    * If the character is a special regular expression character, it is escaped with "\\".
    */
   const wildcardToRegExp = memoize(function wildcardToRegExp(operand) {
+      if (operand === "*") {
+          return /.+/;
+      }
       let exp = "";
       let predecessor = "";
       for (let char of operand) {
@@ -2788,9 +2787,9 @@
       }
       return new RegExp("^" + exp + "$", "i");
   });
-  function evaluatePredicate(value, criterion) {
+  function evaluatePredicate(value = "", criterion) {
       const { operator, operand } = criterion;
-      if (value === undefined || operand === undefined || value === null || operand === null) {
+      if (operand === undefined || value === null || operand === null) {
           return false;
       }
       if (typeof operand === "number" && operator === "=") {
@@ -5895,19 +5894,22 @@
   }
 
   function isValidLocale(locale) {
-      if (!(locale &&
-          typeof locale === "object" &&
-          typeof locale.name === "string" &&
-          typeof locale.code === "string" &&
-          typeof locale.thousandsSeparator === "string" &&
-          typeof locale.decimalSeparator === "string" &&
-          typeof locale.dateFormat === "string" &&
-          typeof locale.timeFormat === "string" &&
-          typeof locale.formulaArgSeparator === "string")) {
+      if (!locale ||
+          typeof locale !== "object" ||
+          !(!locale.thousandsSeparator || typeof locale.thousandsSeparator === "string")) {
           return false;
       }
-      if (!Object.values(locale).every((v) => v)) {
-          return false;
+      for (const property of [
+          "code",
+          "name",
+          "decimalSeparator",
+          "dateFormat",
+          "timeFormat",
+          "formulaArgSeparator",
+      ]) {
+          if (!locale[property] || typeof locale[property] !== "string") {
+              return false;
+          }
       }
       if (locale.formulaArgSeparator === locale.decimalSeparator) {
           return false;
@@ -6005,7 +6007,10 @@
       if (locale.decimalSeparator === "." || !isNumber(content, locale)) {
           return content;
       }
-      return content.replace(locale.thousandsSeparator, "").replace(locale.decimalSeparator, ".");
+      if (locale.thousandsSeparator) {
+          content = content.replaceAll(locale.thousandsSeparator, "");
+      }
+      return content.replace(locale.decimalSeparator, ".");
   }
   /**
    * Change a content string from the given locale to its canonical form (en_US locale). Also convert date string.
@@ -6994,6 +6999,146 @@
       return undefined;
   }
 
+  var State;
+  (function (State) {
+      /**
+       * Initial state.
+       * Expecting any reference for the left part of a range
+       * e.g. "A1", "1", "A", "Sheet1!A1", "Sheet1!A"
+       */
+      State[State["LeftRef"] = 0] = "LeftRef";
+      /**
+       * Expecting any reference for the right part of a range
+       * e.g. "A1", "1", "A", "Sheet1!A1", "Sheet1!A"
+       */
+      State[State["RightRef"] = 1] = "RightRef";
+      /**
+       * Expecting the separator without any constraint on the right part
+       */
+      State[State["Separator"] = 2] = "Separator";
+      /**
+       * Expecting the separator for a full column range
+       */
+      State[State["FullColumnSeparator"] = 3] = "FullColumnSeparator";
+      /**
+       * Expecting the separator for a full row range
+       */
+      State[State["FullRowSeparator"] = 4] = "FullRowSeparator";
+      /**
+       * Expecting the right part of a full column range
+       * e.g. "1", "A1"
+       */
+      State[State["RightColumnRef"] = 5] = "RightColumnRef";
+      /**
+       * Expecting the right part of a full row range
+       * e.g. "A", "A1"
+       */
+      State[State["RightRowRef"] = 6] = "RightRowRef";
+      /**
+       * Final state. A range has been matched
+       */
+      State[State["Found"] = 7] = "Found";
+  })(State || (State = {}));
+  const goTo = (state, guard = () => true) => [
+      {
+          goTo: state,
+          guard,
+      },
+  ];
+  const goToMulti = (state, guard = () => true) => ({
+      goTo: state,
+      guard,
+  });
+  const machine = {
+      [State.LeftRef]: {
+          REFERENCE: goTo(State.Separator),
+          NUMBER: goTo(State.FullRowSeparator),
+          SYMBOL: [
+              goToMulti(State.FullColumnSeparator, (token) => isColReference(token.value)),
+              goToMulti(State.FullRowSeparator, (token) => isRowReference(token.value)),
+          ],
+      },
+      [State.FullColumnSeparator]: {
+          SPACE: goTo(State.FullColumnSeparator),
+          OPERATOR: goTo(State.RightColumnRef, (token) => token.value === ":"),
+      },
+      [State.FullRowSeparator]: {
+          SPACE: goTo(State.FullRowSeparator),
+          OPERATOR: goTo(State.RightRowRef, (token) => token.value === ":"),
+      },
+      [State.Separator]: {
+          SPACE: goTo(State.Separator),
+          OPERATOR: goTo(State.RightRef, (token) => token.value === ":"),
+      },
+      [State.RightRef]: {
+          SPACE: goTo(State.RightRef),
+          NUMBER: goTo(State.Found),
+          REFERENCE: goTo(State.Found, (token) => isSingleCellReference(token.value)),
+          SYMBOL: goTo(State.Found, (token) => isColHeader(token.value) || isRowHeader(token.value)),
+      },
+      [State.RightColumnRef]: {
+          SPACE: goTo(State.RightColumnRef),
+          SYMBOL: goTo(State.Found, (token) => isColHeader(token.value)),
+          REFERENCE: goTo(State.Found, (token) => isSingleCellReference(token.value)),
+      },
+      [State.RightRowRef]: {
+          SPACE: goTo(State.RightRowRef),
+          NUMBER: goTo(State.Found),
+          REFERENCE: goTo(State.Found, (token) => isSingleCellReference(token.value)),
+          SYMBOL: goTo(State.Found, (token) => isRowHeader(token.value)),
+      },
+      [State.Found]: {},
+  };
+  /**
+   * Check if the list of tokens starts with a sequence of tokens representing
+   * a range.
+   * If a range is found, the sequence is removed from the list and is returned
+   * as a single token.
+   */
+  function matchReference(tokens) {
+      let head = 0;
+      let transitions = machine[State.LeftRef];
+      let matchedTokens = "";
+      while (transitions !== undefined) {
+          const token = tokens[head++];
+          if (!token) {
+              return null;
+          }
+          const transition = transitions[token.type]?.find((transition) => transition.guard(token));
+          const nextState = transition ? transition.goTo : undefined;
+          switch (nextState) {
+              case undefined:
+                  return null;
+              case State.Found:
+                  matchedTokens += token.value;
+                  tokens.splice(0, head);
+                  return {
+                      type: "REFERENCE",
+                      value: matchedTokens,
+                  };
+              default:
+                  transitions = machine[nextState];
+                  matchedTokens += token.value;
+                  break;
+          }
+      }
+      return null;
+  }
+  /**
+   * Take the result of the tokenizer and transform it to be usable in the
+   * manipulations of range
+   *
+   * @param formula
+   */
+  function rangeTokenize(formula, locale = DEFAULT_LOCALE) {
+      const tokens = tokenize(formula, locale);
+      const result = [];
+      while (tokens.length) {
+          result.push(matchReference(tokens) || tokens.shift());
+      }
+      return result;
+  }
+
   const functionRegex = /[a-zA-Z0-9\_]+(\.[a-zA-Z0-9\_]+)*/;
   const UNARY_OPERATORS_PREFIX = ["-", "+"];
   const UNARY_OPERATORS_POSTFIX = ["%"];
@@ -7142,7 +7287,7 @@
    * Parse an expression (as a string) into an AST.
    */
   function parse(str) {
-      return parseTokens(tokenize(str));
+      return parseTokens(rangeTokenize(str));
   }
   function parseTokens(tokens) {
       tokens = tokens.filter((x) => x.type !== "SPACE");
@@ -7278,146 +7423,6 @@
       return needParenthesis ? `(${astToFormula(rightOperation)})` : astToFormula(rightOperation);
   }
 
-  var State;
-  (function (State) {
-      /**
-       * Initial state.
-       * Expecting any reference for the left part of a range
-       * e.g. "A1", "1", "A", "Sheet1!A1", "Sheet1!A"
-       */
-      State[State["LeftRef"] = 0] = "LeftRef";
-      /**
-       * Expecting any reference for the right part of a range
-       * e.g. "A1", "1", "A", "Sheet1!A1", "Sheet1!A"
-       */
-      State[State["RightRef"] = 1] = "RightRef";
-      /**
-       * Expecting the separator without any constraint on the right part
-       */
-      State[State["Separator"] = 2] = "Separator";
-      /**
-       * Expecting the separator for a full column range
-       */
-      State[State["FullColumnSeparator"] = 3] = "FullColumnSeparator";
-      /**
-       * Expecting the separator for a full row range
-       */
-      State[State["FullRowSeparator"] = 4] = "FullRowSeparator";
-      /**
-       * Expecting the right part of a full column range
-       * e.g. "1", "A1"
-       */
-      State[State["RightColumnRef"] = 5] = "RightColumnRef";
-      /**
-       * Expecting the right part of a full row range
-       * e.g. "A", "A1"
-       */
-      State[State["RightRowRef"] = 6] = "RightRowRef";
-      /**
-       * Final state. A range has been matched
-       */
-      State[State["Found"] = 7] = "Found";
-  })(State || (State = {}));
-  const goTo = (state, guard = () => true) => [
-      {
-          goTo: state,
-          guard,
-      },
-  ];
-  const goToMulti = (state, guard = () => true) => ({
-      goTo: state,
-      guard,
-  });
-  const machine = {
-      [State.LeftRef]: {
-          REFERENCE: goTo(State.Separator),
-          NUMBER: goTo(State.FullRowSeparator),
-          SYMBOL: [
-              goToMulti(State.FullColumnSeparator, (token) => isColReference(token.value)),
-              goToMulti(State.FullRowSeparator, (token) => isRowReference(token.value)),
-          ],
-      },
-      [State.FullColumnSeparator]: {
-          SPACE: goTo(State.FullColumnSeparator),
-          OPERATOR: goTo(State.RightColumnRef, (token) => token.value === ":"),
-      },
-      [State.FullRowSeparator]: {
-          SPACE: goTo(State.FullRowSeparator),
-          OPERATOR: goTo(State.RightRowRef, (token) => token.value === ":"),
-      },
-      [State.Separator]: {
-          SPACE: goTo(State.Separator),
-          OPERATOR: goTo(State.RightRef, (token) => token.value === ":"),
-      },
-      [State.RightRef]: {
-          SPACE: goTo(State.RightRef),
-          NUMBER: goTo(State.Found),
-          REFERENCE: goTo(State.Found, (token) => isSingleCellReference(token.value)),
-          SYMBOL: goTo(State.Found, (token) => isColHeader(token.value) || isRowHeader(token.value)),
-      },
-      [State.RightColumnRef]: {
-          SPACE: goTo(State.RightColumnRef),
-          SYMBOL: goTo(State.Found, (token) => isColHeader(token.value)),
-          REFERENCE: goTo(State.Found, (token) => isSingleCellReference(token.value)),
-      },
-      [State.RightRowRef]: {
-          SPACE: goTo(State.RightRowRef),
-          NUMBER: goTo(State.Found),
-          REFERENCE: goTo(State.Found, (token) => isSingleCellReference(token.value)),
-          SYMBOL: goTo(State.Found, (token) => isRowHeader(token.value)),
-      },
-      [State.Found]: {},
-  };
-  /**
-   * Check if the list of tokens starts with a sequence of tokens representing
-   * a range.
-   * If a range is found, the sequence is removed from the list and is returned
-   * as a single token.
-   */
-  function matchReference(tokens) {
-      let head = 0;
-      let transitions = machine[State.LeftRef];
-      let matchedTokens = "";
-      while (transitions !== undefined) {
-          const token = tokens[head++];
-          if (!token) {
-              return null;
-          }
-          const transition = transitions[token.type]?.find((transition) => transition.guard(token));
-          const nextState = transition ? transition.goTo : undefined;
-          switch (nextState) {
-              case undefined:
-                  return null;
-              case State.Found:
-                  matchedTokens += token.value;
-                  tokens.splice(0, head);
-                  return {
-                      type: "REFERENCE",
-                      value: matchedTokens,
-                  };
-              default:
-                  transitions = machine[nextState];
-                  matchedTokens += token.value;
-                  break;
-          }
-      }
-      return null;
-  }
-  /**
-   * Take the result of the tokenizer and transform it to be usable in the
-   * manipulations of range
-   *
-   * @param formula
-   */
-  function rangeTokenize(formula, locale = DEFAULT_LOCALE) {
-      const tokens = tokenize(formula, locale);
-      const result = [];
-      while (tokens.length) {
-          result.push(matchReference(tokens) || tokens.shift());
-      }
-      return result;
-  }
-
   /**
    * Add the following information on tokens:
    * - length
@@ -9835,7 +9840,6 @@ stores.inject(MyMetaStore, storeInstance);
       };
       canvas = owl.useRef("graphContainer");
       chart;
-      currentRuntime;
       get background() {
           return this.chartRuntime.background;
       }
@@ -9852,18 +9856,11 @@ stores.inject(MyMetaStore, storeInstance);
       setup() {
           owl.onMounted(() => {
               const runtime = this.chartRuntime;
-              this.currentRuntime = runtime;
               // Note: chartJS modify the runtime in place, so it's important to give it a copy
               this.createChart(deepCopy(runtime.chartJsConfig));
           });
           owl.onWillUnmount(() => this.chart?.destroy());
-          owl.useEffect(() => {
-              const runtime = this.chartRuntime;
-              if (!deepEquals(runtime, this.currentRuntime, "ignoreFunctions")) {
-                  this.currentRuntime = runtime;
-                  this.updateChartJs(deepCopy(runtime));
-              }
-          });
+          owl.useEffect(() => this.updateChartJs(deepCopy(this.chartRuntime)), () => [this.chartRuntime]);
       }
       createChart(chartData) {
           const canvas = this.canvas.el;
@@ -22697,7 +22694,7 @@ stores.inject(MyMetaStore, storeInstance);
   /**
    * Get a default chart js configuration
    */
-  function getDefaultChartJsRuntime(chart, labels, fontColor, { format, locale, truncateLabels }) {
+  function getDefaultChartJsRuntime(chart, labels, fontColor, { format, locale, truncateLabels = true }) {
       const chartTitle = chart.title.text ? chart.title : { ...chart.title, content: "" };
       const options = {
           // https://www.chartjs.org/docs/latest/general/responsive.html
@@ -23926,10 +23923,6 @@ stores.inject(MyMetaStore, storeInstance);
       const colors = new ColorGenerator();
       const definition = chart.getDefinition();
       for (let [index, { label, data }] of dataSetsValues.entries()) {
-          if (["linear", "time"].includes(axisType)) {
-              // Replace empty string labels by undefined to make sure chartJS doesn't decide that "" is the same as 0
-              data = data.map((y, index) => ({ x: labels[index] || undefined, y }));
-          }
           const color = colors.next();
           let backgroundRGBA = colorToRGBA(color);
           if (stacked) {
@@ -23945,6 +23938,10 @@ stores.inject(MyMetaStore, storeInstance);
                   return value;
               });
           }
+          if (["linear", "time"].includes(axisType)) {
+              // Replace empty string labels by undefined to make sure chartJS doesn't decide that "" is the same as 0
+              data = data.map((y, index) => ({ x: labels[index] || undefined, y }));
+          }
           const backgroundColor = rgbaToHex(backgroundRGBA);
           const dataset = {
               label,
@@ -36262,7 +36259,13 @@ stores.inject(MyMetaStore, storeInstance);
       }
       async loadLocales() {
           this.loadedLocales = (await this.env.loadLocales())
-              .filter(isValidLocale)
+              .filter((locale) => {
+              const isValid = isValidLocale(locale);
+              if (!isValid) {
+                  console.warn(`Invalid locale: ${locale["code"]} ${locale}`);
+              }
+              return isValid;
+          })
               .sort((a, b) => a.name.localeCompare(b.name));
       }
       get numberFormatPreview() {
@@ -41414,6 +41417,7 @@ stores.inject(MyMetaStore, storeInstance);
               return;
           }
           if (clipboardData.types.indexOf(ClipboardMIMEType.PlainText) > -1) {
+              ev.preventDefault();
               const content = clipboardData.getData(ClipboardMIMEType.PlainText);
               const target = this.env.model.getters.getSelectedZones();
               const clipboardString = this.env.model.getters.getClipboardTextContent();
@@ -45125,7 +45129,7 @@ stores.inject(MyMetaStore, storeInstance);
       return relsFile;
   }
 
-  const EXCEL_IMPORT_VERSION = 16;
+  const EXCEL_IMPORT_VERSION = 17;
   class XlsxReader {
       warningManager;
       xmls;
@@ -45257,7 +45261,7 @@ stores.inject(MyMetaStore, storeInstance);
    * a breaking change is made in the way the state is handled, and an upgrade
    * function should be defined
    */
-  const CURRENT_VERSION = 16;
+  const CURRENT_VERSION = 17;
   const INITIAL_SHEET_ID = "Sheet1";
   /**
    * This function tries to load anything that could look like a valid
@@ -46204,6 +46208,21 @@ stores.inject(MyMetaStore, storeInstance);
               return [];
           return Object.keys(sheetBorders).map((index) => parseInt(index, 10));
       }
+      /**
+       * Get all the rows which contains at least a border
+       */
+      getRowsWithBorders(sheetId) {
+          const sheetBorders = this.borders[sheetId]?.filter(isDefined);
+          if (!sheetBorders)
+              return [];
+          const rowsWithBorders = new Set();
+          for (const rowBorders of sheetBorders) {
+              for (const rowBorder in rowBorders) {
+                  rowsWithBorders.add(parseInt(rowBorder, 10));
+              }
+          }
+          return Array.from(rowsWithBorders);
+      }
       /**
        * Get the range of all the rows in the sheet
        */
@@ -46253,7 +46272,7 @@ stores.inject(MyMetaStore, storeInstance);
                   destructive: false,
               });
           }
-          this.getRowsRange(sheetId)
+          this.getRowsWithBorders(sheetId)
               .filter((row) => row >= start)
               .sort((a, b) => (delta < 0 ? a - b : b - a)) // start by the end when moving up
               .forEach((row) => {
@@ -52585,7 +52604,7 @@ stores.inject(MyMetaStore, storeInstance);
           return this.sheets[position.sheetId].getValue(position) === 1;
       }
       clear() {
-          const insertions = this.insertions;
+          const insertions = [...this];
           this.insertions = [];
           for (const sheetId in this.sheets) {
               this.sheets[sheetId].clear();
@@ -52923,6 +52942,7 @@ stores.inject(MyMetaStore, storeInstance);
           }
           finally {
               this.cellsBeingComputed.delete(cellId);
+              this.nextPositionsToUpdate.delete(position);
           }
       }
       computeAndSave(position) {
@@ -52949,8 +52969,33 @@ stores.inject(MyMetaStore, storeInstance);
           forEachSpreadPositionInMatrix(nbColumns, nbRows, 
           // thanks to the isMatrix check above, we know that formulaReturn is MatrixFunctionReturn
           this.spreadValues(formulaPosition, formulaReturn));
+          this.invalidatePositionsDependingOnSpread(formulaPosition, nbColumns, nbRows);
           return createEvaluatedCell(nullValueToZeroValue(formulaReturn[0][0]), this.getters.getLocale(), cellData);
       }
+      invalidatePositionsDependingOnSpread(arrayFormulaPosition, nbColumns, nbRows) {
+          // the result matrix is split in 2 zones to exclude the array formula position
+          const top = arrayFormulaPosition.row;
+          const left = arrayFormulaPosition.col;
+          const bottom = top + nbRows - 1;
+          const leftColumnZone = {
+              top: top + 1,
+              bottom,
+              left,
+              right: left,
+          };
+          const rightPartZone = {
+              top,
+              bottom,
+              left: left + 1,
+              right: left + nbColumns - 1,
+          };
+          const sheetId = arrayFormulaPosition.sheetId;
+          const invalidatedPositions = this.formulaDependencies().getCellsDependingOn([
+              { sheetId, zone: rightPartZone },
+              { sheetId, zone: leftColumnZone },
+          ]);
+          this.nextPositionsToUpdate.addMany(invalidatedPositions);
+      }
       assertSheetHasEnoughSpaceToSpreadFormulaResult({ sheetId, col, row }, matrixResult) {
           const numberOfCols = this.getters.getNumberCols(sheetId);
           const numberOfRows = this.getters.getNumberRows(sheetId);
@@ -53005,9 +53050,6 @@ stores.inject(MyMetaStore, storeInstance);
               const cell = this.getters.getCell(position);
               const evaluatedCell = createEvaluatedCell(nullValueToZeroValue(matrixResult[i][j]), this.getters.getLocale(), cell);
               this.evaluatedCells.set(position, evaluatedCell);
-              // check if formula dependencies present in the spread zone
-              // if so, they need to be recomputed
-              this.nextPositionsToUpdate.addMany(this.getCellsDependingOn([position]));
           };
       }
       invalidateSpreading(position) {
@@ -67045,9 +67087,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.3.7";
-  __info__.date = "2024-07-02T10:38:52.530Z";
-  __info__.hash = "c570df3";
+  __info__.version = "17.3.9";
+  __info__.date = "2024-07-11T06:37:05.603Z";
+  __info__.hash = "2346a16";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);