@odoo/o-spreadsheet 17.3.3 → 17.3.5

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.3
- * @date 2024-06-14T10:02:58.082Z
- * @hash c690e9f
+ * @version 17.3.5
+ * @date 2024-06-26T11:05:50.339Z
+ * @hash 54eb151
  */
 
 (function (exports, owl) {
@@ -1897,9 +1897,10 @@
           sortedValues[indexLow] * (indexSup - percentIndex));
   }
 
-  // define a mock translation function, when o-spreadsheet runs in standalone it doesn't translate any string
-  let _translate = (s) => s;
-  let _loaded = () => false;
+  const defaultTranslate = (s) => s;
+  const defaultLoaded = () => false;
+  let _translate = defaultTranslate;
+  let _loaded = defaultLoaded;
   function sprintf(s, ...values) {
       if (values.length === 1 && typeof values[0] === "object" && !(values[0] instanceof String)) {
           const valuesDict = values[0];
@@ -1911,7 +1912,8 @@
       return s;
   }
   /***
-   * Allow to inject a translation function from outside o-spreadsheet.
+   * Allow to inject a translation function from outside o-spreadsheet. This should be called before instantiating
+   * a model.
    * @param tfn the function that will do the translation
    * @param loaded a function that returns true when the translation is loaded
    */
@@ -1919,6 +1921,18 @@
       _translate = tfn;
       _loaded = loaded;
   }
+  /**
+   * If no translation function has been set, this will mark the translation are loaded.
+   *
+   * By default, the translations should not be set as loaded, otherwise top-level translated constants will never be
+   * translated. But if by the time the model is instantiated no custom translation function has been set, we can set
+   * the default translation function as loaded so o-spreadsheet can be run in standalone with no translations.
+   */
+  function setDefaultTranslationMethod() {
+      if (_translate === defaultTranslate && _loaded === defaultLoaded) {
+          _loaded = () => true;
+      }
+  }
   const _t = function (s, ...values) {
       if (!_loaded()) {
           return new LazyTranslatedString(s, values);
@@ -9460,10 +9474,8 @@ stores.inject(MyMetaStore, storeInstance);
               const refRange = this.getters.getRangeFromSheetXC(activeSheetId, xc);
               return isEqual(this.getters.expandZone(activeSheetId, refRange.zone), oldZone);
           });
-          // this function assumes that the previous range is always found because
-          // it's called when changing a highlight, which exists by definition
           if (!previousRefToken) {
-              throw new Error("Previous range not found");
+              return;
           }
           const previousRange = this.getters.getRangeFromSheetXC(activeSheetId, previousRefToken.value);
           this.selectionStart = previousRefToken.start;
@@ -34064,6 +34076,7 @@ stores.inject(MyMetaStore, storeInstance);
       currentSearchRegex = null;
       isSearchDirty = false;
       initialShowFormulaState;
+      preserveSelectedMatchIndex = false;
       // fixme: why do we make selectedMatchIndex on top of a selected
       // property in the matches?
       selectedMatchIndex = null;
@@ -34173,7 +34186,9 @@ stores.inject(MyMetaStore, storeInstance);
        * refresh the matches according to the current search options
        */
       refreshSearch(jumpToMatchSheet = true) {
-          this.selectedMatchIndex = null;
+          if (!this.preserveSelectedMatchIndex) {
+              this.selectedMatchIndex = null;
+          }
           this.findMatches();
           this.selectNextCell(Direction.current, jumpToMatchSheet);
       }
@@ -34272,10 +34287,16 @@ stores.inject(MyMetaStore, storeInstance);
           const selectedMatch = matches[nextIndex];
           // Switch to the sheet where the match is located
           if (jumpToMatchSheet && this.getters.getActiveSheetId() !== selectedMatch.sheetId) {
+              // We set `preserveSelectedMatchIndex` to true to avoid resetting the selected search
+              // index in the `refreshSearch` function when a new sheet is activated. The reason being
+              // that, when we automatically go back to previous sheet while performing a search, the
+              // search index is reset to the first occurrence each time.
+              this.preserveSelectedMatchIndex = true;
               this.model.dispatch("ACTIVATE_SHEET", {
                   sheetIdFrom: this.getters.getActiveSheetId(),
                   sheetIdTo: selectedMatch.sheetId,
               });
+              this.preserveSelectedMatchIndex = false;
               // We do not want to reset the selection at finalize in this case
               this.isSearchDirty = false;
           }
