@odoo/o-spreadsheet 17.2.5 → 17.2.7

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.2.5
- * @date 2024-04-26T07:41:13.193Z
- * @hash a730f5c
+ * @version 17.2.7
+ * @date 2024-05-15T09:20:44.429Z
+ * @hash 57e89fa
  */
 
 (function (exports, owl) {
@@ -553,7 +553,7 @@
   /**
    * Compares two objects.
    */
-  function deepEquals(o1, o2) {
+  function deepEquals(o1, o2, ignoreFunctions) {
       if (o1 === o2)
           return true;
       if ((o1 && !o2) || (o2 && !o1))
@@ -569,13 +569,16 @@
           }
       }
       for (const key in o1) {
-          if (typeof o1[key] !== typeof o2[key])
+          const typeOfO1Key = typeof o1[key];
+          if (typeOfO1Key !== typeof o2[key])
               return false;
-          if (typeof o1[key] === "object") {
-              if (!deepEquals(o1[key], o2[key]))
+          if (typeOfO1Key === "object") {
+              if (!deepEquals(o1[key], o2[key], ignoreFunctions))
                   return false;
           }
           else {
+              if (ignoreFunctions && typeOfO1Key === "function")
+                  return true;
               if (o1[key] !== o2[key])
                   return false;
           }
@@ -3683,21 +3686,22 @@
           xc = xc.split("!").at(-1);
       }
       if (xc.includes("$")) {
-          xc = xc.replace(/\$/g, "");
+          xc = xc.replaceAll("$", "");
       }
-      let ranges;
+      let firstRangePart = "";
+      let secondRangePart;
       if (xc.includes(":")) {
-          ranges = xc.split(":").map((x) => x.trim());
+          [firstRangePart, secondRangePart] = xc.split(":");
+          firstRangePart = firstRangePart.trim();
+          secondRangePart = secondRangePart.trim();
       }
       else {
-          ranges = [xc.trim()];
+          firstRangePart = xc.trim();
       }
       let top, bottom, left, right;
       let fullCol = false;
       let fullRow = false;
       let hasHeader = false;
-      const firstRangePart = ranges[0];
-      const secondRangePart = ranges[1] && ranges[1];
       if (isColReference(firstRangePart)) {
           left = right = lettersToNumber(firstRangePart);
           top = bottom = 0;
@@ -3714,7 +3718,7 @@
           top = bottom = c.row;
           hasHeader = true;
       }
-      if (ranges.length === 2) {
+      if (secondRangePart) {
           if (isColReference(secondRangePart)) {
               right = lettersToNumber(secondRangePart);
               fullCol = true;
@@ -5198,7 +5202,7 @@
       }
       return null;
   }
-  const separatorRegexp = /^[\w\.!\$]+/;
+  const SYMBOL_CHARS = new Set("ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789_.!$");
   /**
    * A "Symbol" is just basically any word-like element that can appear in a
    * formula, which is not a string. So:
@@ -5238,11 +5242,8 @@
               };
           }
       }
-      const match = chars.remaining().match(separatorRegexp);
-      if (match) {
-          const value = match[0];
-          result += value;
-          chars.advanceBy(value.length);
+      while (chars.current && SYMBOL_CHARS.has(chars.current)) {
+          result += chars.shift();
       }
       if (result.length) {
           const value = result;
@@ -6794,7 +6795,7 @@
   function matchReference(tokens) {
       let head = 0;
       let transitions = machine[State.LeftRef];
-      const matchedTokens = [];
+      let matchedTokens = "";
       while (transitions !== undefined) {
           const token = tokens[head++];
           if (!token) {
@@ -6806,15 +6807,15 @@
               case undefined:
                   return null;
               case State.Found:
-                  matchedTokens.push(token);
+                  matchedTokens += token.value;
                   tokens.splice(0, head);
                   return {
                       type: "REFERENCE",
-                      value: concat(matchedTokens.map((token) => token.value)),
+                      value: matchedTokens,
                   };
               default:
                   transitions = machine[nextState];
-                  matchedTokens.push(token);
+                  matchedTokens += token.value;
                   break;
           }
       }
@@ -8905,8 +8906,7 @@ stores.inject(MyMetaStore, storeInstance);
               const exactMatch = proposals?.find((p) => p.text === tokenAtCursor.value);
               // remove tokens that are likely to be other parts of the formula that slipped in the token if it's a string
               const searchTerm = tokenAtCursor.value.replace(/[ ,\(\)]/g, "");
-              const initialContent = this.initialContent;
-              if (exactMatch && exactMatch.text !== initialContent) {
+              if (exactMatch && this._currentContent !== this.initialContent) {
                   // this means the user has chosen a proposal
                   return;
               }
@@ -8914,7 +8914,7 @@ stores.inject(MyMetaStore, storeInstance);
                   proposals &&
                   !["ARG_SEPARATOR", "LEFT_PAREN"].includes(tokenAtCursor.type)) {
                   const filteredProposals = fuzzyLookup(searchTerm, proposals, (p) => p.fuzzySearchKey || p.text);
-                  if (!exactMatch) {
+                  if (!exactMatch || filteredProposals.length > 1) {
                       proposals = filteredProposals;
                   }
               }
@@ -9049,6 +9049,7 @@ stores.inject(MyMetaStore, storeInstance);
       };
       canvas = owl.useRef("graphContainer");
       chart;
+      currentRuntime;
       get background() {
           return this.chartRuntime.background;
       }
@@ -9065,9 +9066,18 @@ stores.inject(MyMetaStore, storeInstance);
       setup() {
           owl.onMounted(() => {
               const runtime = this.chartRuntime;
-              this.createChart(runtime.chartJsConfig);
+              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(this.chartRuntime), () => [this.chartRuntime]);
       }
       createChart(chartData) {
           const canvas = this.canvas.el;
@@ -9089,8 +9099,7 @@ stores.inject(MyMetaStore, storeInstance);
           this.chart.config.options.plugins.tooltip = chartData.options.plugins.tooltip;
           this.chart.config.options.plugins.legend = chartData.options.plugins.legend;
           this.chart.config.options.scales = chartData.options?.scales;
-          // ?
-          this.chart.update("active");
+          this.chart.update();
       }
   }
 
@@ -19453,7 +19462,7 @@ stores.inject(MyMetaStore, storeInstance);
   /**
    * Get a default chart js configuration
    */
-  function getDefaultChartJsRuntime(chart, labels, fontColor, { format, locale }) {
+  function getDefaultChartJsRuntime(chart, labels, fontColor, { format, locale, truncateLabels }) {
       const options = {
           // https://www.chartjs.org/docs/latest/general/responsive.html
           responsive: true,
@@ -19500,7 +19509,7 @@ stores.inject(MyMetaStore, storeInstance);
           type: chart.type,
           options,
           data: {
-              labels: labels.map(truncateLabel),
+              labels: truncateLabels ? labels.map(truncateLabel) : labels,
               datasets: [],
           },
           platform: undefined,
@@ -20266,9 +20275,9 @@ stores.inject(MyMetaStore, storeInstance);
       }
       return isInstalled;
   }
-  function getLineOrScatterConfiguration(chart, labels, localeFormat) {
+  function getLineOrScatterConfiguration(chart, labels, options) {
       const fontColor = chartFontColor(chart.background);
-      const config = getDefaultChartJsRuntime(chart, labels, fontColor, localeFormat);
+      const config = getDefaultChartJsRuntime(chart, labels, fontColor, options);
       const legend = {
           labels: {
               color: fontColor,
@@ -20311,7 +20320,7 @@ stores.inject(MyMetaStore, storeInstance);
                       value = Number(value);
                       if (isNaN(value))
                           return value;
-                      const { locale, format } = localeFormat;
+                      const { locale, format } = options;
                       return formatValue(value, {
                           locale,
                           format: !format && Math.abs(value) >= 1000 ? "#,##" : format,
@@ -20344,9 +20353,10 @@ stores.inject(MyMetaStore, storeInstance);
           ({ labels, dataSetsValues } = aggregateDataForLabels(labels, dataSetsValues));
       }
       const locale = getters.getLocale();
+      const truncateLabels = axisType === "category";
       const dataSetFormat = getChartDatasetFormat(getters, chart.dataSets);
-      const localeFormat = { format: dataSetFormat, locale };
-      const config = getLineOrScatterConfiguration(chart, labels, localeFormat);
+      const options = { format: dataSetFormat, locale, truncateLabels };
+      const config = getLineOrScatterConfiguration(chart, labels, options);
       const labelFormat = getChartLabelFormat(getters, chart.labelRange);
       if (axisType === "time") {
           const axis = {
@@ -32900,7 +32910,8 @@ stores.inject(MyMetaStore, storeInstance);
       get checkBoxCellPositions() {
           return this.env.model.getters
               .getVisibleCellPositions()
-              .filter(this.env.model.getters.isCellValidCheckbox);
+              .filter((position) => this.env.model.getters.isCellValidCheckbox(position) &&
+              !this.env.model.getters.isFilterHeader(position));
       }
       get listIconsCellPositions() {
           if (this.env.model.getters.isReadonly()) {
@@ -32908,7 +32919,8 @@ stores.inject(MyMetaStore, storeInstance);
           }
           return this.env.model.getters
               .getVisibleCellPositions()
-              .filter(this.env.model.getters.cellHasListDataValidationIcon);
+              .filter((position) => this.env.model.getters.cellHasListDataValidationIcon(position) &&
+              !this.env.model.getters.isFilterHeader(position));
       }
   }
 
@@ -47421,9 +47433,10 @@ stores.inject(MyMetaStore, storeInstance);
               }
               if (!content) {
                   // The previous content could have blocked some array formulas
-                  impactedPositions.addMany(this.getArrayFormulasBlockedBy(position));
+                  impactedPositions.add(position);
               }
           }
+          impactedPositions.addMany(this.getArrayFormulasBlockedBy(impactedPositions));
           return impactedPositions;
       }
       buildDependencyGraph() {
@@ -47465,23 +47478,25 @@ stores.inject(MyMetaStore, storeInstance);
           return positions;
       }
       /**
-       * Return the position of formulas blocked by the given position
+       * Return the position of formulas blocked by the given positions
        * as well as all their dependencies.
        */
-      getArrayFormulasBlockedBy(position) {
-          if (!this.spreadingRelations.hasArrayFormulaResult(position)) {
-              return [];
-          }
-          const arrayFormulas = this.spreadingRelations.getFormulaPositionsSpreadingOn(position);
-          const positions = this.createEmptyPositionSet();
-          positions.addMany(arrayFormulas);
-          const arrayFormulaPosition = this.getArrayFormulaSpreadingOn(position);
-          if (arrayFormulaPosition) {
-              // ignore the formula spreading on the position. Keep only the blocked ones
-              positions.delete(arrayFormulaPosition);
+      getArrayFormulasBlockedBy(positions) {
+          const arrayFormulaPositions = this.createEmptyPositionSet();
+          for (const position of positions) {
+              if (!this.spreadingRelations.hasArrayFormulaResult(position)) {
+                  continue;
+              }
+              const arrayFormulas = this.spreadingRelations.getFormulaPositionsSpreadingOn(position);
+              arrayFormulaPositions.addMany(arrayFormulas);
+              const arrayFormulaPosition = this.getArrayFormulaSpreadingOn(position);
+              if (arrayFormulaPosition) {
+                  // ignore the formula spreading on the position. Keep only the blocked ones
+                  arrayFormulaPositions.delete(arrayFormulaPosition);
+              }
           }
-          positions.addMany(this.getCellsDependingOn(positions));
-          return positions;
+          arrayFormulaPositions.addMany(this.getCellsDependingOn(arrayFormulaPositions));
+          return arrayFormulaPositions;
       }
       nextPositionsToUpdate = new PositionSet({});
       cellsBeingComputed = new Set();
@@ -47611,6 +47626,7 @@ stores.inject(MyMetaStore, storeInstance);
           if (!this.spreadingRelations.isArrayFormula(position)) {
               return;
           }
+          const invalidated = this.createEmptyPositionSet();
           for (const child of this.spreadingRelations.getArrayResultPositions(position)) {
               const content = this.getters.getCell(child)?.content;
               if (content) {
@@ -47618,10 +47634,11 @@ stores.inject(MyMetaStore, storeInstance);
                   // there's still a collision
                   continue;
               }
+              invalidated.add(child);
               this.evaluatedCells.delete(child);
-              this.nextPositionsToUpdate.addMany(this.getCellsDependingOn([child]));
-              this.nextPositionsToUpdate.addMany(this.getArrayFormulasBlockedBy(child));
           }
+          this.nextPositionsToUpdate.addMany(this.getCellsDependingOn(invalidated));
+          this.nextPositionsToUpdate.addMany(this.getArrayFormulasBlockedBy(invalidated));
           this.spreadingRelations.removeNode(position);
       }
       // ----------------------------------------------------------
@@ -47928,7 +47945,7 @@ stores.inject(MyMetaStore, storeInstance);
                   ? getItemId(newFormat, data.formats)
                   : exportedCellData.format;
               let content;
-              if (isFormula && formulaCell instanceof FormulaCellWithDependencies) {
+              if (isExported && isFormula && formulaCell instanceof FormulaCellWithDependencies) {
                   content = formulaCell.contentWithFixedReferences;
               }
               else {
@@ -60599,9 +60616,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.2.5";
-  __info__.date = "2024-04-26T07:41:13.193Z";
-  __info__.hash = "a730f5c";
+  __info__.version = "17.2.7";
+  __info__.date = "2024-05-15T09:20:44.429Z";
+  __info__.hash = "57e89fa";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);