@odoo/o-spreadsheet 17.2.3 → 17.2.4

package/dist/o-spreadsheet.iife.js

@@ -3,9 +3,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.2.3
- * @date 2024-04-10T12:28:36.552Z
- * @hash ab5e22d
+ * @version 17.2.4
+ * @date 2024-04-18T16:41:38.407Z
+ * @hash 0c66038
  */
 
 (function (exports, owl) {
@@ -2508,6 +2508,15 @@
       }
       return generateMatrix(matrix.length, matrix[0].length, (col, row) => fn(matrix[col][row]));
   }
+  function matrixForEach(matrix, fn) {
+      const numberOfCols = matrix.length;
+      const numberOfRows = matrix[0]?.length ?? 0;
+      for (let col = 0; col < numberOfCols; col++) {
+          for (let row = 0; row < numberOfRows; row++) {
+              fn(matrix[col][row]);
+          }
+      }
+  }
   function transposeMatrix(matrix) {
       if (!matrix.length) {
           return [];
@@ -18635,7 +18644,7 @@ stores.inject(MyMetaStore, storeInstance);
           }
           const descr = addMetaInfoFromArg(addDescr);
           validateArguments(descr.args);
-          this.mapping[name] = addErrorHandling(addResultHandling(addInputHandling(descr)), name);
+          this.mapping[name] = addErrorHandling(addResultHandling(addInputHandling(descr), name), name);
           super.add(name, descr);
           return this;
       }
@@ -18674,9 +18683,7 @@ stores.inject(MyMetaStore, storeInstance);
       // so we fallback to a generic error
       if (hasStringValue(e) && isEvaluationError(e.value)) {
           if (hasStringMessage(e)) {
-              if (e.message?.includes("[[FUNCTION_NAME]]")) {
-                  e.message = e.message.replace("[[FUNCTION_NAME]]", functionName);
-              }
+              replaceFunctionNamePlaceholder(e, functionName);
           }
           return e;
       }
@@ -18691,21 +18698,29 @@ stores.inject(MyMetaStore, storeInstance);
       return (obj?.message !== undefined &&
           typeof obj.message === "string");
   }
-  function addResultHandling(compute) {
-      return function (...args) {
+  function addResultHandling(compute, functionName) {
+      return function computeWithResultHandling(...args) {
           const result = compute.apply(this, args);
           if (!isMatrix(result)) {
               if (typeof result === "object" && result !== null && "value" in result) {
+                  replaceFunctionNamePlaceholder(result, functionName);
                   return result;
               }
               return { value: result };
           }
           if (typeof result[0][0] === "object" && result[0][0] !== null && "value" in result[0][0]) {
+              matrixForEach(result, (result) => replaceFunctionNamePlaceholder(result, functionName));
               return result;
           }
           return matrixMap(result, (row) => ({ value: row }));
       };
   }
+  function replaceFunctionNamePlaceholder(fPayload, functionName) {
+      // for performance reasons: change in place and only if needed
+      if (fPayload.message?.includes("[[FUNCTION_NAME]]")) {
+          fPayload.message = fPayload.message.replace("[[FUNCTION_NAME]]", functionName);
+      }
+  }
   const functionRegistry = new FunctionRegistry();
   for (let category of categories) {
       const fns = category.functions;
@@ -19473,7 +19488,7 @@ stores.inject(MyMetaStore, storeInstance);
                           const xLabel = tooltipItem.dataset?.label || tooltipItem.label;
                           // tooltipItem.parsed.y can be an object or a number for pie charts
                           const yLabel = tooltipItem.parsed.y ?? tooltipItem.parsed;
-                          const toolTipFormat = !format && yLabel > 1000 ? "#,##" : format;
+                          const toolTipFormat = !format && Math.abs(yLabel) >= 1000 ? "#,##" : format;
                           const yLabelStr = formatValue(yLabel, { format: toolTipFormat, locale });
                           return xLabel ? `${xLabel}: ${yLabelStr}` : yLabelStr;
                       },
@@ -19773,7 +19788,10 @@ stores.inject(MyMetaStore, storeInstance);
                       if (isNaN(value))
                           return value;
                       const { locale, format } = localeFormat;
-                      return formatValue(value, { locale, format: !format && value > 1000 ? "#,##" : format });
+                      return formatValue(value, {
+                          locale,
+                          format: !format && Math.abs(value) >= 1000 ? "#,##" : format,
+                      });
                   },
               },
           },
@@ -20294,7 +20312,10 @@ stores.inject(MyMetaStore, storeInstance);
                       if (isNaN(value))
                           return value;
                       const { locale, format } = localeFormat;
-                      return formatValue(value, { locale, format: !format && value > 1000 ? "#,##" : format });
+                      return formatValue(value, {
+                          locale,
+                          format: !format && Math.abs(value) >= 1000 ? "#,##" : format,
+                      });
                   },
               },
           },
