@odoo/o-spreadsheet 17.1.5 → 17.1.7

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.1.5
- * @date 2024-02-16T14:23:54.651Z
- * @hash 29d8ad6
+ * @version 17.1.7
+ * @date 2024-03-12T07:56:10.783Z
+ * @hash 3a9f334
  */
 
 (function (exports, owl) {
@@ -6099,8 +6099,10 @@
           if (x === cell) {
               found = true;
           }
-          const cellValue = evaluateLiteral(x?.content, { locale: DEFAULT_LOCALE });
-          if (filter(cellValue)) {
+          const cellValue = x?.isFormula
+              ? undefined
+              : evaluateLiteral(x?.content, { locale: DEFAULT_LOCALE });
+          if (cellValue && filter(cellValue)) {
               group.push(cellValue);
           }
           else {
@@ -6324,11 +6326,14 @@
    * Transform CSS properties into a CSS string.
    */
   function cssPropertiesToCss(attributes) {
-      const str = Object.entries(attributes)
-          .filter(([attName, attValue]) => attValue !== undefined)
-          .map(([attName, attValue]) => `${attName}:${attValue};`)
-          .join(" ");
-      return str;
+      let styleStr = "";
+      for (const attName in attributes) {
+          if (!attributes[attName]) {
+              continue;
+          }
+          styleStr += `${attName}:${attributes[attName]}; `;
+      }
+      return styleStr;
   }
   function getElementMargins(el) {
       const style = window.getComputedStyle(el);
@@ -13052,6 +13057,18 @@
       },
       isExported: true,
   };
+  // -----------------------------------------------------------------------------
+  // INT
+  // -----------------------------------------------------------------------------
+  const INT = {
+      description: _t("Rounds a number down to the nearest integer that is less than or equal to it."),
+      args: [arg("value (number)", _t("The number to round down to the nearest integer."))],
+      returns: ["NUMBER"],
+      compute: function (value) {
+          return Math.floor(toNumber(value, this.locale));
+      },
+      isExported: true,
+  };
 
   var math = /*#__PURE__*/Object.freeze({
     __proto__: null,
@@ -13085,6 +13102,7 @@
     FLOOR: FLOOR,
     FLOOR_MATH: FLOOR_MATH,
     FLOOR_PRECISE: FLOOR_PRECISE,
+    INT: INT,
     ISEVEN: ISEVEN,
     ISODD: ISODD,
     ISO_CEILING: ISO_CEILING,
@@ -18611,10 +18629,11 @@
   const DEFAULT_MATCH_MODE = 0;
   const DEFAULT_SEARCH_MODE = 1;
   const DEFAULT_ABSOLUTE_RELATIVE_MODE = 1;
-  function assertAvailable(variable, searchKey) {
-      if (variable === undefined) {
-          throw new NotAvailableError(_t("Did not find value '%s' in [[FUNCTION_NAME]] evaluation.", toString(searchKey)));
-      }
+  function valueNotAvailable(searchKey) {
+      return {
+          value: CellErrorType.NotAvailable,
+          message: _t("Did not find value '%s' in [[FUNCTION_NAME]] evaluation.", toString(searchKey)),
+      };
   }
   // -----------------------------------------------------------------------------
   // ADDRESS
@@ -18711,7 +18730,9 @@
               ? dichotomicSearch(range, searchKey, "nextSmaller", "asc", range.length, getValueFromRange)
               : linearSearch(range, searchKey, "strict", range.length, getValueFromRange);
           const col = range[colIndex];
-          assertAvailable(col, searchKey?.value);
+          if (col === undefined) {
+              return valueNotAvailable(searchKey);
+          }
           return col[_index - 1];
       },
       isExported: true,
@@ -18768,11 +18789,11 @@
               : (range, index) => range[index][0].value;
           const rangeLength = verticalSearch ? nbRow : nbCol;
           const index = dichotomicSearch(searchArray, searchKey, "nextSmaller", "asc", rangeLength, getElement);
-          if (index === -1)
-              assertAvailable(undefined, searchKey?.value);
-          verticalSearch
-              ? assertAvailable(searchArray[0][index], searchKey?.value)
-              : assertAvailable(searchArray[index][nbRow - 1], searchKey?.value);
+          if (index === -1 ||
+              (verticalSearch && searchArray[0][index] === undefined) ||
+              (!verticalSearch && searchArray[index][nbRow - 1] === undefined)) {
+              return valueNotAvailable(searchKey);
+          }
           if (resultRange === undefined) {
               return verticalSearch ? searchArray[nbCol - 1][index] : searchArray[index][nbRow - 1];
           }
@@ -18822,7 +18843,10 @@
                   index = dichotomicSearch(range, searchKey, "nextGreater", "desc", rangeLen, getElement);
                   break;
           }
-          assertAvailable(nbCol === 1 ? range[0][index] : range[index], searchKey);
+          if ((nbCol === 1 && range[0][index] === undefined) ||
+              (nbCol !== 1 && range[index] === undefined)) {
+              return valueNotAvailable(searchKey);
+          }
           return index + 1;
       },
       isExported: true,
@@ -18881,7 +18905,9 @@
               ? dichotomicSearch(range, searchKey, "nextSmaller", "asc", range[0].length, getValueFromRange)
               : linearSearch(range, searchKey, "strict", range[0].length, getValueFromRange);
           const value = range[_index - 1][rowIndex];
-          assertAvailable(value, searchKey);
+          if (value === undefined) {
+              return valueNotAvailable(searchKey);
+          }
           return value;
       },
       isExported: true,
@@ -18931,7 +18957,9 @@
                   ? returnRange.map((col) => [col[index]])
                   : [returnRange[index]];
           }
-          assertAvailable(defaultValue, searchKey);
+          if (defaultValue === undefined) {
+              return valueNotAvailable(searchKey);
+          }
           return [[defaultValue]];
       },
       isExported: true,
