@odoo/o-spreadsheet 16.4.13 → 16.4.14

package/dist/o-spreadsheet.cjs.js

@@ -18569,6 +18569,17 @@ const SELECTION_CONTAINS_FILTER = (env) => {
 //------------------------------------------------------------------------------
 const IS_ONLY_ONE_RANGE = (env) => {
     return env.model.getters.getSelectedZones().length === 1;
+};
+const CAN_INSERT_HEADER = (env, dimension) => {
+    if (!IS_ONLY_ONE_RANGE(env)) {
+        return false;
+    }
+    const activeHeaders = dimension === "COL" ? env.model.getters.getActiveCols() : env.model.getters.getActiveRows();
+    const ortogonalActiveHeaders = dimension === "COL" ? env.model.getters.getActiveRows() : env.model.getters.getActiveCols();
+    const sheetId = env.model.getters.getActiveSheetId();
+    const zone = env.model.getters.getSelectedZone();
+    const allSheetSelected = isEqual(zone, env.model.getters.getSheetZone(sheetId));
+    return isConsecutive(activeHeaders) && (ortogonalActiveHeaders.size === 0 || allSheetSelected);
 };
 
 const undo = {
@@ -18758,9 +18769,7 @@ const insertRow = {
         const number = getRowsNumber(env);
         return number === 1 ? _lt("Insert row") : _lt("Insert %s rows", number.toString());
     },
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveRows()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveCols().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "ROW"),
     icon: "o-spreadsheet-Icon.INSERT_ROW",
 };
 const rowInsertRowBefore = {
@@ -18769,9 +18778,7 @@ const rowInsertRowBefore = {
         return number === 1 ? _lt("Insert row above") : _lt("Insert %s rows above", number.toString());
     },
     execute: INSERT_ROWS_BEFORE_ACTION,
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveRows()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveCols().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "ROW"),
     icon: "o-spreadsheet-Icon.INSERT_ROW_BEFORE",
 };
 const topBarInsertRowsBefore = {
@@ -18802,9 +18809,7 @@ const rowInsertRowsAfter = {
         const number = getRowsNumber(env);
         return number === 1 ? _lt("Insert row below") : _lt("Insert %s rows below", number.toString());
     },
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveRows()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveCols().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "ROW"),
     icon: "o-spreadsheet-Icon.INSERT_ROW_AFTER",
 };
 const topBarInsertRowsAfter = {
@@ -18822,9 +18827,7 @@ const insertCol = {
         const number = getColumnsNumber(env);
         return number === 1 ? _lt("Insert column") : _lt("Insert %s columns", number.toString());
     },
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveCols()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveRows().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "COL"),
     icon: "o-spreadsheet-Icon.INSERT_COL",
 };
 const colInsertColsBefore = {
@@ -18835,9 +18838,7 @@ const colInsertColsBefore = {
             : _lt("Insert %s columns left", number.toString());
     },
     execute: INSERT_COLUMNS_BEFORE_ACTION,
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveCols()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveRows().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "COL"),
     icon: "o-spreadsheet-Icon.INSERT_COL_BEFORE",
 };
 const topBarInsertColsBefore = {
@@ -18870,9 +18871,7 @@ const colInsertColsAfter = {
             : _lt("Insert %s columns right", number.toString());
     },
     execute: INSERT_COLUMNS_AFTER_ACTION,
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveCols()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveRows().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "COL"),
     icon: "o-spreadsheet-Icon.INSERT_COL_AFTER",
 };
 const topBarInsertColsAfter = {
@@ -22830,7 +22829,7 @@ css /* scss */ `
       }
       .o-input-count {
         width: fit-content;
-        padding: 4 0 4 4;
+        padding: 4px 0 4px 4px;
       }
     }
   }
@@ -33077,6 +33076,13 @@ class FiltersPlugin extends CorePlugin {
     }
     onDeleteColumnsRows(cmd) {
         for (const table of this.getFilterTables(cmd.sheetId)) {
+            // Remove the filter tables whose data filter headers are in the removed rows.
+            if (cmd.dimension === "ROW" && cmd.elements.includes(table.zone.top)) {
+                const tables = { ...this.tables[cmd.sheetId] };
+                delete tables[table.id];
+                this.history.update("tables", cmd.sheetId, tables);
+                continue;
+            }
             const zone = reduceZoneOnDeletion(table.zone, cmd.dimension === "COL" ? "left" : "top", cmd.elements);
             if (!zone) {
                 const tables = { ...this.tables[cmd.sheetId] };
@@ -36525,22 +36531,20 @@ class EvaluationChartPlugin extends UIPlugin {
             invalidateCFEvaluationCommands.has(cmd.type) ||
             cmd.type === "EVALUATE_CELLS" ||
             cmd.type === "UPDATE_CELL") {
-            if (cmd.type !== "UNDO" && cmd.type !== "REDO") {
-                for (const chartId in this.charts) {
-                    this.history.update("charts", chartId, undefined);
-                }
+            for (const chartId in this.charts) {
+                this.charts[chartId] = undefined;
             }
         }
         switch (cmd.type) {
             case "UPDATE_CHART":
             case "CREATE_CHART":
             case "DELETE_FIGURE":
-                this.history.update("charts", cmd.id, undefined);
+                this.charts[cmd.id] = undefined;
                 break;
             case "DELETE_SHEET":
                 for (let chartId in this.charts) {
                     if (!this.getters.isChartDefined(chartId)) {
-                        this.history.update("charts", chartId, undefined);
+                        this.charts[chartId] = undefined;
                     }
                 }
                 break;
@@ -36552,7 +36556,7 @@ class EvaluationChartPlugin extends UIPlugin {
             if (!chart) {
                 throw new Error(`No chart for the given id: ${figureId}`);
             }
-            this.history.update("charts", figureId, this.createRuntimeChart(chart));
+            this.charts[figureId] = this.createRuntimeChart(chart);
         }
         return this.charts[figureId];
     }
@@ -43016,6 +43020,8 @@ class FilterEvaluationPlugin extends UIPlugin {
             case "EVALUATE_CELLS":
             case "ACTIVATE_SHEET":
             case "REMOVE_FILTER_TABLE":
+            case "ADD_COLUMNS_ROWS":
+            case "REMOVE_COLUMNS_ROWS":
                 this.isEvaluationDirty = true;
                 break;
             case "START":
@@ -50772,6 +50778,6 @@ exports.setTranslationMethod = setTranslationMethod;
 exports.tokenize = tokenize;
 
 
-__info__.version = '16.4.13';
-__info__.date = '2023-11-03T12:21:08.124Z';
-__info__.hash = '001985e';
+__info__.version = '16.4.14';
+__info__.date = '2023-11-16T11:59:51.900Z';
+__info__.hash = '66c6f47';

package/dist/o-spreadsheet.d.ts

@@ -3819,11 +3819,11 @@ declare class CustomColorsPlugin extends UIPlugin<CustomColorState> {
 }
 
 interface EvaluationChartState {
-    readonly charts: Immutable<Record<UID, ChartRuntime | undefined>>;
+    charts: Record<UID, ChartRuntime | undefined>;
 }
 declare class EvaluationChartPlugin extends UIPlugin<EvaluationChartState> {
     static getters: readonly ["getChartRuntime", "getBackgroundOfSingleCellChart"];
-    readonly charts: Immutable<Record<UID, ChartRuntime | undefined>>;
+    charts: Record<UID, ChartRuntime | undefined>;
     private createRuntimeChart;
     handle(cmd: CoreViewCommand): void;
     getChartRuntime(figureId: UID): ChartRuntime;

package/dist/o-spreadsheet.esm.js

@@ -18565,6 +18565,17 @@ const SELECTION_CONTAINS_FILTER = (env) => {
 //------------------------------------------------------------------------------
 const IS_ONLY_ONE_RANGE = (env) => {
     return env.model.getters.getSelectedZones().length === 1;
+};
+const CAN_INSERT_HEADER = (env, dimension) => {
+    if (!IS_ONLY_ONE_RANGE(env)) {
+        return false;
+    }
+    const activeHeaders = dimension === "COL" ? env.model.getters.getActiveCols() : env.model.getters.getActiveRows();
+    const ortogonalActiveHeaders = dimension === "COL" ? env.model.getters.getActiveRows() : env.model.getters.getActiveCols();
+    const sheetId = env.model.getters.getActiveSheetId();
+    const zone = env.model.getters.getSelectedZone();
+    const allSheetSelected = isEqual(zone, env.model.getters.getSheetZone(sheetId));
+    return isConsecutive(activeHeaders) && (ortogonalActiveHeaders.size === 0 || allSheetSelected);
 };
 
 const undo = {
@@ -18754,9 +18765,7 @@ const insertRow = {
         const number = getRowsNumber(env);
         return number === 1 ? _lt("Insert row") : _lt("Insert %s rows", number.toString());
     },
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveRows()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveCols().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "ROW"),
     icon: "o-spreadsheet-Icon.INSERT_ROW",
 };
 const rowInsertRowBefore = {
@@ -18765,9 +18774,7 @@ const rowInsertRowBefore = {
         return number === 1 ? _lt("Insert row above") : _lt("Insert %s rows above", number.toString());
     },
     execute: INSERT_ROWS_BEFORE_ACTION,
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveRows()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveCols().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "ROW"),
     icon: "o-spreadsheet-Icon.INSERT_ROW_BEFORE",
 };
 const topBarInsertRowsBefore = {
@@ -18798,9 +18805,7 @@ const rowInsertRowsAfter = {
         const number = getRowsNumber(env);
         return number === 1 ? _lt("Insert row below") : _lt("Insert %s rows below", number.toString());
     },
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveRows()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveCols().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "ROW"),
     icon: "o-spreadsheet-Icon.INSERT_ROW_AFTER",
 };
 const topBarInsertRowsAfter = {
@@ -18818,9 +18823,7 @@ const insertCol = {
         const number = getColumnsNumber(env);
         return number === 1 ? _lt("Insert column") : _lt("Insert %s columns", number.toString());
     },
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveCols()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveRows().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "COL"),
     icon: "o-spreadsheet-Icon.INSERT_COL",
 };
 const colInsertColsBefore = {
@@ -18831,9 +18834,7 @@ const colInsertColsBefore = {
             : _lt("Insert %s columns left", number.toString());
     },
     execute: INSERT_COLUMNS_BEFORE_ACTION,
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveCols()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveRows().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "COL"),
     icon: "o-spreadsheet-Icon.INSERT_COL_BEFORE",
 };
 const topBarInsertColsBefore = {
@@ -18866,9 +18867,7 @@ const colInsertColsAfter = {
             : _lt("Insert %s columns right", number.toString());
     },
     execute: INSERT_COLUMNS_AFTER_ACTION,
-    isVisible: (env) => isConsecutive(env.model.getters.getActiveCols()) &&
-        IS_ONLY_ONE_RANGE(env) &&
-        env.model.getters.getActiveRows().size === 0,
+    isVisible: (env) => CAN_INSERT_HEADER(env, "COL"),
     icon: "o-spreadsheet-Icon.INSERT_COL_AFTER",
 };
 const topBarInsertColsAfter = {
@@ -22826,7 +22825,7 @@ css /* scss */ `
       }
       .o-input-count {
         width: fit-content;
-        padding: 4 0 4 4;
+        padding: 4px 0 4px 4px;
       }
     }
   }
