@odoo/o-spreadsheet 17.0.40 → 17.0.42

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 17.0.40
- * @date 2024-11-13T15:03:13.024Z
- * @hash a14bf60
+ * @version 17.0.42
+ * @date 2024-11-28T09:02:39.394Z
+ * @hash aa06a38
  */
 
 'use strict';
@@ -2797,7 +2797,9 @@ function linearSearch(data, target, mode, numberOfValues, getValueInData, revers
             }
         }
     }
-    return reverseSearch ? numberOfValues - closestMatchIndex - 1 : closestMatchIndex;
+    return reverseSearch && closestMatchIndex !== -1
+        ? numberOfValues - closestMatchIndex - 1
+        : closestMatchIndex;
 }
 /**
  * Normalize a value.
@@ -18732,7 +18734,7 @@ const VLOOKUP = {
             ? dichotomicSearch(range, searchKey?.value, "nextSmaller", "asc", range[0].length, getValueFromRange)
             : linearSearch(range, searchKey?.value, "strict", range[0].length, getValueFromRange);
         const value = range[_index - 1][rowIndex];
-        assertAvailable(value, searchKey);
+        assertAvailable(value, searchKey?.value);
         return value;
     },
     isExported: true,
@@ -18780,7 +18782,7 @@ const XLOOKUP = {
                 : [returnRange[index]];
         }
         const _defaultValue = defaultValue?.();
-        assertAvailable(_defaultValue, searchKey);
+        assertAvailable(_defaultValue, searchKey?.value);
         return [[_defaultValue]];
     },
     isExported: true,
@@ -41683,7 +41685,7 @@ class Evaluator {
         const arrayFormulas = this.spreadingRelations.getFormulaPositionsSpreadingOn(positionId);
         const cells = new JetSet(arrayFormulas);
         const arrayFormulaPositionId = this.getArrayFormulaSpreadingOnId(positionId);
-        if (arrayFormulaPositionId) {
+        if (arrayFormulaPositionId !== undefined) {
             // ignore the formula spreading on the position. Keep only the blocked ones
             cells.delete(arrayFormulaPositionId);
         }
@@ -41706,9 +41708,15 @@ class Evaluator {
             }
             for (let i = 0; i < positionIds.length; ++i) {
                 const cell = positionIds[i];
+                if (this.nextPositionsToUpdate.has(cell)) {
+                    continue;
+                }
                 this.setEvaluatedCell(cell, this.computeCell(cell));
             }
         }
+        if (currentIteration >= MAX_ITERATION) {
+            console.warn("Maximum iteration reached while evaluating cells");
+        }
     }
     setEvaluatedCell(positionId, evaluatedCell) {
         this.evaluatedCells.set(positionId, evaluatedCell);
@@ -41740,7 +41748,6 @@ class Evaluator {
         }
         finally {
             this.cellsBeingComputed.delete(cellId);
-            this.nextPositionsToUpdate.delete(positionId);
         }
     }
     computeAndSave(position) {
@@ -41810,6 +41817,7 @@ class Evaluator {
             { sheetId, zone: rightPartZone },
             { sheetId, zone: leftColumnZone },
         ]);
+        invalidatedPositions.delete(this.encoder.encode(arrayFormulaPosition));
         this.nextPositionsToUpdate.addMany(invalidatedPositions);
     }
     assertSheetHasEnoughSpaceToSpreadFormulaResult({ sheetId, col, row }, matrixResult) {
@@ -41876,6 +41884,7 @@ class Evaluator {
             this.evaluatedCells.delete(child);
             this.nextPositionsToUpdate.addMany(this.getCellsDependingOn([child]));
             this.nextPositionsToUpdate.addMany(this.getArrayFormulasBlockedBy(child));
+            this.spreadingRelations.removeNode(child);
         }
         this.spreadingRelations.removeNode(positionId);
     }
@@ -44503,7 +44512,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];
@@ -57158,7 +57167,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);
@@ -57661,6 +57671,6 @@ exports.setTranslationMethod = setTranslationMethod;
 exports.tokenize = tokenize;
 
 
-__info__.version = "17.0.40";
-__info__.date = "2024-11-13T15:03:13.024Z";
-__info__.hash = "a14bf60";
+__info__.version = "17.0.42";
+__info__.date = "2024-11-28T09:02:39.394Z";
+__info__.hash = "aa06a38";

package/dist/o-spreadsheet.d.ts

@@ -852,7 +852,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
      */

