@odoo/o-spreadsheet 16.0.51 → 16.0.52

package/dist/o-spreadsheet.cjs.js

@@ -35112,6 +35112,8 @@ class GridSelectionPlugin extends UIPlugin {
                 this.selectedFigureId = null;
                 break;
         }
+    }
+    finalize() {
         /** Any change to the selection has to be  reflected in the selection processor. */
         this.selection.resetDefaultAnchor(this, deepCopy(this.gridSelection.anchor));
     }
@@ -37111,7 +37113,6 @@ class SheetViewPlugin extends UIPlugin {
     }
     handle(cmd) {
         var _a;
-        this.cleanViewports();
         switch (cmd.type) {
             case "START":
                 this.selection.observe(this, {
@@ -37156,6 +37157,9 @@ class SheetViewPlugin extends UIPlugin {
                     this.sheetsWithDirtyViewports.add(cmd.sheetId);
                 }
                 break;
+            case "DELETE_SHEET":
+                this.cleanViewports();
+                break;
             case "ACTIVATE_SHEET":
                 this.setViewports();
                 this.refreshViewport(cmd.sheetIdTo);
@@ -37472,6 +37476,7 @@ class SheetViewPlugin extends UIPlugin {
         }
     }
     resetSheetViews() {
+        this.cleanViewports();
         for (let sheetId of Object.keys(this.viewports)) {
             const position = this.getters.getSheetPosition(sheetId);
             this.resetViewports(sheetId);
@@ -41857,30 +41862,27 @@ function addFormula(cell) {
     const isExported = tokens
         .filter((tk) => tk.type === "FUNCTION")
         .every((tk) => functions[tk.value.toUpperCase()].isExported);
+    const type = getCellType(cell.value);
     if (isExported) {
-        let cycle = escapeXml ``;
         const XlsxFormula = adaptFormulaToExcel(formula);
-        // hack for cycles : if we don't set a value (be it 0 or #VALUE!), it will appear as invisible on excel,
-        // Making it very hard for the client to find where the recursion is.
-        if (cell.value === CellErrorType.CircularDependency) {
-            attrs.push(["t", "str"]);
-            cycle = escapeXml /*xml*/ `<v>${cell.value}</v>`;
-        }
-        node = escapeXml /*xml*/ `<f> ${XlsxFormula} </f> ${cycle}`;
+        node = escapeXml /*xml*/ `
+      <f>
+        ${XlsxFormula}
+      </f>
+      ${escapeXml /*xml*/ `<v>${cell.value}</v>`}
+    `;
+        attrs.push(["t", type]);
         return { attrs, node };
     }
     else {
-        // Shouldn't we always output the value then ?
-        const value = cell.value;
         // If the cell contains a non-exported formula and that is evaluates to
         // nothing* ,we don't export it.
         // * non-falsy value are relevant and so are 0 and FALSE, which only leaves
         // the empty string.
-        if (value === "")
+        if (cell.value === "")
             return undefined;
-        const type = getCellType(value);
         attrs.push(["t", type]);
-        node = escapeXml /*xml*/ `<v>${value}</v>`;
+        node = escapeXml /*xml*/ `<v>${cell.value}</v>`;
         return { attrs, node };
     }
 }
@@ -43441,6 +43443,6 @@ exports.setTranslationMethod = setTranslationMethod;
 exports.tokenize = tokenize;
 
 
-__info__.version = '16.0.51';
-__info__.date = '2024-08-19T08:08:50.605Z';
-__info__.hash = '2b60022';
+__info__.version = '16.0.52';
+__info__.date = '2024-09-05T09:28:56.522Z';
+__info__.hash = '2d666d7';

package/dist/o-spreadsheet.d.ts

@@ -4105,6 +4105,7 @@ declare class GridSelectionPlugin extends UIPlugin {
     allowDispatch(cmd: Command): CommandResult;
     private handleEvent;
     handle(cmd: Command): void;
+    finalize(): void;
     isGridSelectionActive(): boolean;
     getActiveSheet(): Readonly<Sheet>;
     getActiveSheetId(): UID;

package/dist/o-spreadsheet.esm.js

@@ -35108,6 +35108,8 @@ class GridSelectionPlugin extends UIPlugin {
                 this.selectedFigureId = null;
                 break;
         }
+    }
+    finalize() {
         /** Any change to the selection has to be  reflected in the selection processor. */
         this.selection.resetDefaultAnchor(this, deepCopy(this.gridSelection.anchor));
     }
@@ -37107,7 +37109,6 @@ class SheetViewPlugin extends UIPlugin {
     }
     handle(cmd) {
         var _a;
-        this.cleanViewports();
         switch (cmd.type) {
             case "START":
                 this.selection.observe(this, {
@@ -37152,6 +37153,9 @@ class SheetViewPlugin extends UIPlugin {
                     this.sheetsWithDirtyViewports.add(cmd.sheetId);
                 }
                 break;
+            case "DELETE_SHEET":
+                this.cleanViewports();
+                break;
             case "ACTIVATE_SHEET":
                 this.setViewports();
                 this.refreshViewport(cmd.sheetIdTo);
@@ -37468,6 +37472,7 @@ class SheetViewPlugin extends UIPlugin {
         }
     }
     resetSheetViews() {
+        this.cleanViewports();
         for (let sheetId of Object.keys(this.viewports)) {
             const position = this.getters.getSheetPosition(sheetId);
             this.resetViewports(sheetId);
@@ -41853,30 +41858,27 @@ function addFormula(cell) {
     const isExported = tokens
         .filter((tk) => tk.type === "FUNCTION")
         .every((tk) => functions[tk.value.toUpperCase()].isExported);
+    const type = getCellType(cell.value);
     if (isExported) {
-        let cycle = escapeXml ``;
         const XlsxFormula = adaptFormulaToExcel(formula);
-        // hack for cycles : if we don't set a value (be it 0 or #VALUE!), it will appear as invisible on excel,
-        // Making it very hard for the client to find where the recursion is.
-        if (cell.value === CellErrorType.CircularDependency) {
-            attrs.push(["t", "str"]);
-            cycle = escapeXml /*xml*/ `<v>${cell.value}</v>`;
-        }
-        node = escapeXml /*xml*/ `<f> ${XlsxFormula} </f> ${cycle}`;
+        node = escapeXml /*xml*/ `
+      <f>
+        ${XlsxFormula}
+      </f>
+      ${escapeXml /*xml*/ `<v>${cell.value}</v>`}
+    `;
+        attrs.push(["t", type]);
         return { attrs, node };
     }
     else {
-        // Shouldn't we always output the value then ?
-        const value = cell.value;
         // If the cell contains a non-exported formula and that is evaluates to
         // nothing* ,we don't export it.
         // * non-falsy value are relevant and so are 0 and FALSE, which only leaves
         // the empty string.
-        if (value === "")
+        if (cell.value === "")
             return undefined;
-        const type = getCellType(value);
         attrs.push(["t", type]);
-        node = escapeXml /*xml*/ `<v>${value}</v>`;
+        node = escapeXml /*xml*/ `<v>${cell.value}</v>`;
         return { attrs, node };
     }
 }
@@ -43409,6 +43411,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.51';
-__info__.date = '2024-08-19T08:08:50.605Z';
-__info__.hash = '2b60022';
+__info__.version = '16.0.52';
+__info__.date = '2024-09-05T09:28:56.522Z';
+__info__.hash = '2d666d7';

package/dist/o-spreadsheet.iife.js

@@ -35109,6 +35109,8 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   this.selectedFigureId = null;
                   break;
           }
+      }
+      finalize() {
           /** Any change to the selection has to be  reflected in the selection processor. */
           this.selection.resetDefaultAnchor(this, deepCopy(this.gridSelection.anchor));
       }
@@ -37108,7 +37110,6 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       }
       handle(cmd) {
           var _a;
-          this.cleanViewports();
           switch (cmd.type) {
               case "START":
                   this.selection.observe(this, {
@@ -37153,6 +37154,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                       this.sheetsWithDirtyViewports.add(cmd.sheetId);
                   }
                   break;
+              case "DELETE_SHEET":
+                  this.cleanViewports();
+                  break;
               case "ACTIVATE_SHEET":
                   this.setViewports();
                   this.refreshViewport(cmd.sheetIdTo);
@@ -37469,6 +37473,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           }
       }
       resetSheetViews() {
+          this.cleanViewports();
           for (let sheetId of Object.keys(this.viewports)) {
               const position = this.getters.getSheetPosition(sheetId);
               this.resetViewports(sheetId);
@@ -41854,30 +41859,27 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       const isExported = tokens
           .filter((tk) => tk.type === "FUNCTION")
           .every((tk) => functions[tk.value.toUpperCase()].isExported);
+      const type = getCellType(cell.value);
       if (isExported) {
-          let cycle = escapeXml ``;
           const XlsxFormula = adaptFormulaToExcel(formula);
-          // hack for cycles : if we don't set a value (be it 0 or #VALUE!), it will appear as invisible on excel,
-          // Making it very hard for the client to find where the recursion is.
-          if (cell.value === CellErrorType.CircularDependency) {
-              attrs.push(["t", "str"]);
-              cycle = escapeXml /*xml*/ `<v>${cell.value}</v>`;
-          }
-          node = escapeXml /*xml*/ `<f> ${XlsxFormula} </f> ${cycle}`;
+          node = escapeXml /*xml*/ `
+      <f>
+        ${XlsxFormula}
+      </f>
+      ${escapeXml /*xml*/ `<v>${cell.value}</v>`}
+    `;
+          attrs.push(["t", type]);
           return { attrs, node };
       }
       else {
-          // Shouldn't we always output the value then ?
-          const value = cell.value;
           // If the cell contains a non-exported formula and that is evaluates to
           // nothing* ,we don't export it.
           // * non-falsy value are relevant and so are 0 and FALSE, which only leaves
           // the empty string.
-          if (value === "")
+          if (cell.value === "")
               return undefined;
-          const type = getCellType(value);
           attrs.push(["t", type]);
-          node = escapeXml /*xml*/ `<v>${value}</v>`;
+          node = escapeXml /*xml*/ `<v>${cell.value}</v>`;
           return { attrs, node };
       }
   }
@@ -43440,9 +43442,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
   Object.defineProperty(exports, '__esModule', { value: true });
 
 
-  __info__.version = '16.0.51';
-  __info__.date = '2024-08-19T08:08:50.605Z';
-  __info__.hash = '2b60022';
+  __info__.version = '16.0.52';
+  __info__.date = '2024-09-05T09:28:56.522Z';
+  __info__.hash = '2d666d7';
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);