@odoo/o-spreadsheet 17.2.25 → 17.2.27

package/dist/o-spreadsheet.cjs.js

@@ -3,9 +3,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.2.25
- * @date 2024-11-08T12:13:19.966Z
- * @hash 40d42f8
+ * @version 17.2.27
+ * @date 2024-11-22T14:17:16.936Z
+ * @hash 139e5a3
  */
 
 'use strict';
@@ -2933,7 +2933,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.
@@ -19695,11 +19697,14 @@ function getDefaultChartJsRuntime(chart, labels, fontColor, { format, locale, tr
 function getChartLabelFormat(getters, range) {
     if (!range)
         return undefined;
-    return getters.getEvaluatedCell({
-        sheetId: range.sheetId,
-        col: range.zone.left,
-        row: range.zone.top,
-    }).format;
+    const { sheetId, zone: { left, top, bottom }, } = range;
+    for (let row = top; row <= bottom; row++) {
+        const format = getters.getEvaluatedCell({ sheetId, col: left, row }).format;
+        if (format) {
+            return format;
+        }
+    }
+    return undefined;
 }
 function getChartLabelValues(getters, dataSets, labelRange) {
     let labels = { values: [], formattedValues: [] };
@@ -20415,11 +20420,7 @@ function canBeDateChart(labelRange, getters) {
     if (!labelRange || !canBeLinearChart(labelRange, getters)) {
         return false;
     }
-    const labelFormat = getters.getEvaluatedCell({
-        sheetId: labelRange.sheetId,
-        col: labelRange.zone.left,
-        row: labelRange.zone.top,
-    }).format;
+    const labelFormat = getChartLabelFormat(getters, labelRange);
     return Boolean(labelFormat && timeFormatLuxonCompatible.test(labelFormat));
 }
 function canBeLinearChart(labelRange, getters) {
@@ -27218,6 +27219,7 @@ class ColorPicker extends owl.Component {
         currentColor: { type: String, optional: true },
         maxHeight: { type: Number, optional: true },
         anchorRect: Object,
+        disableNoColor: { type: Boolean, optional: true },
     };
     static defaultProps = { currentColor: "" };
     static components = { Popover };
@@ -27401,6 +27403,7 @@ class ColorPickerWidget extends owl.Component {
         disabled: { type: Boolean, optional: true },
         dropdownMaxHeight: { type: Number, optional: true },
         class: { type: String, optional: true },
+        disableNoColor: { type: Boolean, optional: true },
     };
     static components = { ColorPicker };
     colorPickerButtonRef = owl.useRef("colorPickerButton");
@@ -28959,6 +28962,9 @@ class ConditionalFormattingEditor extends owl.Component {
         }
     }
     setColorScaleColor(target, color) {
+        if (!isColorValid(color)) {
+            return;
+        }
         const point = this.state.rules.colorScale[target];
         if (point) {
             point.color = Number.parseInt(color.substr(1), 16);
@@ -47827,12 +47833,18 @@ 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);
                 }
             }
         }
+        if (currentIteration >= MAX_ITERATION) {
+            console.warn("Maximum iteration reached while evaluating cells");
+        }
     }
     computeCell(position) {
         const evaluation = this.evaluatedCells.get(position);
@@ -47864,7 +47876,6 @@ class Evaluator {
         }
         finally {
             this.cellsBeingComputed.delete(cellId);
-            this.nextPositionsToUpdate.delete(position);
         }
     }
     computeAndSave(position) {
@@ -47914,6 +47925,7 @@ class Evaluator {
             { sheetId, zone: rightPartZone },
             { sheetId, zone: leftColumnZone },
         ]);
+        invalidatedPositions.delete(arrayFormulaPosition);
         this.nextPositionsToUpdate.addMany(invalidatedPositions);
     }
     assertSheetHasEnoughSpaceToSpreadFormulaResult({ sheetId, col, row }, matrixResult) {
@@ -48958,25 +48970,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;
     }
@@ -50448,7 +50446,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];
@@ -51554,6 +51552,42 @@ class SheetUIPlugin 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,
@@ -54787,7 +54821,8 @@ const featurePluginRegistry = new Registry()
     .add("collaborative", CollaborativePlugin)
     .add("history", HistoryPlugin)
     .add("data_cleanup", DataCleanupPlugin)
