@odoo/o-spreadsheet 17.4.5 → 17.4.6

package/dist/o-spreadsheet.cjs.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.4.5
- * @date 2024-08-19T08:12:00.492Z
- * @hash 28579ad
+ * @version 17.4.6
+ * @date 2024-08-26T13:50:49.623Z
+ * @hash f5648d9
  */
 
 'use strict';
@@ -242,6 +242,7 @@ const PIVOT_TABLE_CONFIG = {
     bandedRows: true,
     bandedColumns: false,
     styleId: "TableStyleMedium5",
+    automaticAutofill: true,
 };
 
 //------------------------------------------------------------------------------
@@ -35173,7 +35174,6 @@ class FindAndReplaceStore extends SpreadsheetStore {
         searchScope: "activeSheet",
         specificRange: undefined,
     };
-    updateSearchContent = debounce(this._updateSearchContent.bind(this), 200);
     constructor(get) {
         super(get);
         this.initialShowFormulaState = this.model.getters.shouldShowFormulas();
@@ -35182,7 +35182,6 @@ class FindAndReplaceStore extends SpreadsheetStore {
         highlightStore.register(this);
         this.onDispose(() => {
             this.model.dispatch("SET_FORMULA_VISIBILITY", { show: this.initialShowFormulaState });
-            this.updateSearchContent.stopDebounce();
             highlightStore.unRegister(this);
         });
     }
@@ -35196,7 +35195,7 @@ class FindAndReplaceStore extends SpreadsheetStore {
                 return this.specificRangeMatches;
         }
     }