package/dist/o-spreadsheet.esm.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 17.0.40
- * @date 2024-11-13T15:03:13.024Z
- * @hash a14bf60
+ * @version 17.0.42
+ * @date 2024-11-28T09:02:39.394Z
+ * @hash aa06a38
  */
 
 import { Component, useRef, onMounted, useEffect, onWillPatch, useState, onWillUpdateProps, onPatched, useComponent, useExternalListener, onWillUnmount, onWillStart, xml, useChildSubEnv, useSubEnv, markRaw } from '@odoo/owl';
@@ -2795,7 +2795,9 @@ function linearSearch(data, target, mode, numberOfValues, getValueInData, revers
             }
         }
     }
-    return reverseSearch ? numberOfValues - closestMatchIndex - 1 : closestMatchIndex;
+    return reverseSearch && closestMatchIndex !== -1
+        ? numberOfValues - closestMatchIndex - 1
+        : closestMatchIndex;
 }
 /**
  * Normalize a value.
@@ -18730,7 +18732,7 @@ const VLOOKUP = {
             ? dichotomicSearch(range, searchKey?.value, "nextSmaller", "asc", range[0].length, getValueFromRange)
             : linearSearch(range, searchKey?.value, "strict", range[0].length, getValueFromRange);
         const value = range[_index - 1][rowIndex];
-        assertAvailable(value, searchKey);
+        assertAvailable(value, searchKey?.value);
         return value;
     },
     isExported: true,
@@ -18778,7 +18780,7 @@ const XLOOKUP = {
                 : [returnRange[index]];
         }
         const _defaultValue = defaultValue?.();
-        assertAvailable(_defaultValue, searchKey);
+        assertAvailable(_defaultValue, searchKey?.value);
         return [[_defaultValue]];
     },
     isExported: true,
@@ -41681,7 +41683,7 @@ class Evaluator {
         const arrayFormulas = this.spreadingRelations.getFormulaPositionsSpreadingOn(positionId);
         const cells = new JetSet(arrayFormulas);
         const arrayFormulaPositionId = this.getArrayFormulaSpreadingOnId(positionId);
-        if (arrayFormulaPositionId) {
+        if (arrayFormulaPositionId !== undefined) {
             // ignore the formula spreading on the position. Keep only the blocked ones
             cells.delete(arrayFormulaPositionId);
         }
@@ -41704,9 +41706,15 @@ class Evaluator {
             }
             for (let i = 0; i < positionIds.length; ++i) {
                 const cell = positionIds[i];
+                if (this.nextPositionsToUpdate.has(cell)) {
+                    continue;
+                }
                 this.setEvaluatedCell(cell, this.computeCell(cell));
             }
         }
+        if (currentIteration >= MAX_ITERATION) {
+            console.warn("Maximum iteration reached while evaluating cells");
+        }
     }
     setEvaluatedCell(positionId, evaluatedCell) {
         this.evaluatedCells.set(positionId, evaluatedCell);
@@ -41738,7 +41746,6 @@ class Evaluator {
         }
         finally {
             this.cellsBeingComputed.delete(cellId);
-            this.nextPositionsToUpdate.delete(positionId);
         }
     }
     computeAndSave(position) {
@@ -41808,6 +41815,7 @@ class Evaluator {
             { sheetId, zone: rightPartZone },
             { sheetId, zone: leftColumnZone },
         ]);
+        invalidatedPositions.delete(this.encoder.encode(arrayFormulaPosition));
         this.nextPositionsToUpdate.addMany(invalidatedPositions);
     }
     assertSheetHasEnoughSpaceToSpreadFormulaResult({ sheetId, col, row }, matrixResult) {
@@ -41874,6 +41882,7 @@ class Evaluator {
             this.evaluatedCells.delete(child);
             this.nextPositionsToUpdate.addMany(this.getCellsDependingOn([child]));
             this.nextPositionsToUpdate.addMany(this.getArrayFormulasBlockedBy(child));
+            this.spreadingRelations.removeNode(child);
         }
         this.spreadingRelations.removeNode(positionId);
     }
@@ -44501,7 +44510,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];
@@ -57156,7 +57165,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);
@@ -57624,6 +57634,6 @@ const constants = {
 export { AbstractChart, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, __info__, addFunction, astToFormula, compile, compileTokens, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, hooks, invalidateCFEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, parseTokens, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, tokenize };
 
 
-__info__.version = "17.0.40";
-__info__.date = "2024-11-13T15:03:13.024Z";
-__info__.hash = "a14bf60";
+__info__.version = "17.0.42";
+__info__.date = "2024-11-28T09:02:39.394Z";
+__info__.hash = "aa06a38";

package/dist/o-spreadsheet.iife.js

@@ -2,9 +2,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.0.40
- * @date 2024-11-13T15:03:13.024Z
- * @hash a14bf60
+ * @version 17.0.42
+ * @date 2024-11-28T09:02:39.394Z
+ * @hash aa06a38
  */
 
 (function (exports, owl) {
@@ -2796,7 +2796,9 @@
               }
           }
       }
