@odoo/o-spreadsheet 18.0.20 → 18.0.21

package/dist/o-spreadsheet.iife.js

@@ -2,9 +2,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 18.0.20
- * @date 2025-03-19T08:21:32.426Z
- * @hash 3f48d8b
+ * @version 18.0.21
+ * @date 2025-03-26T12:49:46.872Z
+ * @hash c687e1c
  */
 
 (function (exports, owl) {
@@ -1112,7 +1112,10 @@
       }
       else if (stringVals.length === 4) {
           const alpha = parseFloat(stringVals.pop() || "1");
-          alphaHex = Math.round((alpha || 1) * 255);
+          if (isNaN(alpha)) {
+              throw new Error("invalid alpha value");
+          }
+          alphaHex = Math.round(alpha * 255);
       }
       const vals = stringVals.map((val) => parseInt(val, 10));
       if (alphaHex !== 255) {
@@ -6836,7 +6839,7 @@
    */
   function canonicalizeNumberContent(content, locale) {
       return content.startsWith("=")
-          ? canonicalizeFormula$1(content, locale)
+          ? canonicalizeFormula(content, locale)
           : canonicalizeNumberLiteral(content, locale);
   }
   /**
@@ -6851,7 +6854,7 @@
    */
   function canonicalizeContent(content, locale) {
       return content.startsWith("=")
-          ? canonicalizeFormula$1(content, locale)
+          ? canonicalizeFormula(content, locale)
           : canonicalizeLiteral(content, locale);
   }
   /**
@@ -6867,15 +6870,21 @@
           ? localizeFormula(content, locale)
           : localizeLiteral(content, locale);
   }
+  /** Change a number string to its canonical form (en_US locale) */
+  function canonicalizeNumberValue(content, locale) {
+      return content.startsWith("=")
+          ? canonicalizeFormula(content, locale)
+          : canonicalizeNumberLiteral(content, locale);
+  }
   /** Change a formula to its canonical form (en_US locale) */
-  function canonicalizeFormula$1(formula, locale) {
-      return _localizeFormula$1(formula, locale, DEFAULT_LOCALE);
+  function canonicalizeFormula(formula, locale) {
+      return _localizeFormula(formula, locale, DEFAULT_LOCALE);
   }
   /** Change a formula from the canonical form to the given locale */
   function localizeFormula(formula, locale) {
-      return _localizeFormula$1(formula, DEFAULT_LOCALE, locale);
+      return _localizeFormula(formula, DEFAULT_LOCALE, locale);
   }
-  function _localizeFormula$1(formula, fromLocale, toLocale) {
+  function _localizeFormula(formula, fromLocale, toLocale) {
       if (fromLocale.formulaArgSeparator === toLocale.formulaArgSeparator &&
           fromLocale.decimalSeparator === toLocale.decimalSeparator) {
           return formula;
@@ -7031,37 +7040,6 @@
       return locale.dateFormat + " " + locale.timeFormat;
   }
 
-  /** Change a number string to its canonical form (en_US locale) */
-  function canonicalizeNumberValue(content, locale) {
-      return content.startsWith("=")
-          ? canonicalizeFormula(content, locale)
-          : canonicalizeNumberLiteral(content, locale);
-  }
-  /** Change a formula to its canonical form (en_US locale) */
-  function canonicalizeFormula(formula, locale) {
-      return _localizeFormula(formula, locale, DEFAULT_LOCALE);
-  }
-  function _localizeFormula(formula, fromLocale, toLocale) {
-      if (fromLocale.formulaArgSeparator === toLocale.formulaArgSeparator &&
-          fromLocale.decimalSeparator === toLocale.decimalSeparator) {
-          return formula;
-      }
-      const tokens = tokenize(formula, fromLocale);
-      let localizedFormula = "";
-      for (const token of tokens) {
-          if (token.type === "NUMBER") {
-              localizedFormula += token.value.replace(fromLocale.decimalSeparator, toLocale.decimalSeparator);
-          }
-          else if (token.type === "ARG_SEPARATOR") {
-              localizedFormula += toLocale.formulaArgSeparator;
-          }
-          else {
-              localizedFormula += token.value;
-          }
-      }
-      return localizedFormula;
-  }
-
   function boolAnd(args) {
       let foundBoolean = false;
       let acc = true;
@@ -10072,9 +10050,9 @@ stores.inject(MyMetaStore, storeInstance);
   /** In XLSX color format (no #)  */
   const AUTO_COLOR = "000000";
   const XLSX_ICONSET_MAP = {
-      arrow: "3Arrows",
+      arrows: "3Arrows",
       smiley: "3Symbols",
-      dot: "3TrafficLights1",
+      dots: "3TrafficLights1",
   };
   const NAMESPACE = {
       styleSheet: "http://schemas.openxmlformats.org/spreadsheetml/2006/main",
@@ -10746,6 +10724,7 @@ stores.inject(MyMetaStore, storeInstance);
   };
   /** Map between legend position in XLSX file and human readable position  */
   const DRAWING_LEGEND_POSITION_CONVERSION_MAP = {
+      none: "none",
       b: "bottom",
       t: "top",
       l: "left",
@@ -13502,7 +13481,7 @@ stores.inject(MyMetaStore, storeInstance);
                       default: "ffffff",
                   }).asString(),
                   legendPosition: DRAWING_LEGEND_POSITION_CONVERSION_MAP[this.extractChildAttr(rootChartElement, "c:legendPos", "val", {
-                      default: "b",
+                      default: "none",
                   }).asString()],
                   stacked: barChartGrouping === "stacked",
                   fontColor: "000000",
@@ -13536,7 +13515,7 @@ stores.inject(MyMetaStore, storeInstance);
                   default: "ffffff",
               }).asString(),
               legendPosition: DRAWING_LEGEND_POSITION_CONVERSION_MAP[this.extractChildAttr(chartElement, "c:legendPos", "val", {
-                  default: "b",
+                  default: "none",
               }).asString()],
               stacked: barChartGrouping === "stacked",
               fontColor: "000000",
@@ -16728,7 +16707,8 @@ stores.inject(MyMetaStore, storeInstance);
           }
       }
       else if (dataSets.length === 1) {
-          for (let i = 0; i < getData(getters, dataSets[0]).length; i++) {
+          const dataLength = getData(getters, dataSets[0]).length;
+          for (let i = 0; i < dataLength; i++) {
               labels.formattedValues.push("");
               labels.values.push("");
           }
@@ -25974,11 +25954,26 @@ stores.inject(MyMetaStore, storeInstance);
           const _searchFor = toString(searchFor).toLowerCase();
           const _textToSearch = toString(textToSearch).toLowerCase();
           const _startingAt = toNumber(startingAt, this.locale);
-          assert(() => _textToSearch !== "", _t("The text_to_search must be non-empty."));
-          assert(() => _startingAt >= 1, _t("The starting_at (%s) must be greater than or equal to 1.", _startingAt.toString()));
+          if (_textToSearch === "") {
+              return {
+                  value: CellErrorType.GenericError,
+                  message: _t("The text_to_search must be non-empty."),
+              };
+          }
+          if (_startingAt < 1) {
+              return {
+                  value: CellErrorType.GenericError,
+                  message: _t("The starting_at (%s) must be greater than or equal to 1.", _startingAt),
+              };
+          }
           const result = _textToSearch.indexOf(_searchFor, _startingAt - 1);
-          assert(() => result >= 0, _t("In [[FUNCTION_NAME]] evaluation, cannot find '%s' within '%s'.", _searchFor, _textToSearch));
-          return result + 1;
+          if (result === -1) {
+              return {
+                  value: CellErrorType.GenericError,
+                  message: _t("In [[FUNCTION_NAME]] evaluation, cannot find '%s' within '%s'.", _searchFor, _textToSearch),
+              };
+          }
+          return { value: result + 1 };
       },
       isExported: true,
   };
@@ -27828,11 +27823,14 @@ stores.inject(MyMetaStore, storeInstance);
       }
   }
   function compileTokensOrThrow(tokens) {
-      const { dependencies, constantValues, symbols } = formulaArguments(tokens);
-      const cacheKey = compilationCacheKey(tokens, dependencies, constantValues);
+      const { dependencies, literalValues, symbols } = formulaArguments(tokens);
+      const cacheKey = compilationCacheKey(tokens);
       if (!functionCache[cacheKey]) {
           const ast = parseTokens([...tokens]);
           const scope = new Scope();
+          let stringCount = 0;
+          let numberCount = 0;
+          let dependencyCount = 0;
           if (ast.type === "BIN_OPERATION" && ast.value === ":") {
               throw new BadExpressionError(_t("Invalid formula"));
           }
@@ -27905,16 +27903,15 @@ stores.inject(MyMetaStore, storeInstance);
                   case "BOOLEAN":
                       return code.return(`{ value: ${ast.value} }`);
                   case "NUMBER":
-                      return code.return(`{ value: this.constantValues.numbers[${constantValues.numbers.indexOf(ast.value)}] }`);
+                      return code.return(`this.literalValues.numbers[${numberCount++}]`);
                   case "STRING":
-                      return code.return(`{ value: this.constantValues.strings[${constantValues.strings.indexOf(ast.value)}] }`);
+                      return code.return(`this.literalValues.strings[${stringCount++}]`);
                   case "REFERENCE":
-                      const referenceIndex = dependencies.indexOf(ast.value);
                       if ((!isMeta && ast.value.includes(":")) || hasRange) {
-                          return code.return(`range(deps[${referenceIndex}])`);
+                          return code.return(`range(deps[${dependencyCount++}])`);
                       }
                       else {
-                          return code.return(`ref(deps[${referenceIndex}], ${isMeta ? "true" : "false"})`);
+                          return code.return(`ref(deps[${dependencyCount++}], ${isMeta ? "true" : "false"})`);
                       }
                   case "FUNCALL":
                       const args = compileFunctionArgs(ast).map((arg) => arg.assignResultToVariable());
@@ -27946,7 +27943,7 @@ stores.inject(MyMetaStore, storeInstance);
       const compiledFormula = {
           execute: functionCache[cacheKey],
           dependencies,
-          constantValues,
+          literalValues,
           symbols,
           tokens,
           isBadExpression: false,
@@ -27958,33 +27955,31 @@ stores.inject(MyMetaStore, storeInstance);
    * References, numbers and strings are replaced with placeholders because
    * the compiled formula does not depend on their actual value.
    * Both `=A1+1+"2"` and `=A2+2+"3"` are compiled to the exact same function.
-   *
    * Spaces are also ignored to compute the cache key.
    *
-   * A formula `=A1+A2+SUM(2, 2, "2")` have the cache key `=|0|+|1|+SUM(|N0|,|N0|,|S0|)`
+   * A formula `=A1+A2+SUM(2, 2, "2")` have the cache key `=|C|+|C|+SUM(|N|,|N|,|S|)`
    */
-  function compilationCacheKey(tokens, dependencies, constantValues, symbols) {
+  function compilationCacheKey(tokens) {
       let cacheKey = "";
       for (const token of tokens) {
           switch (token.type) {
               case "STRING":
-                  const value = removeStringQuotes(token.value);
-                  cacheKey += `|S${constantValues.strings.indexOf(value)}|`;
+                  cacheKey += "|S|";
                   break;
               case "NUMBER":
-                  cacheKey += `|N${constantValues.numbers.indexOf(parseNumber(token.value, DEFAULT_LOCALE))}|`;
+                  cacheKey += "|N|";
                   break;
               case "REFERENCE":
               case "INVALID_REFERENCE":
                   if (token.value.includes(":")) {
-                      cacheKey += `R|${dependencies.indexOf(token.value)}|`;
+                      cacheKey += "|R|";
                   }
                   else {
-                      cacheKey += `C|${dependencies.indexOf(token.value)}|`;
+                      cacheKey += "|C|";
                   }
                   break;
               case "SPACE":
-                  cacheKey += "";
+                  // ignore spaces
                   break;
               default:
                   cacheKey += token.value;
@@ -27997,7 +27992,7 @@ stores.inject(MyMetaStore, storeInstance);
    * Return formula arguments which are references, strings and numbers.
    */
   function formulaArguments(tokens) {
-      const constantValues = {
+      const literalValues = {
           numbers: [],
           strings: [],
       };
@@ -28011,15 +28006,11 @@ stores.inject(MyMetaStore, storeInstance);
                   break;
               case "STRING":
                   const value = removeStringQuotes(token.value);
-                  if (!constantValues.strings.includes(value)) {
-                      constantValues.strings.push(value);
-                  }
+                  literalValues.strings.push({ value });
                   break;
               case "NUMBER": {
                   const value = parseNumber(token.value, DEFAULT_LOCALE);
-                  if (!constantValues.numbers.includes(value)) {
-                      constantValues.numbers.push(value);
-                  }
+                  literalValues.numbers.push({ value });
                   break;
               }
               case "SYMBOL": {
@@ -28030,7 +28021,7 @@ stores.inject(MyMetaStore, storeInstance);
       }
       return {
           dependencies,
-          constantValues,
+          literalValues,
           symbols,
       };
   }
@@ -68298,7 +68289,9 @@ stores.inject(MyMetaStore, storeInstance);
     border: 1px solid;
     font-family: ${DEFAULT_FONT};
 
-    .o-composer:empty:not(:focus):not(.active)::before {
+    /* In readonly we always show the fx icon if the composer is empty, not matter the focus */
+    .o-composer:empty:not(:focus):not(.active)::before,
+    &.o-topbar-composer-readonly .o-composer:empty::before {
       content: url("data:image/svg+xml,${encodeURIComponent(FX_SVG)}");
       position: relative;
       top: 20%;
@@ -71659,10 +71652,14 @@ stores.inject(MyMetaStore, storeInstance);
               continue;
           }
           const cfValueObjectNodes = cfValueObject.map((attrs) => escapeXml /*xml*/ `<cfvo ${formatAttributes(attrs)} />`);
+          const iconSetAttrs = [["iconSet", getIconSet(rule.icons)]];
+          if (isIconSetReversed(rule.icons)) {
+              iconSetAttrs.push(["reverse", "1"]);
+          }
           conditionalFormats.push(escapeXml /*xml*/ `
       <conditionalFormatting sqref="${range}">
         <cfRule ${formatAttributes(ruleAttributes)}>
-          <iconSet iconSet="${getIconSet(rule.icons)}">
+          <iconSet ${formatAttributes(iconSetAttrs)}>
             ${joinXmlNodes(cfValueObjectNodes)}
           </iconSet>
         </cfRule>
@@ -71680,9 +71677,21 @@ stores.inject(MyMetaStore, storeInstance);
           ["stopIfTrue", cf.stopIfTrue ? 1 : 0],
       ];
   }
+  function isIconSetReversed(iconSet) {
+      const defaultIconSet = ICON_SETS[detectIconsType(iconSet)];
+      return iconSet.upper === defaultIconSet.bad && iconSet.lower === defaultIconSet.good;
+  }
   function getIconSet(iconSet) {
-      return XLSX_ICONSET_MAP[Object.keys(XLSX_ICONSET_MAP).find((key) => iconSet.upper.toLowerCase().startsWith(key)) ||
-          "dots"];
+      return XLSX_ICONSET_MAP[detectIconsType(iconSet)];
+  }
+  /**
+   * Partial detection based on "upper" point only.
+   * We support any arbitrary icon in the set, while excel doesn't allow
+   * mixing icons from different types.
+   */
+  function detectIconsType(iconSet) {
+      const type = Object.keys(ICON_SETS).find((type) => Object.values(ICON_SETS[type]).includes(iconSet.upper)) || "dots";
+      return type;
   }
   function thresholdAttributes(threshold, position) {
       const type = getExcelThresholdType(threshold.type, position);
@@ -73606,9 +73615,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "18.0.20";
-  __info__.date = "2025-03-19T08:21:32.426Z";
-  __info__.hash = "3f48d8b";
+  __info__.version = "18.0.21";
+  __info__.date = "2025-03-26T12:49:46.872Z";
+  __info__.hash = "c687e1c";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);