@@ -33073,6 +33072,13 @@ class FiltersPlugin extends CorePlugin {
     }
     onDeleteColumnsRows(cmd) {
         for (const table of this.getFilterTables(cmd.sheetId)) {
+            // Remove the filter tables whose data filter headers are in the removed rows.
+            if (cmd.dimension === "ROW" && cmd.elements.includes(table.zone.top)) {
+                const tables = { ...this.tables[cmd.sheetId] };
+                delete tables[table.id];
+                this.history.update("tables", cmd.sheetId, tables);
+                continue;
+            }
             const zone = reduceZoneOnDeletion(table.zone, cmd.dimension === "COL" ? "left" : "top", cmd.elements);
             if (!zone) {
                 const tables = { ...this.tables[cmd.sheetId] };
@@ -36521,22 +36527,20 @@ class EvaluationChartPlugin extends UIPlugin {
             invalidateCFEvaluationCommands.has(cmd.type) ||
             cmd.type === "EVALUATE_CELLS" ||
             cmd.type === "UPDATE_CELL") {
-            if (cmd.type !== "UNDO" && cmd.type !== "REDO") {
-                for (const chartId in this.charts) {
-                    this.history.update("charts", chartId, undefined);
-                }
+            for (const chartId in this.charts) {
+                this.charts[chartId] = undefined;
             }
         }
         switch (cmd.type) {
             case "UPDATE_CHART":
             case "CREATE_CHART":
             case "DELETE_FIGURE":
-                this.history.update("charts", cmd.id, undefined);
+                this.charts[cmd.id] = undefined;
                 break;
             case "DELETE_SHEET":
                 for (let chartId in this.charts) {
                     if (!this.getters.isChartDefined(chartId)) {
-                        this.history.update("charts", chartId, undefined);
+                        this.charts[chartId] = undefined;
                     }
                 }
                 break;
@@ -36548,7 +36552,7 @@ class EvaluationChartPlugin extends UIPlugin {
             if (!chart) {
                 throw new Error(`No chart for the given id: ${figureId}`);
             }
-            this.history.update("charts", figureId, this.createRuntimeChart(chart));
+            this.charts[figureId] = this.createRuntimeChart(chart);
         }
         return this.charts[figureId];
     }
@@ -43012,6 +43016,8 @@ class FilterEvaluationPlugin extends UIPlugin {
             case "EVALUATE_CELLS":
             case "ACTIVATE_SHEET":
             case "REMOVE_FILTER_TABLE":
+            case "ADD_COLUMNS_ROWS":
+            case "REMOVE_COLUMNS_ROWS":
                 this.isEvaluationDirty = true;
                 break;
             case "START":
@@ -50736,6 +50742,6 @@ const constants = {
 export { AbstractChart, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, __info__, addFunction, astToFormula, compile, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, invalidateCFEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, tokenize };
 
 
-__info__.version = '16.4.13';
-__info__.date = '2023-11-03T12:21:08.124Z';
-__info__.hash = '001985e';
+__info__.version = '16.4.14';
+__info__.date = '2023-11-16T11:59:51.900Z';
+__info__.hash = '66c6f47';

package/dist/o-spreadsheet.iife.js

@@ -18566,6 +18566,17 @@
   //------------------------------------------------------------------------------
   const IS_ONLY_ONE_RANGE = (env) => {
       return env.model.getters.getSelectedZones().length === 1;
+  };
+  const CAN_INSERT_HEADER = (env, dimension) => {
+      if (!IS_ONLY_ONE_RANGE(env)) {
+          return false;
+      }
+      const activeHeaders = dimension === "COL" ? env.model.getters.getActiveCols() : env.model.getters.getActiveRows();
+      const ortogonalActiveHeaders = dimension === "COL" ? env.model.getters.getActiveRows() : env.model.getters.getActiveCols();
+      const sheetId = env.model.getters.getActiveSheetId();
+      const zone = env.model.getters.getSelectedZone();
+      const allSheetSelected = isEqual(zone, env.model.getters.getSheetZone(sheetId));
+      return isConsecutive(activeHeaders) && (ortogonalActiveHeaders.size === 0 || allSheetSelected);
   };
 
   const undo = {
@@ -18755,9 +18766,7 @@
           const number = getRowsNumber(env);
           return number === 1 ? _lt("Insert row") : _lt("Insert %s rows", number.toString());
       },
-      isVisible: (env) => isConsecutive(env.model.getters.getActiveRows()) &&
-          IS_ONLY_ONE_RANGE(env) &&
-          env.model.getters.getActiveCols().size === 0,
+      isVisible: (env) => CAN_INSERT_HEADER(env, "ROW"),
       icon: "o-spreadsheet-Icon.INSERT_ROW",
   };
   const rowInsertRowBefore = {
@@ -18766,9 +18775,7 @@
           return number === 1 ? _lt("Insert row above") : _lt("Insert %s rows above", number.toString());
       },
       execute: INSERT_ROWS_BEFORE_ACTION,
-      isVisible: (env) => isConsecutive(env.model.getters.getActiveRows()) &&
-          IS_ONLY_ONE_RANGE(env) &&
-          env.model.getters.getActiveCols().size === 0,
+      isVisible: (env) => CAN_INSERT_HEADER(env, "ROW"),
       icon: "o-spreadsheet-Icon.INSERT_ROW_BEFORE",
   };
   const topBarInsertRowsBefore = {
@@ -18799,9 +18806,7 @@
           const number = getRowsNumber(env);
           return number === 1 ? _lt("Insert row below") : _lt("Insert %s rows below", number.toString());
       },
-      isVisible: (env) => isConsecutive(env.model.getters.getActiveRows()) &&
-          IS_ONLY_ONE_RANGE(env) &&
-          env.model.getters.getActiveCols().size === 0,
+      isVisible: (env) => CAN_INSERT_HEADER(env, "ROW"),
       icon: "o-spreadsheet-Icon.INSERT_ROW_AFTER",
   };
   const topBarInsertRowsAfter = {
@@ -18819,9 +18824,7 @@
           const number = getColumnsNumber(env);
           return number === 1 ? _lt("Insert column") : _lt("Insert %s columns", number.toString());
       },
-      isVisible: (env) => isConsecutive(env.model.getters.getActiveCols()) &&
-          IS_ONLY_ONE_RANGE(env) &&
-          env.model.getters.getActiveRows().size === 0,
+      isVisible: (env) => CAN_INSERT_HEADER(env, "COL"),
       icon: "o-spreadsheet-Icon.INSERT_COL",
   };
   const colInsertColsBefore = {
@@ -18832,9 +18835,7 @@
               : _lt("Insert %s columns left", number.toString());
       },
       execute: INSERT_COLUMNS_BEFORE_ACTION,
-      isVisible: (env) => isConsecutive(env.model.getters.getActiveCols()) &&
-          IS_ONLY_ONE_RANGE(env) &&
-          env.model.getters.getActiveRows().size === 0,
+      isVisible: (env) => CAN_INSERT_HEADER(env, "COL"),
       icon: "o-spreadsheet-Icon.INSERT_COL_BEFORE",
   };
   const topBarInsertColsBefore = {
@@ -18867,9 +18868,7 @@
               : _lt("Insert %s columns right", number.toString());
       },
       execute: INSERT_COLUMNS_AFTER_ACTION,
-      isVisible: (env) => isConsecutive(env.model.getters.getActiveCols()) &&
-          IS_ONLY_ONE_RANGE(env) &&
-          env.model.getters.getActiveRows().size === 0,
+      isVisible: (env) => CAN_INSERT_HEADER(env, "COL"),
       icon: "o-spreadsheet-Icon.INSERT_COL_AFTER",
   };
   const topBarInsertColsAfter = {
@@ -22827,7 +22826,7 @@
       }
       .o-input-count {
         width: fit-content;
-        padding: 4 0 4 4;
+        padding: 4px 0 4px 4px;
       }
     }
   }
