@odoo/o-spreadsheet 18.0.4 → 18.0.5

package/dist/o-spreadsheet.cjs.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.4
- * @date 2024-11-13T15:08:50.620Z
- * @hash 88e1aee
+ * @version 18.0.5
+ * @date 2024-11-22T14:19:19.721Z
+ * @hash 9eb34d9
  */
 
 'use strict';
@@ -130,7 +130,6 @@ const SELECTION_BORDER_COLOR = "#3266ca";
 const HEADER_BORDER_COLOR = "#C0C0C0";
 const CELL_BORDER_COLOR = "#E2E3E3";
 const BACKGROUND_CHART_COLOR = "#FFFFFF";
-const BORDER_CHART_COLOR = "#FFFFFF";
 const DISABLED_TEXT_COLOR = "#CACACA";
 const DEFAULT_COLOR_SCALE_MIDPOINT_COLOR = 0xb6d7a8;
 const LINK_COLOR = "#017E84";
@@ -3224,7 +3223,6 @@ var ClipboardMIMEType;
 (function (ClipboardMIMEType) {
     ClipboardMIMEType["PlainText"] = "text/plain";
     ClipboardMIMEType["Html"] = "text/html";
-    ClipboardMIMEType["OSpreadsheet"] = "web application/o-spreadsheet";
 })(ClipboardMIMEType || (ClipboardMIMEType = {}));
 
 function isSheetDependent(cmd) {
@@ -4334,7 +4332,9 @@ function linearSearch(data, target, mode, numberOfValues, getValueInData, revers
             return reverseSearch ? numberOfValues - i - 1 : i;
         }
     }
