@odoo/o-spreadsheet 16.0.52 → 16.0.54

package/dist/o-spreadsheet.cjs.js

@@ -18638,7 +18638,10 @@ function parsePrefix(current, tokens) {
                 return { type: "FUNCALL", value: current.value, args };
             }
         case "INVALID_REFERENCE":
-            throw new InvalidReferenceError();
+            return {
+                type: "REFERENCE",
+                value: CellErrorType.InvalidReference,
+            };
         case "REFERENCE":
             if (((_b = tokens[0]) === null || _b === void 0 ? void 0 : _b.value) === ":" && ((_c = tokens[1]) === null || _c === void 0 ? void 0 : _c.type) === "REFERENCE") {
                 tokens.shift();
@@ -33954,11 +33957,7 @@ class FindAndReplacePlugin extends UIPlugin {
                 }
                 if (cell &&
                     this.currentSearchRegex &&
-                    this.currentSearchRegex.test(this.searchOptions.searchFormulas
-                        ? cell.isFormula()
-                            ? cell.content
-                            : String(cell.evaluated.value)
-                        : String(cell.evaluated.value))) {
+                    this.currentSearchRegex.test(this.searchOptions.searchFormulas ? cell.content : String(cell.evaluated.value))) {
                     const position = this.getters.getCellPosition(cell.id);
                     const match = { col: position.col, row: position.row, selected: false };
                     matches.push(match);
@@ -36596,6 +36595,7 @@ class Session extends EventBus {
             case "REMOTE_REVISION":
             case "REVISION_REDONE":
             case "REVISION_UNDONE":
+            case "SNAPSHOT_CREATED":
                 return this.processedRevisions.has(message.nextRevisionId);
             default:
                 return false;
@@ -41863,13 +41863,14 @@ function addFormula(cell) {
         .filter((tk) => tk.type === "FUNCTION")
         .every((tk) => functions[tk.value.toUpperCase()].isExported);
     const type = getCellType(cell.value);
+    const exportedValue = adaptFormulaValueToExcel(cell.value);
     if (isExported) {
         const XlsxFormula = adaptFormulaToExcel(formula);
         node = escapeXml /*xml*/ `
       <f>
         ${XlsxFormula}
       </f>
-      ${escapeXml /*xml*/ `<v>${cell.value}</v>`}
+      ${escapeXml /*xml*/ `<v>${exportedValue}</v>`}
     `;
         attrs.push(["t", type]);
         return { attrs, node };
@@ -41879,10 +41880,10 @@ function addFormula(cell) {
         // nothing* ,we don't export it.
         // * non-falsy value are relevant and so are 0 and FALSE, which only leaves
         // the empty string.
-        if (cell.value === "")
+        if (exportedValue === "")
             return undefined;
         attrs.push(["t", type]);
-        node = escapeXml /*xml*/ `<v>${cell.value}</v>`;
+        node = escapeXml /*xml*/ `<v>${exportedValue}</v>`;
         return { attrs, node };
     }
 }
@@ -41917,8 +41918,14 @@ function adaptFormulaToExcel(formulaText) {
         ast = addMissingRequiredArgs(ast);
         return ast;
     });
+    ast = convertAstNodes(ast, "REFERENCE", (ast) => {
+        return ast.value === CellErrorType.InvalidReference ? { ...ast, value: "#REF!" } : ast;
+    });
     return ast ? astToFormula(ast) : formulaText;
 }
+function adaptFormulaValueToExcel(formulaValue) {
+    return formulaValue === CellErrorType.InvalidReference ? "#REF!" : formulaValue;
+}
 /**
  * Some Excel function need required args that might not be mandatory in o-spreadsheet.
  * This adds those missing args.
@@ -43106,6 +43113,9 @@ class Model extends EventBus {
         this.selection.observe(this, {
             handleEvent: () => this.trigger("update"),
         });
+        // move in "Loading" mode where we ignore redundant calls to finalize triggered by the
+        // replay of stateUpdateMessages in the session
+        this.isLoading = true;
         // This should be done after construction of LocalHistory due to order of
         // events
         this.setupSessionEvents();
@@ -43118,6 +43128,9 @@ class Model extends EventBus {
         // mark all models as "raw", so they will not be turned into reactive objects
         // by owl, since we do not rely on reactivity
         owl.markRaw(this);
+        this.isLoading = false;
+        // ensure propre recomputation of plugin states after the message replays
+        this.finalize();
     }
     joinSession() {
         this.session.join(this.config.client);
@@ -43227,6 +43240,9 @@ class Model extends EventBus {
         return DispatchResult.Success;
     }
     finalize() {
+        if (this.isLoading) {
+            return;
+        }
         this.status = 3 /* Status.Finalizing */;
         for (const h of this.handlers) {
             h.finalize();
@@ -43443,6 +43459,6 @@ exports.setTranslationMethod = setTranslationMethod;
 exports.tokenize = tokenize;
 
 
-__info__.version = '16.0.52';
-__info__.date = '2024-09-05T09:28:56.522Z';
-__info__.hash = '2d666d7';
+__info__.version = '16.0.54';
+__info__.date = '2024-11-08T12:11:01.946Z';
+__info__.hash = 'fdfa7dc';

package/dist/o-spreadsheet.d.ts

@@ -1078,6 +1078,7 @@ declare class Model extends EventBus<any> implements CommandDispatcher {
      * can just iterate on it
      */
     private renderers;
+    private isLoading;
     /**
      * Internal status of the model. Important for command handling coordination
      */

package/dist/o-spreadsheet.esm.js

@@ -18634,7 +18634,10 @@ function parsePrefix(current, tokens) {
                 return { type: "FUNCALL", value: current.value, args };
             }
         case "INVALID_REFERENCE":
-            throw new InvalidReferenceError();
+            return {
+                type: "REFERENCE",
+                value: CellErrorType.InvalidReference,
+            };
         case "REFERENCE":
             if (((_b = tokens[0]) === null || _b === void 0 ? void 0 : _b.value) === ":" && ((_c = tokens[1]) === null || _c === void 0 ? void 0 : _c.type) === "REFERENCE") {
                 tokens.shift();
@@ -33950,11 +33953,7 @@ class FindAndReplacePlugin extends UIPlugin {
                 }
                 if (cell &&
                     this.currentSearchRegex &&
-                    this.currentSearchRegex.test(this.searchOptions.searchFormulas
-                        ? cell.isFormula()
-                            ? cell.content
-                            : String(cell.evaluated.value)
-                        : String(cell.evaluated.value))) {
+                    this.currentSearchRegex.test(this.searchOptions.searchFormulas ? cell.content : String(cell.evaluated.value))) {
                     const position = this.getters.getCellPosition(cell.id);
                     const match = { col: position.col, row: position.row, selected: false };
                     matches.push(match);
@@ -36592,6 +36591,7 @@ class Session extends EventBus {
             case "REMOTE_REVISION":
             case "REVISION_REDONE":
             case "REVISION_UNDONE":
+            case "SNAPSHOT_CREATED":
                 return this.processedRevisions.has(message.nextRevisionId);
             default:
                 return false;
@@ -41859,13 +41859,14 @@ function addFormula(cell) {
         .filter((tk) => tk.type === "FUNCTION")
         .every((tk) => functions[tk.value.toUpperCase()].isExported);
     const type = getCellType(cell.value);
+    const exportedValue = adaptFormulaValueToExcel(cell.value);
     if (isExported) {
         const XlsxFormula = adaptFormulaToExcel(formula);
         node = escapeXml /*xml*/ `
       <f>
         ${XlsxFormula}
       </f>
-      ${escapeXml /*xml*/ `<v>${cell.value}</v>`}
+      ${escapeXml /*xml*/ `<v>${exportedValue}</v>`}
     `;
         attrs.push(["t", type]);
         return { attrs, node };
@@ -41875,10 +41876,10 @@ function addFormula(cell) {
         // nothing* ,we don't export it.
         // * non-falsy value are relevant and so are 0 and FALSE, which only leaves
         // the empty string.
-        if (cell.value === "")
+        if (exportedValue === "")
             return undefined;
         attrs.push(["t", type]);
-        node = escapeXml /*xml*/ `<v>${cell.value}</v>`;
+        node = escapeXml /*xml*/ `<v>${exportedValue}</v>`;
         return { attrs, node };
     }
 }
@@ -41913,8 +41914,14 @@ function adaptFormulaToExcel(formulaText) {
         ast = addMissingRequiredArgs(ast);
         return ast;
     });
+    ast = convertAstNodes(ast, "REFERENCE", (ast) => {
+        return ast.value === CellErrorType.InvalidReference ? { ...ast, value: "#REF!" } : ast;
+    });
     return ast ? astToFormula(ast) : formulaText;
 }
+function adaptFormulaValueToExcel(formulaValue) {
+    return formulaValue === CellErrorType.InvalidReference ? "#REF!" : formulaValue;
+}
 /**
  * Some Excel function need required args that might not be mandatory in o-spreadsheet.
  * This adds those missing args.
@@ -43102,6 +43109,9 @@ class Model extends EventBus {
         this.selection.observe(this, {
             handleEvent: () => this.trigger("update"),
         });
+        // move in "Loading" mode where we ignore redundant calls to finalize triggered by the
+        // replay of stateUpdateMessages in the session
+        this.isLoading = true;
         // This should be done after construction of LocalHistory due to order of
         // events
         this.setupSessionEvents();
@@ -43114,6 +43124,9 @@ class Model extends EventBus {
         // mark all models as "raw", so they will not be turned into reactive objects
         // by owl, since we do not rely on reactivity
         markRaw(this);
+        this.isLoading = false;
+        // ensure propre recomputation of plugin states after the message replays
+        this.finalize();
     }
     joinSession() {
         this.session.join(this.config.client);
@@ -43223,6 +43236,9 @@ class Model extends EventBus {
         return DispatchResult.Success;
     }
     finalize() {
+        if (this.isLoading) {
+            return;
+        }
         this.status = 3 /* Status.Finalizing */;
         for (const h of this.handlers) {
             h.finalize();
@@ -43411,6 +43427,6 @@ function addFunction(functionName, functionDescription) {
 export { AbstractChart, CommandResult, CorePlugin, DATETIME_FORMAT, DispatchResult, EvaluationError, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, __info__, addFunction, astToFormula, cellTypes, compile, components, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, invalidateEvaluationCommands, load, parse, readonlyAllowedCommands, registries, setTranslationMethod, tokenize };
 
 
-__info__.version = '16.0.52';
-__info__.date = '2024-09-05T09:28:56.522Z';
-__info__.hash = '2d666d7';
+__info__.version = '16.0.54';
+__info__.date = '2024-11-08T12:11:01.946Z';
+__info__.hash = 'fdfa7dc';

package/dist/o-spreadsheet.iife.js

@@ -18635,7 +18635,10 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   return { type: "FUNCALL", value: current.value, args };
               }
           case "INVALID_REFERENCE":
-              throw new InvalidReferenceError();
+              return {
+                  type: "REFERENCE",
+                  value: CellErrorType.InvalidReference,
+              };
           case "REFERENCE":
               if (((_b = tokens[0]) === null || _b === void 0 ? void 0 : _b.value) === ":" && ((_c = tokens[1]) === null || _c === void 0 ? void 0 : _c.type) === "REFERENCE") {
                   tokens.shift();
@@ -33951,11 +33954,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   }
                   if (cell &&
                       this.currentSearchRegex &&
-                      this.currentSearchRegex.test(this.searchOptions.searchFormulas
-                          ? cell.isFormula()
-                              ? cell.content
-                              : String(cell.evaluated.value)
-                          : String(cell.evaluated.value))) {
+                      this.currentSearchRegex.test(this.searchOptions.searchFormulas ? cell.content : String(cell.evaluated.value))) {
                       const position = this.getters.getCellPosition(cell.id);
                       const match = { col: position.col, row: position.row, selected: false };
                       matches.push(match);
@@ -36593,6 +36592,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               case "REMOTE_REVISION":
               case "REVISION_REDONE":
               case "REVISION_UNDONE":
+              case "SNAPSHOT_CREATED":
                   return this.processedRevisions.has(message.nextRevisionId);
               default:
                   return false;
@@ -41860,13 +41860,14 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           .filter((tk) => tk.type === "FUNCTION")
           .every((tk) => functions[tk.value.toUpperCase()].isExported);
       const type = getCellType(cell.value);
+      const exportedValue = adaptFormulaValueToExcel(cell.value);
       if (isExported) {
           const XlsxFormula = adaptFormulaToExcel(formula);
           node = escapeXml /*xml*/ `
       <f>
         ${XlsxFormula}
       </f>
-      ${escapeXml /*xml*/ `<v>${cell.value}</v>`}
+      ${escapeXml /*xml*/ `<v>${exportedValue}</v>`}
     `;
           attrs.push(["t", type]);
           return { attrs, node };
@@ -41876,10 +41877,10 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           // nothing* ,we don't export it.
           // * non-falsy value are relevant and so are 0 and FALSE, which only leaves
           // the empty string.
-          if (cell.value === "")
+          if (exportedValue === "")
               return undefined;
           attrs.push(["t", type]);
-          node = escapeXml /*xml*/ `<v>${cell.value}</v>`;
+          node = escapeXml /*xml*/ `<v>${exportedValue}</v>`;
           return { attrs, node };
       }
   }
@@ -41914,8 +41915,14 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           ast = addMissingRequiredArgs(ast);
           return ast;
       });
+      ast = convertAstNodes(ast, "REFERENCE", (ast) => {
+          return ast.value === CellErrorType.InvalidReference ? { ...ast, value: "#REF!" } : ast;
+      });
       return ast ? astToFormula(ast) : formulaText;
   }
+  function adaptFormulaValueToExcel(formulaValue) {
+      return formulaValue === CellErrorType.InvalidReference ? "#REF!" : formulaValue;
+  }
   /**
    * Some Excel function need required args that might not be mandatory in o-spreadsheet.
    * This adds those missing args.
@@ -43103,6 +43110,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           this.selection.observe(this, {
               handleEvent: () => this.trigger("update"),
           });
+          // move in "Loading" mode where we ignore redundant calls to finalize triggered by the
+          // replay of stateUpdateMessages in the session
+          this.isLoading = true;
           // This should be done after construction of LocalHistory due to order of
           // events
           this.setupSessionEvents();
@@ -43115,6 +43125,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           // mark all models as "raw", so they will not be turned into reactive objects
           // by owl, since we do not rely on reactivity
           owl.markRaw(this);
+          this.isLoading = false;
+          // ensure propre recomputation of plugin states after the message replays
+          this.finalize();
       }
       joinSession() {
           this.session.join(this.config.client);
@@ -43224,6 +43237,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           return DispatchResult.Success;
       }
       finalize() {
+          if (this.isLoading) {
+              return;
+          }
           this.status = 3 /* Status.Finalizing */;
           for (const h of this.handlers) {
               h.finalize();
@@ -43442,9 +43458,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
   Object.defineProperty(exports, '__esModule', { value: true });
 
 
-  __info__.version = '16.0.52';
-  __info__.date = '2024-09-05T09:28:56.522Z';
-  __info__.hash = '2d666d7';
+  __info__.version = '16.0.54';
+  __info__.date = '2024-11-08T12:11:01.946Z';
+  __info__.hash = 'fdfa7dc';
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);