@odoo/o-spreadsheet 17.1.24 → 17.1.26

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.24
- * @date 2024-07-18T14:21:58.787Z
- * @hash 46a558e
+ * @version 17.1.26
+ * @date 2024-08-02T08:23:08.499Z
+ * @hash 4ba7e55
  */
 
 (function (exports, owl) {
@@ -480,7 +480,7 @@
       let timeout;
       return function () {
           const context = this;
-          const args = arguments;
+          const args = Array.from(arguments);
           function later() {
               timeout = null;
               if (!immediate) {
@@ -683,7 +683,7 @@
    */
   function largeMax(array) {
       let len = array.length;
-      if (len < 100000)
+      if (len < 100_000)
           return Math.max(...array);
       let max = -Infinity;
       while (len--) {
@@ -697,7 +697,7 @@
    */
   function largeMin(array) {
       let len = array.length;
-      if (len < 100000)
+      if (len < 100_000)
           return Math.min(...array);
       let min = +Infinity;
       while (len--) {
@@ -2614,13 +2614,16 @@
       }
       return new RegExp("^" + exp + "$", "i");
   }
-  function evaluatePredicate(value = "", criterion) {
+  function evaluatePredicate(value = "", criterion, locale) {
       const { operator, operand } = criterion;
       if (operand === undefined || value === null || operand === null) {
           return false;
       }
       if (typeof operand === "number" && operator === "=") {
-          return value.toString() === operand.toString();
+          if (typeof value === "string" && (isNumber(value, locale) || isDateTime(value, locale))) {
+              return toNumber(value, locale) === operand;
+          }
+          return value === operand;
       }
       if (operator === "<>" || operator === "=") {
           let result;
@@ -2698,7 +2701,7 @@
               for (let k = 0; k < countArg - 1; k += 2) {
                   const criteriaValue = toMatrix(args[k])[i][j].value;
                   const criterion = predicates[k / 2];
-                  validatedPredicates = evaluatePredicate(criteriaValue ?? undefined, criterion);
+                  validatedPredicates = evaluatePredicate(criteriaValue ?? undefined, criterion, locale);
                   if (!validatedPredicates) {
                       break;
                   }
@@ -5980,7 +5983,7 @@
   function numberCell(value, localeFormat) {
       return {
           type: CellValueType.number,
-          value: value || 0,
+          value: value || 0, // necessary to avoid "-0" and NaN values,
           format: localeFormat.format,
           isAutoSummable: true,
           defaultAlign: "right",
@@ -8471,8 +8474,8 @@
   function getDefaultChartJsRuntime(chart, labels, fontColor, { format, locale, truncateLabels = true }) {
       const options = {
           // https://www.chartjs.org/docs/latest/general/responsive.html
-          responsive: true,
-          maintainAspectRatio: false,
+          responsive: true, // will resize when its container is resized
+          maintainAspectRatio: false, // doesn't maintain the aspect ration (width/height =2 by default) so the user has the choice of the exact layout
           layout: {
               padding: { left: 20, right: 20, top: chart.title ? 10 : 25, bottom: 10 },
           },
@@ -8518,7 +8521,7 @@
               labels: truncateLabels ? labels.map(truncateLabel) : labels,
               datasets: [],
           },
-          platform: undefined,
+          platform: undefined, // This key is optional and will be set by chart.js
           plugins: [],
       };
   }
@@ -8782,7 +8785,7 @@
           },
           y: {
               position: chart.verticalAxisPosition,
-              beginAtZero: true,
+              beginAtZero: true, // the origin of the y axis is always zero
               ticks: {
                   color: fontColor,
                   callback: (value) => {
@@ -9436,7 +9439,7 @@
           },
           y: {
               position: chart.verticalAxisPosition,
-              beginAtZero: true,
+              beginAtZero: true, // the origin of the y axis is always zero
               ticks: {
                   color: fontColor,
                   callback: (value) => {
@@ -9524,7 +9527,7 @@
           const dataset = {
               label,
               data,
-              tension: 0,
+              tension: 0, // 0 -> render straight lines, which is much faster
               borderColor: color,
               backgroundColor,
               pointBackgroundColor: color,
@@ -9635,7 +9638,7 @@
               ...this.getDefinition(),
               backgroundColor: toXlsxHexColor(this.background || BACKGROUND_CHART_COLOR),
               fontColor: toXlsxHexColor(chartFontColor(this.background)),
-              verticalAxisPosition: "left",
+              verticalAxisPosition: "left", //TODO ExcelChartDefinition should be adapted, but can be done later
               dataSets,
               labelRange,
           };
@@ -27202,7 +27205,9 @@
           argToFocus: 0,
       });
       compositionActive = false;
+      spreadsheetRect = useSpreadsheetRect();
       get assistantStyle() {
+          const composerRect = this.composerRef.el.getBoundingClientRect();
           const assistantStyle = {};
           assistantStyle["min-width"] = `${this.props.rect?.width || ASSISTANT_WIDTH}px`;
           if (this.autoCompleteState.type === "function") {
@@ -27225,6 +27230,10 @@
                   assistantStyle.right = `0px`;
               }
           }
+          if (this.props.focus === "contentFocus" &&
+              composerRect.left + ASSISTANT_WIDTH > this.spreadsheetRect.width) {
+              assistantStyle.right = "0px";
+          }
           return cssPropertiesToCss(assistantStyle);
       }
       // we can't allow input events to be triggered while we remove and add back the content of the composer in processContent
@@ -28870,10 +28879,13 @@
           row: undefined,
       };
       const { Date } = window;
-      let x = 0;
-      let y = 0;
+      let x = undefined;
+      let y = undefined;
       let lastMoved = 0;
       function getPosition() {
+          if (x === undefined || y === undefined) {
+              return { col: -1, row: -1 };
+          }
           const col = env.model.getters.getColIndex(x);
           const row = env.model.getters.getRowIndex(y);
           return { col, row };
@@ -30055,7 +30067,7 @@
       onScroll(offset) {
           const { scrollX } = this.env.model.getters.getActiveSheetDOMScrollInfo();
           this.env.model.dispatch("SET_VIEWPORT_OFFSET", {
-              offsetX: scrollX,
+              offsetX: scrollX, // offsetX is the same
               offsetY: offset,
           });
       }
@@ -30268,8 +30280,8 @@
           "Ctrl+Shift+L": () => this.setHorizontalAlign("left"),
           "Ctrl+Shift+R": () => this.setHorizontalAlign("right"),
           "Ctrl+Shift+V": () => PASTE_AS_VALUE_ACTION(this.env),
-          "Ctrl+Shift+<": () => this.clearFormatting(),
-          "Ctrl+<": () => this.clearFormatting(),
+          "Ctrl+Shift+<": () => this.clearFormatting(), // for qwerty
+          "Ctrl+<": () => this.clearFormatting(), // for azerty
           "Ctrl+Shift+ ": () => {
               this.env.model.selection.selectAll();
           },
@@ -31067,10 +31079,10 @@
   const CF_TYPE_CONVERSION_MAP = {
       aboveAverage: undefined,
       expression: undefined,
-      cellIs: undefined,
-      colorScale: undefined,
+      cellIs: undefined, // exist but isn't an operator in o_spreadsheet
+      colorScale: undefined, // exist but isn't an operator in o_spreadsheet
       dataBar: undefined,
-      iconSet: undefined,
+      iconSet: undefined, // exist but isn't an operator in o_spreadsheet
       top10: undefined,
       uniqueValues: undefined,
       duplicateValues: undefined,
@@ -31305,7 +31317,7 @@
       61: "993366",
       62: "333399",
       63: "333333",
-      64: "000000",
+      64: "000000", // system foreground
       65: "FFFFFF", // system background
   };
   const IMAGE_MIMETYPE_TO_EXTENSION_MAPPING = {
@@ -34975,12 +34987,13 @@
                   this.clearBorders(cmd.sheetId, cmd.target);
                   break;
               case "REMOVE_COLUMNS_ROWS":
-                  for (let el of [...cmd.elements].sort((a, b) => b - a)) {
+                  const elements = [...cmd.elements].sort((a, b) => b - a);
+                  for (const group of groupConsecutive(elements)) {
                       if (cmd.dimension === "COL") {
-                          this.shiftBordersHorizontally(cmd.sheetId, el + 1, -1);
+                          this.shiftBordersHorizontally(cmd.sheetId, group[group.length - 1] + 1, -group.length);
                       }
                       else {
-                          this.shiftBordersVertically(cmd.sheetId, el + 1, -1);
+                          this.shiftBordersVertically(cmd.sheetId, group[group.length - 1] + 1, -group.length);
                       }
                   }
                   break;
@@ -40012,12 +40025,7 @@
                           });
                       }
                       if (colIndex > deletedColumn) {
-                          this.dispatch("UPDATE_CELL_POSITION", {
-                              sheetId: sheet.id,
-                              cellId: cellId,
-                              col: colIndex - 1,
-                              row: rowIndex,
-                          });
+                          this.setNewPosition(cellId, sheet.id, colIndex - 1, rowIndex);
                       }
                   }
               }
@@ -40027,7 +40035,7 @@
        * Move the cells after a column or rows insertion
        */
       moveCellsOnAddition(sheet, addedElement, quantity, dimension) {
-          const commands = [];
+          const updates = [];
           for (let rowIndex = 0; rowIndex < sheet.rows.length; rowIndex++) {
               const row = sheet.rows[rowIndex];
               if (dimension !== "rows" || rowIndex >= addedElement) {
@@ -40036,20 +40044,20 @@
                       const cellId = row.cells[i];
                       if (cellId) {
                           if (dimension === "rows" || colIndex >= addedElement) {
-                              commands.push({
-                                  type: "UPDATE_CELL_POSITION",
+                              updates.push({
                                   sheetId: sheet.id,
                                   cellId: cellId,
                                   col: colIndex + (dimension === "columns" ? quantity : 0),
                                   row: rowIndex + (dimension === "rows" ? quantity : 0),
+                                  type: "UPDATE_CELL_POSITION",
                               });
                           }
                       }
                   }
               }
           }
-          for (let cmd of commands.reverse()) {
-              this.dispatch(cmd.type, cmd);
+          for (let update of updates.reverse()) {
+              this.updateCellPosition(update);
           }
       }
       /**
@@ -40082,12 +40090,7 @@
                       const colIndex = Number(i);
                       const cellId = row.cells[i];
                       if (cellId) {
-                          this.dispatch("UPDATE_CELL_POSITION", {
-                              sheetId: sheet.id,
-                              cellId: cellId,
-                              col: colIndex,
-                              row: rowIndex - numberRows,
-                          });
+                          this.setNewPosition(cellId, sheet.id, colIndex, rowIndex - numberRows);
                       }
                   }
               }
@@ -42696,17 +42699,17 @@
    */
   function sortWithClusters(colorsToSort) {
       const clusters = [
-          { leadColor: rgba(255, 0, 0), colors: [] },
-          { leadColor: rgba(255, 128, 0), colors: [] },
-          { leadColor: rgba(128, 128, 0), colors: [] },
-          { leadColor: rgba(128, 255, 0), colors: [] },
-          { leadColor: rgba(0, 255, 0), colors: [] },
-          { leadColor: rgba(0, 255, 128), colors: [] },
-          { leadColor: rgba(0, 255, 255), colors: [] },
-          { leadColor: rgba(0, 127, 255), colors: [] },
-          { leadColor: rgba(0, 0, 255), colors: [] },
-          { leadColor: rgba(127, 0, 255), colors: [] },
-          { leadColor: rgba(128, 0, 128), colors: [] },
+          { leadColor: rgba(255, 0, 0), colors: [] }, // red
+          { leadColor: rgba(255, 128, 0), colors: [] }, // orange
+          { leadColor: rgba(128, 128, 0), colors: [] }, // yellow
+          { leadColor: rgba(128, 255, 0), colors: [] }, // chartreuse
+          { leadColor: rgba(0, 255, 0), colors: [] }, // green
+          { leadColor: rgba(0, 255, 128), colors: [] }, // spring green
+          { leadColor: rgba(0, 255, 255), colors: [] }, // cyan
+          { leadColor: rgba(0, 127, 255), colors: [] }, // azure
+          { leadColor: rgba(0, 0, 255), colors: [] }, // blue
+          { leadColor: rgba(127, 0, 255), colors: [] }, // violet
+          { leadColor: rgba(128, 0, 128), colors: [] }, // magenta
           { leadColor: rgba(255, 0, 128), colors: [] }, // rose
       ];
       for (const color of colorsToSort.map(colorToRGBA)) {
@@ -52493,7 +52496,7 @@
               .map((sheetEl) => sheetEl.getBoundingClientRect())
               .map((rect) => ({
               x: rect.x,
-              width: rect.width - 1,
+              width: rect.width - 1, // -1 to compensate negative margin
               y: rect.y,
               height: rect.height,
           }));
@@ -52715,7 +52718,7 @@
           }
           return cssPropertiesToCss({
               top: `${groupBox.headerRect.height / 2}px`,
-              left: `calc(50% - 1px)`,
+              left: `calc(50% - 1px)`, // -1px: we want the border to be on the center
               width: `30%`,
               height: `calc(100% - ${groupBox.headerRect.height / 2}px)`,
               "border-left": `1px solid ${HEADER_GROUPING_BORDER_COLOR}`,
@@ -52767,7 +52770,7 @@
               return "";
           }
           return cssPropertiesToCss({
-              top: `calc(50% - 1px)`,
+              top: `calc(50% - 1px)`, // -1px: we want the border to be on the center
               left: `${groupBox.headerRect.width / 2}px`,
               width: `calc(100% - ${groupBox.headerRect.width / 2}px)`,
               height: `30%`,
@@ -54178,7 +54181,7 @@
           const gridColSize = GROUP_LAYER_WIDTH * this.rowLayers.length;
           const gridRowSize = GROUP_LAYER_WIDTH * this.colLayers.length;
           return cssPropertiesToCss({
-              "grid-template-columns": `${gridColSize ? gridColSize + 2 : 0}px auto`,
+              "grid-template-columns": `${gridColSize ? gridColSize + 2 : 0}px auto`, // +2: margins
               "grid-template-rows": `${gridRowSize ? gridRowSize + 2 : 0}px auto`,
           });
       }
@@ -55623,7 +55626,7 @@
   }
 
   class StateObserver {
-      changes = [];
+      changes;
       commands = [];
       /**
        * Record the changes which could happen in the given callback, save them in a
@@ -55655,7 +55658,7 @@
           if (value[key] === val) {
               return;
           }
-          this.changes.push({
+          this.changes?.push({
               path: args,
               before: value[key],
               after: val,
@@ -56786,7 +56789,7 @@
           const colHeaderXc = toXC(tableZone.left + i, tableZone.top);
           const colName = sheetData.cells[colHeaderXc]?.content || `col${i}`;
           const colAttributes = [
-              ["id", i + 1],
+              ["id", i + 1], // id cannot be 0
               ["name", colName],
           ];
           columns.push(escapeXml /*xml*/ `<tableColumn ${formatAttributes(colAttributes)}/>`);
@@ -57878,9 +57881,9 @@
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.1.24";
-  __info__.date = "2024-07-18T14:21:58.787Z";
-  __info__.hash = "46a558e";
+  __info__.version = "17.1.26";
+  __info__.date = "2024-08-02T08:23:08.499Z";
+  __info__.hash = "4ba7e55";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);