@@ -33074,6 +33073,13 @@
       }
       onDeleteColumnsRows(cmd) {
           for (const table of this.getFilterTables(cmd.sheetId)) {
+              // Remove the filter tables whose data filter headers are in the removed rows.
+              if (cmd.dimension === "ROW" && cmd.elements.includes(table.zone.top)) {
+                  const tables = { ...this.tables[cmd.sheetId] };
+                  delete tables[table.id];
+                  this.history.update("tables", cmd.sheetId, tables);
+                  continue;
+              }
               const zone = reduceZoneOnDeletion(table.zone, cmd.dimension === "COL" ? "left" : "top", cmd.elements);
               if (!zone) {
                   const tables = { ...this.tables[cmd.sheetId] };
@@ -36522,22 +36528,20 @@
               invalidateCFEvaluationCommands.has(cmd.type) ||
               cmd.type === "EVALUATE_CELLS" ||
               cmd.type === "UPDATE_CELL") {
-              if (cmd.type !== "UNDO" && cmd.type !== "REDO") {
-                  for (const chartId in this.charts) {
-                      this.history.update("charts", chartId, undefined);
-                  }
+              for (const chartId in this.charts) {
+                  this.charts[chartId] = undefined;
               }
           }
           switch (cmd.type) {
               case "UPDATE_CHART":
               case "CREATE_CHART":
               case "DELETE_FIGURE":
-                  this.history.update("charts", cmd.id, undefined);
+                  this.charts[cmd.id] = undefined;
                   break;
               case "DELETE_SHEET":
                   for (let chartId in this.charts) {
                       if (!this.getters.isChartDefined(chartId)) {
-                          this.history.update("charts", chartId, undefined);
+                          this.charts[chartId] = undefined;
                       }
                   }
                   break;
@@ -36549,7 +36553,7 @@
               if (!chart) {
                   throw new Error(`No chart for the given id: ${figureId}`);
               }
-              this.history.update("charts", figureId, this.createRuntimeChart(chart));
+              this.charts[figureId] = this.createRuntimeChart(chart);
           }
           return this.charts[figureId];
       }
@@ -43013,6 +43017,8 @@
               case "EVALUATE_CELLS":
               case "ACTIVATE_SHEET":
               case "REMOVE_FILTER_TABLE":
+              case "ADD_COLUMNS_ROWS":
+              case "REMOVE_COLUMNS_ROWS":
                   this.isEvaluationDirty = true;
                   break;
               case "START":
@@ -50771,9 +50777,9 @@
   Object.defineProperty(exports, '__esModule', { value: true });
 
 
-  __info__.version = '16.4.13';
-  __info__.date = '2023-11-03T12:21:08.124Z';
-  __info__.hash = '001985e';
+  __info__.version = '16.4.14';
+  __info__.date = '2023-11-16T11:59:51.900Z';
+  __info__.hash = '66c6f47';
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);