@odoo/o-spreadsheet 16.1.9 → 16.1.10

package/dist/o-spreadsheet.cjs.js

@@ -3628,6 +3628,8 @@ exports.CommandResult = void 0;
     CommandResult[CommandResult["InvalidSelectionStep"] = 83] = "InvalidSelectionStep";
     CommandResult[CommandResult["DuplicatedChartId"] = 84] = "DuplicatedChartId";
     CommandResult[CommandResult["ChartDoesNotExist"] = 85] = "ChartDoesNotExist";
+    CommandResult[CommandResult["InvalidHeaderIndex"] = 86] = "InvalidHeaderIndex";
+    CommandResult[CommandResult["InvalidQuantity"] = 87] = "InvalidQuantity";
 })(exports.CommandResult || (exports.CommandResult = {}));
 
 var DIRECTION;
@@ -14497,7 +14499,7 @@ const POW = {
       base (number) ${_lt("The number to raise to the exponent power.")}
       exponent (number) ${_lt("The exponent to raise base to.")}
     `),
-    returns: ["BOOLEAN"],
+    returns: ["NUMBER"],
     compute: function (base, exponent) {
         return POWER.compute(base, exponent);
     },
@@ -25191,7 +25193,7 @@ function extractStyle(cell, data) {
     if (cell.style) {
         style = data.styles[cell.style];
     }
-    const format = cell.format ? data.formats[cell.format] : undefined;
+    const format = extractFormat(cell, data);
     const exportedBorder = {};
     if (cell.border) {
         const border = data.borders[cell.border];
@@ -25225,6 +25227,22 @@ function extractStyle(cell, data) {
     styles.font["italic"] = !!(style === null || style === void 0 ? void 0 : style.italic) || undefined;
     return styles;
 }
+function extractFormat(cell, data) {
+    if (cell.format) {
+        return data.formats[cell.format];
+    }
+    if (cell.isFormula) {
+        const tokens = tokenize(cell.content || "");
+        const functions = functionRegistry.content;
+        const isExported = tokens
+            .filter((tk) => tk.type === "FUNCTION")
+            .every((tk) => functions[tk.value.toUpperCase()].isExported);
+        if (!isExported) {
+            return cell.computedFormat;
+        }
+    }
+    return undefined;
+}
 function normalizeStyle(construct, styles) {
     const { id: fontId } = pushElement(styles["font"], construct.fonts);
     const { id: fillId } = pushElement(styles["fill"], construct.fills);
@@ -29975,9 +29993,16 @@ class HeaderVisibilityPlugin extends CorePlugin {
                 const elements = cmd.dimension === "COL"
                     ? this.getters.getNumberCols(cmd.sheetId)
                     : this.getters.getNumberRows(cmd.sheetId);
-                return (hiddenGroup || []).flat().concat(cmd.elements).length < elements
-                    ? 0 /* CommandResult.Success */
-                    : 66 /* CommandResult.TooManyHiddenElements */;
+                const hiddenElements = new Set((hiddenGroup || []).flat().concat(cmd.elements));
+                if (hiddenElements.size >= elements) {
+                    return 66 /* CommandResult.TooManyHiddenElements */;
+                }
+                else if (Math.min(...cmd.elements) < 0 || Math.max(...cmd.elements) > elements) {
+                    return 86 /* CommandResult.InvalidHeaderIndex */;
+                }
+                else {
+                    return 0 /* CommandResult.Success */;
+                }
             }
             case "REMOVE_COLUMNS_ROWS":
                 if (!this.getters.tryGetSheet(cmd.sheetId)) {
@@ -31141,6 +31166,28 @@ class SheetPlugin extends CorePlugin {
                 return this.orderedSheetIds.length > 1
                     ? 0 /* CommandResult.Success */
                     : 9 /* CommandResult.NotEnoughSheets */;
+            case "ADD_COLUMNS_ROWS":
+                const elements = cmd.dimension === "COL"
+                    ? this.getNumberCols(cmd.sheetId)
+                    : this.getNumberRows(cmd.sheetId);
+                if (cmd.base < 0 || cmd.base > elements) {
+                    return 86 /* CommandResult.InvalidHeaderIndex */;
+                }
+                else if (cmd.quantity <= 0) {
+                    return 87 /* CommandResult.InvalidQuantity */;
+                }
+                return 0 /* CommandResult.Success */;
+            case "REMOVE_COLUMNS_ROWS": {
+                const elements = cmd.dimension === "COL"
+                    ? this.getNumberCols(cmd.sheetId)
+                    : this.getNumberRows(cmd.sheetId);
+                if (Math.min(...cmd.elements) < 0 || Math.max(...cmd.elements) > elements) {
+                    return 86 /* CommandResult.InvalidHeaderIndex */;
+                }
+                else {
+                    return 0 /* CommandResult.Success */;
+                }
+            }
             case "FREEZE_ROWS": {
                 return this.checkValidations(cmd, this.checkRowFreezeQuantity, this.checkRowFreezeOverlapMerge);
             }
@@ -32389,8 +32436,12 @@ class EvaluationPlugin extends UIPlugin {
                 const cell = this.getters.getCell(position);
                 if (cell) {
                     const exportedCellData = sheet.cells[xc];
-                    exportedCellData.value = this.getEvaluatedCell(position).value;
+                    const evaluatedCell = this.getEvaluatedCell(position);
+                    exportedCellData.value = evaluatedCell.value;
                     exportedCellData.isFormula = cell.isFormula && !this.isBadExpression(cell.content);
+                    if (cell.format !== evaluatedCell.format) {
+                        exportedCellData.computedFormat = evaluatedCell.format;
+                    }
                 }
             }
         }
@@ -42568,12 +42619,9 @@ function addFormula(cell) {
     else {
         // Shouldn't we always output the value then ?
         const value = cell.value;
-        // what if value = 0? Is this condition correct?
-        if (value) {
-            const type = getCellType(value);
-            attrs.push(["t", type]);
-            node = escapeXml /*xml*/ `<v>${value}</v>`;
-        }
+        const type = getCellType(value);
+        attrs.push(["t", type]);
+        node = escapeXml /*xml*/ `<v>${value}</v>`;
         return { attrs, node };
     }
 }
@@ -44183,6 +44231,6 @@ exports.setTranslationMethod = setTranslationMethod;
 exports.tokenize = tokenize;
 
 
-__info__.version = '16.1.9';
-__info__.date = '2023-04-21T07:29:41.080Z';
-__info__.hash = '686bffa';
+__info__.version = '16.1.10';
+__info__.date = '2023-04-28T12:24:14.323Z';
+__info__.hash = '9632927';

package/dist/o-spreadsheet.d.ts

@@ -800,6 +800,7 @@ interface ExcelWorkbookData extends WorkbookData {
 interface ExcelCellData extends CellData {
     value: CellValue;
     isFormula: Boolean;
+    computedFormat?: Format;
 }
 interface ExcelSheetData extends Omit<SheetData, "figureTables"> {
     cells: {
@@ -2254,7 +2255,9 @@ declare const enum CommandResult {
     DuplicatedFigureId = 82,
     InvalidSelectionStep = 83,
     DuplicatedChartId = 84,
-    ChartDoesNotExist = 85
+    ChartDoesNotExist = 85,
+    InvalidHeaderIndex = 86,
+    InvalidQuantity = 87
 }
 interface CommandHandler<T> {
     allowDispatch(command: T): CommandResult | CommandResult[];
@@ -3496,7 +3499,7 @@ declare class HeaderSizePlugin extends CorePlugin<HeaderSizeState> implements He
 declare class HeaderVisibilityPlugin extends CorePlugin {
     static getters: readonly ["canRemoveHeaders", "getHiddenColsGroups", "getHiddenRowsGroups", "isRowHiddenByUser", "isColHiddenByUser"];
     private readonly hiddenHeaders;
-    allowDispatch(cmd: Command): CommandResult.Success | CommandResult.NotEnoughElements | CommandResult.InvalidSheetId | CommandResult.TooManyHiddenElements;
+    allowDispatch(cmd: Command): CommandResult.Success | CommandResult.NotEnoughElements | CommandResult.InvalidSheetId | CommandResult.TooManyHiddenElements | CommandResult.InvalidHeaderIndex;
     handle(cmd: Command): void;
     canRemoveHeaders(sheetId: UID, dimension: Dimension, elements: HeaderIndex[]): boolean;
     isRowHiddenByUser(sheetId: UID, index: HeaderIndex): boolean;

package/dist/o-spreadsheet.esm.js

@@ -3624,6 +3624,8 @@ var CommandResult;
     CommandResult[CommandResult["InvalidSelectionStep"] = 83] = "InvalidSelectionStep";
     CommandResult[CommandResult["DuplicatedChartId"] = 84] = "DuplicatedChartId";
     CommandResult[CommandResult["ChartDoesNotExist"] = 85] = "ChartDoesNotExist";
+    CommandResult[CommandResult["InvalidHeaderIndex"] = 86] = "InvalidHeaderIndex";
+    CommandResult[CommandResult["InvalidQuantity"] = 87] = "InvalidQuantity";
 })(CommandResult || (CommandResult = {}));
 
 var DIRECTION;
@@ -14493,7 +14495,7 @@ const POW = {
       base (number) ${_lt("The number to raise to the exponent power.")}
       exponent (number) ${_lt("The exponent to raise base to.")}
     `),
