@odoo/o-spreadsheet 15.2.14 → 15.2.16

package/dist/o-spreadsheet.cjs.js

@@ -18786,13 +18786,18 @@ class EvaluationPlugin extends UIPlugin {
     // Getters
     // ---------------------------------------------------------------------------
     evaluateFormula(formulaString, sheetId = this.getters.getActiveSheetId()) {
-        const compiledFormula = compile(formulaString);
-        const params = this.getFormulaParameters(() => { });
-        const ranges = [];
-        for (let xc of compiledFormula.dependencies) {
-            ranges.push(this.getters.getRangeFromSheetXC(sheetId, xc));
+        try {
+            const compiledFormula = compile(formulaString);
+            const params = this.getFormulaParameters(() => { });
+            const ranges = [];
+            for (let xc of compiledFormula.dependencies) {
+                ranges.push(this.getters.getRangeFromSheetXC(sheetId, xc));
+            }
+            return compiledFormula.execute(ranges, sheetId, ...params);
+        }
+        catch {
+            return "#ERROR";
         }
-        return compiledFormula.execute(ranges, sheetId, ...params);
     }
     /**
      * Return the value of each cell in the range as they are displayed in the grid.
@@ -18904,10 +18909,8 @@ class EvaluationPlugin extends UIPlugin {
          * that are actually present in the grid.
          */
         function _range(range) {
+            assertRangeValid(range);
             const sheetId = range.sheetId;
-            if (!isZoneValid(range.zone)) {
-                throw new Error(_lt("Invalid reference"));
-            }
             // Performance issue: Avoid fetching data on positions that are out of the spreadsheet
             // e.g. A1:ZZZ9999 in a sheet with 10 cols and 10 rows should ignore everything past J10 and return a 10x10 array
             const sheetZone = {
@@ -18946,21 +18949,16 @@ class EvaluationPlugin extends UIPlugin {
          */
         function refFn(position, references, sheetId, isMeta, functionName, paramNumber) {
             const range = references[position];
+            assertRangeValid(range);
             if (isMeta) {
                 return evalContext.getters.getRangeString(range, sheetId);
             }
-            if (!isZoneValid(range.zone)) {
-                throw new Error(_lt("Invalid reference"));
-            }
             // if the formula definition could have accepted a range, we would pass through the _range function and not here
             if (range.zone.bottom !== range.zone.top || range.zone.left !== range.zone.right) {
                 throw new Error(paramNumber
                     ? _lt("Function %s expects the parameter %s to be a single value or a single cell reference, not a range.", functionName.toString(), paramNumber.toString())
                     : _lt("Function %s expects its parameters to be single values or single cell references, not ranges.", functionName.toString()));
             }
-            if (range.invalidSheetName) {
-                throw new Error(_lt("Invalid sheet name: %s", range.invalidSheetName));
-            }
             return readCell(range);
         }
         /**
@@ -18973,6 +18971,14 @@ class EvaluationPlugin extends UIPlugin {
         function range(position, references, sheetId) {
             return _range(references[position]);
         }
+        function assertRangeValid(range) {
+            if (!isZoneValid(range.zone)) {
+                throw new Error(_lt("Invalid reference"));
+            }
+            if (range.invalidSheetName) {
+                throw new Error(_lt("Invalid sheet name: %s", range.invalidSheetName));
+            }
+        }
         return [refFn, range, evalContext];
     }
     /**
@@ -19285,25 +19291,26 @@ class EvaluationConditionalFormatPlugin extends UIPlugin {
          */
         this.rulePredicate = {
             CellIsRule: (cell, rule) => {
+                var _a, _b, _c, _d, _e, _f;
                 if (cell && cell.evaluated.type === CellValueType.error) {
                     return false;
                 }
                 const values = rule.values.map(parsePrimitiveContent);
                 switch (rule.operator) {
                     case "IsEmpty":
-                        return !isDefined(cell) || cell.evaluated.value.toString().trim() === "";
+                        return !isDefined(cell) || ((_a = cell.evaluated.value) === null || _a === void 0 ? void 0 : _a.toString().trim()) === "";
                     case "IsNotEmpty":
-                        return isDefined(cell) && cell.evaluated.value.toString().trim() !== "";
+                        return isDefined(cell) && ((_b = cell.evaluated.value) === null || _b === void 0 ? void 0 : _b.toString().trim()) !== "";
                     case "BeginsWith":
                         if (!cell && values[0] === "") {
                             return false;
                         }
-                        return (isDefined(cell) && (cell === null || cell === void 0 ? void 0 : cell.evaluated.value.toString().startsWith(values[0].toString())));
+                        return (isDefined(cell) && ((_c = cell.evaluated.value) === null || _c === void 0 ? void 0 : _c.toString().startsWith(values[0].toString())));
                     case "EndsWith":
                         if (!cell && values[0] === "") {
                             return false;
                         }
-                        return isDefined(cell) && cell.evaluated.value.toString().endsWith(values[0].toString());
+                        return isDefined(cell) && ((_d = cell.evaluated.value) === null || _d === void 0 ? void 0 : _d.toString().endsWith(values[0].toString()));
                     case "Between":
                         return (isDefined(cell) &&
                             cell.evaluated.value >= values[0] &&
@@ -19313,11 +19320,11 @@ class EvaluationConditionalFormatPlugin extends UIPlugin {
                             cell.evaluated.value >= values[0] &&
                             cell.evaluated.value <= values[1]);
                     case "ContainsText":
-                        return (isDefined(cell) && cell.evaluated.value.toString().indexOf(values[0].toString()) > -1);
+                        return (isDefined(cell) && ((_e = cell.evaluated.value) === null || _e === void 0 ? void 0 : _e.toString().indexOf(values[0].toString())) > -1);
                     case "NotContains":
                         return (!isDefined(cell) ||
                             !cell.evaluated.value ||
-                            cell.evaluated.value.toString().indexOf(values[0].toString()) == -1);
+                            ((_f = cell.evaluated.value) === null || _f === void 0 ? void 0 : _f.toString().indexOf(values[0].toString())) == -1);
                     case "GreaterThan":
                         return isDefined(cell) && cell.evaluated.value > values[0];
                     case "GreaterThanOrEqual":
@@ -19426,39 +19433,34 @@ class EvaluationConditionalFormatPlugin extends UIPlugin {
         this.computedIcons[activeSheetId] = {};
         const computedStyle = this.computedStyles[activeSheetId];
         for (let cf of this.getters.getConditionalFormats(activeSheetId)) {
-            try {
-                switch (cf.rule.type) {
-                    case "ColorScaleRule":
-                        for (let range of cf.ranges) {
-                            this.applyColorScale(range, cf.rule);
-                        }
-                        break;
-                    case "IconSetRule":
-                        for (let range of cf.ranges) {
-                            this.applyIcon(range, cf.rule);
-                        }
-                        break;
-                    default:
-                        for (let ref of cf.ranges) {
-                            const zone = toZone(ref);
-                            for (let row = zone.top; row <= zone.bottom; row++) {
-                                for (let col = zone.left; col <= zone.right; col++) {
-                                    const pr = this.rulePredicate[cf.rule.type];
-                                    let cell = this.getters.getCell(activeSheetId, col, row);
-                                    if (pr && pr(cell, cf.rule)) {
-                                        if (!computedStyle[col])
-                                            computedStyle[col] = [];
-                                        // we must combine all the properties of all the CF rules applied to the given cell
-                                        computedStyle[col][row] = Object.assign(((_a = computedStyle[col]) === null || _a === void 0 ? void 0 : _a[row]) || {}, cf.rule.style);
-                                    }
+            switch (cf.rule.type) {
+                case "ColorScaleRule":
+                    for (let range of cf.ranges) {
+                        this.applyColorScale(range, cf.rule);
+                    }
+                    break;
+                case "IconSetRule":
+                    for (let range of cf.ranges) {
+                        this.applyIcon(range, cf.rule);
+                    }
+                    break;
+                default:
+                    for (let ref of cf.ranges) {
+                        const zone = toZone(ref);
+                        for (let row = zone.top; row <= zone.bottom; row++) {
+                            for (let col = zone.left; col <= zone.right; col++) {
+                                const pr = this.rulePredicate[cf.rule.type];
+                                let cell = this.getters.getCell(activeSheetId, col, row);
+                                if (pr && pr(cell, cf.rule)) {
+                                    if (!computedStyle[col])
+                                        computedStyle[col] = [];
+                                    // we must combine all the properties of all the CF rules applied to the given cell
+                                    computedStyle[col][row] = Object.assign(((_a = computedStyle[col]) === null || _a === void 0 ? void 0 : _a[row]) || {}, cf.rule.style);
                                 }
                             }
                         }
-                        break;
-                }
-            }
-            catch (_) {
-                // we don't care about the errors within the evaluation of a rule
+                    }
+                    break;
             }
         }
     }
@@ -19477,7 +19479,7 @@ class EvaluationConditionalFormatPlugin extends UIPlugin {
                 return this.getters.evaluateFormula(`=PERCENTILE(${range},${Number(threshold.value) / 100})`);
             case "formula":
                 const value = threshold.value && this.getters.evaluateFormula(threshold.value);
-                return !(value instanceof Promise) ? value : null;
+                return typeof value === "number" ? value : null;
             default:
                 return null;
         }
@@ -31440,6 +31442,6 @@ exports.registries = registries;
 exports.setTranslationMethod = setTranslationMethod;
 
 
-__info__.version = '15.2.14';
-__info__.date = '2024-01-12T13:37:05.872Z';
-__info__.hash = '6515f7c';
+__info__.version = '15.2.16';
+__info__.date = '2024-01-29T13:48:09.354Z';
+__info__.hash = '446a537';

package/dist/o-spreadsheet.esm.js

@@ -18763,13 +18763,18 @@ class EvaluationPlugin extends UIPlugin {
     // Getters
     // ---------------------------------------------------------------------------
     evaluateFormula(formulaString, sheetId = this.getters.getActiveSheetId()) {
-        const compiledFormula = compile(formulaString);
-        const params = this.getFormulaParameters(() => { });
-        const ranges = [];
-        for (let xc of compiledFormula.dependencies) {
-            ranges.push(this.getters.getRangeFromSheetXC(sheetId, xc));
+        try {
+            const compiledFormula = compile(formulaString);
+            const params = this.getFormulaParameters(() => { });
+            const ranges = [];
+            for (let xc of compiledFormula.dependencies) {
+                ranges.push(this.getters.getRangeFromSheetXC(sheetId, xc));
+            }
+            return compiledFormula.execute(ranges, sheetId, ...params);
+        }
+        catch {
+            return "#ERROR";
         }
-        return compiledFormula.execute(ranges, sheetId, ...params);
     }
     /**
      * Return the value of each cell in the range as they are displayed in the grid.
@@ -18881,10 +18886,8 @@ class EvaluationPlugin extends UIPlugin {
          * that are actually present in the grid.
          */
         function _range(range) {
+            assertRangeValid(range);
             const sheetId = range.sheetId;
-            if (!isZoneValid(range.zone)) {
-                throw new Error(_lt("Invalid reference"));
-            }
             // Performance issue: Avoid fetching data on positions that are out of the spreadsheet
             // e.g. A1:ZZZ9999 in a sheet with 10 cols and 10 rows should ignore everything past J10 and return a 10x10 array
             const sheetZone = {
@@ -18923,21 +18926,16 @@ class EvaluationPlugin extends UIPlugin {
          */
         function refFn(position, references, sheetId, isMeta, functionName, paramNumber) {
             const range = references[position];
+            assertRangeValid(range);
             if (isMeta) {
                 return evalContext.getters.getRangeString(range, sheetId);
             }
-            if (!isZoneValid(range.zone)) {
-                throw new Error(_lt("Invalid reference"));
-            }
             // if the formula definition could have accepted a range, we would pass through the _range function and not here
             if (range.zone.bottom !== range.zone.top || range.zone.left !== range.zone.right) {
                 throw new Error(paramNumber
                     ? _lt("Function %s expects the parameter %s to be a single value or a single cell reference, not a range.", functionName.toString(), paramNumber.toString())
                     : _lt("Function %s expects its parameters to be single values or single cell references, not ranges.", functionName.toString()));
             }
-            if (range.invalidSheetName) {
-                throw new Error(_lt("Invalid sheet name: %s", range.invalidSheetName));
-            }
             return readCell(range);
         }
         /**
@@ -18950,6 +18948,14 @@ class EvaluationPlugin extends UIPlugin {
         function range(position, references, sheetId) {
             return _range(references[position]);
         }
+        function assertRangeValid(range) {
+            if (!isZoneValid(range.zone)) {
+                throw new Error(_lt("Invalid reference"));
+            }
+            if (range.invalidSheetName) {
+                throw new Error(_lt("Invalid sheet name: %s", range.invalidSheetName));
+            }
+        }
         return [refFn, range, evalContext];
     }
     /**
@@ -19262,25 +19268,26 @@ class EvaluationConditionalFormatPlugin extends UIPlugin {
          */
         this.rulePredicate = {
             CellIsRule: (cell, rule) => {
+                var _a, _b, _c, _d, _e, _f;
                 if (cell && cell.evaluated.type === CellValueType.error) {
                     return false;
                 }
                 const values = rule.values.map(parsePrimitiveContent);
                 switch (rule.operator) {
                     case "IsEmpty":
-                        return !isDefined(cell) || cell.evaluated.value.toString().trim() === "";
+                        return !isDefined(cell) || ((_a = cell.evaluated.value) === null || _a === void 0 ? void 0 : _a.toString().trim()) === "";
                     case "IsNotEmpty":
-                        return isDefined(cell) && cell.evaluated.value.toString().trim() !== "";
+                        return isDefined(cell) && ((_b = cell.evaluated.value) === null || _b === void 0 ? void 0 : _b.toString().trim()) !== "";
                     case "BeginsWith":
                         if (!cell && values[0] === "") {
                             return false;
                         }
-                        return (isDefined(cell) && (cell === null || cell === void 0 ? void 0 : cell.evaluated.value.toString().startsWith(values[0].toString())));
+                        return (isDefined(cell) && ((_c = cell.evaluated.value) === null || _c === void 0 ? void 0 : _c.toString().startsWith(values[0].toString())));
                     case "EndsWith":
                         if (!cell && values[0] === "") {
                             return false;
                         }
-                        return isDefined(cell) && cell.evaluated.value.toString().endsWith(values[0].toString());
+                        return isDefined(cell) && ((_d = cell.evaluated.value) === null || _d === void 0 ? void 0 : _d.toString().endsWith(values[0].toString()));
                     case "Between":
                         return (isDefined(cell) &&
                             cell.evaluated.value >= values[0] &&
@@ -19290,11 +19297,11 @@ class EvaluationConditionalFormatPlugin extends UIPlugin {
                             cell.evaluated.value >= values[0] &&
                             cell.evaluated.value <= values[1]);
                     case "ContainsText":
-                        return (isDefined(cell) && cell.evaluated.value.toString().indexOf(values[0].toString()) > -1);
+                        return (isDefined(cell) && ((_e = cell.evaluated.value) === null || _e === void 0 ? void 0 : _e.toString().indexOf(values[0].toString())) > -1);
                     case "NotContains":
                         return (!isDefined(cell) ||
                             !cell.evaluated.value ||
-                            cell.evaluated.value.toString().indexOf(values[0].toString()) == -1);
+                            ((_f = cell.evaluated.value) === null || _f === void 0 ? void 0 : _f.toString().indexOf(values[0].toString())) == -1);
                     case "GreaterThan":
                         return isDefined(cell) && cell.evaluated.value > values[0];
                     case "GreaterThanOrEqual":
@@ -19403,39 +19410,34 @@ class EvaluationConditionalFormatPlugin extends UIPlugin {
         this.computedIcons[activeSheetId] = {};
         const computedStyle = this.computedStyles[activeSheetId];
         for (let cf of this.getters.getConditionalFormats(activeSheetId)) {
-            try {
-                switch (cf.rule.type) {
-                    case "ColorScaleRule":
-                        for (let range of cf.ranges) {
-                            this.applyColorScale(range, cf.rule);
-                        }
-                        break;
-                    case "IconSetRule":
-                        for (let range of cf.ranges) {
-                            this.applyIcon(range, cf.rule);
-                        }
-                        break;
-                    default:
-                        for (let ref of cf.ranges) {
-                            const zone = toZone(ref);
-                            for (let row = zone.top; row <= zone.bottom; row++) {
-                                for (let col = zone.left; col <= zone.right; col++) {
-                                    const pr = this.rulePredicate[cf.rule.type];
-                                    let cell = this.getters.getCell(activeSheetId, col, row);
-                                    if (pr && pr(cell, cf.rule)) {
-                                        if (!computedStyle[col])
-                                            computedStyle[col] = [];
-                                        // we must combine all the properties of all the CF rules applied to the given cell
-                                        computedStyle[col][row] = Object.assign(((_a = computedStyle[col]) === null || _a === void 0 ? void 0 : _a[row]) || {}, cf.rule.style);
-                                    }
+            switch (cf.rule.type) {
+                case "ColorScaleRule":
+                    for (let range of cf.ranges) {
+                        this.applyColorScale(range, cf.rule);
+                    }
+                    break;
+                case "IconSetRule":
+                    for (let range of cf.ranges) {
+                        this.applyIcon(range, cf.rule);
+                    }
+                    break;
+                default:
+                    for (let ref of cf.ranges) {
+                        const zone = toZone(ref);
+                        for (let row = zone.top; row <= zone.bottom; row++) {
+                            for (let col = zone.left; col <= zone.right; col++) {
+                                const pr = this.rulePredicate[cf.rule.type];
+                                let cell = this.getters.getCell(activeSheetId, col, row);
+                                if (pr && pr(cell, cf.rule)) {
+                                    if (!computedStyle[col])
+                                        computedStyle[col] = [];
+                                    // we must combine all the properties of all the CF rules applied to the given cell
+                                    computedStyle[col][row] = Object.assign(((_a = computedStyle[col]) === null || _a === void 0 ? void 0 : _a[row]) || {}, cf.rule.style);
                                 }
                             }
                         }
-                        break;
-                }
-            }
-            catch (_) {
-                // we don't care about the errors within the evaluation of a rule
+                    }
+                    break;
             }
         }
     }
@@ -19454,7 +19456,7 @@ class EvaluationConditionalFormatPlugin extends UIPlugin {
                 return this.getters.evaluateFormula(`=PERCENTILE(${range},${Number(threshold.value) / 100})`);
             case "formula":
                 const value = threshold.value && this.getters.evaluateFormula(threshold.value);
-                return !(value instanceof Promise) ? value : null;
+                return typeof value === "number" ? value : null;
             default:
                 return null;
         }
@@ -31394,6 +31396,6 @@ const helpers = {
 export { CommandResult, CorePlugin, DATETIME_FORMAT, DataSource, DispatchResult, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, DEBUG as __DEBUG__, __info__, astToFormula, cellTypes, compile, convertAstNodes, coreTypes, functionCache, helpers, invalidateEvaluationCommands, parse, readonlyAllowedCommands, registries, setTranslationMethod };
 
 
-__info__.version = '15.2.14';
-__info__.date = '2024-01-12T13:37:05.872Z';
-__info__.hash = '6515f7c';
+__info__.version = '15.2.16';
+__info__.date = '2024-01-29T13:48:09.354Z';
+__info__.hash = '446a537';

package/dist/o-spreadsheet.iife.js

@@ -18783,13 +18783,18 @@
       // Getters
       // ---------------------------------------------------------------------------
       evaluateFormula(formulaString, sheetId = this.getters.getActiveSheetId()) {
-          const compiledFormula = compile(formulaString);
-          const params = this.getFormulaParameters(() => { });
-          const ranges = [];
-          for (let xc of compiledFormula.dependencies) {
-              ranges.push(this.getters.getRangeFromSheetXC(sheetId, xc));
+          try {
+              const compiledFormula = compile(formulaString);
+              const params = this.getFormulaParameters(() => { });
+              const ranges = [];
+              for (let xc of compiledFormula.dependencies) {
+                  ranges.push(this.getters.getRangeFromSheetXC(sheetId, xc));
+              }
+              return compiledFormula.execute(ranges, sheetId, ...params);
+          }
+          catch {
+              return "#ERROR";
           }
-          return compiledFormula.execute(ranges, sheetId, ...params);
       }
       /**
        * Return the value of each cell in the range as they are displayed in the grid.
@@ -18901,10 +18906,8 @@
            * that are actually present in the grid.
            */
           function _range(range) {
+              assertRangeValid(range);
               const sheetId = range.sheetId;
-              if (!isZoneValid(range.zone)) {
-                  throw new Error(_lt("Invalid reference"));
-              }
               // Performance issue: Avoid fetching data on positions that are out of the spreadsheet
               // e.g. A1:ZZZ9999 in a sheet with 10 cols and 10 rows should ignore everything past J10 and return a 10x10 array
               const sheetZone = {
@@ -18943,21 +18946,16 @@
            */
           function refFn(position, references, sheetId, isMeta, functionName, paramNumber) {
               const range = references[position];
+              assertRangeValid(range);
               if (isMeta) {
                   return evalContext.getters.getRangeString(range, sheetId);
               }
-              if (!isZoneValid(range.zone)) {
-                  throw new Error(_lt("Invalid reference"));
-              }
               // if the formula definition could have accepted a range, we would pass through the _range function and not here
               if (range.zone.bottom !== range.zone.top || range.zone.left !== range.zone.right) {
                   throw new Error(paramNumber
                       ? _lt("Function %s expects the parameter %s to be a single value or a single cell reference, not a range.", functionName.toString(), paramNumber.toString())
                       : _lt("Function %s expects its parameters to be single values or single cell references, not ranges.", functionName.toString()));
               }
-              if (range.invalidSheetName) {
-                  throw new Error(_lt("Invalid sheet name: %s", range.invalidSheetName));
-              }
               return readCell(range);
           }
           /**
@@ -18970,6 +18968,14 @@
           function range(position, references, sheetId) {
               return _range(references[position]);
           }
+          function assertRangeValid(range) {
+              if (!isZoneValid(range.zone)) {
+                  throw new Error(_lt("Invalid reference"));
+              }
+              if (range.invalidSheetName) {
+                  throw new Error(_lt("Invalid sheet name: %s", range.invalidSheetName));
+              }
+          }
           return [refFn, range, evalContext];
       }
       /**
@@ -19282,25 +19288,26 @@
            */
           this.rulePredicate = {
               CellIsRule: (cell, rule) => {
+                  var _a, _b, _c, _d, _e, _f;
                   if (cell && cell.evaluated.type === CellValueType.error) {
                       return false;
                   }
                   const values = rule.values.map(parsePrimitiveContent);
                   switch (rule.operator) {
                       case "IsEmpty":
-                          return !isDefined(cell) || cell.evaluated.value.toString().trim() === "";
+                          return !isDefined(cell) || ((_a = cell.evaluated.value) === null || _a === void 0 ? void 0 : _a.toString().trim()) === "";
                       case "IsNotEmpty":
-                          return isDefined(cell) && cell.evaluated.value.toString().trim() !== "";
+                          return isDefined(cell) && ((_b = cell.evaluated.value) === null || _b === void 0 ? void 0 : _b.toString().trim()) !== "";
                       case "BeginsWith":
                           if (!cell && values[0] === "") {
                               return false;
                           }
-                          return (isDefined(cell) && (cell === null || cell === void 0 ? void 0 : cell.evaluated.value.toString().startsWith(values[0].toString())));
+                          return (isDefined(cell) && ((_c = cell.evaluated.value) === null || _c === void 0 ? void 0 : _c.toString().startsWith(values[0].toString())));
                       case "EndsWith":
                           if (!cell && values[0] === "") {
                               return false;
                           }
-                          return isDefined(cell) && cell.evaluated.value.toString().endsWith(values[0].toString());
+                          return isDefined(cell) && ((_d = cell.evaluated.value) === null || _d === void 0 ? void 0 : _d.toString().endsWith(values[0].toString()));
                       case "Between":
                           return (isDefined(cell) &&
                               cell.evaluated.value >= values[0] &&
@@ -19310,11 +19317,11 @@
                               cell.evaluated.value >= values[0] &&
                               cell.evaluated.value <= values[1]);
                       case "ContainsText":
-                          return (isDefined(cell) && cell.evaluated.value.toString().indexOf(values[0].toString()) > -1);
+                          return (isDefined(cell) && ((_e = cell.evaluated.value) === null || _e === void 0 ? void 0 : _e.toString().indexOf(values[0].toString())) > -1);
                       case "NotContains":
                           return (!isDefined(cell) ||
                               !cell.evaluated.value ||
-                              cell.evaluated.value.toString().indexOf(values[0].toString()) == -1);
+                              ((_f = cell.evaluated.value) === null || _f === void 0 ? void 0 : _f.toString().indexOf(values[0].toString())) == -1);
                       case "GreaterThan":
                           return isDefined(cell) && cell.evaluated.value > values[0];
                       case "GreaterThanOrEqual":
@@ -19423,39 +19430,34 @@
           this.computedIcons[activeSheetId] = {};
           const computedStyle = this.computedStyles[activeSheetId];
           for (let cf of this.getters.getConditionalFormats(activeSheetId)) {
-              try {
-                  switch (cf.rule.type) {
-                      case "ColorScaleRule":
-                          for (let range of cf.ranges) {
-                              this.applyColorScale(range, cf.rule);
-                          }
-                          break;
-                      case "IconSetRule":
-                          for (let range of cf.ranges) {
-                              this.applyIcon(range, cf.rule);
-                          }
-                          break;
-                      default:
-                          for (let ref of cf.ranges) {
-                              const zone = toZone(ref);
-                              for (let row = zone.top; row <= zone.bottom; row++) {
-                                  for (let col = zone.left; col <= zone.right; col++) {
-                                      const pr = this.rulePredicate[cf.rule.type];
-                                      let cell = this.getters.getCell(activeSheetId, col, row);
-                                      if (pr && pr(cell, cf.rule)) {
-                                          if (!computedStyle[col])
-                                              computedStyle[col] = [];
-                                          // we must combine all the properties of all the CF rules applied to the given cell
-                                          computedStyle[col][row] = Object.assign(((_a = computedStyle[col]) === null || _a === void 0 ? void 0 : _a[row]) || {}, cf.rule.style);
-                                      }
+              switch (cf.rule.type) {
+                  case "ColorScaleRule":
+                      for (let range of cf.ranges) {
+                          this.applyColorScale(range, cf.rule);
+                      }
+                      break;
+                  case "IconSetRule":
+                      for (let range of cf.ranges) {
+                          this.applyIcon(range, cf.rule);
+                      }
+                      break;
+                  default:
+                      for (let ref of cf.ranges) {
+                          const zone = toZone(ref);
+                          for (let row = zone.top; row <= zone.bottom; row++) {
+                              for (let col = zone.left; col <= zone.right; col++) {
+                                  const pr = this.rulePredicate[cf.rule.type];
+                                  let cell = this.getters.getCell(activeSheetId, col, row);
+                                  if (pr && pr(cell, cf.rule)) {
+                                      if (!computedStyle[col])
+                                          computedStyle[col] = [];
+                                      // we must combine all the properties of all the CF rules applied to the given cell
+                                      computedStyle[col][row] = Object.assign(((_a = computedStyle[col]) === null || _a === void 0 ? void 0 : _a[row]) || {}, cf.rule.style);
                                   }
                               }
                           }
-                          break;
-                  }
-              }
-              catch (_) {
-                  // we don't care about the errors within the evaluation of a rule
+                      }
+                      break;
               }
           }
       }
@@ -19474,7 +19476,7 @@
                   return this.getters.evaluateFormula(`=PERCENTILE(${range},${Number(threshold.value) / 100})`);
               case "formula":
                   const value = threshold.value && this.getters.evaluateFormula(threshold.value);
-                  return !(value instanceof Promise) ? value : null;
+                  return typeof value === "number" ? value : null;
               default:
                   return null;
           }
@@ -31439,9 +31441,9 @@
   Object.defineProperty(exports, '__esModule', { value: true });
 
 
-  __info__.version = '15.2.14';
-  __info__.date = '2024-01-12T13:37:05.872Z';
-  __info__.hash = '6515f7c';
+  __info__.version = '15.2.16';
+  __info__.date = '2024-01-29T13:48:09.354Z';
+  __info__.hash = '446a537';
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);