@@ -20634,7 +20655,7 @@ stores.inject(MyMetaStore, storeInstance);
           const percentage = calculatePercentage(data, dataIndex);
           const xLabel = tooltipItem.label || tooltipItem.dataset.label;
           const yLabel = tooltipItem.parsed.y ?? tooltipItem.parsed;
-          const toolTipFormat = !format && yLabel > 1000 ? "#,##" : format;
+          const toolTipFormat = !format && yLabel >= 1000 ? "#,##" : format;
           const yLabelStr = formatValue(yLabel, { format: toolTipFormat, locale });
           return xLabel ? `${xLabel}: ${yLabelStr} (${percentage}%)` : `${yLabelStr} (${percentage}%)`;
       };
@@ -22886,6 +22907,7 @@ stores.inject(MyMetaStore, storeInstance);
                       this.save();
                   }
                   ev.stopPropagation();
+                  ev.preventDefault();
                   break;
               case "Escape":
                   this.cancel();
@@ -33162,6 +33184,10 @@ stores.inject(MyMetaStore, storeInstance);
       height: 0px;
     }
   }
+  .o-figure-container {
+    -webkit-user-select: none; // safari
+    user-select: none;
+  }
 `;
   /**
    * Each figure ⭐ is positioned inside a container `div` placed and sized
@@ -52085,7 +52111,7 @@ stores.inject(MyMetaStore, storeInstance);
           "isFilterActive",
       ];
       filterValues = {};
-      hiddenRows = new Set();
+      hiddenRows = {};
       isEvaluationDirty = false;
       allowDispatch(cmd) {
           switch (cmd.type) {
@@ -52129,11 +52155,11 @@ stores.inject(MyMetaStore, storeInstance);
               case "UNFOLD_HEADER_GROUP":
               case "FOLD_ALL_HEADER_GROUPS":
               case "UNFOLD_ALL_HEADER_GROUPS":
-                  this.updateHiddenRows();
+                  this.updateHiddenRows(cmd.sheetId);
                   break;
               case "UPDATE_FILTER":
                   this.updateFilter(cmd);
-                  this.updateHiddenRows();
+                  this.updateHiddenRows(cmd.sheetId);
                   break;
               case "DUPLICATE_SHEET":
                   const filterValues = {};
@@ -52153,15 +52179,14 @@ stores.inject(MyMetaStore, storeInstance);
       }
       finalize() {
           if (this.isEvaluationDirty) {
-              this.updateHiddenRows();
+              for (const sheetId of this.getters.getSheetIds()) {
+                  this.updateHiddenRows(sheetId);
+              }
               this.isEvaluationDirty = false;
           }
       }
       isRowFiltered(sheetId, row) {
-          if (sheetId !== this.getters.getActiveSheetId()) {
-              return false;
-          }
-          return this.hiddenRows.has(row);
+          return !!this.hiddenRows[sheetId]?.has(row);
       }
       getFilterHiddenValues(position) {
           const id = this.getters.getFilterId(position);
@@ -52188,8 +52213,7 @@ stores.inject(MyMetaStore, storeInstance);
               this.filterValues[sheetId] = {};
           this.filterValues[sheetId][id] = hiddenValues;
       }
-      updateHiddenRows() {
-          const sheetId = this.getters.getActiveSheetId();
+      updateHiddenRows(sheetId) {
           const filters = this.getters
               .getFilters(sheetId)
               .sort((filter1, filter2) => filter1.rangeWithHeaders.zone.top - filter2.rangeWithHeaders.zone.top);
@@ -52211,7 +52235,7 @@ stores.inject(MyMetaStore, storeInstance);
                   }
               }
           }
-          this.hiddenRows = hiddenRows;
+          this.hiddenRows[sheetId] = hiddenRows;
       }
       getCellValueAsString(sheetId, col, row) {
           const value = this.getters.getEvaluatedCell({ sheetId, col, row }).formattedValue;
@@ -53330,13 +53354,14 @@ stores.inject(MyMetaStore, storeInstance);
           }
       }
       handleEvent(event) {
+          const sheetId = this.getters.getActiveSheetId();
           if (event.options.scrollIntoView) {
               let { col, row } = findCellInNewZone(event.previousAnchor.zone, event.anchor.zone);
               if (event.mode === "updateAnchor") {
                   const oldZone = event.previousAnchor.zone;
                   const newZone = event.anchor.zone;
                   // altering a zone should not move the viewport in a dimension that wasn't changed
-                  const { top, bottom, left, right } = this.getters.getActiveMainViewport();
+                  const { top, bottom, left, right } = this.getMainInternalViewport(sheetId);
                   if (oldZone.left === newZone.left && oldZone.right === newZone.right) {
                       col = left > col || col > right ? left : col;
                   }
@@ -53344,7 +53369,6 @@ stores.inject(MyMetaStore, storeInstance);
                       row = top > row || row > bottom ? top : row;
                   }
               }
-              const sheetId = this.getters.getActiveSheetId();
               col = Math.min(col, this.getters.getNumberCols(sheetId) - 1);
               row = Math.min(row, this.getters.getNumberRows(sheetId) - 1);
               if (!this.sheetsWithDirtyViewports.has(sheetId)) {
@@ -53381,16 +53405,16 @@ stores.inject(MyMetaStore, storeInstance);
                   this.setSheetViewOffset(cmd.offsetX, cmd.offsetY);
                   break;
               case "SHIFT_VIEWPORT_DOWN":
-                  const { top } = this.getActiveMainViewport();
                   const sheetId = this.getters.getActiveSheetId();
-                  const shiftedOffsetY = this.clipOffsetY(this.getters.getRowDimensions(sheetId, top).start + this.sheetViewHeight);
-                  this.shiftVertically(shiftedOffsetY);
+                  const { top, viewportHeight, offsetCorrectionY } = this.getMainInternalViewport(sheetId);
+                  const topRowDims = this.getters.getRowDimensions(sheetId, top);
+                  this.shiftVertically(topRowDims.start + viewportHeight - offsetCorrectionY);
                   break;
               case "SHIFT_VIEWPORT_UP": {
-                  const { top } = this.getActiveMainViewport();
                   const sheetId = this.getters.getActiveSheetId();
-                  const shiftedOffsetY = this.clipOffsetY(this.getters.getRowDimensions(sheetId, top).end - this.sheetViewHeight);
-                  this.shiftVertically(shiftedOffsetY);
+                  const { top, viewportHeight, offsetCorrectionY } = this.getMainInternalViewport(sheetId);
+                  const topRowDims = this.getters.getRowDimensions(sheetId, top);
+                  this.shiftVertically(topRowDims.end - offsetCorrectionY - viewportHeight);
                   break;
               }
               case "REMOVE_TABLE":
@@ -53813,17 +53837,6 @@ stores.inject(MyMetaStore, storeInstance);
           const { maxOffsetX, maxOffsetY } = this.getMaximumSheetOffset();
           Object.values(this.getSubViewports(sheetId)).forEach((viewport) => viewport.setViewportOffset(clip(offsetX, 0, maxOffsetX), clip(offsetY, 0, maxOffsetY)));
       }
-      /**
-       * Clip the vertical offset within the allowed range.
-       * Not above the sheet, nor below the sheet.
-       */
-      clipOffsetY(offsetY) {
-          const { height } = this.getMainViewportRect();
-          const maxOffset = height - this.sheetViewHeight;
-          offsetY = Math.min(offsetY, maxOffset);
-          offsetY = Math.max(offsetY, 0);
-          return offsetY;
-      }
       getViewportOffset(sheetId) {
           return {
               x: this.viewports[sheetId]?.bottomRight.offsetScrollbarX || 0,
@@ -53879,12 +53892,15 @@ stores.inject(MyMetaStore, storeInstance);
        * viewport top.
        */
       shiftVertically(offset) {
-          const { top } = this.getActiveMainViewport();
+          const sheetId = this.getters.getActiveSheetId();
+          const { top } = this.getMainInternalViewport(sheetId);
           const { scrollX } = this.getActiveSheetScrollInfo();
           this.setSheetViewOffset(scrollX, offset);
           const { anchor } = this.getters.getSelection();
-          const deltaRow = this.getActiveMainViewport().top - top;
-          this.selection.selectCell(anchor.cell.col, anchor.cell.row + deltaRow);
+          if (anchor.cell.row >= this.getters.getPaneDivisions(sheetId).ySplit) {
+              const deltaRow = this.getMainInternalViewport(sheetId).top - top;
+              this.selection.selectCell(anchor.cell.col, anchor.cell.row + deltaRow);
+          }
       }
       getVisibleFigures() {
           const sheetId = this.getters.getActiveSheetId();
@@ -54803,7 +54819,6 @@ stores.inject(MyMetaStore, storeInstance);
     cursor: pointer;
   }
 `;
-  let tKey = 1;
   class SpreadsheetDashboard extends owl.Component {
       static template = "o-spreadsheet-SpreadsheetDashboard";
       static props = {};
@@ -54882,13 +54897,9 @@ stores.inject(MyMetaStore, storeInstance);
                       coordinates: rect,
                       position: { col, row },
                       action,
-                      // we can't rely on position only because a row or a column could
-                      // be inserted at any time.
-                      tKey: `${tKey}-${col}-${row}`,
                   });
               }
           }
-          tKey++;
           return cells;
       }
       getClickableAction(position) {
@@ -60228,9 +60239,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.2.3";
-  __info__.date = "2024-04-10T12:28:36.552Z";
-  __info__.hash = "ab5e22d";
+  __info__.version = "17.2.4";
+  __info__.date = "2024-04-18T16:41:38.407Z";
+  __info__.hash = "0c66038";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);