-    returns: ["BOOLEAN"],
+    returns: ["NUMBER"],
     compute: function (base, exponent) {
         return POWER.compute(base, exponent);
     },
@@ -25187,7 +25189,7 @@ function extractStyle(cell, data) {
     if (cell.style) {
         style = data.styles[cell.style];
     }
-    const format = cell.format ? data.formats[cell.format] : undefined;
+    const format = extractFormat(cell, data);
     const exportedBorder = {};
     if (cell.border) {
         const border = data.borders[cell.border];
@@ -25221,6 +25223,22 @@ function extractStyle(cell, data) {
     styles.font["italic"] = !!(style === null || style === void 0 ? void 0 : style.italic) || undefined;
     return styles;
 }
+function extractFormat(cell, data) {
+    if (cell.format) {
+        return data.formats[cell.format];
+    }
+    if (cell.isFormula) {
+        const tokens = tokenize(cell.content || "");
+        const functions = functionRegistry.content;
+        const isExported = tokens
+            .filter((tk) => tk.type === "FUNCTION")
+            .every((tk) => functions[tk.value.toUpperCase()].isExported);
+        if (!isExported) {
+            return cell.computedFormat;
+        }
+    }
+    return undefined;
+}
 function normalizeStyle(construct, styles) {
     const { id: fontId } = pushElement(styles["font"], construct.fonts);
     const { id: fillId } = pushElement(styles["fill"], construct.fills);
@@ -29971,9 +29989,16 @@ class HeaderVisibilityPlugin extends CorePlugin {
                 const elements = cmd.dimension === "COL"
                     ? this.getters.getNumberCols(cmd.sheetId)
                     : this.getters.getNumberRows(cmd.sheetId);
-                return (hiddenGroup || []).flat().concat(cmd.elements).length < elements
-                    ? 0 /* CommandResult.Success */
-                    : 66 /* CommandResult.TooManyHiddenElements */;
+                const hiddenElements = new Set((hiddenGroup || []).flat().concat(cmd.elements));
+                if (hiddenElements.size >= elements) {
+                    return 66 /* CommandResult.TooManyHiddenElements */;
+                }
+                else if (Math.min(...cmd.elements) < 0 || Math.max(...cmd.elements) > elements) {
+                    return 86 /* CommandResult.InvalidHeaderIndex */;
+                }
+                else {
+                    return 0 /* CommandResult.Success */;
+                }
             }
             case "REMOVE_COLUMNS_ROWS":
                 if (!this.getters.tryGetSheet(cmd.sheetId)) {
@@ -31137,6 +31162,28 @@ class SheetPlugin extends CorePlugin {
                 return this.orderedSheetIds.length > 1
                     ? 0 /* CommandResult.Success */
                     : 9 /* CommandResult.NotEnoughSheets */;
+            case "ADD_COLUMNS_ROWS":
+                const elements = cmd.dimension === "COL"
+                    ? this.getNumberCols(cmd.sheetId)
+                    : this.getNumberRows(cmd.sheetId);
+                if (cmd.base < 0 || cmd.base > elements) {
+                    return 86 /* CommandResult.InvalidHeaderIndex */;
+                }
+                else if (cmd.quantity <= 0) {
+                    return 87 /* CommandResult.InvalidQuantity */;
+                }
+                return 0 /* CommandResult.Success */;
+            case "REMOVE_COLUMNS_ROWS": {
+                const elements = cmd.dimension === "COL"
+                    ? this.getNumberCols(cmd.sheetId)
+                    : this.getNumberRows(cmd.sheetId);
+                if (Math.min(...cmd.elements) < 0 || Math.max(...cmd.elements) > elements) {
+                    return 86 /* CommandResult.InvalidHeaderIndex */;
+                }
+                else {
+                    return 0 /* CommandResult.Success */;
+                }
+            }
             case "FREEZE_ROWS": {
                 return this.checkValidations(cmd, this.checkRowFreezeQuantity, this.checkRowFreezeOverlapMerge);
             }
@@ -32385,8 +32432,12 @@ class EvaluationPlugin extends UIPlugin {
                 const cell = this.getters.getCell(position);
                 if (cell) {
                     const exportedCellData = sheet.cells[xc];
-                    exportedCellData.value = this.getEvaluatedCell(position).value;
+                    const evaluatedCell = this.getEvaluatedCell(position);
+                    exportedCellData.value = evaluatedCell.value;
                     exportedCellData.isFormula = cell.isFormula && !this.isBadExpression(cell.content);
+                    if (cell.format !== evaluatedCell.format) {
+                        exportedCellData.computedFormat = evaluatedCell.format;
+                    }
                 }
             }
         }
@@ -42564,12 +42615,9 @@ function addFormula(cell) {
     else {
         // Shouldn't we always output the value then ?
         const value = cell.value;
-        // what if value = 0? Is this condition correct?
-        if (value) {
-            const type = getCellType(value);
-            attrs.push(["t", type]);
-            node = escapeXml /*xml*/ `<v>${value}</v>`;
-        }
+        const type = getCellType(value);
+        attrs.push(["t", type]);
+        node = escapeXml /*xml*/ `<v>${value}</v>`;
         return { attrs, node };
     }
 }
@@ -44151,6 +44199,6 @@ function addFunction(functionName, functionDescription) {
 export { AbstractChart, CommandResult, CorePlugin, DATETIME_FORMAT, DispatchResult, EvaluationError, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, __info__, addFunction, astToFormula, compile, components, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, invalidateEvaluationCommands, links, load, parse, readonlyAllowedCommands, registries, setTranslationMethod, tokenize };
 
 
-__info__.version = '16.1.9';
-__info__.date = '2023-04-21T07:29:41.080Z';
-__info__.hash = '686bffa';
+__info__.version = '16.1.10';
+__info__.date = '2023-04-28T12:24:14.323Z';
+__info__.hash = '9632927';

package/dist/o-spreadsheet.iife.js

@@ -3625,6 +3625,8 @@
       CommandResult[CommandResult["InvalidSelectionStep"] = 83] = "InvalidSelectionStep";
       CommandResult[CommandResult["DuplicatedChartId"] = 84] = "DuplicatedChartId";
       CommandResult[CommandResult["ChartDoesNotExist"] = 85] = "ChartDoesNotExist";
+      CommandResult[CommandResult["InvalidHeaderIndex"] = 86] = "InvalidHeaderIndex";
+      CommandResult[CommandResult["InvalidQuantity"] = 87] = "InvalidQuantity";
   })(exports.CommandResult || (exports.CommandResult = {}));
 
   var DIRECTION;
@@ -14494,7 +14496,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       base (number) ${_lt("The number to raise to the exponent power.")}
       exponent (number) ${_lt("The exponent to raise base to.")}
     `),
-      returns: ["BOOLEAN"],
+      returns: ["NUMBER"],
       compute: function (base, exponent) {
           return POWER.compute(base, exponent);
       },
@@ -25188,7 +25190,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       if (cell.style) {
           style = data.styles[cell.style];
       }
-      const format = cell.format ? data.formats[cell.format] : undefined;
+      const format = extractFormat(cell, data);
       const exportedBorder = {};
       if (cell.border) {
           const border = data.borders[cell.border];
@@ -25222,6 +25224,22 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       styles.font["italic"] = !!(style === null || style === void 0 ? void 0 : style.italic) || undefined;
       return styles;
   }
+  function extractFormat(cell, data) {
+      if (cell.format) {
+          return data.formats[cell.format];
+      }
+      if (cell.isFormula) {
+          const tokens = tokenize(cell.content || "");
+          const functions = functionRegistry.content;
+          const isExported = tokens
+              .filter((tk) => tk.type === "FUNCTION")
+              .every((tk) => functions[tk.value.toUpperCase()].isExported);
+          if (!isExported) {
+              return cell.computedFormat;
+          }
+      }
+      return undefined;
+  }
   function normalizeStyle(construct, styles) {
       const { id: fontId } = pushElement(styles["font"], construct.fonts);
       const { id: fillId } = pushElement(styles["fill"], construct.fills);
@@ -29972,9 +29990,16 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   const elements = cmd.dimension === "COL"
                       ? this.getters.getNumberCols(cmd.sheetId)
                       : this.getters.getNumberRows(cmd.sheetId);
-                  return (hiddenGroup || []).flat().concat(cmd.elements).length < elements
-                      ? 0 /* CommandResult.Success */
-                      : 66 /* CommandResult.TooManyHiddenElements */;
+                  const hiddenElements = new Set((hiddenGroup || []).flat().concat(cmd.elements));
+                  if (hiddenElements.size >= elements) {
+                      return 66 /* CommandResult.TooManyHiddenElements */;
+                  }
+                  else if (Math.min(...cmd.elements) < 0 || Math.max(...cmd.elements) > elements) {
+                      return 86 /* CommandResult.InvalidHeaderIndex */;
+                  }
+                  else {
+                      return 0 /* CommandResult.Success */;
+                  }
               }
               case "REMOVE_COLUMNS_ROWS":
                   if (!this.getters.tryGetSheet(cmd.sheetId)) {
@@ -31138,6 +31163,28 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   return this.orderedSheetIds.length > 1
                       ? 0 /* CommandResult.Success */
                       : 9 /* CommandResult.NotEnoughSheets */;
+              case "ADD_COLUMNS_ROWS":
+                  const elements = cmd.dimension === "COL"
+                      ? this.getNumberCols(cmd.sheetId)
+                      : this.getNumberRows(cmd.sheetId);
+                  if (cmd.base < 0 || cmd.base > elements) {
+                      return 86 /* CommandResult.InvalidHeaderIndex */;
+                  }
+                  else if (cmd.quantity <= 0) {
+                      return 87 /* CommandResult.InvalidQuantity */;
+                  }
+                  return 0 /* CommandResult.Success */;
+              case "REMOVE_COLUMNS_ROWS": {
+                  const elements = cmd.dimension === "COL"
+                      ? this.getNumberCols(cmd.sheetId)
+                      : this.getNumberRows(cmd.sheetId);
+                  if (Math.min(...cmd.elements) < 0 || Math.max(...cmd.elements) > elements) {
+                      return 86 /* CommandResult.InvalidHeaderIndex */;
+                  }
+                  else {
+                      return 0 /* CommandResult.Success */;
+                  }
+              }
               case "FREEZE_ROWS": {
                   return this.checkValidations(cmd, this.checkRowFreezeQuantity, this.checkRowFreezeOverlapMerge);
               }
@@ -32386,8 +32433,12 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   const cell = this.getters.getCell(position);
                   if (cell) {
                       const exportedCellData = sheet.cells[xc];
-                      exportedCellData.value = this.getEvaluatedCell(position).value;
+                      const evaluatedCell = this.getEvaluatedCell(position);
+                      exportedCellData.value = evaluatedCell.value;
                       exportedCellData.isFormula = cell.isFormula && !this.isBadExpression(cell.content);
+                      if (cell.format !== evaluatedCell.format) {
+                          exportedCellData.computedFormat = evaluatedCell.format;
+                      }
                   }
               }
           }
@@ -42565,12 +42616,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       else {
           // Shouldn't we always output the value then ?
           const value = cell.value;
-          // what if value = 0? Is this condition correct?
-          if (value) {
-              const type = getCellType(value);
-              attrs.push(["t", type]);
-              node = escapeXml /*xml*/ `<v>${value}</v>`;
-          }
+          const type = getCellType(value);
+          attrs.push(["t", type]);
+          node = escapeXml /*xml*/ `<v>${value}</v>`;
           return { attrs, node };
       }
   }
@@ -44182,9 +44230,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
   Object.defineProperty(exports, '__esModule', { value: true });
 
 
-  __info__.version = '16.1.9';
-  __info__.date = '2023-04-21T07:29:41.080Z';
-  __info__.hash = '686bffa';
+  __info__.version = '16.1.10';
+  __info__.date = '2023-04-28T12:24:14.323Z';
+  __info__.hash = '9632927';
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);