-    _updateSearchContent(toSearch) {
+    updateSearchContent(toSearch) {
         this._updateSearch(toSearch, this.searchOptions);
     }
     updateSearchOptions(searchOptions) {
@@ -35212,9 +35211,6 @@ class FindAndReplaceStore extends SpreadsheetStore {
     selectNextMatch() {
         this.selectNextCell(Direction.next);
     }
-    get pendingSearch() {
-        return this.updateSearchContent.isDebouncePending();
-    }
     handle(cmd) {
         switch (cmd.type) {
             case "SET_FORMULA_VISIBILITY":
@@ -35540,6 +35536,7 @@ class FindAndReplacePanel extends owl.Component {
     searchInput = owl.useRef("searchInput");
     store;
     state;
+    updateSearchContent;
     get hasSearchResult() {
         return this.store.selectedMatchIndex !== null;
     }
@@ -35571,12 +35568,14 @@ class FindAndReplacePanel extends owl.Component {
         this.store = useLocalStore(FindAndReplaceStore);
         this.state = owl.useState({ dataRange: "" });
         owl.onMounted(() => this.searchInput.el?.focus());
+        owl.onWillUnmount(() => this.updateSearchContent.stopDebounce());
+        this.updateSearchContent = debounce(this.store.updateSearchContent, 200);
     }
     onFocusSearch() {
         this.updateDataRange();
     }
     onSearchInput(ev) {
-        this.store.updateSearchContent(ev.target?.value || "");
+        this.updateSearchContent(ev.target.value);
     }
     onKeydownSearch(ev) {
         if (ev.key === "Enter") {
@@ -35615,6 +35614,9 @@ class FindAndReplacePanel extends owl.Component {
         const specificRange = this.env.model.getters.getRangeFromSheetXC(this.env.model.getters.getActiveSheetId(), this.state.dataRange);
         this.store.updateSearchOptions({ specificRange });
     }
+    get pendingSearch() {
+        return this.updateSearchContent.isDebouncePending();
+    }
 }
 
 css /* scss */ `
@@ -36668,6 +36670,12 @@ function orderDataEntriesKeys(groups, dimension) {
  * Used to order two values
  */
 function compareDimensionValues(dimension, a, b) {
+    if (a === "null") {
+        return dimension.order === "asc" ? 1 : -1;
+    }
+    if (b === "null") {
+        return dimension.order === "asc" ? -1 : 1;
+    }
     if (dimension.type === "integer" || dimension.type === "date") {
         return dimension.order === "asc" ? Number(a) - Number(b) : Number(b) - Number(a);
     }
@@ -37030,7 +37038,7 @@ class SpreadsheetPivot {
         if (cell.type === CellValueType.error) {
             throw new EvaluationError(_t("The pivot cannot be created because cell %s contains an error", toXC(col, row)));
         }
-        if (cell.type === CellValueType.empty) {
+        if (cell.type === CellValueType.empty || cell.value === "") {
             throw new EvaluationError(_t("The pivot cannot be created because cell %s is empty", toXC(col, row)));
         }
         if (cell.value === "__count") {
@@ -37376,15 +37384,11 @@ class PivotSpreadsheetSidePanel extends owl.Component {
     }
     onSelectionConfirmed() {
         if (this.state.range) {
-            const { sheetName, xc } = splitReference(this.state.range);
-            const sheetId = sheetName
-                ? this.env.model.getters.getSheetIdByName(sheetName)
-                : this.env.model.getters.getActiveSheetId();
-            if (!sheetId) {
+            const range = this.env.model.getters.getRangeFromSheetXC(this.env.model.getters.getActiveSheetId(), this.state.range);
+            if (range.invalidSheetName || range.invalidXc) {
                 return;
             }
-            const zone = toZone(xc);
-            const dataSet = { sheetId, zone };
+            const dataSet = { sheetId: range.sheetId, zone: range.zone };
             this.store.update({ dataSet });
             // Immediately apply the update to recompute the pivot fields
             this.store.applyUpdate();
@@ -38317,10 +38321,6 @@ class TablePanel extends owl.Component {
         });
     }
     onRangeChanged(ranges) {
-        if (!ranges[0] || !ranges[0].match(rangeReference)) {
-            this.state.tableZoneErrors = ["InvalidRange" /* CommandResult.InvalidRange */];
-            return;
-        }
         const sheetId = this.env.model.getters.getActiveSheetId();
         this.state.tableXc = ranges[0];
         const newTableRange = this.env.model.getters.getRangeFromSheetXC(sheetId, this.state.tableXc);
@@ -38360,11 +38360,6 @@ class TablePanel extends owl.Component {
             const newTableRange = updatedTable.range;
             this.state.tableXc = this.env.model.getters.getRangeString(newTableRange, sheetId);
         }
-        else {
-            const oldTableRange = this.props.table.range;
-            this.state.tableXc = this.env.model.getters.getRangeString(oldTableRange, sheetId);
-        }
-        this.state.tableZoneErrors = [];
     }
     deleteTable() {
         const sheetId = this.env.model.getters.getActiveSheetId();
@@ -49251,7 +49246,10 @@ class HeaderSizePlugin extends CorePlugin {
     handle(cmd) {
         switch (cmd.type) {
             case "CREATE_SHEET": {
-                this.history.update("sizes", cmd.sheetId, { COL: [], ROW: [] });
+                this.history.update("sizes", cmd.sheetId, {
+                    COL: Array(this.getters.getNumberCols(cmd.sheetId)).fill(undefined),
+                    ROW: Array(this.getters.getNumberRows(cmd.sheetId)).fill(undefined),
+                });
                 break;
             }
             case "DUPLICATE_SHEET":
@@ -55795,10 +55793,6 @@ class PivotUIPlugin extends UIPlugin {
                 this.setupPivot(cmd.pivotId, { recreate: true });
                 break;
             }
-            case "REMOVE_PIVOT": {
-                delete this.pivots[cmd.pivotId];
-                break;
-            }
             case "DELETE_SHEET":
             case "UPDATE_CELL": {
                 this.unusedPivots = undefined;
@@ -55940,6 +55934,9 @@ class PivotUIPlugin extends UIPlugin {
         }
     }
     getPivot(pivotId) {
+        if (!this.getters.isExistingPivot(pivotId)) {
+            throw new Error(`pivot ${pivotId} not found`);
+        }
         return this.pivots[pivotId];
     }
     isPivotUnused(pivotId) {
@@ -58020,7 +58017,7 @@ class InsertPivotPlugin extends UIPlugin {
             sheetId,
             col,
             row,
-            content: `=PIVOT("${pivotFormulaId}")`,
+            content: `=PIVOT(${pivotFormulaId})`,
         });
         const zone = {
             left: col,
@@ -59411,6 +59408,7 @@ class ClipboardPlugin extends UIPlugin {
                 this.status = "invisible";
                 if (this._isCutOperation) {
                     this.copiedData = undefined;
+                    this._isCutOperation = false;
                 }
                 break;
             }
@@ -59508,6 +59506,7 @@ class ClipboardPlugin extends UIPlugin {
             }
             case "ACTIVATE_PAINT_FORMAT": {
                 const zones = this.getters.getSelectedZones();
+                this._isCutOperation = false;
                 this.copiedData = this.copy(zones);
                 this.status = "visible";
                 if (cmd.persistent) {
@@ -68461,6 +68460,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "17.4.5";
-__info__.date = "2024-08-19T08:12:00.492Z";
-__info__.hash = "28579ad";
+__info__.version = "17.4.6";
+__info__.date = "2024-08-26T13:50:49.623Z";
+__info__.hash = "f5648d9";

package/dist/o-spreadsheet.d.ts

@@ -9102,15 +9102,13 @@ declare class FindAndReplaceStore extends SpreadsheetStore implements HighlightP
     toSearch: string;
     toReplace: string;
     searchOptions: SearchOptions;
-    updateSearchContent: DebouncedFunction<(toSearch: string) => void>;
     constructor(get: Get);
     get searchMatches(): CellPosition[];
-    private _updateSearchContent;
+    updateSearchContent(toSearch: string): void;
     updateSearchOptions(searchOptions: Partial<SearchOptions>): void;
     searchFormulas(showFormula: boolean): void;
     selectPreviousMatch(): void;
     selectNextMatch(): void;
-    get pendingSearch(): boolean;
     handle(cmd: Command): void;
     finalize(): void;
     get allSheetMatchesCount(): number;
@@ -11598,6 +11596,7 @@ declare const constants: {
         bandedRows: boolean;
         bandedColumns: boolean;
         styleId: string;
+        automaticAutofill: boolean;
     };
     ChartTerms: {
         Series: string;

package/dist/o-spreadsheet.esm.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.4.5
- * @date 2024-08-19T08:12:00.492Z
- * @hash 28579ad
+ * @version 17.4.6
+ * @date 2024-08-26T13:50:49.623Z
+ * @hash f5648d9
  */
 
 import { useEnv, useSubEnv, onWillUnmount, useComponent, status, Component, useRef, onMounted, useEffect, useState, onPatched, onWillPatch, onWillUpdateProps, useExternalListener, onWillStart, xml, useChildSubEnv, markRaw, toRaw } from '@odoo/owl';
@@ -240,6 +240,7 @@ const PIVOT_TABLE_CONFIG = {
     bandedRows: true,
     bandedColumns: false,
     styleId: "TableStyleMedium5",
+    automaticAutofill: true,
 };
 
 //------------------------------------------------------------------------------
@@ -35171,7 +35172,6 @@ class FindAndReplaceStore extends SpreadsheetStore {
         searchScope: "activeSheet",
         specificRange: undefined,
     };
-    updateSearchContent = debounce(this._updateSearchContent.bind(this), 200);
     constructor(get) {
         super(get);
         this.initialShowFormulaState = this.model.getters.shouldShowFormulas();
@@ -35180,7 +35180,6 @@ class FindAndReplaceStore extends SpreadsheetStore {
         highlightStore.register(this);
         this.onDispose(() => {
             this.model.dispatch("SET_FORMULA_VISIBILITY", { show: this.initialShowFormulaState });
-            this.updateSearchContent.stopDebounce();
             highlightStore.unRegister(this);
         });
     }
@@ -35194,7 +35193,7 @@ class FindAndReplaceStore extends SpreadsheetStore {
                 return this.specificRangeMatches;
         }
     }
-    _updateSearchContent(toSearch) {
+    updateSearchContent(toSearch) {
         this._updateSearch(toSearch, this.searchOptions);
     }
     updateSearchOptions(searchOptions) {
@@ -35210,9 +35209,6 @@ class FindAndReplaceStore extends SpreadsheetStore {
     selectNextMatch() {
         this.selectNextCell(Direction.next);
     }
-    get pendingSearch() {
-        return this.updateSearchContent.isDebouncePending();
-    }
     handle(cmd) {
         switch (cmd.type) {
             case "SET_FORMULA_VISIBILITY":
@@ -35538,6 +35534,7 @@ class FindAndReplacePanel extends Component {
     searchInput = useRef("searchInput");
     store;
     state;
+    updateSearchContent;
     get hasSearchResult() {
         return this.store.selectedMatchIndex !== null;
     }
@@ -35569,12 +35566,14 @@ class FindAndReplacePanel extends Component {
         this.store = useLocalStore(FindAndReplaceStore);
         this.state = useState({ dataRange: "" });
         onMounted(() => this.searchInput.el?.focus());
+        onWillUnmount(() => this.updateSearchContent.stopDebounce());
+        this.updateSearchContent = debounce(this.store.updateSearchContent, 200);
     }
     onFocusSearch() {
         this.updateDataRange();
     }
     onSearchInput(ev) {
-        this.store.updateSearchContent(ev.target?.value || "");
+        this.updateSearchContent(ev.target.value);
     }
     onKeydownSearch(ev) {
         if (ev.key === "Enter") {
@@ -35613,6 +35612,9 @@ class FindAndReplacePanel extends Component {
         const specificRange = this.env.model.getters.getRangeFromSheetXC(this.env.model.getters.getActiveSheetId(), this.state.dataRange);
         this.store.updateSearchOptions({ specificRange });
     }
+    get pendingSearch() {
+        return this.updateSearchContent.isDebouncePending();
+    }
 }
 
 css /* scss */ `
@@ -36666,6 +36668,12 @@ function orderDataEntriesKeys(groups, dimension) {
  * Used to order two values
  */
 function compareDimensionValues(dimension, a, b) {
+    if (a === "null") {
+        return dimension.order === "asc" ? 1 : -1;
+    }
+    if (b === "null") {
+        return dimension.order === "asc" ? -1 : 1;
+    }
     if (dimension.type === "integer" || dimension.type === "date") {
         return dimension.order === "asc" ? Number(a) - Number(b) : Number(b) - Number(a);
     }
@@ -37028,7 +37036,7 @@ class SpreadsheetPivot {
         if (cell.type === CellValueType.error) {
             throw new EvaluationError(_t("The pivot cannot be created because cell %s contains an error", toXC(col, row)));
         }
-        if (cell.type === CellValueType.empty) {
+        if (cell.type === CellValueType.empty || cell.value === "") {
             throw new EvaluationError(_t("The pivot cannot be created because cell %s is empty", toXC(col, row)));
         }
         if (cell.value === "__count") {
@@ -37374,15 +37382,11 @@ class PivotSpreadsheetSidePanel extends Component {
     }
     onSelectionConfirmed() {
         if (this.state.range) {
-            const { sheetName, xc } = splitReference(this.state.range);
-            const sheetId = sheetName
-                ? this.env.model.getters.getSheetIdByName(sheetName)
-                : this.env.model.getters.getActiveSheetId();
-            if (!sheetId) {
+            const range = this.env.model.getters.getRangeFromSheetXC(this.env.model.getters.getActiveSheetId(), this.state.range);
+            if (range.invalidSheetName || range.invalidXc) {
                 return;
             }
-            const zone = toZone(xc);
-            const dataSet = { sheetId, zone };
+            const dataSet = { sheetId: range.sheetId, zone: range.zone };
             this.store.update({ dataSet });
             // Immediately apply the update to recompute the pivot fields
             this.store.applyUpdate();
@@ -38315,10 +38319,6 @@ class TablePanel extends Component {
         });
     }
     onRangeChanged(ranges) {
-        if (!ranges[0] || !ranges[0].match(rangeReference)) {
-            this.state.tableZoneErrors = ["InvalidRange" /* CommandResult.InvalidRange */];
-            return;
-        }
         const sheetId = this.env.model.getters.getActiveSheetId();
         this.state.tableXc = ranges[0];
         const newTableRange = this.env.model.getters.getRangeFromSheetXC(sheetId, this.state.tableXc);
@@ -38358,11 +38358,6 @@ class TablePanel extends Component {
             const newTableRange = updatedTable.range;
             this.state.tableXc = this.env.model.getters.getRangeString(newTableRange, sheetId);
         }
-        else {
-            const oldTableRange = this.props.table.range;
-            this.state.tableXc = this.env.model.getters.getRangeString(oldTableRange, sheetId);
-        }
-        this.state.tableZoneErrors = [];
     }
     deleteTable() {
         const sheetId = this.env.model.getters.getActiveSheetId();
@@ -49249,7 +49244,10 @@ class HeaderSizePlugin extends CorePlugin {
     handle(cmd) {
         switch (cmd.type) {
             case "CREATE_SHEET": {
-                this.history.update("sizes", cmd.sheetId, { COL: [], ROW: [] });
+                this.history.update("sizes", cmd.sheetId, {
+                    COL: Array(this.getters.getNumberCols(cmd.sheetId)).fill(undefined),
+                    ROW: Array(this.getters.getNumberRows(cmd.sheetId)).fill(undefined),
+                });
                 break;
             }
             case "DUPLICATE_SHEET":
@@ -55793,10 +55791,6 @@ class PivotUIPlugin extends UIPlugin {
                 this.setupPivot(cmd.pivotId, { recreate: true });
                 break;
             }
-            case "REMOVE_PIVOT": {
-                delete this.pivots[cmd.pivotId];
-                break;
-            }
             case "DELETE_SHEET":
             case "UPDATE_CELL": {
                 this.unusedPivots = undefined;
@@ -55938,6 +55932,9 @@ class PivotUIPlugin extends UIPlugin {
         }
     }
     getPivot(pivotId) {
+        if (!this.getters.isExistingPivot(pivotId)) {
+            throw new Error(`pivot ${pivotId} not found`);
+        }
         return this.pivots[pivotId];
     }
     isPivotUnused(pivotId) {
@@ -58018,7 +58015,7 @@ class InsertPivotPlugin extends UIPlugin {
             sheetId,
             col,
             row,
-            content: `=PIVOT("${pivotFormulaId}")`,
+            content: `=PIVOT(${pivotFormulaId})`,
         });
         const zone = {
             left: col,
@@ -59409,6 +59406,7 @@ class ClipboardPlugin extends UIPlugin {
                 this.status = "invisible";
                 if (this._isCutOperation) {
                     this.copiedData = undefined;
+                    this._isCutOperation = false;
                 }
                 break;
             }
@@ -59506,6 +59504,7 @@ class ClipboardPlugin extends UIPlugin {
             }
             case "ACTIVATE_PAINT_FORMAT": {
                 const zones = this.getters.getSelectedZones();
+                this._isCutOperation = false;
                 this.copiedData = this.copy(zones);
                 this.status = "visible";
                 if (cmd.persistent) {
@@ -68416,6 +68415,6 @@ const constants = {
 export { AbstractCellClipboardHandler, AbstractChart, AbstractFigureClipboardHandler, CellErrorType, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, PivotRuntimeDefinition, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, SpreadsheetPivotTable, 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.4.5";
-__info__.date = "2024-08-19T08:12:00.492Z";
-__info__.hash = "28579ad";
+__info__.version = "17.4.6";
+__info__.date = "2024-08-26T13:50:49.623Z";
+__info__.hash = "f5648d9";

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.4.5
- * @date 2024-08-19T08:12:00.492Z
- * @hash 28579ad
+ * @version 17.4.6
+ * @date 2024-08-26T13:50:49.623Z
+ * @hash f5648d9
  */
 
 (function (exports, owl) {
@@ -241,6 +241,7 @@
       bandedRows: true,
       bandedColumns: false,
       styleId: "TableStyleMedium5",
+      automaticAutofill: true,
   };
 
   //------------------------------------------------------------------------------
@@ -35172,7 +35173,6 @@ stores.inject(MyMetaStore, storeInstance);
           searchScope: "activeSheet",
           specificRange: undefined,
       };
-      updateSearchContent = debounce(this._updateSearchContent.bind(this), 200);
       constructor(get) {
           super(get);
           this.initialShowFormulaState = this.model.getters.shouldShowFormulas();
@@ -35181,7 +35181,6 @@ stores.inject(MyMetaStore, storeInstance);
           highlightStore.register(this);
           this.onDispose(() => {
               this.model.dispatch("SET_FORMULA_VISIBILITY", { show: this.initialShowFormulaState });
-              this.updateSearchContent.stopDebounce();
               highlightStore.unRegister(this);
           });
       }
@@ -35195,7 +35194,7 @@ stores.inject(MyMetaStore, storeInstance);
                   return this.specificRangeMatches;
           }
       }
-      _updateSearchContent(toSearch) {
+      updateSearchContent(toSearch) {
           this._updateSearch(toSearch, this.searchOptions);
       }
       updateSearchOptions(searchOptions) {
@@ -35211,9 +35210,6 @@ stores.inject(MyMetaStore, storeInstance);
       selectNextMatch() {
           this.selectNextCell(Direction.next);
       }
-      get pendingSearch() {
-          return this.updateSearchContent.isDebouncePending();
-      }
       handle(cmd) {
           switch (cmd.type) {
               case "SET_FORMULA_VISIBILITY":
@@ -35539,6 +35535,7 @@ stores.inject(MyMetaStore, storeInstance);
       searchInput = owl.useRef("searchInput");
       store;
       state;
+      updateSearchContent;
       get hasSearchResult() {
           return this.store.selectedMatchIndex !== null;
       }
@@ -35570,12 +35567,14 @@ stores.inject(MyMetaStore, storeInstance);
           this.store = useLocalStore(FindAndReplaceStore);
           this.state = owl.useState({ dataRange: "" });
           owl.onMounted(() => this.searchInput.el?.focus());
+          owl.onWillUnmount(() => this.updateSearchContent.stopDebounce());
+          this.updateSearchContent = debounce(this.store.updateSearchContent, 200);
       }
       onFocusSearch() {
           this.updateDataRange();
       }
       onSearchInput(ev) {
-          this.store.updateSearchContent(ev.target?.value || "");
+          this.updateSearchContent(ev.target.value);
       }
       onKeydownSearch(ev) {
           if (ev.key === "Enter") {
@@ -35614,6 +35613,9 @@ stores.inject(MyMetaStore, storeInstance);
           const specificRange = this.env.model.getters.getRangeFromSheetXC(this.env.model.getters.getActiveSheetId(), this.state.dataRange);
           this.store.updateSearchOptions({ specificRange });
       }
+      get pendingSearch() {
+          return this.updateSearchContent.isDebouncePending();
+      }
   }
 
   css /* scss */ `
@@ -36667,6 +36669,12 @@ stores.inject(MyMetaStore, storeInstance);
    * Used to order two values
    */
   function compareDimensionValues(dimension, a, b) {
+      if (a === "null") {
+          return dimension.order === "asc" ? 1 : -1;
+      }
+      if (b === "null") {
+          return dimension.order === "asc" ? -1 : 1;
+      }
       if (dimension.type === "integer" || dimension.type === "date") {
           return dimension.order === "asc" ? Number(a) - Number(b) : Number(b) - Number(a);
       }
@@ -37029,7 +37037,7 @@ stores.inject(MyMetaStore, storeInstance);
           if (cell.type === CellValueType.error) {
               throw new EvaluationError(_t("The pivot cannot be created because cell %s contains an error", toXC(col, row)));
           }
-          if (cell.type === CellValueType.empty) {
+          if (cell.type === CellValueType.empty || cell.value === "") {
               throw new EvaluationError(_t("The pivot cannot be created because cell %s is empty", toXC(col, row)));
           }
           if (cell.value === "__count") {
@@ -37375,15 +37383,11 @@ stores.inject(MyMetaStore, storeInstance);
       }
       onSelectionConfirmed() {
           if (this.state.range) {
-              const { sheetName, xc } = splitReference(this.state.range);
-              const sheetId = sheetName
-                  ? this.env.model.getters.getSheetIdByName(sheetName)
-                  : this.env.model.getters.getActiveSheetId();
-              if (!sheetId) {
+              const range = this.env.model.getters.getRangeFromSheetXC(this.env.model.getters.getActiveSheetId(), this.state.range);
+              if (range.invalidSheetName || range.invalidXc) {
                   return;
               }
-              const zone = toZone(xc);
-              const dataSet = { sheetId, zone };
+              const dataSet = { sheetId: range.sheetId, zone: range.zone };
               this.store.update({ dataSet });
               // Immediately apply the update to recompute the pivot fields
               this.store.applyUpdate();
@@ -38316,10 +38320,6 @@ stores.inject(MyMetaStore, storeInstance);
           });
       }
       onRangeChanged(ranges) {
-          if (!ranges[0] || !ranges[0].match(rangeReference)) {
-              this.state.tableZoneErrors = ["InvalidRange" /* CommandResult.InvalidRange */];
-              return;
-          }
           const sheetId = this.env.model.getters.getActiveSheetId();
           this.state.tableXc = ranges[0];
           const newTableRange = this.env.model.getters.getRangeFromSheetXC(sheetId, this.state.tableXc);
@@ -38359,11 +38359,6 @@ stores.inject(MyMetaStore, storeInstance);
               const newTableRange = updatedTable.range;
               this.state.tableXc = this.env.model.getters.getRangeString(newTableRange, sheetId);
           }
-          else {
-              const oldTableRange = this.props.table.range;
-              this.state.tableXc = this.env.model.getters.getRangeString(oldTableRange, sheetId);
-          }
-          this.state.tableZoneErrors = [];
       }
       deleteTable() {
           const sheetId = this.env.model.getters.getActiveSheetId();
@@ -49250,7 +49245,10 @@ stores.inject(MyMetaStore, storeInstance);
       handle(cmd) {
           switch (cmd.type) {
               case "CREATE_SHEET": {
-                  this.history.update("sizes", cmd.sheetId, { COL: [], ROW: [] });
+                  this.history.update("sizes", cmd.sheetId, {
+                      COL: Array(this.getters.getNumberCols(cmd.sheetId)).fill(undefined),
+                      ROW: Array(this.getters.getNumberRows(cmd.sheetId)).fill(undefined),
+                  });
                   break;
               }
               case "DUPLICATE_SHEET":
@@ -55794,10 +55792,6 @@ stores.inject(MyMetaStore, storeInstance);
                   this.setupPivot(cmd.pivotId, { recreate: true });
                   break;
               }
-              case "REMOVE_PIVOT": {
-                  delete this.pivots[cmd.pivotId];
-                  break;
-              }
               case "DELETE_SHEET":
               case "UPDATE_CELL": {
                   this.unusedPivots = undefined;
@@ -55939,6 +55933,9 @@ stores.inject(MyMetaStore, storeInstance);
           }
       }
       getPivot(pivotId) {
+          if (!this.getters.isExistingPivot(pivotId)) {
+              throw new Error(`pivot ${pivotId} not found`);
+          }
           return this.pivots[pivotId];
       }
       isPivotUnused(pivotId) {
@@ -58019,7 +58016,7 @@ stores.inject(MyMetaStore, storeInstance);
               sheetId,
               col,
               row,
-              content: `=PIVOT("${pivotFormulaId}")`,
+              content: `=PIVOT(${pivotFormulaId})`,
           });
           const zone = {
               left: col,
@@ -59410,6 +59407,7 @@ stores.inject(MyMetaStore, storeInstance);
                   this.status = "invisible";
                   if (this._isCutOperation) {
                       this.copiedData = undefined;
+                      this._isCutOperation = false;
                   }
                   break;
               }
@@ -59507,6 +59505,7 @@ stores.inject(MyMetaStore, storeInstance);
               }
               case "ACTIVATE_PAINT_FORMAT": {
                   const zones = this.getters.getSelectedZones();
+                  this._isCutOperation = false;
                   this.copiedData = this.copy(zones);
                   this.status = "visible";
                   if (cmd.persistent) {
@@ -68460,9 +68459,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.4.5";
-  __info__.date = "2024-08-19T08:12:00.492Z";
-  __info__.hash = "28579ad";
+  __info__.version = "17.4.6";
+  __info__.date = "2024-08-26T13:50:49.623Z";
+  __info__.hash = "f5648d9";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);