-      return reverseSearch ? numberOfValues - closestMatchIndex - 1 : closestMatchIndex;
+      return reverseSearch && closestMatchIndex !== -1
+          ? numberOfValues - closestMatchIndex - 1
+          : closestMatchIndex;
   }
   /**
    * Normalize a value.
@@ -18731,7 +18733,7 @@
               ? dichotomicSearch(range, searchKey?.value, "nextSmaller", "asc", range[0].length, getValueFromRange)
               : linearSearch(range, searchKey?.value, "strict", range[0].length, getValueFromRange);
           const value = range[_index - 1][rowIndex];
-          assertAvailable(value, searchKey);
+          assertAvailable(value, searchKey?.value);
           return value;
       },
       isExported: true,
@@ -18779,7 +18781,7 @@
                   : [returnRange[index]];
           }
           const _defaultValue = defaultValue?.();
-          assertAvailable(_defaultValue, searchKey);
+          assertAvailable(_defaultValue, searchKey?.value);
           return [[_defaultValue]];
       },
       isExported: true,
@@ -41682,7 +41684,7 @@
           const arrayFormulas = this.spreadingRelations.getFormulaPositionsSpreadingOn(positionId);
           const cells = new JetSet(arrayFormulas);
           const arrayFormulaPositionId = this.getArrayFormulaSpreadingOnId(positionId);
-          if (arrayFormulaPositionId) {
+          if (arrayFormulaPositionId !== undefined) {
               // ignore the formula spreading on the position. Keep only the blocked ones
               cells.delete(arrayFormulaPositionId);
           }
@@ -41705,9 +41707,15 @@
               }
               for (let i = 0; i < positionIds.length; ++i) {
                   const cell = positionIds[i];
+                  if (this.nextPositionsToUpdate.has(cell)) {
+                      continue;
+                  }
                   this.setEvaluatedCell(cell, this.computeCell(cell));
               }
           }
+          if (currentIteration >= MAX_ITERATION) {
+              console.warn("Maximum iteration reached while evaluating cells");
+          }
       }
       setEvaluatedCell(positionId, evaluatedCell) {
           this.evaluatedCells.set(positionId, evaluatedCell);
@@ -41739,7 +41747,6 @@
           }
           finally {
               this.cellsBeingComputed.delete(cellId);
-              this.nextPositionsToUpdate.delete(positionId);
           }
       }
       computeAndSave(position) {
@@ -41809,6 +41816,7 @@
               { sheetId, zone: rightPartZone },
               { sheetId, zone: leftColumnZone },
           ]);
+          invalidatedPositions.delete(this.encoder.encode(arrayFormulaPosition));
           this.nextPositionsToUpdate.addMany(invalidatedPositions);
       }
       assertSheetHasEnoughSpaceToSpreadFormulaResult({ sheetId, col, row }, matrixResult) {
@@ -41875,6 +41883,7 @@
               this.evaluatedCells.delete(child);
               this.nextPositionsToUpdate.addMany(this.getCellsDependingOn([child]));
               this.nextPositionsToUpdate.addMany(this.getArrayFormulasBlockedBy(child));
+              this.spreadingRelations.removeNode(child);
           }
           this.spreadingRelations.removeNode(positionId);
       }
@@ -44502,7 +44511,7 @@
        * 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];
@@ -57157,7 +57166,8 @@
           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);
@@ -57660,9 +57670,9 @@
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.0.40";
-  __info__.date = "2024-11-13T15:03:13.024Z";
-  __info__.hash = "a14bf60";
+  __info__.version = "17.0.42";
+  __info__.date = "2024-11-28T09:02:39.394Z";
+  __info__.hash = "aa06a38";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);