-    .add("table_autofill", TableAutofillPlugin);
+    .add("table_autofill", TableAutofillPlugin)
+    .add("datavalidation_insert", DataValidationInsertionPlugin);
 // Plugins which have a state, but which should not be shared in collaborative
 const statefulUIPluginRegistry = new Registry()
     .add("selection", GridSelectionPlugin)
@@ -60415,7 +60450,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);
@@ -60969,6 +61005,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "17.2.25";
-__info__.date = "2024-11-08T12:13:19.966Z";
-__info__.hash = "40d42f8";
+__info__.version = "17.2.27";
+__info__.date = "2024-11-22T14:17:16.936Z";
+__info__.hash = "139e5a3";

package/dist/o-spreadsheet.d.ts

@@ -3451,7 +3451,7 @@ declare class Session extends EventBus<CollaborativeEvent> {
     /**
      * Notify the server that the user client left the collaborative session
      */
-    leave(data: Lazy<WorkbookData>): Promise<void>;
+    leave(data?: Lazy<WorkbookData>): Promise<void>;
     /**
      * Send a snapshot of the spreadsheet to the collaboration server
      */
@@ -3703,7 +3703,6 @@ declare class EvaluationDataValidationPlugin extends UIPlugin {
     static getters: readonly ["getDataValidationInvalidCriterionValueMessage", "getInvalidDataValidationMessage", "getValidationResultForCellValue", "isCellValidCheckbox", "isDataValidationInvalid"];
     validationResults: Record<UID, SheetValidationResult>;
     handle(cmd: CoreViewCommand): void;
-    private setContentToBooleanCells;
     isDataValidationInvalid(cellPosition: CellPosition): boolean;
     getInvalidDataValidationMessage(cellPosition: CellPosition): string | undefined;
     /**
@@ -5932,6 +5931,7 @@ interface ColorPickerProps {
     maxHeight?: Pixel;
     onColorPicked: (color: Color) => void;
     currentColor: Color;
+    disableNoColor?: boolean;
 }
 declare class ColorPicker extends Component<ColorPickerProps, SpreadsheetChildEnv> {
     static template: string;
@@ -5946,6 +5946,10 @@ declare class ColorPicker extends Component<ColorPickerProps, SpreadsheetChildEn
             optional: boolean;
         };
         anchorRect: ObjectConstructor;
+        disableNoColor: {
+            type: BooleanConstructor;
+            optional: boolean;
+        };
     };
     static defaultProps: {
         currentColor: string;
@@ -5986,6 +5990,7 @@ interface Props$I {
     disabled?: boolean;
     dropdownMaxHeight?: Pixel;
     class?: string;
+    disableNoColor?: boolean;
 }
 declare class ColorPickerWidget extends Component<Props$I, SpreadsheetChildEnv> {
     static template: string;
@@ -6014,6 +6019,10 @@ declare class ColorPickerWidget extends Component<Props$I, SpreadsheetChildEnv>
             type: StringConstructor;
             optional: boolean;
         };
+        disableNoColor: {
+            type: BooleanConstructor;
+            optional: boolean;
+        };
     };
     static components: {
         ColorPicker: typeof ColorPicker;

package/dist/o-spreadsheet.esm.js

@@ -3,9 +3,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.2.25
- * @date 2024-11-08T12:13:19.966Z
- * @hash 40d42f8
+ * @version 17.2.27
+ * @date 2024-11-22T14:17:16.936Z
+ * @hash 139e5a3
  */
 
 import { useEnv, useSubEnv, onWillUnmount, useComponent, status, Component, useRef, onMounted, useEffect, useState, onPatched, onWillPatch, onWillUpdateProps, useExternalListener, onWillStart, xml, useChildSubEnv, markRaw } from '@odoo/owl';
@@ -2931,7 +2931,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.
@@ -19693,11 +19695,14 @@ function getDefaultChartJsRuntime(chart, labels, fontColor, { format, locale, tr
 function getChartLabelFormat(getters, range) {
     if (!range)
         return undefined;
-    return getters.getEvaluatedCell({
-        sheetId: range.sheetId,
-        col: range.zone.left,
-        row: range.zone.top,
-    }).format;
+    const { sheetId, zone: { left, top, bottom }, } = range;
+    for (let row = top; row <= bottom; row++) {
+        const format = getters.getEvaluatedCell({ sheetId, col: left, row }).format;
+        if (format) {
+            return format;
+        }
+    }
+    return undefined;
 }
 function getChartLabelValues(getters, dataSets, labelRange) {
     let labels = { values: [], formattedValues: [] };
@@ -20413,11 +20418,7 @@ function canBeDateChart(labelRange, getters) {
     if (!labelRange || !canBeLinearChart(labelRange, getters)) {
         return false;
     }
-    const labelFormat = getters.getEvaluatedCell({
-        sheetId: labelRange.sheetId,
-        col: labelRange.zone.left,
-        row: labelRange.zone.top,
-    }).format;
+    const labelFormat = getChartLabelFormat(getters, labelRange);
     return Boolean(labelFormat && timeFormatLuxonCompatible.test(labelFormat));
 }
 function canBeLinearChart(labelRange, getters) {
@@ -27216,6 +27217,7 @@ class ColorPicker extends Component {
         currentColor: { type: String, optional: true },
         maxHeight: { type: Number, optional: true },
         anchorRect: Object,
+        disableNoColor: { type: Boolean, optional: true },
     };
     static defaultProps = { currentColor: "" };
     static components = { Popover };
@@ -27399,6 +27401,7 @@ class ColorPickerWidget extends Component {
         disabled: { type: Boolean, optional: true },
         dropdownMaxHeight: { type: Number, optional: true },
         class: { type: String, optional: true },
+        disableNoColor: { type: Boolean, optional: true },
     };
     static components = { ColorPicker };
     colorPickerButtonRef = useRef("colorPickerButton");
@@ -28957,6 +28960,9 @@ class ConditionalFormattingEditor extends Component {
         }
     }
     setColorScaleColor(target, color) {
+        if (!isColorValid(color)) {
+            return;
+        }
         const point = this.state.rules.colorScale[target];
         if (point) {
             point.color = Number.parseInt(color.substr(1), 16);
@@ -47825,12 +47831,18 @@ 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);
                 }
             }
         }
+        if (currentIteration >= MAX_ITERATION) {
+            console.warn("Maximum iteration reached while evaluating cells");
+        }
     }
     computeCell(position) {
         const evaluation = this.evaluatedCells.get(position);
@@ -47862,7 +47874,6 @@ class Evaluator {
         }
         finally {
             this.cellsBeingComputed.delete(cellId);
-            this.nextPositionsToUpdate.delete(position);
         }
     }
     computeAndSave(position) {
@@ -47912,6 +47923,7 @@ class Evaluator {
             { sheetId, zone: rightPartZone },
             { sheetId, zone: leftColumnZone },
         ]);
+        invalidatedPositions.delete(arrayFormulaPosition);
         this.nextPositionsToUpdate.addMany(invalidatedPositions);
     }
     assertSheetHasEnoughSpaceToSpreadFormulaResult({ sheetId, col, row }, matrixResult) {
@@ -48956,25 +48968,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;
     }
@@ -50446,7 +50444,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];
@@ -51552,6 +51550,42 @@ class SheetUIPlugin 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,
@@ -54785,7 +54819,8 @@ const featurePluginRegistry = new Registry()
     .add("collaborative", CollaborativePlugin)
     .add("history", HistoryPlugin)
     .add("data_cleanup", DataCleanupPlugin)
-    .add("table_autofill", TableAutofillPlugin);
+    .add("table_autofill", TableAutofillPlugin)
+    .add("datavalidation_insert", DataValidationInsertionPlugin);
 // Plugins which have a state, but which should not be shared in collaborative
 const statefulUIPluginRegistry = new Registry()
     .add("selection", GridSelectionPlugin)
@@ -60413,7 +60448,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);
@@ -60926,6 +60962,6 @@ const constants = {
 export { AbstractCellClipboardHandler, AbstractChart, AbstractFigureClipboardHandler, CellErrorType, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, __info__, addFunction, addRenderingLayer, astToFormula, compile, compileTokens, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, hooks, invalidateCFEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, parseTokens, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, stores, tokenColors, tokenize };
 
 
-__info__.version = "17.2.25";
-__info__.date = "2024-11-08T12:13:19.966Z";
-__info__.hash = "40d42f8";
+__info__.version = "17.2.27";
+__info__.date = "2024-11-22T14:17:16.936Z";
+__info__.hash = "139e5a3";