@@ -19703,8 +19731,7 @@
   function addErrorHandling(compute, functionName) {
       return function (...args) {
           try {
-              const computeFormula = compute.bind(this);
-              return computeFormula(...args);
+              return compute.apply(this, args);
           }
           catch (e) {
               return handleError(e, functionName);
@@ -19738,8 +19765,7 @@
   }
   function addResultHandling(compute) {
       return function (...args) {
-          const computeResult = compute.bind(this);
-          const result = computeResult(...args);
+          const result = compute.apply(this, args);
           if (!isMatrix(result)) {
               if (typeof result === "object" && result !== null && "value" in result) {
                   return result;
@@ -21706,7 +21732,7 @@
               ? existingSelectionRanges
               : this.props.ranges().map((xc, id) => ({
                   xc,
-                  id,
+                  id: id + 1,
                   isFocused: false,
               }));
           return ranges.map((range) => ({
@@ -27746,7 +27772,11 @@
           }
       }
       updateCellReferenceVisibility() {
-          if (this.isCellReferenceVisible || this.env.model.getters.getEditionMode() === "inactive") {
+          if (this.env.model.getters.getEditionMode() === "inactive") {
+              this.isCellReferenceVisible = false;
+              return;
+          }
+          if (this.isCellReferenceVisible) {
               return;
           }
           const sheetId = this.env.model.getters.getActiveSheetId();
@@ -27775,19 +27805,21 @@
           slots: Object,
       };
       get iconStyle() {
-          const x = this.getIconHorizontalPosition();
-          const y = this.getIconVerticalPosition();
+          const cellPosition = this.props.cellPosition;
+          const merge = this.env.model.getters.getMerge(cellPosition);
+          const zone = merge || positionToZone(cellPosition);
+          const rect = this.env.model.getters.getVisibleRectWithoutHeaders(zone);
+          const x = this.getIconHorizontalPosition(rect, cellPosition);
+          const y = this.getIconVerticalPosition(rect, cellPosition);
           return cssPropertiesToCss({
               top: `${y + (this.props.offset?.y || 0)}px`,
               left: `${x + (this.props.offset?.x || 0)}px`,
           });
       }
-      getIconVerticalPosition() {
-          const { sheetId, row } = this.props.cellPosition;
-          const merge = this.env.model.getters.getMerge(this.props.cellPosition);
-          const start = this.env.model.getters.getRowDimensionsInViewport(sheetId, row).start;
-          const end = this.env.model.getters.getRowDimensionsInViewport(sheetId, merge ? merge.bottom : row).end;
-          const cell = this.env.model.getters.getCell(this.props.cellPosition);
+      getIconVerticalPosition(rect, cellPosition) {
+          const start = rect.y;
+          const end = rect.y + rect.height;
+          const cell = this.env.model.getters.getCell(cellPosition);
           const align = this.props.verticalAlign || cell?.style?.verticalAlign || DEFAULT_VERTICAL_ALIGN;
           switch (align) {
               case "bottom":
@@ -27799,13 +27831,11 @@
                   return end - GRID_ICON_EDGE_LENGTH - centeringOffset;
           }
       }
-      getIconHorizontalPosition() {
-          const { sheetId, col } = this.props.cellPosition;
-          const merge = this.env.model.getters.getMerge(this.props.cellPosition);
-          const start = this.env.model.getters.getColDimensionsInViewport(sheetId, col).start;
-          const end = this.env.model.getters.getColDimensionsInViewport(sheetId, merge ? merge.right : col).end;
-          const cell = this.env.model.getters.getCell(this.props.cellPosition);
-          const evaluatedCell = this.env.model.getters.getEvaluatedCell(this.props.cellPosition);
+      getIconHorizontalPosition(rect, cellPosition) {
+          const start = rect.x;
+          const end = rect.x + rect.width;
+          const cell = this.env.model.getters.getCell(cellPosition);
+          const evaluatedCell = this.env.model.getters.getEvaluatedCell(cellPosition);
           const align = this.props.horizontalAlign || cell?.style?.align || evaluatedCell.defaultAlign;
           switch (align) {
               case "right":
@@ -27889,6 +27919,8 @@
     height: ${CHECKBOX_WIDTH}px;
     accent-color: #808080;
     margin: ${MARGIN}px;
+    /** required to prevent the checkbox position to be sensible to the font-size (affects Firefox) */
+    position: absolute;
   }
 `;
   class DataValidationCheckbox extends owl.Component {
@@ -27907,7 +27939,7 @@
       }
       get isDisabled() {
           const cell = this.env.model.getters.getCell(this.props.cellPosition);
-          return !!cell?.isFormula;
+          return this.env.model.getters.isReadonly() || !!cell?.isFormula;
       }
   }
 
@@ -27947,12 +27979,17 @@
       static props = {};
       static components = { GridCellIcon, DataValidationCheckbox, DataValidationListIcon };
       get checkBoxCellPositions() {
-          return this.env.model.getters.getDataValidationCheckBoxCellPositions();
+          return this.env.model.getters
+              .getVisibleCellPositions()
+              .filter(this.env.model.getters.isCellValidCheckbox);
       }
       get listIconsCellPositions() {
-          return this.env.model.getters.isReadonly()
-              ? []
-              : this.env.model.getters.getDataValidationListCellsPositions();
+          if (this.env.model.getters.isReadonly()) {
+              return [];
+          }
+          return this.env.model.getters
+              .getVisibleCellPositions()
+              .filter(this.env.model.getters.cellHasListDataValidationIcon);
       }
   }
 
@@ -34314,29 +34351,12 @@
           from: 12,
           to: 12.5,
           applyMigration(data) {
-              for (let sheet of data.sheets || []) {
-                  let knownDataFilterZones = [];
-                  for (let filterTable of sheet.filterTables || []) {
-                      const zone = toZone(filterTable.range);
-                      // See commit message for the details
-                      const intersectZoneIndex = knownDataFilterZones.findIndex((knownZone) => overlap(knownZone, zone));
-                      if (intersectZoneIndex !== -1) {
-                          knownDataFilterZones[intersectZoneIndex] = zone;
-                      }
-                      else {
-                          knownDataFilterZones.push(zone);
-                      }
-                  }
-                  sheet.filterTables = knownDataFilterZones.map((zone) => ({
-                      range: zoneToXc(zone),
-                  }));
-              }
-              return data;
+              return fixOverlappingFilters(data);
           },
       },
       {
           description: "Change Border description structure",
-          from: 12,
+          from: 12.5,
           to: 13,
           applyMigration(data) {
               for (const borderId in data.borders) {
@@ -34367,6 +34387,14 @@
               return data;
           },
       },
+      {
+          description: "Fix datafilter duplication (post saas-16.4)",
+          from: 14,
+          to: 14.5,
+          applyMigration(data) {
+              return fixOverlappingFilters(data);
+          },
+      },
   ];
   /**
    * This function is used to repair faulty data independently of the migration.
@@ -34524,6 +34552,26 @@
       }
       return messages;
   }
+  function fixOverlappingFilters(data) {
+      for (let sheet of data.sheets || []) {
+          let knownDataFilterZones = [];
+          for (let filterTable of sheet.filterTables || []) {
+              const zone = toZone(filterTable.range);
+              // See commit message of https://github.com/odoo/o-spreadsheet/pull/3632 of more details
+              const intersectZoneIndex = knownDataFilterZones.findIndex((knownZone) => overlap(knownZone, zone));
+              if (intersectZoneIndex !== -1) {
+                  knownDataFilterZones[intersectZoneIndex] = zone;
+              }
+              else {
+                  knownDataFilterZones.push(zone);
+              }
+          }
+          sheet.filterTables = knownDataFilterZones.map((zone) => ({
+              range: zoneToXc(zone),
+          }));
+      }
+      return data;
+  }
   // -----------------------------------------------------------------------------
   // Helpers
   // -----------------------------------------------------------------------------
@@ -39045,7 +39093,6 @@
           "getNumberHeaders",
           "getGridLinesVisibility",
           "getNextSheetName",
-          "isEmpty",
           "getSheetSize",
           "getSheetZone",
           "getPaneDivisions",
@@ -39420,14 +39467,6 @@
       // ---------------------------------------------------------------------------
       // Row/Col manipulation
       // ---------------------------------------------------------------------------
-      /**
-       * Check if a zone only contains empty cells
-       */
-      isEmpty(sheetId, zone) {
-          return positions(zone)
-              .map(({ col, row }) => this.getCell({ sheetId, col, row }))
-              .every((cell) => !cell || cell.content === "");
-      }
       getCommandZones(cmd) {
           const zones = [];
           if ("zone" in cmd) {
@@ -41542,7 +41581,7 @@
                   : evaluateLiteral(cell.content, localeFormat);
           }
           catch (e) {
-              return createEvaluatedCell(e.value, localeFormat, e.message);
+              return createEvaluatedCell(e?.value || CellErrorType.GenericError, localeFormat, e?.message || implementationErrorMessage);
           }
           finally {
               this.cellsBeingComputed.delete(cellId);
@@ -41876,6 +41915,7 @@
           "getEvaluatedCellsInZone",
           "getSpreadPositionsOf",
           "getArrayFormulaSpreadingOn",
+          "isEmpty",
       ];
       shouldRebuildDependenciesGraph = true;
       evaluator;
@@ -41981,6 +42021,14 @@
       getArrayFormulaSpreadingOn(position) {
           return this.evaluator.getArrayFormulaSpreadingOn(position);
       }
+      /**
+       * Check if a zone only contains empty cells
+       */
+      isEmpty(sheetId, zone) {
+          return positions(zone)
+              .map(({ col, row }) => this.getEvaluatedCell({ sheetId, col, row }))
+              .every((cell) => cell.type === CellValueType.empty);
+      }
       // ---------------------------------------------------------------------------
       // Export
       // ---------------------------------------------------------------------------
@@ -42628,8 +42676,6 @@
   class EvaluationDataValidationPlugin extends UIPlugin {
       static getters = [
           "getDataValidationInvalidCriterionValueMessage",
-          "getDataValidationCheckBoxCellPositions",
-          "getDataValidationListCellsPositions",
           "getInvalidDataValidationMessage",
           "getValidationResultForCellValue",
           "isCellValidCheckbox",
@@ -42693,19 +42739,6 @@
           const error = this.getRuleErrorForCellValue(cellValue, cellPosition, rule);
           return error ? { error, rule, isValid: false } : VALID_RESULT;
       }
-      getDataValidationCheckBoxCellPositions() {
-          const rules = this.getters
-              .getDataValidationRules(this.getters.getActiveSheetId())
-              .filter((rule) => rule.criterion.type === "isBoolean");
-          return getCellPositionsInRanges(rules.map((rule) => rule.ranges).flat()).filter((position) => this.isCellValidCheckbox(position));
-      }
-      getDataValidationListCellsPositions() {
-          const rules = this.getters
-              .getDataValidationRules(this.getters.getActiveSheetId())
-              .filter((rule) => (rule.criterion.type === "isValueInList" || rule.criterion.type === "isValueInRange") &&
-              rule.criterion.displayStyle === "arrow");
-          return getCellPositionsInRanges(rules.map((rule) => rule.ranges).flat());
-      }
       getValidationResultForCell(cellPosition) {
           const { col, row, sheetId } = cellPosition;
           if (!this.validationResults[sheetId]) {
@@ -45674,6 +45707,8 @@
                   sheetIdFrom: this.getters.getActiveSheetId(),
                   sheetIdTo: selectedMatch.sheetId,
               });
+              // We do not want to reset the selection at finalize in this case
+              this.isSearchDirty = false;
           }
           // we want grid selection to capture the selection stream
           this.selection.getBackToDefault();
@@ -46975,19 +47010,17 @@
                   break;
               case "FOCUS_RANGE":
               case "CHANGE_RANGE":
-                  if (cmd.id !== this.focusedInputId) {
-                      const input = this.inputs[cmd.id];
-                      const range = input.ranges.find((range) => range.id === cmd.rangeId);
-                      if (this.isRangeValid(range?.xc || "A1")) {
-                          const sheetId = this.getters.getActiveSheetId();
-                          const zone = this.getters.getRangeFromSheetXC(sheetId, range?.xc || "A1").zone;
-                          this.selection.capture(input, { cell: { col: zone.left, row: zone.top }, zone }, {
-                              handleEvent: input.handleEvent.bind(input),
-                              release: () => (this.focusedInputId = null),
-                          });
-                      }
-                      this.focusedInputId = cmd.id;
+                  const input = this.inputs[cmd.id];
+                  const range = input.ranges.find((range) => range.id === cmd.rangeId);
+                  if (range) {
+                      const sheetId = this.getters.getActiveSheetId();
+                      const zone = this.getters.getRangeFromSheetXC(sheetId, range?.xc || "A1").zone;
+                      this.selection.capture(input, { cell: { col: zone.left, row: zone.top }, zone }, {
+                          handleEvent: input.handleEvent.bind(input),
+                          release: () => (this.focusedInputId = null),
+                      });
                   }
+                  this.focusedInputId = cmd.id;
                   break;
           }
           this.currentInput?.handle(cmd);
@@ -50406,6 +50439,8 @@
           "getEdgeScrollRow",
           "getVisibleFigures",
           "getVisibleRect",
+          "getVisibleRectWithoutHeaders",
+          "getVisibleCellPositions",
           "getColRowOffsetInViewport",
           "getMainViewportCoordinates",
           "getActiveSheetScrollInfo",
@@ -50649,6 +50684,26 @@
           const viewports = this.getSubViewports(sheetId);
           return [...new Set(viewports.map((v) => range(v.top, v.bottom + 1)).flat())].filter((row) => !this.getters.isHeaderHidden(sheetId, "ROW", row));
       }
+      /**
+       * Get the positions of all the cells that are visible in the viewport, taking merges into account.
+       */
+      getVisibleCellPositions() {
+          const visibleCols = this.getSheetViewVisibleCols();
+          const visibleRows = this.getSheetViewVisibleRows();
+          const sheetId = this.getters.getActiveSheetId();
+          const positions = [];
+          for (const col of visibleCols) {
+              for (const row of visibleRows) {
+                  const position = { sheetId, col, row };
+                  const mainPosition = this.getters.getMainCellPosition(position);
+                  if (mainPosition.row !== row || mainPosition.col !== col) {
+                      continue;
+                  }
+                  positions.push(position);
+              }
+          }
+          return positions;
+      }
       /**
        * Return the main viewport maximum size relative to the client size.
        */
@@ -50768,6 +50823,13 @@
        * Computes the coordinates and size to draw the zone on the canvas
        */
       getVisibleRect(zone) {
+          const rect = this.getVisibleRectWithoutHeaders(zone);
+          return { ...rect, x: rect.x + this.gridOffsetX, y: rect.y + this.gridOffsetY };
+      }
+      /**
+       * Computes the coordinates and size to draw the zone without taking the grid offset into account
+       */
+      getVisibleRectWithoutHeaders(zone) {
           const sheetId = this.getters.getActiveSheetId();
           const viewportRects = this.getSubViewports(sheetId)
               .map((viewport) => viewport.getRect(zone))
@@ -50779,12 +50841,7 @@
           const y = Math.min(...viewportRects.map((rect) => rect.y));
           const width = Math.max(...viewportRects.map((rect) => rect.x + rect.width)) - x;
           const height = Math.max(...viewportRects.map((rect) => rect.y + rect.height)) - y;
-          return {
-              x: x + this.gridOffsetX,
-              y: y + this.gridOffsetY,
-              width,
-              height,
-          };
+          return { x, y, width, height };
       }
       /**
        * Returns the position of the MainViewport relatively to the start of the grid (without headers)
@@ -51008,13 +51065,8 @@
                   }
                   break;
               case "UPDATE_CELL":
-                  if ("content" in cmd || "format" in cmd) {
-                      this.headerPositions = {};
-                      this.isDirty = true;
-                  }
-                  else {
-                      this.headerPositions[cmd.sheetId] = this.computeHeaderPositionsOfSheet(cmd.sheetId);
-                  }
+                  this.headerPositions = {};
+                  this.isDirty = true;
                   break;
               case "UPDATE_FILTER":
               case "REMOVE_FILTER_TABLE":
@@ -57223,9 +57275,9 @@
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.1.5";
-  __info__.date = "2024-02-16T14:23:54.651Z";
-  __info__.hash = "29d8ad6";
+  __info__.version = "17.1.7";
+  __info__.date = "2024-03-12T07:56:10.783Z";
+  __info__.hash = "3a9f334";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);