-    return reverseSearch ? numberOfValues - closestMatchIndex - 1 : closestMatchIndex;
+    return reverseSearch && closestMatchIndex !== -1
+        ? numberOfValues - closestMatchIndex - 1
+        : closestMatchIndex;
 }
 /**
  * Normalize a value.
@@ -6147,6 +6147,22 @@ function getPasteZones(target, content) {
     const width = content[0].length, height = content.length;
     return target.map((t) => splitZoneForPaste(t, width, height)).flat();
 }
+function parseOSClipboardContent(content) {
+    if (!content[ClipboardMIMEType.Html]) {
+        return {
+            text: content[ClipboardMIMEType.PlainText],
+        };
+    }
+    const htmlDocument = new DOMParser().parseFromString(content[ClipboardMIMEType.Html], "text/html");
+    const oSheetClipboardData = htmlDocument
+        .querySelector("div")
+        ?.getAttribute("data-osheet-clipboard");
+    const spreadsheetContent = oSheetClipboardData && JSON.parse(oSheetClipboardData);
+    return {
+        text: content[ClipboardMIMEType.PlainText],
+        data: spreadsheetContent,
+    };
+}
 
 class ClipboardHandler {
     getters;
@@ -6168,7 +6184,7 @@ class ClipboardHandler {
     getPasteTarget(sheetId, target, content, options) {
         return { zones: [], sheetId };
     }
-    convertOSClipboardData(data) {
+    convertTextToClipboardData(data) {
         return;
     }
 }
@@ -7998,7 +8014,7 @@ class CellClipboardHandler extends AbstractCellClipboardHandler {
             this.dispatch("CLEAR_CELL", target);
         }
     }
-    convertOSClipboardData(text) {
+    convertTextToClipboardData(text) {
         const locale = this.getters.getLocale();
         const copiedData = {
             cells: [],
@@ -8106,6 +8122,7 @@ class ChartClipboardHandler extends AbstractFigureClipboardHandler {
 
 class ConditionalFormatClipboardHandler extends AbstractCellClipboardHandler {
     uuidGenerator = new UuidGenerator();
+    queuedChanges = {};
     copy(data) {
         if (!data.zones.length) {
             return;
@@ -8127,6 +8144,7 @@ class ConditionalFormatClipboardHandler extends AbstractCellClipboardHandler {
         return { cfRules };
     }
     paste(target, clippedContent, options) {
+        this.queuedChanges = {};
         if (options.pasteOption === "asValue") {
             return;
         }
@@ -8138,6 +8156,7 @@ class ConditionalFormatClipboardHandler extends AbstractCellClipboardHandler {
         else {
             this.pasteFromCut(sheetId, zones, clippedContent);
         }
+        this.executeQueuedChanges();
     }
     pasteFromCut(sheetId, target, content) {
         const selection = target[0];
@@ -8177,34 +8196,56 @@ class ConditionalFormatClipboardHandler extends AbstractCellClipboardHandler {
      * Add or remove cells to a given conditional formatting rule.
      */
     adaptCFRules(sheetId, cf, toAdd, toRemove) {
-        const newRangesXc = this.getters.getAdaptedCfRanges(sheetId, cf, toAdd, toRemove);
-        if (!newRangesXc) {
-            return;
+        if (!this.queuedChanges[sheetId]) {
+            this.queuedChanges[sheetId] = [];
         }
-        if (newRangesXc.length === 0) {
-            this.dispatch("REMOVE_CONDITIONAL_FORMAT", { id: cf.id, sheetId });
-            return;
+        const queuedChange = this.queuedChanges[sheetId].find((queued) => queued.cf.id === cf.id);
+        if (!queuedChange) {
+            this.queuedChanges[sheetId].push({ toAdd, toRemove, cf });
+        }
+        else {
+            queuedChange.toAdd.push(...toAdd);
+            queuedChange.toRemove.push(...toRemove);
+        }
+    }
+    executeQueuedChanges() {
+        for (const sheetId in this.queuedChanges) {
+            for (const { toAdd, toRemove, cf } of this.queuedChanges[sheetId]) {
+                const newRangesXc = this.getters.getAdaptedCfRanges(sheetId, cf, toAdd, toRemove);
+                if (!newRangesXc) {
+                    continue;
+                }
+                if (newRangesXc.length === 0) {
+                    this.dispatch("REMOVE_CONDITIONAL_FORMAT", { id: cf.id, sheetId });
+                    continue;
+                }
+                this.dispatch("ADD_CONDITIONAL_FORMAT", {
+                    cf: {
+                        id: cf.id,
+                        rule: cf.rule,
+                        stopIfTrue: cf.stopIfTrue,
+                    },
+                    ranges: newRangesXc,
+                    sheetId,
+                });
+            }
         }
-        this.dispatch("ADD_CONDITIONAL_FORMAT", {
-            cf: {
-                id: cf.id,
-                rule: cf.rule,
-                stopIfTrue: cf.stopIfTrue,
-            },
-            ranges: newRangesXc,
-            sheetId,
-        });
     }
     getCFToCopyTo(targetSheetId, originCF) {
-        const cfInTarget = this.getters
+        let targetCF = this.getters
             .getConditionalFormats(targetSheetId)
             .find((cf) => cf.stopIfTrue === originCF.stopIfTrue && deepEquals(cf.rule, originCF.rule));
-        return cfInTarget ? cfInTarget : { ...originCF, id: this.uuidGenerator.uuidv4(), ranges: [] };
+        const queuedCfs = this.queuedChanges[targetSheetId];
+        if (!targetCF && queuedCfs) {
+            targetCF = queuedCfs.find((queued) => queued.cf.stopIfTrue === originCF.stopIfTrue && deepEquals(queued.cf.rule, originCF.rule))?.cf;
+        }
+        return targetCF || { ...originCF, id: this.uuidGenerator.uuidv4(), ranges: [] };
     }
 }
 
 class DataValidationClipboardHandler extends AbstractCellClipboardHandler {
     uuidGenerator = new UuidGenerator();
+    queuedChanges = {};
     copy(data) {
         const { rowsIndexes, columnsIndexes } = data;
         const sheetId = data.sheetId;
@@ -8221,6 +8262,7 @@ class DataValidationClipboardHandler extends AbstractCellClipboardHandler {
         return { dvRules };
     }
     paste(target, clippedContent, options) {
+        this.queuedChanges = {};
         if (options.pasteOption) {
             return;
         }
@@ -8232,6 +8274,7 @@ class DataValidationClipboardHandler extends AbstractCellClipboardHandler {
         else {
             this.pasteFromCut(sheetId, zones, clippedContent);
         }
+        this.executeQueuedChanges();
     }
     pasteFromCut(sheetId, target, content) {
         const selection = target[0];
@@ -8278,29 +8321,55 @@ class DataValidationClipboardHandler extends AbstractCellClipboardHandler {
         }
     }
     getDataValidationRuleToCopyTo(targetSheetId, originRule, newId = true) {
-        const ruleInTargetSheet = this.getters
+        let targetRule = this.getters
             .getDataValidationRules(targetSheetId)
             .find((rule) => deepEquals(originRule.criterion, rule.criterion) &&
             originRule.isBlocking === rule.isBlocking);
-        return ruleInTargetSheet
-            ? ruleInTargetSheet
-            : { ...originRule, id: newId ? this.uuidGenerator.uuidv4() : originRule.id, ranges: [] };
+        const queuedRules = this.queuedChanges[targetSheetId];
+        if (!targetRule && queuedRules) {
+            targetRule = queuedRules.find((queued) => deepEquals(originRule.criterion, queued.rule.criterion) &&
+                originRule.isBlocking === queued.rule.isBlocking)?.rule;
+        }
+        return (targetRule || {
+            ...originRule,
+            id: newId ? this.uuidGenerator.uuidv4() : originRule.id,
+            ranges: [],
+        });
     }
     /**
      * Add or remove XCs to a given data validation rule.
      */
     adaptDataValidationRule(sheetId, rule, toAdd, toRemove) {
-        const dvZones = rule.ranges.map((range) => range.zone);
-        const newDvZones = recomputeZones([...dvZones, ...toAdd], toRemove);
-        if (newDvZones.length === 0) {
-            this.dispatch("REMOVE_DATA_VALIDATION_RULE", { sheetId, id: rule.id });
-            return;
+        if (!this.queuedChanges[sheetId]) {
+            this.queuedChanges[sheetId] = [];
+        }
+        const queuedChange = this.queuedChanges[sheetId].find((queued) => queued.rule.id === rule.id);
+        if (!queuedChange) {
+            this.queuedChanges[sheetId].push({ toAdd, toRemove, rule });
+        }
+        else {
+            queuedChange.toAdd.push(...toAdd);
+            queuedChange.toRemove.push(...toRemove);
+        }
+    }
+    executeQueuedChanges() {
+        for (const sheetId in this.queuedChanges) {
+            for (const { toAdd, toRemove, rule: dv } of this.queuedChanges[sheetId]) {
+                // Remove the zones first in case the same position is in toAdd and toRemove
+                const dvZones = dv.ranges.map((range) => range.zone);
+                const withRemovedZones = recomputeZones(dvZones, toRemove);
+                const newDvZones = recomputeZones([...withRemovedZones, ...toAdd], []);
+                if (newDvZones.length === 0) {
+                    this.dispatch("REMOVE_DATA_VALIDATION_RULE", { sheetId, id: dv.id });
+                    continue;
+                }
+                this.dispatch("ADD_DATA_VALIDATION_RULE", {
+                    rule: dv,
+                    ranges: newDvZones.map((zone) => this.getters.getRangeDataFromZone(sheetId, zone)),
+                    sheetId,
+                });
+            }
         }
-        this.dispatch("ADD_DATA_VALIDATION_RULE", {
-            rule,
-            ranges: newDvZones.map((zone) => this.getters.getRangeDataFromZone(sheetId, zone)),
-            sheetId,
-        });
     }
 }
 
@@ -10360,6 +10429,10 @@ function makeArg(str, description) {
         description,
         type: types,
     };
+    const acceptErrors = types.includes("ANY") || types.includes("RANGE");
+    if (acceptErrors) {
+        result.acceptErrors = true;
+    }
     if (isOptional) {
         result.optional = true;
     }
@@ -12423,8 +12496,8 @@ const AVERAGEIFS = {
 const COUNT = {
     description: _t("The number of numeric values in dataset."),
     args: [
-        arg("value1 (number, range<number>)", _t("The first value or range to consider when counting.")),
-        arg("value2 (number, range<number>, repeating)", _t("Additional values or ranges to consider when counting.")),
+        arg("value1 (number, any, range<number>)", _t("The first value or range to consider when counting.")),
+        arg("value2 (number, any, range<number>, repeating)", _t("Additional values or ranges to consider when counting.")),
     ],
     compute: function (...values) {
         return countNumbers(values, this.locale);
@@ -12813,6 +12886,7 @@ const PEARSON = {
     },
     isExported: true,
 };
+// CORREL
 // In GSheet, CORREL is just an alias to PEARSON
 const CORREL = PEARSON;
 // -----------------------------------------------------------------------------
@@ -13424,7 +13498,7 @@ function getMatchingCells(database, field, criteria, locale) {
 }
 const databaseArgs = [
     arg("database (range)", _t("The array or range containing the data to consider, structured in such a way that the first row contains the labels for each column's values.")),
-    arg("field (any)", _t("Indicates which column in database contains the values to be extracted and operated on.")),
+    arg("field (number, string)", _t("Indicates which column in database contains the values to be extracted and operated on.")),
     arg("criteria (range)", _t("An array or range containing zero or more criteria to filter the database values by before operating.")),
 ];
 // -----------------------------------------------------------------------------
@@ -17783,7 +17857,7 @@ const IFS = {
     args: [
         arg("condition1 (boolean)", _t("The first condition to be evaluated. This can be a boolean, a number, an array, or a reference to any of those.")),
         arg("value1 (any)", _t("The returned value if condition1 is TRUE.")),
-        arg("condition2 (boolean, repeating)", _t("Additional conditions to be evaluated if the previous ones are FALSE.")),
+        arg("condition2 (boolean, any, repeating)", _t("Additional conditions to be evaluated if the previous ones are FALSE.")),
         arg("value2 (any, repeating)", _t("Additional values to be returned if their corresponding conditions are TRUE.")),
     ],
     compute: function (...values) {
@@ -18022,7 +18096,7 @@ const COLUMNS = {
 const HLOOKUP = {
     description: _t("Horizontal lookup"),
     args: [
-        arg("search_key (any)", _t("The value to search for. For example, 42, 'Cats', or I24.")),
+        arg("search_key (string, number, boolean)", _t("The value to search for. For example, 42, 'Cats', or I24.")),
         arg("range (range)", _t("The range to consider for the search. The first row in the range is searched for the key specified in search_key.")),
         arg("index (number)", _t("The row index of the value to be returned, where the first row in range is numbered 1.")),
         arg(`is_sorted (boolean, default=${DEFAULT_IS_SORTED})`, _t("Indicates whether the row to be searched (the first row of the specified range) is sorted, in which case the closest match for search_key will be returned.")),
@@ -18030,9 +18104,6 @@ const HLOOKUP = {
     compute: function (searchKey, range, index, isSorted = { value: DEFAULT_IS_SORTED }) {
         const _index = Math.trunc(toNumber(index?.value, this.locale));
         assert(() => 1 <= _index && _index <= range[0].length, _t("[[FUNCTION_NAME]] evaluates to an out of bounds range."));
-        if (searchKey && isEvaluationError(searchKey.value)) {
-            return searchKey;
-        }
         const getValueFromRange = (range, index) => range[index][0].value;
         const _isSorted = toBoolean(isSorted.value);
         const colIndex = _isSorted
@@ -18129,7 +18200,7 @@ const INDIRECT = {
 const LOOKUP = {
     description: _t("Look up a value."),
     args: [
-        arg("search_key (any)", _t("The value to search for. For example, 42, 'Cats', or I24.")),
+        arg("search_key (string, number, boolean)", _t("The value to search for. For example, 42, 'Cats', or I24.")),
         arg("search_array (range)", _t("One method of using this function is to provide a single sorted row or column search_array to look through for the search_key with a second argument result_range. The other way is to combine these two arguments into one search_array where the first row or column is searched and a value is returned from the last row or column in the array. If search_key is not found, a non-exact match may be returned.")),
         arg("result_range (range, optional)", _t("The range from which to return a result. The value returned corresponds to the location where search_key is found in search_range. This range must be only a single row or column and should not be used if using the search_result_array method.")),
     ],
@@ -18169,7 +18240,7 @@ const DEFAULT_SEARCH_TYPE = 1;
 const MATCH = {
     description: _t("Position of item in range that matches value."),
     args: [
-        arg("search_key (any)", _t("The value to search for. For example, 42, 'Cats', or I24.")),
+        arg("search_key (string, number, boolean)", _t("The value to search for. For example, 42, 'Cats', or I24.")),
         arg("range (any, range)", _t("The one-dimensional array to be searched.")),
         arg(`search_type (number, default=${DEFAULT_SEARCH_TYPE})`, _t("The search method. 1 (default) finds the largest value less than or equal to search_key when range is sorted in ascending order. 0 finds the exact value when range is unsorted. -1 finds the smallest value greater than or equal to search_key when range is sorted in descending order.")),
     ],
@@ -18244,7 +18315,7 @@ const ROWS = {
 const VLOOKUP = {
     description: _t("Vertical lookup."),
     args: [
-        arg("search_key (any)", _t("The value to search for. For example, 42, 'Cats', or I24.")),
+        arg("search_key (string, number, boolean)", _t("The value to search for. For example, 42, 'Cats', or I24.")),
         arg("range (any, range)", _t("The range to consider for the search. The first column in the range is searched for the key specified in search_key.")),
         arg("index (number)", _t("The column index of the value to be returned, where the first column in range is numbered 1.")),
         arg(`is_sorted (boolean, default=${DEFAULT_IS_SORTED})`, _t("Indicates whether the column to be searched (the first column of the specified range) is sorted, in which case the closest match for search_key will be returned.")),
@@ -18252,9 +18323,6 @@ const VLOOKUP = {
     compute: function (searchKey, range, index, isSorted = { value: DEFAULT_IS_SORTED }) {
         const _index = Math.trunc(toNumber(index?.value, this.locale));
         assert(() => 1 <= _index && _index <= range.length, _t("[[FUNCTION_NAME]] evaluates to an out of bounds range."));
-        if (searchKey && isEvaluationError(searchKey.value)) {
-            return searchKey;
-        }
         const getValueFromRange = (range, index) => range[0][index].value;
         const _isSorted = toBoolean(isSorted.value);
         const rowIndex = _isSorted
@@ -18280,7 +18348,7 @@ const MATCH_MODE = {
 const XLOOKUP = {
     description: _t("Search a range for a match and return the corresponding item from a second range."),
     args: [
-        arg("search_key (any)", _t("The value to search for.")),
+        arg("search_key (string,number,boolean)", _t("The value to search for.")),
         arg("lookup_range (any, range)", _t("The range to consider for the search. Should be a single column or a single row.")),
         arg("return_range (any, range)", _t("The range containing the return value. Should have the same dimensions as lookup_range.")),
         arg("if_not_found (any, optional)", _t("If a valid match is not found, return this value.")),
@@ -18305,9 +18373,6 @@ const XLOOKUP = {
         assert(() => lookupDirection === "col"
             ? returnRange[0].length === lookupRange[0].length
             : returnRange.length === lookupRange.length, _t("return_range should have the same dimensions as lookup_range."));
-        if (searchKey && isEvaluationError(searchKey.value)) {
-            return [[searchKey]];
-        }
         const getElement = lookupDirection === "col"
             ? (range, index) => range[0][index].value
             : (range, index) => range[index][0].value;
@@ -18332,13 +18397,14 @@ const XLOOKUP = {
 //--------------------------------------------------------------------------
 // Pivot functions
 //--------------------------------------------------------------------------
+// PIVOT.VALUE
 const PIVOT_VALUE = {
     description: _t("Get the value from a pivot."),
     args: [
-        arg("pivot_id (string)", _t("ID of the pivot.")),
+        arg("pivot_id (number,string)", _t("ID of the pivot.")),
         arg("measure_name (string)", _t("Name of the measure.")),
         arg("domain_field_name (string,optional,repeating)", _t("Field name.")),
-        arg("domain_value (string,optional,repeating)", _t("Value.")),
+        arg("domain_value (number,string,boolean,optional,repeating)", _t("Value.")),
     ],
     compute: function (formulaId, measureName, ...domainArgs) {
         const _pivotFormulaId = toString(formulaId);
@@ -18365,12 +18431,13 @@ const PIVOT_VALUE = {
         return pivot.getPivotCellValueAndFormat(_measure, domain);
     },
 };
+// PIVOT.HEADER
 const PIVOT_HEADER = {
     description: _t("Get the header of a pivot."),
     args: [
-        arg("pivot_id (string)", _t("ID of the pivot.")),
+        arg("pivot_id (number,string)", _t("ID of the pivot.")),
         arg("domain_field_name (string,optional,repeating)", _t("Field name.")),
-        arg("domain_value (string,optional,repeating)", _t("Value.")),
+        arg("domain_value (number,string,value,optional,repeating)", _t("Value.")),
     ],
     compute: function (pivotId, ...domainArgs) {
         const _pivotFormulaId = toString(pivotId);
@@ -18617,8 +18684,8 @@ const getNeutral = { number: 0, string: "", boolean: false };
 const EQ = {
     description: _t("Equal."),
     args: [
-        arg("value1 (any)", _t("The first value.")),
-        arg("value2 (any)", _t("The value to test against value1 for equality.")),
+        arg("value1 (string, number, boolean)", _t("The first value.")),
+        arg("value2 (string, number, boolean)", _t("The value to test against value1 for equality.")),
     ],
     compute: function (value1, value2) {
         let _value1 = isEmpty(value1) ? getNeutral[typeof value2?.value] : value1?.value;
@@ -18629,12 +18696,6 @@ const EQ = {
         if (typeof _value2 === "string") {
             _value2 = _value2.toUpperCase();
         }
-        if (isEvaluationError(_value1)) {
-            throw value1;
-        }
-        if (isEvaluationError(_value2)) {
-            throw value2;
-        }
         return _value1 === _value2;
     },
 };
@@ -18669,8 +18730,8 @@ function applyRelationalOperator(value1, value2, cb) {
 const GT = {
     description: _t("Strictly greater than."),
     args: [
-        arg("value1 (any)", _t("The value to test as being greater than value2.")),
-        arg("value2 (any)", _t("The second value.")),
+        arg("value1 (number, string, boolean)", _t("The value to test as being greater than value2.")),
+        arg("value2 (number, string, boolean)", _t("The second value.")),
     ],
     compute: function (value1, value2) {
         return applyRelationalOperator(value1, value2, (v1, v2) => {
@@ -18684,8 +18745,8 @@ const GT = {
 const GTE = {
     description: _t("Greater than or equal to."),
     args: [
-        arg("value1 (any)", _t("The value to test as being greater than or equal to value2.")),
-        arg("value2 (any)", _t("The second value.")),
+        arg("value1 (number, string, boolean)", _t("The value to test as being greater than or equal to value2.")),
+        arg("value2 (number, string, boolean)", _t("The second value.")),
     ],
     compute: function (value1, value2) {
         return applyRelationalOperator(value1, value2, (v1, v2) => {
@@ -18699,8 +18760,8 @@ const GTE = {
 const LT = {
     description: _t("Less than."),
     args: [
-        arg("value1 (any)", _t("The value to test as being less than value2.")),
-        arg("value2 (any)", _t("The second value.")),
+        arg("value1 (number, string, boolean)", _t("The value to test as being less than value2.")),
+        arg("value2 (number, string, boolean)", _t("The second value.")),
     ],
     compute: function (value1, value2) {
         return !GTE.compute.bind(this)(value1, value2);
@@ -18712,8 +18773,8 @@ const LT = {
 const LTE = {
     description: _t("Less than or equal to."),
     args: [
-        arg("value1 (any)", _t("The value to test as being less than or equal to value2.")),
-        arg("value2 (any)", _t("The second value.")),
+        arg("value1 (number, string, boolean)", _t("The value to test as being less than or equal to value2.")),
+        arg("value2 (number, string, boolean)", _t("The second value.")),
     ],
     compute: function (value1, value2) {
         return !GT.compute.bind(this)(value1, value2);
@@ -18757,8 +18818,8 @@ const MULTIPLY = {
 const NE = {
     description: _t("Not equal."),
     args: [
-        arg("value1 (any)", _t("The first value.")),
-        arg("value2 (any)", _t("The value to test against value1 for inequality.")),
+        arg("value1 (string, number, boolean)", _t("The first value.")),
+        arg("value2 (string, number, boolean)", _t("The value to test against value1 for inequality.")),
     ],
     compute: function (value1, value2) {
         return !EQ.compute.bind(this)(value1, value2);
@@ -19672,6 +19733,16 @@ function createComputeFunction(descr, functionName) {
         });
     }
     function errorHandlingCompute(...args) {
+        for (let i = 0; i < args.length; i++) {
+            const arg = args[i];
+            const argDefinition = descr.args[descr.getArgToFocus(i + 1) - 1];
+            // Early exit if the argument is an error and the function does not accept errors
+            // We only check scalar arguments, not matrix arguments for performance reasons.
+            // Casting helpers are responsible for handling errors in matrix arguments.
+            if (!argDefinition.acceptErrors && !isMatrix(arg) && isEvaluationError(arg?.value)) {
+                return arg;
+            }
+        }
         try {
             return computeFunctionToObject.apply(this, args);
         }
@@ -20611,10 +20682,10 @@ css /* scss */ `
     }
 
     .fa-stack {
-      // reset stack size which is doubled by default
-      width: 1em;
-      height: 1em;
-      line-height: 1em;
+      /* reset stack size which is doubled by default */
+      width: ${CLOSE_ICON_RADIUS * 2}px;
+      height: ${CLOSE_ICON_RADIUS * 2}px;
+      line-height: ${CLOSE_ICON_RADIUS * 2}px;
     }
 
     .force-open-assistant {
@@ -27814,17 +27885,15 @@ function interactivePasteFromOS(env, target, clipboardContent, pasteOption) {
         });
     }
     catch (error) {
-        const parsedSpreadsheetContent = clipboardContent[ClipboardMIMEType.OSpreadsheet]
-            ? JSON.parse(clipboardContent[ClipboardMIMEType.OSpreadsheet])
-            : {};
-        if (parsedSpreadsheetContent.version && parsedSpreadsheetContent.version !== CURRENT_VERSION) {
+        const parsedSpreadsheetContent = clipboardContent.data;
+        if (parsedSpreadsheetContent?.version !== CURRENT_VERSION) {
             env.raiseError(_t("An unexpected error occurred while pasting content.\
           This is probably due to a spreadsheet version mismatch."));
         }
         result = env.model.dispatch("PASTE_FROM_OS_CLIPBOARD", {
             target,
             clipboardContent: {
-                [ClipboardMIMEType.PlainText]: clipboardContent[ClipboardMIMEType.PlainText],
+                text: clipboardContent.text,
             },
             pasteOption,
         });
@@ -28552,8 +28621,8 @@ function createBarChartRuntime(chart, getters) {
         const dataset = {
             label,
             data,
-            borderColor: BORDER_CHART_COLOR,
-            borderWidth: 1,
+            borderColor: definition.background || BACKGROUND_CHART_COLOR,
+            borderWidth: definition.stacked ? 1 : 0,
             backgroundColor: color,
         };
         config.data.datasets.push(dataset);
@@ -32652,13 +32721,11 @@ async function paste$1(env, pasteOption) {
     const osClipboard = await env.clipboard.read();
     switch (osClipboard.status) {
         case "ok":
-            const htmlDocument = new DOMParser().parseFromString(osClipboard.content[ClipboardMIMEType.Html] ?? "<div></div>", "text/html");
-            const osClipboardSpreadsheetContent = osClipboard.content[ClipboardMIMEType.OSpreadsheet] || "{}";
-            const clipboardId = JSON.parse(osClipboardSpreadsheetContent).clipboardId ??
-                htmlDocument.querySelector("div")?.getAttribute("data-clipboard-id");
+            const clipboardContent = parseOSClipboardContent(osClipboard.content);
+            const clipboardId = clipboardContent.data?.clipboardId;
             const target = env.model.getters.getSelectedZones();
             if (env.model.getters.getClipboardId() !== clipboardId) {
-                interactivePasteFromOS(env, target, osClipboard.content, pasteOption);
+                interactivePasteFromOS(env, target, clipboardContent, pasteOption);
             }
             else {
                 interactivePaste(env, target, pasteOption);
@@ -49794,25 +49861,20 @@ class Grid extends owl.Component {
         if (!clipboardData) {
             return;
         }
-        const clipboardDataTextContent = clipboardData?.getData(ClipboardMIMEType.PlainText);
-        const clipboardDataHtmlContent = clipboardData?.getData(ClipboardMIMEType.Html);
-        const htmlDocument = new DOMParser().parseFromString(clipboardDataHtmlContent ?? "<div></div>", "text/html");
-        const osClipboardSpreadsheetContent = clipboardData.getData(ClipboardMIMEType.OSpreadsheet) || "{}";
+        const osClipboard = {
+            content: {
+                [ClipboardMIMEType.PlainText]: clipboardData?.getData(ClipboardMIMEType.PlainText),
+                [ClipboardMIMEType.Html]: clipboardData?.getData(ClipboardMIMEType.Html),
+            },
+        };
         const target = this.env.model.getters.getSelectedZones();
         const isCutOperation = this.env.model.getters.isCutOperation();
-        const clipboardId = JSON.parse(osClipboardSpreadsheetContent).clipboardId ??
-            htmlDocument.querySelector("div")?.getAttribute("data-clipboard-id");
+        const clipboardContent = parseOSClipboardContent(osClipboard.content);
+        const clipboardId = clipboardContent.data?.clipboardId;
         if (this.env.model.getters.getClipboardId() === clipboardId) {
             interactivePaste(this.env, target);
         }
         else {
-            const clipboardContent = {
-                [ClipboardMIMEType.PlainText]: clipboardDataTextContent,
-                [ClipboardMIMEType.Html]: clipboardDataHtmlContent,
-            };
-            if (osClipboardSpreadsheetContent !== "{}") {
-                clipboardContent[ClipboardMIMEType.OSpreadsheet] = osClipboardSpreadsheetContent;
-            }
             interactivePasteFromOS(this.env, target, clipboardContent);
         }
         if (isCutOperation) {
@@ -51695,8 +51757,9 @@ class ConditionalFormatPlugin extends CorePlugin {
         if (replaceIndex > -1) {
             currentRanges = rules[replaceIndex].ranges.map(toUnboundedZone);
         }
-        currentRanges = currentRanges.concat(toAdd);
-        return recomputeZones(currentRanges, toRemove).map((zone) => this.getters.getRangeDataFromZone(sheetId, zone));
+        // Remove the zones first in case the same position is in toAdd and toRemove
+        const withRemovedZones = recomputeZones(currentRanges, toRemove);
+        return recomputeZones([...toAdd, ...withRemovedZones], []).map((zone) => this.getters.getRangeDataFromZone(sheetId, zone));
     }
     // ---------------------------------------------------------------------------
     // Private
@@ -57290,6 +57353,9 @@ class Evaluator {
             }
             for (let i = 0; i < positions.length; ++i) {
                 const position = positions[i];
+                if (this.nextPositionsToUpdate.has(position)) {
+                    continue;
+                }
                 const evaluatedCell = this.computeCell(position);
                 if (evaluatedCell !== EMPTY_CELL) {
                     this.evaluatedCells.set(position, evaluatedCell);
@@ -57297,6 +57363,9 @@ class Evaluator {
             }
             onIterationEndEvaluationRegistry.getAll().forEach((callback) => callback(this.getters));
         }
+        if (currentIteration >= MAX_ITERATION) {
+            console.warn("Maximum iteration reached while evaluating cells");
+        }
     }
     computeCell(position) {
         const evaluation = this.evaluatedCells.get(position);
@@ -57331,7 +57400,6 @@ class Evaluator {
         }
         finally {
             this.cellsBeingComputed.delete(cellId);
-            this.nextPositionsToUpdate.delete(position);
         }
     }
     computeAndSave(position) {
@@ -57367,6 +57435,7 @@ class Evaluator {
     invalidatePositionsDependingOnSpread(sheetId, resultZone) {
         // the result matrix is split in 2 zones to exclude the array formula position
         const invalidatedPositions = this.formulaDependencies().getCellsDependingOn(excludeTopLeft(resultZone).map((zone) => ({ sheetId, zone })));
+        invalidatedPositions.delete({ sheetId, col: resultZone.left, row: resultZone.top });
         this.nextPositionsToUpdate.addMany(invalidatedPositions);
     }
     assertSheetHasEnoughSpaceToSpreadFormulaResult({ sheetId, col, row }, matrixResult) {
@@ -58489,25 +58558,11 @@ class EvaluationDataValidationPlugin extends UIPlugin {
         }
         switch (cmd.type) {
             case "ADD_DATA_VALIDATION_RULE":
-                const ranges = cmd.ranges.map((range) => this.getters.getRangeFromRangeData(range));
-                if (cmd.rule.criterion.type === "isBoolean") {
-                    this.setContentToBooleanCells({ ...cmd.rule, ranges });
-                }
-                delete this.validationResults[cmd.sheetId];
-                break;
             case "REMOVE_DATA_VALIDATION_RULE":
                 delete this.validationResults[cmd.sheetId];
                 break;
         }
     }
-    setContentToBooleanCells(rule) {
-        for (const position of getCellPositionsInRanges(rule.ranges)) {
-            const evaluatedCell = this.getters.getEvaluatedCell(position);
-            if (evaluatedCell.type !== CellValueType.boolean) {
-                this.dispatch("UPDATE_CELL", { ...position, content: "FALSE" });
-            }
-        }
-    }
     isDataValidationInvalid(cellPosition) {
         return !this.getValidationResultForCell(cellPosition).isValid;
     }
@@ -60987,7 +61042,7 @@ class Session extends EventBus {
      * Notify the server that the user client left the collaborative session
      */
     async leave(data) {
-        if (Object.keys(this.clients).length === 1 && this.processedRevisions.size) {
+        if (data && Object.keys(this.clients).length === 1 && this.processedRevisions.size) {
             await this.snapshot(data());
         }
         delete this.clients[this.clientId];
@@ -62542,6 +62597,42 @@ class CellComputedStylePlugin extends UIPlugin {
     }
 }
 
+class DataValidationInsertionPlugin extends UIPlugin {
+    handle(cmd) {
+        switch (cmd.type) {
+            case "ADD_DATA_VALIDATION_RULE":
+                if (cmd.rule.criterion.type === "isBoolean") {
+                    const ranges = cmd.ranges.map((range) => this.getters.getRangeFromRangeData(range));
+                    for (const position of getCellPositionsInRanges(ranges)) {
+                        const cell = this.getters.getCell(position);
+                        const evaluatedCell = this.getters.getEvaluatedCell(position);
+                        if (!cell?.content) {
+                            this.dispatch("UPDATE_CELL", { ...position, content: "FALSE" });
+                            // In this case, a cell has been updated in the core plugin but
+                            // not yet evaluated. This can occur after a paste operation.
+                        }
+                        else if (cell?.content && evaluatedCell.type === CellValueType.empty) {
+                            let value;
+                            if (cell.content.startsWith("=")) {
+                                const result = this.getters.evaluateFormula(position.sheetId, cell.content);
+                                value = (isMatrix(result) ? result[0][0] : result)?.toString();
+                            }
+                            else {
+                                value = cell.content;
+                            }
+                            if (!value || !isBoolean(value)) {
+                                this.dispatch("UPDATE_CELL", { ...position, content: "FALSE" });
+                            }
+                        }
+                        else if (evaluatedCell.type !== CellValueType.boolean) {
+                            this.dispatch("UPDATE_CELL", { ...position, content: "FALSE" });
+                        }
+                    }
+                }
+        }
+    }
+}
+
 const genericRepeatsTransforms = [
     repeatSheetDependantCommand,
     repeatTargetDependantCommand,
@@ -63195,7 +63286,7 @@ class ClipboardPlugin extends UIPlugin {
                 const zones = this.getters.getSelectedZones();
                 return this.isCutAllowedOn(zones);
             case "PASTE_FROM_OS_CLIPBOARD": {
-                const copiedData = this.convertOSClipboardData(cmd.clipboardContent[ClipboardMIMEType.PlainText] ?? "");
+                const copiedData = this.convertTextToClipboardData(cmd.clipboardContent.text ?? "");
                 const pasteOption = cmd.pasteOption;
                 return this.isPasteAllowed(cmd.target, copiedData, { pasteOption, isCutOperation: false });
             }
@@ -63248,12 +63339,9 @@ class ClipboardPlugin extends UIPlugin {
                 break;
             case "PASTE_FROM_OS_CLIPBOARD": {
                 this._isCutOperation = false;
-                if (cmd.clipboardContent[ClipboardMIMEType.OSpreadsheet]) {
-                    this.copiedData = JSON.parse(cmd.clipboardContent[ClipboardMIMEType.OSpreadsheet]);
-                }
-                else {
-                    this.copiedData = this.convertOSClipboardData(cmd.clipboardContent[ClipboardMIMEType.PlainText] ?? "");
-                }
+                this.copiedData =
+                    cmd.clipboardContent.data ||
+                        this.convertTextToClipboardData(cmd.clipboardContent.text ?? "");
                 const pasteOption = cmd.pasteOption;
                 this.paste(cmd.target, this.copiedData, {
                     pasteOption,
@@ -63384,11 +63472,11 @@ class ClipboardPlugin extends UIPlugin {
                 }
         }
     }
-    convertOSClipboardData(clipboardData) {
+    convertTextToClipboardData(clipboardData) {
         const handlers = this.selectClipboardHandlers({ figureId: true }).concat(this.selectClipboardHandlers({}));
         let copiedData = {};
         for (const { handlerName, handler } of handlers) {
-            const data = handler.convertOSClipboardData(clipboardData);
+            const data = handler.convertTextToClipboardData(clipboardData);
             copiedData[handlerName] = data;
             const minimalKeys = ["sheetId", "cells", "zones", "figureId"];
             for (const key of minimalKeys) {
@@ -63557,21 +63645,20 @@ class ClipboardPlugin extends UIPlugin {
         return {
             [ClipboardMIMEType.PlainText]: this.getPlainTextContent(),
             [ClipboardMIMEType.Html]: this.getHTMLContent(),
-            [ClipboardMIMEType.OSpreadsheet]: this.getSerializedGridData(),
         };
     }
-    getSerializedGridData() {
+    getSheetData() {
         const data = {
             version: CURRENT_VERSION,
             clipboardId: this.clipboardId,
         };
         if (this.copiedData && "figureId" in this.copiedData) {
-            return JSON.stringify(data);
+            return data;
         }
-        return JSON.stringify({
+        return {
             ...data,
             ...this.copiedData,
-        });
+        };
     }
     getPlainTextContent() {
         if (!this.copiedData?.cells) {
@@ -63588,31 +63675,36 @@ class ClipboardPlugin extends UIPlugin {
             .join("\n") || "\t");
     }
     getHTMLContent() {
-        if (!this.copiedData?.cells) {
-            return `<div data-clipboard-id="${this.clipboardId}">\t</div>`;
+        let innerHTML = "";
+        const cells = this.copiedData?.cells;
+        if (!cells) {
+            innerHTML = "\t";
         }
-        const cells = this.copiedData.cells;
-        if (cells.length === 1 && cells[0].length === 1) {
-            return `<div data-clipboard-id="${this.clipboardId}">${this.getters.getCellText(cells[0][0].position)}</div>`;
+        else if (cells.length === 1 && cells[0].length === 1) {
+            innerHTML = `${this.getters.getCellText(cells[0][0].position)}`;
         }
-        if (!cells[0][0]) {
+        else if (!cells[0][0]) {
             return "";
         }
-        let htmlTable = `<div data-clipboard-id="${this.clipboardId}"><table border="1" style="border-collapse:collapse">`;
-        for (const row of cells) {
-            htmlTable += "<tr>";
-            for (const cell of row) {
-                if (!cell) {
-                    continue;
+        else {
+            let htmlTable = `<table border="1" style="border-collapse:collapse">`;
+            for (const row of cells) {
+                htmlTable += "<tr>";
+                for (const cell of row) {
+                    if (!cell) {
+                        continue;
+                    }
+                    const cssStyle = cssPropertiesToCss(cellStyleToCss(this.getters.getCellComputedStyle(cell.position)));
+                    const cellText = this.getters.getCellText(cell.position);
+                    htmlTable += `<td style="${cssStyle}">` + xmlEscape(cellText) + "</td>";
                 }
-                const cssStyle = cssPropertiesToCss(cellStyleToCss(this.getters.getCellComputedStyle(cell.position)));
-                const cellText = this.getters.getCellText(cell.position);
-                htmlTable += `<td style="${cssStyle}">` + xmlEscape(cellText) + "</td>";
+                htmlTable += "</tr>";
             }
-            htmlTable += "</tr>";
+            htmlTable += "</table>";
+            innerHTML = htmlTable;
         }
-        htmlTable += "</table></div>";
-        return htmlTable;
+        const serializedData = JSON.stringify(this.getSheetData());
+        return `<div data-osheet-clipboard='${xmlEscape(serializedData)}'>${innerHTML}</div>`;
     }
     isCutOperation() {
         return this._isCutOperation ?? false;
@@ -65599,7 +65691,8 @@ const featurePluginRegistry = new Registry()
     .add("history", HistoryPlugin)
     .add("data_cleanup", DataCleanupPlugin)
     .add("table_autofill", TableAutofillPlugin)
-    .add("table_ui_resize", TableResizeUI);
+    .add("table_ui_resize", TableResizeUI)
+    .add("datavalidation_insert", DataValidationInsertionPlugin);
 // Plugins which have a state, but which should not be shared in collaborative
 const statefulUIPluginRegistry = new Registry()
     .add("selection", GridSelectionPlugin)
@@ -67859,10 +67952,6 @@ class WebClipboardWrapper {
             [ClipboardMIMEType.PlainText]: this.getBlob(content, ClipboardMIMEType.PlainText),
             [ClipboardMIMEType.Html]: this.getBlob(content, ClipboardMIMEType.Html),
         };
-        const spreadsheetData = content[ClipboardMIMEType.OSpreadsheet];
-        if (spreadsheetData) {
-            clipboardItemData[ClipboardMIMEType.OSpreadsheet] = this.getBlob(content, ClipboardMIMEType.OSpreadsheet);
-        }
         return [new ClipboardItem(clipboardItemData)];
     }
     getBlob(clipboardContent, type) {
@@ -71866,7 +71955,8 @@ class Model extends EventBus {
         this.session.join(this.config.client);
     }
     async leaveSession() {
-        await this.session.leave(lazy(() => this.exportData()));
+        const snapshot = this.getters.isReadonly() ? undefined : lazy(() => this.exportData());
+        await this.session.leave(snapshot);
     }
     setupUiPlugin(Plugin) {
         const plugin = new Plugin(this.uiPluginConfig);
@@ -72480,6 +72570,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "18.0.4";
-__info__.date = "2024-11-13T15:08:50.620Z";
-__info__.hash = "88e1aee";
+__info__.version = "18.0.5";
+__info__.date = "2024-11-22T14:19:19.721Z";
+__info__.hash = "9eb34d9";