@odoo/o-spreadsheet 17.2.6 → 17.2.8

package/dist/o-spreadsheet.esm.js

@@ -3,12 +3,12 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.2.6
- * @date 2024-05-07T10:41:20.332Z
- * @hash a4f800c
+ * @version 17.2.8
+ * @date 2024-05-24T11:29:49.320Z
+ * @hash bfbcaa0
  */
 
-import { reactive, useEnv, useSubEnv, useState, onWillUnmount, markRaw, toRaw, Component, useRef, onMounted, useEffect, onPatched, useComponent, onWillPatch, onWillUpdateProps, useExternalListener, onWillStart, xml, useChildSubEnv } from '@odoo/owl';
+import { useEnv, useSubEnv, onWillUnmount, useComponent, status, Component, useRef, onMounted, useEffect, useState, onPatched, onWillPatch, onWillUpdateProps, useExternalListener, onWillStart, xml, useChildSubEnv, markRaw } from '@odoo/owl';
 
 const CANVAS_SHIFT = 0.5;
 // Colors
@@ -502,6 +502,26 @@ function debounce(func, wait, immediate) {
     };
     return debounced;
 }
+/**
+ * Creates a batched version of a callback so that all calls to it in the same
+ * microtick will only call the original callback once.
+ *
+ * @param callback the callback to batch
+ * @returns a batched version of the original callback
+ *
+ * Copied from odoo/owl repo.
+ */
+function batched(callback) {
+    let scheduled = false;
+    return async (...args) => {
+        if (!scheduled) {
+            scheduled = true;
+            await Promise.resolve();
+            scheduled = false;
+            callback(...args);
+        }
+    };
+}
 /*
  * Concatenate an array of strings.
  */
@@ -576,8 +596,9 @@ function deepEquals(o1, o2, ignoreFunctions) {
                 return false;
         }
         else {
-            if (ignoreFunctions && typeOfO1Key === "function")
-                return true;
+            if (ignoreFunctions && typeOfO1Key === "function") {
+                continue;
+            }
             if (o1[key] !== o2[key])
                 return false;
         }
@@ -1828,7 +1849,7 @@ class LazyTranslatedString extends String {
     }
     valueOf() {
         const str = super.valueOf();
-        return _loaded() ? sprintf(_translate(str), ...this.values) : str;
+        return _loaded() ? sprintf(_translate(str), ...this.values) : sprintf(str, ...this.values);
     }
     toString() {
         return this.valueOf();
@@ -5201,7 +5222,7 @@ function tokenizeString(chars) {
     }
     return null;
 }