@@ -34925,7 +34946,7 @@ stores.inject(MyMetaStore, storeInstance);
   function createPivotDimension(fields, dimension) {
       const field = fields[dimension.name];
       const type = field?.type ?? "integer";
-      const granularity = field && isDateField(field) ? dimension.granularity ?? "month_number" : undefined;
+      const granularity = field && isDateField(field) ? dimension.granularity : undefined;
       return {
           /**
            * Get the display name of the dimension
@@ -35328,10 +35349,13 @@ stores.inject(MyMetaStore, storeInstance);
   }
 
   function createDate(dimension, value, locale) {
-      const granularity = dimension.granularity || "month_number";
-      if (!(granularity in MAP_VALUE_DIMENSION_DATE)) {
+      const granularity = dimension.granularity;
+      if (!granularity || !(granularity in MAP_VALUE_DIMENSION_DATE)) {
           throw new Error(`Unknown date granularity: ${granularity}`);
       }
+      if (value === null) {
+          return null;
+      }
       if (!MAP_VALUE_DIMENSION_DATE[granularity].set.has(value)) {
           MAP_VALUE_DIMENSION_DATE[granularity].set.add(value);
           const date = toJsDate(value, locale);
@@ -35558,7 +35582,8 @@ stores.inject(MyMetaStore, storeInstance);
           if (!finalCell) {
               return { value: "" };
           }
-          if (finalCell.value === null) {
+          // Value can be null but stringified (e.g. an empty date, as for now every date is stringified)
+          if (finalCell.value === null || finalCell.value === `${null}`) {
               return { value: _t("(Undefined)") };
           }
           if (dimension.type === "date") {
@@ -35594,10 +35619,15 @@ stores.inject(MyMetaStore, storeInstance);
           if (!operator) {
               throw new Error(`Aggregator ${aggregator} does not exist`);
           }
-          return {
-              value: values.length ? operator.fn([values], this.getters.getLocale()) : "",
-              format: operator.format(values[0]),
-          };
+          try {
+              return {
+                  value: values.length ? operator.fn([values], this.getters.getLocale()) : "",
+                  format: operator.format(values[0]),
+              };
+          }
+          catch (e) {
+              return handleError(e, aggregator.toUpperCase());
+          }
       }
       getPossibleFieldValues(groupBy) {
           //TODO This method should be implemented for the autocomplete feature
@@ -52839,7 +52869,6 @@ stores.inject(MyMetaStore, storeInstance);
           if (!this.blockedArrayFormulas.has(position)) {
               this.invalidateSpreading(position);
           }
-          this.spreadingRelations.removeNode(position);
           const cell = this.getters.getCell(position);
           if (cell === undefined) {
               return EMPTY_CELL;
@@ -52881,6 +52910,7 @@ stores.inject(MyMetaStore, storeInstance);
           this.assertSheetHasEnoughSpaceToSpreadFormulaResult(formulaPosition, formulaReturn);
           const nbColumns = formulaReturn.length;
           const nbRows = formulaReturn[0].length;
+          this.spreadingRelations.removeNode(formulaPosition);
           forEachSpreadPositionInMatrix(nbColumns, nbRows, this.updateSpreadRelation(formulaPosition));
           this.assertNoMergedCellsInSpreadZone(formulaPosition, formulaReturn);
           forEachSpreadPositionInMatrix(nbColumns, nbRows, this.checkCollision(formulaPosition));
@@ -54481,7 +54511,7 @@ stores.inject(MyMetaStore, storeInstance);
       getPivotIdFromPosition(position) {
           const cell = this.getters.getCorrespondingFormulaCell(position);
           if (cell && cell.isFormula) {
-              const pivotFunction = this.getFirstPivotFunction(cell.compiledFormula.tokens);
+              const pivotFunction = this.getFirstPivotFunction(position.sheetId, cell.compiledFormula.tokens);
               if (pivotFunction) {
                   const pivotId = pivotFunction.args[0]?.toString();
                   return pivotId && this.getters.getPivotId(pivotId);
@@ -54489,7 +54519,7 @@ stores.inject(MyMetaStore, storeInstance);
           }
           return undefined;
       }
-      getFirstPivotFunction(tokens) {
+      getFirstPivotFunction(sheetId, tokens) {
           const pivotFunction = getFirstPivotFunction(tokens);
           if (!pivotFunction) {
               return undefined;
@@ -54505,7 +54535,7 @@ stores.inject(MyMetaStore, storeInstance);
                   return argAst.value;
               }
               const argsString = astToFormula(argAst);
-              return this.getters.evaluateFormula(this.getters.getActiveSheetId(), argsString);
+              return this.getters.evaluateFormula(sheetId, argsString);
           });
           return { functionName, args: evaluatedArgs };
       }
@@ -54528,7 +54558,7 @@ stores.inject(MyMetaStore, storeInstance);
               return undefined;
           }
           const mainPosition = this.getters.getCellPosition(cell.id);
-          const result = this.getters.getFirstPivotFunction(cell.compiledFormula.tokens);
+          const result = this.getters.getFirstPivotFunction(position.sheetId, cell.compiledFormula.tokens);
           if (!result) {
               return undefined;
           }
@@ -66306,6 +66336,7 @@ stores.inject(MyMetaStore, storeInstance);
           const start = performance.now();
           console.group("Model creation");
           super();
+          setDefaultTranslationMethod();
           stateUpdateMessages = repairInitialMessages(data, stateUpdateMessages);
           const workbookData = load(data, verboseImport);
           this.state = new StateObserver();
@@ -66982,9 +67013,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.3.3";
-  __info__.date = "2024-06-14T10:02:58.082Z";
-  __info__.hash = "c690e9f";
+  __info__.version = "17.3.5";
+  __info__.date = "2024-06-26T11:05:50.339Z";
+  __info__.hash = "54eb151";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);