@odoo/o-spreadsheet 17.2.8 → 17.2.10

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.8
- * @date 2024-05-24T11:29:49.320Z
- * @hash bfbcaa0
+ * @version 17.2.10
+ * @date 2024-06-10T09:38:54.354Z
+ * @hash 14317ee
  */
 
 (function (exports, owl) {
@@ -294,7 +294,10 @@
    * Check if the object is a plain old javascript object.
    */
   function isPlainObject(obj) {
-      return typeof obj === "object" && obj?.constructor === Object;
+      return (typeof obj === "object" &&
+          obj !== null &&
+          // obj.constructor can be undefined when there's no prototype (`Object.create(null, {})`)
+          (obj?.constructor === Object || obj?.constructor === undefined));
   }
   /**
    * Sanitize the name of a sheet, by eventually removing quotes
@@ -2659,7 +2662,7 @@
           return false;
       }
       if (typeof operand === "number" && operator === "=") {
-          return toString(value) === toString(operand);
+          return value.toString() === operand.toString();
       }
       if (operator === "<>" || operator === "=") {
           let result;
@@ -2719,14 +2722,13 @@
       if (countArg % 2 === 1) {
           throw new EvaluationError(_t("Function [[FUNCTION_NAME]] expects criteria_range and criterion to be in pairs."));
       }
-      const dimRow = args[0].length;
-      const dimCol = args[0][0].length;
+      const firstArg = toMatrix(args[0]);
+      const dimRow = firstArg.length;
+      const dimCol = firstArg[0].length;
       let predicates = [];
       for (let i = 0; i < countArg - 1; i += 2) {
-          const criteriaRange = args[i];
-          if (!isMatrix(criteriaRange) ||
-              criteriaRange.length !== dimRow ||
-              criteriaRange[0].length !== dimCol) {
+          const criteriaRange = toMatrix(args[i]);
+          if (criteriaRange.length !== dimRow || criteriaRange[0].length !== dimCol) {
               throw new EvaluationError(_t("Function [[FUNCTION_NAME]] expects criteria_range to have the same dimension"));
           }
           const description = toString(args[i + 1]);
@@ -2740,7 +2742,7 @@
           for (let j = 0; j < dimCol; j++) {
               let validatedPredicates = true;
               for (let k = 0; k < countArg - 1; k += 2) {
-                  const criteriaValue = args[k][i][j].value;
+                  const criteriaValue = toMatrix(args[k])[i][j].value;
                   const criterion = predicates[k / 2];
                   validatedPredicates = evaluatePredicate(criteriaValue ?? undefined, criterion);
                   if (!validatedPredicates) {
@@ -4457,8 +4459,11 @@
   /**
    * Create a range from a xc. If the xc is empty, this function returns undefined.
    */
-  function createRange(getters, sheetId, range) {
-      return range ? getters.getRangeFromSheetXC(sheetId, range) : undefined;
+  function createValidRange(getters, sheetId, xc) {
+      if (!xc)
+          return;
+      const range = getters.getRangeFromSheetXC(sheetId, xc);
+      return !(range.invalidSheetName || range.invalidXc) ? range : undefined;
   }
   /**
    * Spread multiple colrows zone to one row/col zone and add a many new input range as needed.
@@ -9645,8 +9650,8 @@ stores.inject(MyMetaStore, storeInstance);
       type = "scorecard";
       constructor(definition, sheetId, getters) {
           super(definition, sheetId, getters);
-          this.keyValue = createRange(getters, sheetId, definition.keyValue);
-          this.baseline = createRange(getters, sheetId, definition.baseline);
+          this.keyValue = createValidRange(getters, sheetId, definition.keyValue);
+          this.baseline = createValidRange(getters, sheetId, definition.baseline);
           this.baselineMode = definition.baselineMode;
           this.baselineDescr = definition.baselineDescr;
           this.background = definition.background;
@@ -10216,6 +10221,9 @@ stores.inject(MyMetaStore, storeInstance);
       if (types.some((t) => t.startsWith("RANGE"))) {
           result.acceptMatrix = true;
       }
+      if (types.every((t) => t.startsWith("RANGE"))) {
+          result.acceptMatrixOnly = true;
+      }
       return result;
   }
   /**
@@ -10497,11 +10505,16 @@ stores.inject(MyMetaStore, storeInstance);
       compute: function (array, ...columns) {
           const _array = toMatrix(array);
           const _columns = flattenRowFirst(columns, (item) => toInteger(item?.value, this.locale));
-          assert(() => _columns.every((col) => col > 0 && col <= _array.length), _t("The columns arguments must be between 1 and %s (got %s).", _array.length.toString(), (_columns.find((col) => col <= 0 || col > _array.length) || 0).toString()));
+          const argOutOfRange = _columns.filter((col) => col === 0 || _array.length < Math.abs(col));
+          assert(() => argOutOfRange.length === 0, _t("The columns arguments must be between -%s and %s (got %s), excluding 0.", _array.length.toString(), _array.length.toString(), argOutOfRange.join(",")));
           const result = Array(_columns.length);
           for (let col = 0; col < _columns.length; col++) {
-              const colIndex = _columns[col] - 1; // -1 because columns arguments are 1-indexed
-              result[col] = _array[colIndex];
+              if (_columns[col] > 0) {
+                  result[col] = _array[_columns[col] - 1]; // -1 because columns arguments are 1-indexed
+              }
+              else {
+                  result[col] = _array[_array.length + _columns[col]];
+              }
           }
           return result;
       },
@@ -10522,8 +10535,14 @@ stores.inject(MyMetaStore, storeInstance);
           const _array = toMatrix(array);
           const _rows = flattenRowFirst(rows, (item) => toInteger(item?.value, this.locale));
           const _nbColumns = _array.length;
-          assert(() => _rows.every((row) => row > 0 && row <= _array[0].length), _t("The rows arguments must be between 1 and %s (got %s).", _array[0].length.toString(), (_rows.find((row) => row <= 0 || row > _array[0].length) || 0).toString()));
-          return generateMatrix(_nbColumns, _rows.length, (col, row) => _array[col][_rows[row] - 1]); // -1 because rows arguments are 1-indexed
+          const argOutOfRange = _rows.filter((row) => row === 0 || _array[0].length < Math.abs(row));
+          assert(() => argOutOfRange.length === 0, _t("The rows arguments must be between -%s and %s (got %s), excluding 0.", _array[0].length.toString(), _array[0].length.toString(), argOutOfRange.join(",")));
+          return generateMatrix(_nbColumns, _rows.length, (col, row) => {
+              if (_rows[row] > 0) {
+                  return _array[col][_rows[row] - 1]; // -1 because columns arguments are 1-indexed
+              }
+              return _array[col][_array[col].length + _rows[row]];
+          });
       },
       isExported: true,
   };
@@ -11431,7 +11450,7 @@ stores.inject(MyMetaStore, storeInstance);
       compute: function (range, ...args) {
           let uniqueValues = new Set();
           visitMatchingRanges(args, (i, j) => {
-              const data = range[i][j];
+              const data = range[i]?.[j];
               if (isDefined(data)) {
                   uniqueValues.add(data.value);
               }
@@ -12061,7 +12080,7 @@ stores.inject(MyMetaStore, storeInstance);
           }
           let sum = 0;
           visitMatchingRanges([criteriaRange, criterion], (i, j) => {
-              const value = sumRange[i][j].value;
+              const value = sumRange[i]?.[j]?.value;
               if (typeof value === "number") {
                   sum += value;
               }
@@ -12086,7 +12105,7 @@ stores.inject(MyMetaStore, storeInstance);
       compute: function (sumRange, ...criters) {
           let sum = 0;
           visitMatchingRanges(criters, (i, j) => {
-              const value = sumRange[i][j].value;
+              const value = sumRange[i]?.[j]?.value;
               if (typeof value === "number") {
                   sum += value;
               }
@@ -12633,7 +12652,7 @@ stores.inject(MyMetaStore, storeInstance);
           let count = 0;
           let sum = 0;
           visitMatchingRanges([criteriaRange, criterion], (i, j) => {
-              const value = _averageRange[i][j].value;
+              const value = _averageRange[i]?.[j]?.value;
               if (typeof value === "number") {
                   count += 1;
                   sum += value;
@@ -12662,7 +12681,7 @@ stores.inject(MyMetaStore, storeInstance);
           let count = 0;
           let sum = 0;
           visitMatchingRanges(args, (i, j) => {
-              const value = _averageRange[i][j].value;
+              const value = _averageRange[i]?.[j]?.value;
               if (typeof value === "number") {
                   count += 1;
                   sum += value;
@@ -12967,7 +12986,7 @@ stores.inject(MyMetaStore, storeInstance);
       compute: function (range, ...args) {
           let result = -Infinity;
           visitMatchingRanges(args, (i, j) => {
-              const value = range[i][j].value;
+              const value = range[i]?.[j]?.value;
               if (typeof value === "number") {
                   result = result < value ? value : result;
               }
@@ -13050,7 +13069,7 @@ stores.inject(MyMetaStore, storeInstance);
       compute: function (range, ...args) {
           let result = Infinity;
           visitMatchingRanges(args, (i, j) => {
-              const value = range[i][j].value;
+              const value = range[i]?.[j]?.value;
               if (typeof value === "number") {
                   result = result > value ? value : result;
               }
@@ -18802,6 +18821,9 @@ stores.inject(MyMetaStore, storeInstance);
                   }
                   args[i] = arg[0][0];
               }
+              if (!isMatrix(arg) && argDefinition.acceptMatrixOnly) {
+                  throw new BadExpressionError(_t("Function [[FUNCTION_NAME]] expects the parameter '%s' to be reference to a cell or range.", (i + 1).toString()));
+              }
           }
           return descr.compute.apply(this, args);
       }
@@ -19754,7 +19776,7 @@ stores.inject(MyMetaStore, storeInstance);
       if ("chartJsConfig" in runtime) {
           runtime.chartJsConfig.plugins = [backgroundColorChartJSPlugin];
           // @ts-ignore
-          const chart = new window.Chart(canvas, runtime.chartJsConfig);
+          const chart = new window.Chart(canvas, deepCopy(runtime.chartJsConfig));
           const imgContent = chart.toBase64Image();
           chart.destroy();
           div.remove();
@@ -19804,7 +19826,7 @@ stores.inject(MyMetaStore, storeInstance);
       constructor(definition, sheetId, getters) {
           super(definition, sheetId, getters);
           this.dataSets = createDataSets(getters, definition.dataSets, sheetId, definition.dataSetsHaveTitle);
-          this.labelRange = createRange(getters, sheetId, definition.labelRange);
+          this.labelRange = createValidRange(getters, sheetId, definition.labelRange);
           this.background = definition.background;
           this.verticalAxisPosition = definition.verticalAxisPosition;
           this.legendPosition = definition.legendPosition;
@@ -20048,7 +20070,7 @@ stores.inject(MyMetaStore, storeInstance);
       type = "gauge";
       constructor(definition, sheetId, getters) {
           super(definition, sheetId, getters);
-          this.dataRange = createRange(this.getters, this.sheetId, definition.dataRange);
+          this.dataRange = createValidRange(this.getters, this.sheetId, definition.dataRange);
           this.sectionRule = definition.sectionRule;
           this.background = definition.background;
       }
@@ -20569,7 +20591,7 @@ stores.inject(MyMetaStore, storeInstance);
       constructor(definition, sheetId, getters) {
           super(definition, sheetId, getters);
           this.dataSets = createDataSets(this.getters, definition.dataSets, sheetId, definition.dataSetsHaveTitle);
-          this.labelRange = createRange(this.getters, sheetId, definition.labelRange);
+          this.labelRange = createValidRange(this.getters, sheetId, definition.labelRange);
           this.background = definition.background;
           this.verticalAxisPosition = definition.verticalAxisPosition;
           this.legendPosition = definition.legendPosition;
@@ -20684,7 +20706,7 @@ stores.inject(MyMetaStore, storeInstance);
       constructor(definition, sheetId, getters) {
           super(definition, sheetId, getters);
           this.dataSets = createDataSets(getters, definition.dataSets, sheetId, definition.dataSetsHaveTitle);
-          this.labelRange = createRange(getters, sheetId, definition.labelRange);
+          this.labelRange = createValidRange(getters, sheetId, definition.labelRange);
           this.background = definition.background;
           this.legendPosition = definition.legendPosition;
           this.aggregated = definition.aggregated;
@@ -20886,7 +20908,7 @@ stores.inject(MyMetaStore, storeInstance);
       constructor(definition, sheetId, getters) {
           super(definition, sheetId, getters);
           this.dataSets = createDataSets(this.getters, definition.dataSets, sheetId, definition.dataSetsHaveTitle);
-          this.labelRange = createRange(this.getters, sheetId, definition.labelRange);
+          this.labelRange = createValidRange(this.getters, sheetId, definition.labelRange);
           this.background = definition.background;
           this.verticalAxisPosition = definition.verticalAxisPosition;
           this.legendPosition = definition.legendPosition;
@@ -20972,13 +20994,6 @@ stores.inject(MyMetaStore, storeInstance);
       // have less options than the line chart (it only works with linear labels)
       chartJsConfig.type = "line";
       const configOptions = chartJsConfig.options;
-      configOptions.elements = {
-          point: {
-              radius: 3,
-              hoverRadius: 3,
-              hitRadius: 8,
-          },
-      };
       const locale = getters.getLocale();
       configOptions.plugins.tooltip.callbacks.title = () => "";
       configOptions.plugins.tooltip.callbacks.label = (tooltipItem) => {
@@ -21707,8 +21722,7 @@ stores.inject(MyMetaStore, storeInstance);
    */
   function useSpreadsheetRect() {
       const position = owl.useState({ x: 0, y: 0, width: 0, height: 0 });
-      let spreadsheetElement = document.querySelector(".o-spreadsheet");
-      updatePosition();
+      let spreadsheetElement = null;
       function updatePosition() {
           if (!spreadsheetElement) {
               spreadsheetElement = document.querySelector(".o-spreadsheet");
@@ -26812,7 +26826,7 @@ stores.inject(MyMetaStore, storeInstance);
           }
           const getters = this.env.model.getters;
           const sheetId = getters.getActiveSheetId();
-          const labelRange = createRange(getters, sheetId, this.labelRange);
+          const labelRange = createValidRange(getters, sheetId, this.labelRange);
           const dataSets = createDataSets(getters, this.dataSeriesRanges, sheetId, this.props.definition.dataSetsHaveTitle);
           if (dataSets.length) {
               return dataSets[0].dataRange.zone.top + 1;
@@ -26841,15 +26855,23 @@ stores.inject(MyMetaStore, storeInstance);
       }
   }
 
+  /**
+   * Start listening to pointer events and apply the given callbacks.
+   *
+   * @returns A function to remove the listeners.
+   */
   function startDnd(onMouseMove, onMouseUp, onMouseDown = () => { }) {
-      const _onMouseUp = (ev) => {
-          onMouseUp(ev);
+      const removeListeners = () => {
           window.removeEventListener("pointerdown", onMouseDown);
           window.removeEventListener("pointerup", _onMouseUp);
           window.removeEventListener("dragstart", _onDragStart);
           window.removeEventListener("pointermove", onMouseMove);
           window.removeEventListener("wheel", onMouseMove);
       };
+      const _onMouseUp = (ev) => {
+          onMouseUp(ev);
+          removeListeners();
+      };
       function _onDragStart(ev) {
           ev.preventDefault();
       }
@@ -26861,6 +26883,7 @@ stores.inject(MyMetaStore, storeInstance);
       // preventDefault() is not allowed in passive event handler.
       // https://chromestatus.com/feature/6662647093133312
       window.addEventListener("wheel", onMouseMove, { passive: false });
+      return removeListeners;
   }
   /**
    * Function to be used during a pointerdown event, this function allows to
@@ -28055,6 +28078,7 @@ stores.inject(MyMetaStore, storeInstance);
               state.itemsStyle = {};
               document.body.style.cursor = previousCursor;
               args.onCancel?.();
+              cleanUp();
           };
           const onDragEnd = (itemId, indexAtEnd) => {
               state.draggedItemId = undefined;
@@ -28075,7 +28099,8 @@ stores.inject(MyMetaStore, storeInstance);
               onDragEnd,
               onCancel: state.cancel,
           });
-          startDnd(dndHelper.onMouseMove.bind(dndHelper), dndHelper.onMouseUp.bind(dndHelper));
+          const stopListening = startDnd(dndHelper.onMouseMove.bind(dndHelper), dndHelper.onMouseUp.bind(dndHelper));
+          cleanupFns.push(stopListening);
           const onScroll = dndHelper.onScroll.bind(dndHelper);
           args.containerEl.addEventListener("scroll", onScroll);
           cleanupFns.push(() => args.containerEl.removeEventListener("scroll", onScroll));
@@ -28152,7 +28177,7 @@ stores.inject(MyMetaStore, storeInstance);
           this.moveDraggedItemToPosition(this.currentMousePosition + this.scrollOffset);
       }
       onMouseMove(ev) {
-          if (ev.button !== -1) {
+          if (ev.button > 1) {
               this.onCancel();
               return;
           }
@@ -31322,14 +31347,14 @@ stores.inject(MyMetaStore, storeInstance);
       }
       onKeyDown(ev) {
           const figure = this.props.figure;
-          switch (ev.key) {
+          const keyDownShortcut = keyboardEventToShortcutString(ev);
+          switch (keyDownShortcut) {
               case "Delete":
                   this.env.model.dispatch("DELETE_FIGURE", {
                       sheetId: this.env.model.getters.getActiveSheetId(),
                       id: figure.id,
                   });
                   this.props.onFigureDeleted();
-                  ev.stopPropagation();
                   ev.preventDefault();
                   ev.stopPropagation();
                   break;
@@ -31350,7 +31375,22 @@ stores.inject(MyMetaStore, storeInstance);
                       x: figure.x + delta[0],
                       y: figure.y + delta[1],
                   });
+                  ev.preventDefault();
                   ev.stopPropagation();
+                  break;
+              case "Ctrl+A":
+                  // Maybe in the future we will implement a way to select all figures
+                  ev.preventDefault();
+                  ev.stopPropagation();
+                  break;
+              case "Ctrl+Y":
+              case "Ctrl+Z":
+                  if (keyDownShortcut === "Ctrl+Y") {
+                      this.env.model.dispatch("REQUEST_REDO");
+                  }
+                  else if (keyDownShortcut === "Ctrl+Z") {
+                      this.env.model.dispatch("REQUEST_UNDO");
+                  }
                   ev.preventDefault();
                   ev.stopPropagation();
                   break;
@@ -32393,8 +32433,15 @@ stores.inject(MyMetaStore, storeInstance);
       }
       onPaste(ev) {
           if (this.composerStore.editionMode !== "inactive") {
+              // let the browser clipboard work
               ev.stopPropagation();
           }
+          else {
+              // the user meant to paste in the sheet, not open the composer with the pasted content
+              // While we're not editing, we still have the focus and should therefore prevent
+              // the native "paste" to occur.
+              ev.preventDefault();
+          }
       }
       /*
        * Triggered automatically by the content-editable between the keydown and key up
@@ -32403,9 +32450,6 @@ stores.inject(MyMetaStore, storeInstance);
           if (!this.shouldProcessInputEvents) {
               return;
           }
-          if (ev.inputType === "insertFromPaste" && this.composerStore.editionMode === "inactive") {
-              return;
-          }
           ev.stopPropagation();
           let content;
           if (this.composerStore.editionMode === "inactive") {
@@ -32772,10 +32816,7 @@ stores.inject(MyMetaStore, storeInstance);
       }
       get containerStyle() {
           if (this.composerStore.editionMode === "inactive") {
-              return `
-        position: absolute;
-        z-index: -1000;
-      `;
+              return `z-index: -1000;`;
           }
           const isFormula = this.composerStore.currentContent.startsWith("=");
           const cell = this.env.model.getters.getActiveCell();
@@ -41138,12 +41179,6 @@ stores.inject(MyMetaStore, storeInstance);
                   // detect when an argument need to be evaluated as a meta argument
                   const isMeta = argTypes.includes("META");
                   const hasRange = argTypes.some((t) => isRangeType(t));
-                  const isRangeOnly = argTypes.every((t) => isRangeType(t));
-                  if (isRangeOnly) {
-                      if (!isRangeInput(currentArg)) {
-                          throw new BadExpressionError(_t("Function %s expects the parameter %s to be reference to a cell or range, not a %s.", functionName, (i + 1).toString(), currentArg.type.toLowerCase()));
-                      }
-                  }
                   compiledArgs.push(compileAST(currentArg, isMeta, hasRange, {
                       functionName,
                       paramIndex: i + 1,
@@ -41314,16 +41349,6 @@ stores.inject(MyMetaStore, storeInstance);
   function isRangeType(type) {
       return type.startsWith("RANGE");
   }
-  function isRangeInput(arg) {
-      if (arg.type === "REFERENCE") {
-          return true;
-      }
-      if (arg.type === "FUNCALL") {
-          const fnDef = functions$1[arg.value.toUpperCase()];
-          return fnDef && isRangeType(fnDef.returns[0]);
-      }
-      return false;
-  }
 
   const functions = functionRegistry.content;
   function isExportableToExcel(tokens) {
@@ -44061,6 +44086,9 @@ stores.inject(MyMetaStore, storeInstance);
           if (range.invalidXc) {
               return range.invalidXc;
           }
+          if (!this.getters.tryGetSheet(range.sheetId)) {
+              return CellErrorType.InvalidReference;
+          }
           if (range.zone.bottom - range.zone.top < 0 || range.zone.right - range.zone.left < 0) {
               return CellErrorType.InvalidReference;
           }
@@ -47445,7 +47473,6 @@ stores.inject(MyMetaStore, storeInstance);
    *
    */
   class SpreadingRelation {
-      createEmptyPositionSet;
       /**
        * Internal structure:
        * For something like
@@ -47476,9 +47503,6 @@ stores.inject(MyMetaStore, storeInstance);
        */
       resultsToArrayFormulas = new PositionMap();
       arrayFormulasToResults = new PositionMap();
-      constructor(createEmptyPositionSet) {
-          this.createEmptyPositionSet = createEmptyPositionSet;
-      }
       getFormulaPositionsSpreadingOn(resultPosition) {
           return this.resultsToArrayFormulas.get(resultPosition) || EMPTY_ARRAY;
       }
@@ -47497,13 +47521,13 @@ stores.inject(MyMetaStore, storeInstance);
        */
       addRelation({ arrayFormulaPosition, resultPosition, }) {
           if (!this.resultsToArrayFormulas.has(resultPosition)) {
-              this.resultsToArrayFormulas.set(resultPosition, this.createEmptyPositionSet());
+              this.resultsToArrayFormulas.set(resultPosition, []);
           }
-          this.resultsToArrayFormulas.get(resultPosition)?.add(arrayFormulaPosition);
+          this.resultsToArrayFormulas.get(resultPosition)?.push(arrayFormulaPosition);
           if (!this.arrayFormulasToResults.has(arrayFormulaPosition)) {
-              this.arrayFormulasToResults.set(arrayFormulaPosition, this.createEmptyPositionSet());
+              this.arrayFormulasToResults.set(arrayFormulaPosition, []);
           }
-          this.arrayFormulasToResults.get(arrayFormulaPosition)?.add(resultPosition);
+          this.arrayFormulasToResults.get(arrayFormulaPosition)?.push(resultPosition);
       }
       hasArrayFormulaResult(position) {
           return this.resultsToArrayFormulas.has(position);
@@ -47524,7 +47548,7 @@ stores.inject(MyMetaStore, storeInstance);
       evaluatedCells = new PositionMap();
       formulaDependencies = lazy(new FormulaDependencyGraph(this.createEmptyPositionSet.bind(this)));
       blockedArrayFormulas = new PositionSet({});
-      spreadingRelations = new SpreadingRelation(this.createEmptyPositionSet.bind(this));
+      spreadingRelations = new SpreadingRelation();
       constructor(context, getters) {
           this.context = context;
           this.getters = getters;
@@ -47601,7 +47625,7 @@ stores.inject(MyMetaStore, storeInstance);
       }
       buildDependencyGraph() {
           this.blockedArrayFormulas = this.createEmptyPositionSet();
-          this.spreadingRelations = new SpreadingRelation(this.createEmptyPositionSet.bind(this));
+          this.spreadingRelations = new SpreadingRelation();
           this.formulaDependencies = lazy(() => {
               const dependencies = [...this.getAllCells()].flatMap((position) => this.getDirectDependencies(position)
                   .filter((range) => !range.invalidSheetName && !range.invalidXc)
@@ -48090,16 +48114,22 @@ stores.inject(MyMetaStore, storeInstance);
               let newContent = undefined;
               let newFormat = undefined;
               let isExported = true;
+              const exportedSheetData = data.sheets.find((sheet) => sheet.id === position.sheetId);
               const formulaCell = this.getCorrespondingFormulaCell(position);
               if (formulaCell) {
                   isExported = isExportableToExcel(formulaCell.compiledFormula.tokens);
                   isFormula = isExported;
                   if (!isExported) {
-                      newContent = (value ?? "").toString();
-                      newFormat = evaluatedCell.format;
+                      // 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 !== "") {
+                          newContent = (value ?? "").toString();
+                          newFormat = evaluatedCell.format;
+                      }
                   }
               }
-              const exportedSheetData = data.sheets.find((sheet) => sheet.id === position.sheetId);
               const exportedCellData = exportedSheetData.cells[xc] || {};
               const format = newFormat
                   ? getItemId(newFormat, data.formats)
@@ -52089,7 +52119,7 @@ stores.inject(MyMetaStore, storeInstance);
       paintFormatStatus = "inactive";
       originSheetId;
       copiedData;
-      _isCutOperation;
+      _isCutOperation = false;
       // ---------------------------------------------------------------------------
       // Command Handling
       // ---------------------------------------------------------------------------
@@ -52101,14 +52131,17 @@ stores.inject(MyMetaStore, storeInstance);
               case "PASTE_FROM_OS_CLIPBOARD": {
                   const copiedData = this.convertOSClipboardData(cmd.text);
                   const pasteOption = cmd.pasteOption || (this.paintFormatStatus !== "inactive" ? "onlyFormat" : undefined);
-                  return this.isPasteAllowed(cmd.target, copiedData, { pasteOption });
+                  return this.isPasteAllowed(cmd.target, copiedData, { pasteOption, isCutOperation: false });
               }
               case "PASTE": {
                   if (!this.copiedData) {
                       return "EmptyClipboard" /* CommandResult.EmptyClipboard */;
                   }
                   const pasteOption = cmd.pasteOption || (this.paintFormatStatus !== "inactive" ? "onlyFormat" : undefined);
-                  return this.isPasteAllowed(cmd.target, this.copiedData, { pasteOption });
+                  return this.isPasteAllowed(cmd.target, this.copiedData, {
+                      pasteOption: pasteOption,
+                      isCutOperation: this._isCutOperation,
+                  });
               }
               case "COPY_PASTE_CELLS_ABOVE": {
                   const zones = this.getters.getSelectedZones();
@@ -52126,13 +52159,13 @@ stores.inject(MyMetaStore, storeInstance);
               }
               case "INSERT_CELL": {
                   const { cut, paste } = this.getInsertCellsTargets(cmd.zone, cmd.shiftDimension);
-                  const copiedData = this.copy("CUT", cut);
-                  return this.isPasteAllowed(paste, copiedData, {});
+                  const copiedData = this.copy(cut);
+                  return this.isPasteAllowed(paste, copiedData, { isCutOperation: true });
               }
               case "DELETE_CELL": {
                   const { cut, paste } = this.getDeleteCellsTargets(cmd.zone, cmd.shiftDimension);
-                  const copiedData = this.copy("CUT", cut);
-                  return this.isPasteAllowed(paste, copiedData, {});
+                  const copiedData = this.copy(cut);
+                  return this.isPasteAllowed(paste, copiedData, { isCutOperation: true });
               }
               case "ACTIVATE_PAINT_FORMAT": {
                   if (this.paintFormatStatus !== "inactive") {
@@ -52150,23 +52183,27 @@ stores.inject(MyMetaStore, storeInstance);
                   const zones = this.getters.getSelectedZones();
                   this.status = "visible";
                   this.originSheetId = this.getters.getActiveSheetId();
-                  this.copiedData = this.copy(cmd.type, zones);
+                  this.copiedData = this.copy(zones);
+                  this._isCutOperation = cmd.type === "CUT";
                   break;
               case "PASTE_FROM_OS_CLIPBOARD": {
+                  this._isCutOperation = false;
                   this.copiedData = this.convertOSClipboardData(cmd.text);
                   const pasteOption = cmd.pasteOption || (this.paintFormatStatus !== "inactive" ? "onlyFormat" : undefined);
-                  this.paste(cmd.target, {
+                  this.paste(cmd.target, this.copiedData, {
                       pasteOption,
                       selectTarget: true,
+                      isCutOperation: false,
                   });
                   this.status = "invisible";
                   break;
               }
               case "PASTE": {
                   const pasteOption = cmd.pasteOption || (this.paintFormatStatus !== "inactive" ? "onlyFormat" : undefined);
-                  this.paste(cmd.target, {
+                  this.paste(cmd.target, this.copiedData, {
                       pasteOption,
                       selectTarget: true,
+                      isCutOperation: this._isCutOperation,
                   });
                   if (this.paintFormatStatus === "oneOff") {
                       this.paintFormatStatus = "inactive";
@@ -52187,9 +52224,9 @@ stores.inject(MyMetaStore, storeInstance);
                           top: multipleRowsInSelection ? zone.top : zone.top - 1,
                       };
                       this.originSheetId = this.getters.getActiveSheetId();
-                      this.copiedData = this.copy("COPY", [copyTarget]);
-                      this.paste([zone], {
-                          pasteOption: undefined,
+                      const copiedData = this.copy([copyTarget]);
+                      this.paste([zone], copiedData, {
+                          isCutOperation: false,
                           selectTarget: true,
                       });
                   }
@@ -52204,9 +52241,9 @@ stores.inject(MyMetaStore, storeInstance);
                           left: multipleColsInSelection ? zone.left : zone.left - 1,
                       };
                       this.originSheetId = this.getters.getActiveSheetId();
-                      this.copiedData = this.copy("COPY", [copyTarget]);
-                      this.paste([zone], {
-                          pasteOption: undefined,
+                      const copiedData = this.copy([copyTarget]);
+                      this.paste([zone], copiedData, {
+                          isCutOperation: false,
                           selectTarget: true,
                       });
                   }
@@ -52222,14 +52259,14 @@ stores.inject(MyMetaStore, storeInstance);
                       }
                       break;
                   }
-                  this.copiedData = this.copy("CUT", cut);
-                  this.paste(paste, {});
+                  const copiedData = this.copy(cut);
+                  this.paste(paste, copiedData, { isCutOperation: true });
                   break;
               }
               case "INSERT_CELL": {
                   const { cut, paste } = this.getInsertCellsTargets(cmd.zone, cmd.shiftDimension);
-                  this.copiedData = this.copy("CUT", cut);
-                  this.paste(paste, {});
+                  const copiedData = this.copy(cut);
+                  this.paste(paste, copiedData, { isCutOperation: true });
                   break;
               }
               case "ADD_COLUMNS_ROWS": {
@@ -52261,7 +52298,8 @@ stores.inject(MyMetaStore, storeInstance);
                   break;
               }
               case "REPEAT_PASTE": {
-                  this.paste(cmd.target, {
+                  this.paste(cmd.target, this.copiedData, {
+                      isCutOperation: false,
                       pasteOption: cmd.pasteOption,
                       selectTarget: true,
                   });
@@ -52269,7 +52307,7 @@ stores.inject(MyMetaStore, storeInstance);
               }
               case "ACTIVATE_PAINT_FORMAT": {
                   const zones = this.getters.getSelectedZones();
-                  this.copiedData = this.copy("COPY", zones);
+                  this.copiedData = this.copy(zones);
                   this.status = "visible";
                   if (cmd.persistent) {
                       this.paintFormatStatus = "persistent";
@@ -52300,7 +52338,6 @@ stores.inject(MyMetaStore, storeInstance);
           }
       }
       convertOSClipboardData(clipboardData) {
-          this._isCutOperation = false;
           const handlers = clipboardHandlersRegistries.figureHandlers
               .getAll()
               .map((handler) => new handler(this.getters, this.dispatch));
@@ -52338,7 +52375,6 @@ stores.inject(MyMetaStore, storeInstance);
           for (const handler of this.selectClipboardHandlers(copiedData)) {
               const result = handler.isPasteAllowed(this.getters.getActiveSheetId(), target, copiedData, {
                   ...options,
-                  isCutOperation: this.isCutOperation(),
               });
               if (result !== "Success" /* CommandResult.Success */) {
                   return result;
@@ -52361,9 +52397,8 @@ stores.inject(MyMetaStore, storeInstance);
           }
           return false;
       }
-      copy(operation, zones) {
+      copy(zones) {
           let copiedData = {};
-          this._isCutOperation = operation === "CUT";
           const clipboardData = this.getClipboardData(zones);
           for (const handler of this.selectClipboardHandlers(clipboardData)) {
               const data = handler.copy(clipboardData);
@@ -52371,8 +52406,8 @@ stores.inject(MyMetaStore, storeInstance);
           }
           return copiedData;
       }
-      paste(zones, options) {
-          if (!this.copiedData) {
+      paste(zones, copiedData, options) {
+          if (!copiedData) {
               return;
           }
           let zone = undefined;
@@ -52380,12 +52415,9 @@ stores.inject(MyMetaStore, storeInstance);
           let target = {
               zones,
           };
-          const handlers = this.selectClipboardHandlers(this.copiedData);
+          const handlers = this.selectClipboardHandlers(copiedData);
           for (const handler of handlers) {
-              const currentTarget = handler.getPasteTarget(zones, this.copiedData, {
-                  ...options,
-                  isCutOperation: this.isCutOperation(),
-              });
+              const currentTarget = handler.getPasteTarget(zones, copiedData, options);
               if (currentTarget.figureId) {
                   target.figureId = currentTarget.figureId;
               }
@@ -52401,7 +52433,7 @@ stores.inject(MyMetaStore, storeInstance);
           if (zone !== undefined) {
               this.addMissingDimensions(this.getters.getActiveSheetId(), zone.right - zone.left + 1, zone.bottom - zone.top + 1, zone.left, zone.top);
           }
-          handlers.forEach((handler) => handler.paste(target, this.copiedData, { ...options, isCutOperation: this.isCutOperation() }));
+          handlers.forEach((handler) => handler.paste(target, copiedData, options));
           if (!options?.selectTarget) {
               return;
           }
@@ -54871,6 +54903,7 @@ stores.inject(MyMetaStore, storeInstance);
       sheetDivRef = owl.useRef("sheetDiv");
       sheetNameRef = owl.useRef("sheetNameSpan");
       editionState = "initializing";
+      DOMFocusableElementStore;
       setup() {
           owl.onMounted(() => {
               if (this.isSheetActive) {
@@ -54883,6 +54916,7 @@ stores.inject(MyMetaStore, storeInstance);
                   this.focusInputAndSelectContent();
               }
           });
+          this.DOMFocusableElementStore = useStore(DOMFocusableElementStore);
       }
       focusInputAndSelectContent() {
           if (!this.state.isEditing || !this.sheetNameRef.el)
@@ -54924,9 +54958,11 @@ stores.inject(MyMetaStore, storeInstance);
           if (ev.key === "Enter") {
               ev.preventDefault();
               this.stopEdition();
+              this.DOMFocusableElementStore.focus();
           }
           if (ev.key === "Escape") {
               this.cancelEdition();
+              this.DOMFocusableElementStore.focus();
           }
       }
       onClickSheetName(ev) {
@@ -59649,7 +59685,11 @@ stores.inject(MyMetaStore, storeInstance);
                   let cellNode = escapeXml ``;
                   // Either formula or static value inside the cell
                   if (cell.isFormula) {
-                      ({ attrs: additionalAttrs, node: cellNode } = addFormula(cell));
+                      const res = addFormula(cell);
+                      if (!res) {
+                          continue;
+                      }
+                      ({ attrs: additionalAttrs, node: cellNode } = res);
                   }
                   else if (cell.content && isMarkdownLink(cell.content)) {
                       const { label } = parseMarkdownLink(cell.content);
@@ -60729,9 +60769,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.2.8";
-  __info__.date = "2024-05-24T11:29:49.320Z";
-  __info__.hash = "bfbcaa0";
+  __info__.version = "17.2.10";
+  __info__.date = "2024-06-10T09:38:54.354Z";
+  __info__.hash = "14317ee";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);