-const separatorRegexp = /^[\w\.!\$]+/;
+const SYMBOL_CHARS = new Set("ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789_.!$");
 /**
  * A "Symbol" is just basically any word-like element that can appear in a
  * formula, which is not a string. So:
@@ -5241,11 +5262,8 @@ function tokenizeSymbol(chars) {
             };
         }
     }
-    const match = chars.remaining().match(separatorRegexp);
-    if (match) {
-        const value = match[0];
-        result += value;
-        chars.advanceBy(value.length);
+    while (chars.current && SYMBOL_CHARS.has(chars.current)) {
+        result += chars.shift();
     }
     if (result.length) {
         const value = result;
@@ -6797,7 +6815,7 @@ const machine = {
 function matchReference(tokens) {
     let head = 0;
     let transitions = machine[State.LeftRef];
-    const matchedTokens = [];
+    let matchedTokens = "";
     while (transitions !== undefined) {
         const token = tokens[head++];
         if (!token) {
@@ -6809,15 +6827,15 @@ function matchReference(tokens) {
             case undefined:
                 return null;
             case State.Found:
-                matchedTokens.push(token);
+                matchedTokens += token.value;
                 tokens.splice(0, head);
                 return {
                     type: "REFERENCE",
-                    value: concat(matchedTokens.map((token) => token.value)),
+                    value: matchedTokens,
                 };
             default:
                 transitions = machine[nextState];
-                matchedTokens.push(token);
+                matchedTokens += token.value;
                 break;
         }
     }
@@ -8015,10 +8033,66 @@ function getDateCriterionFormattedValues(criterion, getters) {
     });
 }
 
+/**
+ * This is a generic event bus based on the Owl event bus.
+ * This bus however ensures type safety across events and subscription callbacks.
+ */
+class EventBus {
+    subscriptions = {};
+    /**
+     * Add a listener for the 'eventType' events.
+     *
+     * Note that the 'owner' of this event can be anything, but will more likely
+     * be a component or a class. The idea is that the callback will be called with
+     * the proper owner bound.
+     *
+     * Also, the owner should be kind of unique. This will be used to remove the
+     * listener.
+     */
+    on(type, owner, callback) {
+        if (!callback) {
+            throw new Error("Missing callback");
+        }
+        if (!this.subscriptions[type]) {
+            this.subscriptions[type] = [];
+        }
+        this.subscriptions[type].push({
+            owner,
+            callback,
+        });
+    }
+    /**
+     * Emit an event of type 'eventType'.  Any extra arguments will be passed to
+     * the listeners callback.
+     */
+    trigger(type, payload) {
+        const subs = this.subscriptions[type] || [];
+        for (let i = 0, iLen = subs.length; i < iLen; i++) {
+            const sub = subs[i];
+            sub.callback.call(sub.owner, payload);
+        }
+    }
+    /**
+     * Remove a listener
+     */
+    off(eventType, owner) {
+        const subs = this.subscriptions[eventType];
+        if (subs) {
+            this.subscriptions[eventType] = subs.filter((s) => s.owner !== owner);
+        }
+    }
+    /**
+     * Remove all subscriptions.
+     */
+    clear() {
+        this.subscriptions = {};
+    }
+}
+
 /**
  * A type-safe dependency container
  */
