@odoo/o-spreadsheet 17.2.26 → 17.2.28

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.26
- * @date 2024-11-13T15:04:03.710Z
- * @hash 19b7478
+ * @version 17.2.28
+ * @date 2024-11-28T09:04:19.153Z
+ * @hash fb1cbd8
  */
 
 '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.
@@ -47831,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);
@@ -47868,7 +47876,6 @@ class Evaluator {
         }
         finally {
             this.cellsBeingComputed.delete(cellId);
-            this.nextPositionsToUpdate.delete(position);
         }
     }
     computeAndSave(position) {
@@ -47918,6 +47925,7 @@ class Evaluator {
             { sheetId, zone: rightPartZone },
             { sheetId, zone: leftColumnZone },
         ]);
+        invalidatedPositions.delete(arrayFormulaPosition);
         this.nextPositionsToUpdate.addMany(invalidatedPositions);
     }
     assertSheetHasEnoughSpaceToSpreadFormulaResult({ sheetId, col, row }, matrixResult) {
@@ -47981,6 +47989,7 @@ class Evaluator {
         }
         this.nextPositionsToUpdate.addMany(this.getCellsDependingOn(invalidated));
         this.nextPositionsToUpdate.addMany(this.getArrayFormulasBlockedBy(invalidated));
+        [...invalidated].forEach((inv) => this.spreadingRelations.removeNode(inv));
         this.spreadingRelations.removeNode(position);
     }
     // ----------------------------------------------------------
@@ -48962,25 +48971,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;
     }
@@ -50452,7 +50447,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];
@@ -51558,6 +51553,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,
@@ -54791,7 +54822,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)
@@ -60419,7 +60451,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);
@@ -60973,6 +61006,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "17.2.26";
-__info__.date = "2024-11-13T15:04:03.710Z";
-__info__.hash = "19b7478";
+__info__.version = "17.2.28";
+__info__.date = "2024-11-28T09:04:19.153Z";
+__info__.hash = "fb1cbd8";

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;
     /**

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.26
- * @date 2024-11-13T15:04:03.710Z
- * @hash 19b7478
+ * @version 17.2.28
+ * @date 2024-11-28T09:04:19.153Z
+ * @hash fb1cbd8
  */
 
 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.
@@ -47829,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);
@@ -47866,7 +47874,6 @@ class Evaluator {
         }
         finally {
             this.cellsBeingComputed.delete(cellId);
-            this.nextPositionsToUpdate.delete(position);
         }
     }
     computeAndSave(position) {
@@ -47916,6 +47923,7 @@ class Evaluator {
             { sheetId, zone: rightPartZone },
             { sheetId, zone: leftColumnZone },
         ]);
+        invalidatedPositions.delete(arrayFormulaPosition);
         this.nextPositionsToUpdate.addMany(invalidatedPositions);
     }
     assertSheetHasEnoughSpaceToSpreadFormulaResult({ sheetId, col, row }, matrixResult) {
@@ -47979,6 +47987,7 @@ class Evaluator {
         }
         this.nextPositionsToUpdate.addMany(this.getCellsDependingOn(invalidated));
         this.nextPositionsToUpdate.addMany(this.getArrayFormulasBlockedBy(invalidated));
+        [...invalidated].forEach((inv) => this.spreadingRelations.removeNode(inv));
         this.spreadingRelations.removeNode(position);
     }
     // ----------------------------------------------------------
@@ -48960,25 +48969,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;
     }
@@ -50450,7 +50445,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];
@@ -51556,6 +51551,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,
@@ -54789,7 +54820,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)
@@ -60417,7 +60449,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);
@@ -60930,6 +60963,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.26";
-__info__.date = "2024-11-13T15:04:03.710Z";
-__info__.hash = "19b7478";
+__info__.version = "17.2.28";
+__info__.date = "2024-11-28T09:04:19.153Z";
+__info__.hash = "fb1cbd8";

package/dist/o-spreadsheet.iife.js

@@ -3,9 +3,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.2.26
- * @date 2024-11-13T15:04:03.710Z
- * @hash 19b7478
+ * @version 17.2.28
+ * @date 2024-11-28T09:04:19.153Z
+ * @hash fb1cbd8
  */
 
 (function (exports, owl) {
@@ -2932,7 +2932,9 @@
               return reverseSearch ? numberOfValues - i - 1 : i;
           }
       }
-      return reverseSearch ? numberOfValues - closestMatchIndex - 1 : closestMatchIndex;
+      return reverseSearch && closestMatchIndex !== -1
+          ? numberOfValues - closestMatchIndex - 1
+          : closestMatchIndex;
   }
   /**
    * Normalize a value.
@@ -47830,12 +47832,18 @@ stores.inject(MyMetaStore, storeInstance);
               }
               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);
@@ -47867,7 +47875,6 @@ stores.inject(MyMetaStore, storeInstance);
           }
           finally {
               this.cellsBeingComputed.delete(cellId);
-              this.nextPositionsToUpdate.delete(position);
           }
       }
       computeAndSave(position) {
@@ -47917,6 +47924,7 @@ stores.inject(MyMetaStore, storeInstance);
               { sheetId, zone: rightPartZone },
               { sheetId, zone: leftColumnZone },
           ]);
+          invalidatedPositions.delete(arrayFormulaPosition);
           this.nextPositionsToUpdate.addMany(invalidatedPositions);
       }
       assertSheetHasEnoughSpaceToSpreadFormulaResult({ sheetId, col, row }, matrixResult) {
@@ -47980,6 +47988,7 @@ stores.inject(MyMetaStore, storeInstance);
           }
           this.nextPositionsToUpdate.addMany(this.getCellsDependingOn(invalidated));
           this.nextPositionsToUpdate.addMany(this.getArrayFormulasBlockedBy(invalidated));
+          [...invalidated].forEach((inv) => this.spreadingRelations.removeNode(inv));
           this.spreadingRelations.removeNode(position);
       }
       // ----------------------------------------------------------
@@ -48961,25 +48970,11 @@ stores.inject(MyMetaStore, storeInstance);
           }
           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;
       }
@@ -50451,7 +50446,7 @@ stores.inject(MyMetaStore, storeInstance);
        * 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];
@@ -51557,6 +51552,42 @@ stores.inject(MyMetaStore, storeInstance);
       }
   }
 
+  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,
@@ -54790,7 +54821,8 @@ stores.inject(MyMetaStore, storeInstance);
       .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)
@@ -60418,7 +60450,8 @@ stores.inject(MyMetaStore, storeInstance);
           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);
@@ -60972,9 +61005,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.2.26";
-  __info__.date = "2024-11-13T15:04:03.710Z";
-  __info__.hash = "19b7478";
+  __info__.version = "17.2.28";
+  __info__.date = "2024-11-28T09:04:19.153Z";
+  __info__.hash = "fb1cbd8";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);