-class DependencyContainer {
+class DependencyContainer extends EventBus {
     dependencies = new Map();
     factory = new StoreFactory(this.get.bind(this));
     /**
@@ -8095,15 +8169,12 @@ stores.inject(MyMetaStore, storeInstance);
     }
     return MetaStore;
 }
-class ReactiveStore {
+class DisposableStore {
     get;
+    disposeCallbacks = [];
     constructor(get) {
         this.get = get;
-        return reactive(this);
     }
-}
-class DisposableStore extends ReactiveStore {
-    disposeCallbacks = [];
     onDispose(callback) {
         this.disposeCallbacks.push(callback);
     }
@@ -8123,7 +8194,10 @@ function useStoreProvider() {
     const container = new DependencyContainer();
     useSubEnv({
         __spreadsheet_stores__: container,
-        getStore: container.get.bind(container),
+        getStore: (Store) => {
+            const store = container.get(Store);
+            return proxifyStoreMutation(store, () => container.trigger("store-updated"));
+        },
     });
     return container;
 }
@@ -8133,14 +8207,57 @@ function useStoreProvider() {
 function useStore(Store) {
     const env = useEnv();
     const container = getDependencyContainer(env);
-    return useState(container.get(Store));
+    const store = container.get(Store);
+    return useStoreRenderProxy(container, store);
 }
 function useLocalStore(Store, ...args) {
     const env = useEnv();
     const container = getDependencyContainer(env);
-    const store = useState(container.instantiate(Store, ...args));
+    const store = container.instantiate(Store, ...args);
     onWillUnmount(() => store.dispose());
-    return store;
+    return useStoreRenderProxy(container, store);
+}
+/**
+ * Trigger an event to re-render the app (deep render) when
+ * a store is mutated by invoking one of its mutator methods.
+ */
+function useStoreRenderProxy(container, store) {
+    const component = useComponent();
+    const proxy = proxifyStoreMutation(store, () => {
+        if (status(component) === "mounted") {
+            container.trigger("store-updated");
+        }
+    });
+    return proxy;
+}
+/**
+ * Creates a proxied version of a store object with mutation tracking.
+ * Whenever a mutator method of the store is called, the provided callback function is invoked.
+ */
+function proxifyStoreMutation(store, callback) {
+    const proxy = new Proxy(store, {
+        get(target, property, receiver) {
+            const thisStore = target;
+            // The third argument is `thisStore` (target) instead of `receiver`.
+            // The goal is to always have the same `this` value in getter functions
+            // (when `target[property]` is an accessor property).
+            // `thisStore` is always the same object reference. `receiver` however is the
+            // object on which the property is called, which is the Proxy object which is different for each component.
+            const value = Reflect.get(target, property, thisStore);
+            if (store.mutators.includes(property)) {
+                const functionProxy = new Proxy(value, {
+                    // trap the function call
+                    apply(target, thisArg, argArray) {
+                        Reflect.apply(target, thisStore, argArray);
+                        callback();
+                    },
+                });
+                return functionProxy;
+            }
+            return value;
+        },
+    });
+    return proxy;
 }
 function getDependencyContainer(env) {
     const container = env.__spreadsheet_stores__;
@@ -8152,7 +8269,8 @@ function getDependencyContainer(env) {
 
 const ModelStore = createAbstractStore("Model");
 
-class RendererStore extends ReactiveStore {
+class RendererStore {
+    mutators = ["register", "unRegister"];
     renderers = {};
     register(renderer) {
         if (!renderer.renderingLayers.length) {
@@ -8186,7 +8304,7 @@ class RendererStore extends ReactiveStore {
 class SpreadsheetStore extends DisposableStore {
     // cast the model store as Model to allow model.dispatch to return the DispatchResult
     model = this.get(ModelStore);
-    getters = markRaw(this.model.getters);
+    getters = this.model.getters;
     renderer = this.get(RendererStore);
     constructor(get) {
         super(get);
@@ -8233,6 +8351,7 @@ function drawHighlight(renderingContext, highlight, rect) {
 }
 
 class HighlightStore extends SpreadsheetStore {
+    mutators = ["register", "unRegister"];
     providers = [];
     constructor(get) {
         super(get);
@@ -8263,7 +8382,7 @@ class HighlightStore extends SpreadsheetStore {
         this.providers.push(highlightProvider);
     }
     unRegister(highlightProvider) {
-        this.providers = this.providers.filter((h) => toRaw(h) !== toRaw(highlightProvider));
+        this.providers = this.providers.filter((h) => h !== highlightProvider);
     }
     drawLayer(ctx, layer) {
         if (layer === "Highlights") {
@@ -8279,6 +8398,16 @@ const NotificationStore = createAbstractStore("Notifications");
 
 const CELL_DELETED_MESSAGE = _t("The cell you are trying to edit has been deleted.");
 class ComposerStore extends SpreadsheetStore {
+    mutators = [
+        "startEdition",
+        "setCurrentContent",
+        "stopEdition",
+        "stopComposerRangeSelection",
+        "cancelEdition",
+        "cycleReferences",
+        "changeComposerCursorSelection",
+        "replaceComposerCursorSelection",
+    ];
     col = 0;
     row = 0;
     editionMode = "inactive";
@@ -8293,9 +8422,9 @@ class ComposerStore extends SpreadsheetStore {
     highlightStore = this.get(HighlightStore);
     constructor(get) {
         super(get);
-        this.highlightStore.register(toRaw(this));
+        this.highlightStore.register(this);
         this.onDispose(() => {
-            this.highlightStore.unRegister(toRaw(this));
+            this.highlightStore.unRegister(this);
         });
     }
     canStopEdition() {
@@ -8422,7 +8551,7 @@ class ComposerStore extends SpreadsheetStore {
                 if (this.isSelectingRange) {
                     this.editionMode = "editing";
                 }
-                this.model.selection.resetAnchor(toRaw(this), {
+                this.model.selection.resetAnchor(this, {
                     cell: { col: left, row: top },
                     zone: cmd.zone,
                 });
@@ -8440,7 +8569,7 @@ class ComposerStore extends SpreadsheetStore {
                         row: activePosition.row,
                     });
                     const zone = this.getters.expandZone(cmd.sheetIdTo, positionToZone({ col, row }));
-                    this.model.selection.resetAnchor(toRaw(this), { cell: { col, row }, zone });
+                    this.model.selection.resetAnchor(this, { cell: { col, row }, zone });
                 }
                 break;
             case "DELETE_SHEET":
@@ -8539,7 +8668,7 @@ class ComposerStore extends SpreadsheetStore {
     startComposerRangeSelection() {
         if (this.sheetId === this.getters.getActiveSheetId()) {
             const zone = positionToZone({ col: this.col, row: this.row });
-            this.model.selection.resetAnchor(toRaw(this), {
+            this.model.selection.resetAnchor(this, {
                 cell: { col: this.col, row: this.row },
                 zone,
             });
@@ -8568,7 +8697,7 @@ class ComposerStore extends SpreadsheetStore {
         this.setContent(str || this.initialContent, selection);
         this.colorIndexByRange = {};
         const zone = positionToZone({ col: this.col, row: this.row });
-        this.model.selection.capture(toRaw(this), { cell: { col: this.col, row: this.row }, zone }, {
+        this.model.selection.capture(this, { cell: { col: this.col, row: this.row }, zone }, {
             handleEvent: this.handleEvent.bind(this),
             release: () => {
                 this._stopEdition();
@@ -8676,7 +8805,7 @@ class ComposerStore extends SpreadsheetStore {
             return;
         }
         this.editionMode = "inactive";
-        this.model.selection.release(toRaw(this));
+        this.model.selection.release(this);
     }
     /**
      * Reset the current content to the active cell content
@@ -8998,6 +9127,7 @@ class ComposerStore extends SpreadsheetStore {
 }
 
 class ComposerFocusStore extends SpreadsheetStore {
+    mutators = ["focusTopBarComposer", "focusGridComposerContent", "focusGridComposerCell"];
     composerStore = this.get(ComposerStore);
     topBarFocus = "inactive";
     gridFocusMode = "inactive";
@@ -21454,6 +21584,7 @@ function interactiveAddMerge(env, sheetId, target) {
 }
 
 class HoveredCellStore extends SpreadsheetStore {
+    mutators = ["clear", "hover"];
     col;
     row;
     handle(cmd) {
@@ -21473,6 +21604,7 @@ class HoveredCellStore extends SpreadsheetStore {
 }
 
 class CellPopoverStore extends SpreadsheetStore {
+    mutators = ["open", "close"];
     persistentPopover;
     hoveredCell = this.get(HoveredCellStore);
     handle(cmd) {
@@ -25964,12 +26096,13 @@ function updateSelectionWithArrowKeys(ev, selection) {
 
 // The name is misleading and can be confused with the DOM focus.
 class FocusStore {
+    mutators = ["focus", "unfocus"];
     focusedElement = null;
     focus(element) {
         this.focusedElement = element;
     }
     unfocus(element) {
-        if (this.focusedElement && toRaw(this.focusedElement) === toRaw(element)) {
+        if (this.focusedElement && this.focusedElement === element) {
             this.focusedElement = null;
         }
     }
@@ -25985,6 +26118,16 @@ class FocusStore {
 class SelectionInputStore extends SpreadsheetStore {
     initialRanges;
     inputHasSingleRange;
+    mutators = [
+        "resetWithRanges",
+        "focusById",
+        "unfocus",
+        "addEmptyRange",
+        "removeRange",
+        "changeRange",
+        "reset",
+        "confirm",
+    ];
     ranges = [];
     focusedRangeIndex = null;
     inputSheetId;
@@ -26044,7 +26187,7 @@ class SelectionInputStore extends SpreadsheetStore {
                         row: 0,
                     });
                     const zone = this.getters.expandZone(cmd.sheetIdTo, positionToZone({ col, row }));
-                    this.model.selection.resetAnchor(toRaw(this), { cell: { col, row }, zone });
+                    this.model.selection.resetAnchor(this, { cell: { col, row }, zone });
                 }
                 break;
             }
@@ -26063,7 +26206,7 @@ class SelectionInputStore extends SpreadsheetStore {
                 if (focusIndex !== -1) {
                     this.focus(focusIndex);
                     const { left, top } = newZone;
-                    this.model.selection.resetAnchor(toRaw(this), {
+                    this.model.selection.resetAnchor(this, {
                         cell: { col: left, row: top },
                         zone: newZone,
                     });
@@ -26154,7 +26297,7 @@ class SelectionInputStore extends SpreadsheetStore {
     }
     get hasMainFocus() {
         const focusedElement = this.focusStore.focusedElement;
-        return !!focusedElement && toRaw(focusedElement) === toRaw(this);
+        return !!focusedElement && focusedElement === this;
     }
     get highlights() {
         if (!this.hasMainFocus) {
@@ -26183,7 +26326,7 @@ class SelectionInputStore extends SpreadsheetStore {
     unfocus() {
         this.focusedRangeIndex = null;
         this.focusStore.unfocus(this);
-        this.model.selection.release(toRaw(this));
+        this.model.selection.release(this);
     }
     captureSelection() {
         if (this.focusedRangeIndex === null) {
@@ -26192,7 +26335,7 @@ class SelectionInputStore extends SpreadsheetStore {
         const range = this.ranges[this.focusedRangeIndex];
         const sheetId = this.getters.getActiveSheetId();
         const zone = this.getters.getRangeFromSheetXC(sheetId, range?.xc || "A1").zone;
-        this.model.selection.capture(toRaw(this), { cell: { col: zone.left, row: zone.top }, zone }, {
+        this.model.selection.capture(this, { cell: { col: zone.left, row: zone.top }, zone }, {
             handleEvent: this.handleEvent.bind(this),
             release: this.unfocus.bind(this),
         });
@@ -27660,6 +27803,7 @@ chartSidePanelComponentRegistry
 });
 
 class MainChartPanelStore extends SpreadsheetStore {
+    mutators = ["activatePanel"];
     panel = "configuration";
     activatePanel(panel) {
         this.panel = panel;
@@ -28198,14 +28342,14 @@ function useHoveredElement(ref) {
 
 function useHighlightsOnHover(ref, highlightProvider) {
     const hoverState = useHoveredElement(ref);
-    const env = useEnv();
+    const stores = useStoreProvider();
     useHighlights({
         get highlights() {
             return hoverState.hovered ? highlightProvider.highlights : [];
         },
     });
     useEffect(() => {
-        env.model.dispatch("RENDER_CANVAS");
+        stores.trigger("store-updated");
     }, () => [hoverState.hovered]);
 }
 function useHighlights(highlightProvider) {
@@ -29661,6 +29805,14 @@ var Direction;
     Direction[Direction["next"] = 1] = "next";
 })(Direction || (Direction = {}));
 class FindAndReplaceStore extends SpreadsheetStore {
+    mutators = [
+        "updateSearchOptions",
+        "updateSearchContent",
+        "searchFormulas",
+        "selectPreviousMatch",
+        "selectNextMatch",
+        "replace",
+    ];
     allSheetsMatches = [];
     activeSheetMatches = [];
     specificRangeMatches = [];
@@ -29685,11 +29837,11 @@ class FindAndReplaceStore extends SpreadsheetStore {
         this.initialShowFormulaState = this.model.getters.shouldShowFormulas();
         this.searchOptions.searchFormulas = this.initialShowFormulaState;
         const highlightStore = get(HighlightStore);
-        highlightStore.register(toRaw(this));
+        highlightStore.register(this);
         this.onDispose(() => {
             this.model.dispatch("SET_FORMULA_VISIBILITY", { show: this.initialShowFormulaState });
             this.updateSearchContent.stopDebounce();
-            highlightStore.unRegister(toRaw(this));
+            highlightStore.unRegister(this);
         });
     }
     get searchMatches() {
@@ -31332,6 +31484,7 @@ unGroupHeadersMenuRegistry
 });
 
 class DOMFocusableElementStore {
+    mutators = ["setFocusableElement", "focus"];
     focusableElement = undefined;
     setFocusableElement(element) {
         this.focusableElement = element;
@@ -32068,7 +32221,7 @@ class Composer extends Component {
         "Ctrl+Enter": this.processNewLineEvent,
         Escape: this.processEscapeKey,
         F2: () => console.warn("Not implemented"),
-        F4: this.processF4Key,
+        F4: (ev) => this.processF4Key(ev),
         Tab: (ev) => this.processTabKey(ev, "right"),
         "Shift+Tab": (ev) => this.processTabKey(ev, "left"),
     };
@@ -32190,7 +32343,8 @@ class Composer extends Component {
     processEscapeKey() {
         this.composerStore.cancelEdition();
     }
-    processF4Key() {
+    processF4Key(ev) {
+        ev.stopPropagation();
         this.composerStore.cycleReferences();
         this.processContent();
     }
@@ -34393,13 +34547,6 @@ class GridRenderer {
         this.getters = get(ModelStore).getters;
         this.renderer = get(RendererStore);
         this.renderer.register(this);
-        /**
-         * Mark the instance as raw to avoid reactivity as this class is instanciated
-         * as a Store by `useGridDrawing` (which casts it as reactive).
-         *
-         * Calling `this.` on a reactive instance is significantly slower than on a raw object.
-         */
-        markRaw(this);
     }
     get renderingLayers() {
         return ["Background", "Headers"];
@@ -35029,7 +35176,7 @@ class GridRenderer {
 function useGridDrawing(refName, model, canvasSize) {
     const canvasRef = useRef(refName);
     useEffect(drawGrid);
-    const rendererManager = useStore(RendererStore);
+    const rendererStore = useStore(RendererStore);
     useStore(GridRenderer);
     function drawGrid() {
         const canvas = canvasRef.el;
@@ -35057,7 +35204,11 @@ function useGridDrawing(refName, model, canvasSize) {
         ctx.scale(dpr, dpr);
         for (const layer of OrderedLayers()) {
             model.drawLayer(renderingContext, layer);
-            rendererManager.drawLayer(renderingContext, layer);
+            // @ts-ignore 'drawLayer' is not declated as a mutator because:
+            // it does not mutate anything. Most importantly it's used
+            // during rendering. Invoking a mutator during rendering would
+            // trigger another rendering, ultimately resulting in an infinite loop.
+            rendererStore.drawLayer(renderingContext, layer);
         }
     }
 }
@@ -35459,6 +35610,7 @@ class VerticalScrollBar extends Component {
 }
 
 class SidePanelStore extends SpreadsheetStore {
+    mutators = ["open", "toggle", "close"];
     initialPanelProps = {};
     componentTag = "";
     get isOpen() {
@@ -43287,7 +43439,12 @@ class MergePlugin extends CorePlugin {
      * if they have at least a common cell
      */
     doesIntersectMerge(sheetId, zone) {
-        return positions(zone).some(({ col, row }) => this.getMerge({ sheetId, col, row }) !== undefined);
+        for (const merge of this.getMerges(sheetId)) {
+            if (overlap(zone, merge)) {
+                return true;
+            }
+        }
+        return false;
     }
     /**
      * Returns true if two columns have at least one merge in common
@@ -47435,9 +47592,10 @@ class Evaluator {
             }
             if (!content) {
                 // The previous content could have blocked some array formulas
-                impactedPositions.addMany(this.getArrayFormulasBlockedBy(position));
+                impactedPositions.add(position);
             }
         }
+        impactedPositions.addMany(this.getArrayFormulasBlockedBy(impactedPositions));
         return impactedPositions;
     }
     buildDependencyGraph() {
@@ -47479,23 +47637,25 @@ class Evaluator {
         return positions;
     }
     /**
-     * Return the position of formulas blocked by the given position
+     * Return the position of formulas blocked by the given positions
      * as well as all their dependencies.
      */
-    getArrayFormulasBlockedBy(position) {
-        if (!this.spreadingRelations.hasArrayFormulaResult(position)) {
-            return [];
-        }
-        const arrayFormulas = this.spreadingRelations.getFormulaPositionsSpreadingOn(position);
-        const positions = this.createEmptyPositionSet();
-        positions.addMany(arrayFormulas);
-        const arrayFormulaPosition = this.getArrayFormulaSpreadingOn(position);
-        if (arrayFormulaPosition) {
-            // ignore the formula spreading on the position. Keep only the blocked ones
-            positions.delete(arrayFormulaPosition);
+    getArrayFormulasBlockedBy(positions) {
+        const arrayFormulaPositions = this.createEmptyPositionSet();
+        for (const position of positions) {
+            if (!this.spreadingRelations.hasArrayFormulaResult(position)) {
+                continue;
+            }
+            const arrayFormulas = this.spreadingRelations.getFormulaPositionsSpreadingOn(position);
+            arrayFormulaPositions.addMany(arrayFormulas);
+            const arrayFormulaPosition = this.getArrayFormulaSpreadingOn(position);
+            if (arrayFormulaPosition) {
+                // ignore the formula spreading on the position. Keep only the blocked ones
+                arrayFormulaPositions.delete(arrayFormulaPosition);
+            }
         }
-        positions.addMany(this.getCellsDependingOn(positions));
-        return positions;
+        arrayFormulaPositions.addMany(this.getCellsDependingOn(arrayFormulaPositions));
+        return arrayFormulaPositions;
     }
     nextPositionsToUpdate = new PositionSet({});
     cellsBeingComputed = new Set();
@@ -47625,6 +47785,7 @@ class Evaluator {
         if (!this.spreadingRelations.isArrayFormula(position)) {
             return;
         }
+        const invalidated = this.createEmptyPositionSet();
         for (const child of this.spreadingRelations.getArrayResultPositions(position)) {
             const content = this.getters.getCell(child)?.content;
             if (content) {
@@ -47632,10 +47793,11 @@ class Evaluator {
                 // there's still a collision
                 continue;
             }
+            invalidated.add(child);
             this.evaluatedCells.delete(child);
-            this.nextPositionsToUpdate.addMany(this.getCellsDependingOn([child]));
-            this.nextPositionsToUpdate.addMany(this.getArrayFormulasBlockedBy(child));
         }
+        this.nextPositionsToUpdate.addMany(this.getCellsDependingOn(invalidated));
+        this.nextPositionsToUpdate.addMany(this.getArrayFormulasBlockedBy(invalidated));
         this.spreadingRelations.removeNode(position);
     }
     // ----------------------------------------------------------
@@ -49602,62 +49764,6 @@ class AutomaticSumPlugin extends UIPlugin {
     }
 }
 
-/**
- * This is a generic event bus based on the Owl event bus.
- * This bus however ensures type safety across events and subscription callbacks.
- */
-class EventBus {
-    subscriptions = {};
-    /**
-     * Add a listener for the 'eventType' events.
-     *
-     * Note that the 'owner' of this event can be anything, but will more likely
-     * be a component or a class. The idea is that the callback will be called with
-     * the proper owner bound.
-     *
-     * Also, the owner should be kind of unique. This will be used to remove the
-     * listener.
-     */
-    on(type, owner, callback) {
-        if (!callback) {
-            throw new Error("Missing callback");
-        }
-        if (!this.subscriptions[type]) {
-            this.subscriptions[type] = [];
-        }
-        this.subscriptions[type].push({
-            owner,
-            callback,
-        });
-    }
-    /**
-     * Emit an event of type 'eventType'.  Any extra arguments will be passed to
-     * the listeners callback.
-     */
-    trigger(type, payload) {
-        const subs = this.subscriptions[type] || [];
-        for (let i = 0, iLen = subs.length; i < iLen; i++) {
-            const sub = subs[i];
-            sub.callback.call(sub.owner, payload);
-        }
-    }
-    /**
-     * Remove a listener
-     */
-    off(eventType, owner) {
-        const subs = this.subscriptions[eventType];
-        if (subs) {
-            this.subscriptions[eventType] = subs.filter((s) => s.owner !== owner);
-        }
-    }
-    /**
-     * Remove all subscriptions.
-     */
-    clear() {
-        this.subscriptions = {};
-    }
-}
-
 /*
  * This file contains the specifics transformations
  */
@@ -52128,7 +52234,7 @@ class ClipboardPlugin extends UIPlugin {
             case "ADD_COLUMNS_ROWS": {
                 this.status = "invisible";
                 // If we add a col/row inside or before the cut area, we invalidate the clipboard
-                if (this._isCutOperation !== true) {
+                if (this._isCutOperation !== true || cmd.sheetId !== this.copiedData?.sheetId) {
                     return;
                 }
                 const isClipboardDirty = this.isColRowDirtyingClipboard(cmd.position === "before" ? cmd.base : cmd.base + 1, cmd.dimension);
@@ -52140,7 +52246,7 @@ class ClipboardPlugin extends UIPlugin {
             case "REMOVE_COLUMNS_ROWS": {
                 this.status = "invisible";
                 // If we remove a col/row inside or before the cut area, we invalidate the clipboard
-                if (this._isCutOperation !== true) {
+                if (this._isCutOperation !== true || cmd.sheetId !== this.copiedData?.sheetId) {
                     return;
                 }
                 for (let el of cmd.elements) {
@@ -56775,16 +56881,25 @@ class Spreadsheet extends Component {
             }
         }, () => [this.env.model.getters.getActiveSheetId()]);
         useExternalListener(window, "resize", () => this.render(true));
+        // For some reason, the wheel event is not properly registered inside templates
+        // in Chromium-based browsers based on chromium 125
+        // This hack ensures the event declared in the template is properly registered/working
+        useExternalListener(document.body, "wheel", () => { });
         this.bindModelEvents();
         onWillUpdateProps((nextProps) => {
             if (nextProps.model !== this.props.model) {
                 throw new Error("Changing the props model is not supported at the moment.");
             }
         });
+        const render = batched(this.render.bind(this, true));
         onMounted(() => {
             this.checkViewportSize();
+            stores.on("store-updated", this, render);
+        });
+        onWillUnmount(() => {
+            this.unbindModelEvents();
+            stores.off("store-updated", this);
         });
-        onWillUnmount(() => this.unbindModelEvents());
         onPatched(() => {
             this.checkViewportSize();
         });
@@ -60572,6 +60687,6 @@ const constants = {
 export { AbstractCellClipboardHandler, AbstractChart, AbstractFigureClipboardHandler, CellErrorType, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, __info__, addFunction, addRenderingLayer, astToFormula, compile, compileTokens, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, hooks, invalidateCFEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, parseTokens, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, stores, tokenColors, tokenize };
 
 
-__info__.version = "17.2.6";
-__info__.date = "2024-05-07T10:41:20.332Z";
-__info__.hash = "a4f800c";
+__info__.version = "17.2.8";
+__info__.date = "2024-05-24T11:29:49.320Z";
+__info__.hash = "bfbcaa0";