@odoo/o-spreadsheet 16.1.2 → 16.1.3

package/dist/o_spreadsheet.iife.js

@@ -2824,7 +2824,7 @@
           return `background-color: ${this.background}`;
       }
       get chartRuntime() {
-          const runtime = this.env.model.getters.getChartRuntime(this.env.model.getters.getActiveSheetId(), this.props.figure.id);
+          const runtime = this.env.model.getters.getChartRuntime(this.props.figure.id);
           if (!("chartJsConfig" in runtime)) {
               throw new Error("Unsupported chart runtime");
           }
@@ -3078,7 +3078,7 @@
           this.ctx = document.createElement("canvas").getContext("2d");
       }
       get runtime() {
-          return this.env.model.getters.getChartRuntime(this.env.model.getters.getActiveSheetId(), this.props.figure.id);
+          return this.env.model.getters.getChartRuntime(this.props.figure.id);
       }
       get title() {
           return this.runtime.title;
@@ -3469,7 +3469,7 @@
       CommandResult[CommandResult["InvalidRange"] = 25] = "InvalidRange";
       CommandResult[CommandResult["InvalidZones"] = 26] = "InvalidZones";
       CommandResult[CommandResult["InvalidSheetId"] = 27] = "InvalidSheetId";
-      CommandResult[CommandResult["DuplicatedImageId"] = 28] = "DuplicatedImageId";
+      CommandResult[CommandResult["InvalidFigureId"] = 28] = "InvalidFigureId";
       CommandResult[CommandResult["InputAlreadyFocused"] = 29] = "InputAlreadyFocused";
       CommandResult[CommandResult["MaximumRangesReached"] = 30] = "MaximumRangesReached";
       CommandResult[CommandResult["InvalidChartDefinition"] = 31] = "InvalidChartDefinition";
@@ -3526,6 +3526,7 @@
       CommandResult[CommandResult["DuplicatedFigureId"] = 82] = "DuplicatedFigureId";
       CommandResult[CommandResult["InvalidSelectionStep"] = 83] = "InvalidSelectionStep";
       CommandResult[CommandResult["DuplicatedChartId"] = 84] = "DuplicatedChartId";
+      CommandResult[CommandResult["ChartDoesNotExist"] = 85] = "ChartDoesNotExist";
   })(exports.CommandResult || (exports.CommandResult = {}));
 
   var DIRECTION;
@@ -6066,7 +6067,7 @@
           return registry;
       }
       get chartType() {
-          return this.env.model.getters.getChartType(this.env.model.getters.getActiveSheetId(), this.props.figure.id);
+          return this.env.model.getters.getChartType(this.props.figure.id);
       }
       onContextMenu(ev) {
           const position = {
@@ -18801,7 +18802,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               name: _lt("Reset size"),
               sequence: 3,
               action: () => {
-                  const size = this.env.model.getters.getImage(this.env.model.getters.getActiveSheetId(), this.figureId).size;
+                  const size = this.env.model.getters.getImageSize(this.figureId);
                   const { height, width } = getMaxFigureSize(this.env.model.getters, size);
                   this.env.model.dispatch("UPDATE_FIGURE", {
                       sheetId: this.env.model.getters.getActiveSheetId(),
@@ -18852,8 +18853,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           return this.props.figure.id;
       }
       get getImagePath() {
-          return this.env.model.getters.getImage(this.env.model.getters.getActiveSheetId(), this.figureId)
-              .path;
+          return this.env.model.getters.getImagePath(this.figureId);
       }
   }
   ImageFigure.template = "o-spreadsheet-ImageFigure";
@@ -19808,7 +19808,6 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
   LineBarPieConfigPanel.template = "o-spreadsheet-LineBarPieConfigPanel";
   LineBarPieConfigPanel.components = { SelectionInput };
   LineBarPieConfigPanel.props = {
-      sheetId: String,
       figureId: String,
       definition: Object,
       updateChart: Function,
@@ -20138,7 +20137,6 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
   GaugeChartConfigPanel.template = "o-spreadsheet-GaugeChartConfigPanel";
   GaugeChartConfigPanel.components = { SelectionInput };
   GaugeChartConfigPanel.props = {
-      sheetId: String,
       figureId: String,
       definition: Object,
       updateChart: Function,
@@ -20292,7 +20290,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
 
   class LineConfigPanel extends LineBarPieConfigPanel {
       get canTreatLabelsAsText() {
-          const chart = this.env.model.getters.getChart(this.props.sheetId, this.props.figureId);
+          const chart = this.env.model.getters.getChart(this.props.figureId);
           if (chart && chart instanceof LineChart) {
               return canChartParseLabels(chart.labelRange, this.env.model.getters);
           }
@@ -20369,7 +20367,6 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
   ScorecardChartConfigPanel.template = "o-spreadsheet-ScorecardChartConfigPanel";
   ScorecardChartConfigPanel.components = { SelectionInput };
   ScorecardChartConfigPanel.props = {
-      sheetId: String,
       figureId: String,
       definition: Object,
       updateChart: Function,
@@ -20501,7 +20498,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               else {
                   this.shouldUpdateChart = true;
               }
-              if (!this.env.model.getters.isChartDefined(this.sheetId, this.figureId)) {
+              if (!this.env.model.getters.isChartDefined(this.figureId)) {
                   this.props.onCloseSidePanel();
                   return;
               }
@@ -20522,7 +20519,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           });
       }
       onTypeChange(type) {
-          const context = this.env.model.getters.getContextCreationChart(this.sheetId, this.figureId);
+          const context = this.env.model.getters.getContextCreationChart(this.figureId);
           if (!context) {
               throw new Error("Chart not defined.");
           }
@@ -20534,7 +20531,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           });
       }
       get chartPanel() {
-          const type = this.env.model.getters.getChartType(this.sheetId, this.figureId);
+          const type = this.env.model.getters.getChartType(this.figureId);
           if (!type) {
               throw new Error("Chart not defined.");
           }
@@ -20544,8 +20541,8 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           }
           return chartPanel;
       }
-      getChartDefinition() {
-          return this.env.model.getters.getChartDefinition(this.sheetId, this.figureId);
+      getChartDefinition(figureId = this.figureId) {
+          return this.env.model.getters.getChartDefinition(figureId);
       }
       get chartTypes() {
           return getChartTypes();
@@ -27104,13 +27101,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               data = migrate(data);
           }
       }
-      // sanity check: try to fix missing fields/corrupted state by providing
-      // sensible default values
-      data = Object.assign(createEmptyWorkbookData(), data, { version: CURRENT_VERSION });
-      data.sheets = data.sheets.map((s, i) => Object.assign(createEmptySheet(`Sheet${i + 1}`, `Sheet${i + 1}`), s));
-      if (data.sheets.length === 0) {
-          data.sheets.push(createEmptySheet(INITIAL_SHEET_ID, "Sheet1"));
-      }
+      data = repairData(data);
       return data;
   }
   function migrate(data) {
@@ -27348,6 +27339,48 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           },
       },
   ];
+  /**
+   * This function is used to repair faulty data independently of the migration.
+   */
+  function repairData(data) {
+      data = forceUnicityOfFigure(data);
+      data = setDefaults(data);
+      return data;
+  }
+  /**
+   * Force the unicity of figure ids accross sheets
+   */
+  function forceUnicityOfFigure(data) {
+      if (data.uniqueFigureIds) {
+          return data;
+      }
+      const figureIds = new Set();
+      const uuidGenerator = new UuidGenerator();
+      for (const sheet of data.sheets || []) {
+          for (const figure of sheet.figures || []) {
+              if (figureIds.has(figure.id)) {
+                  figure.id += uuidGenerator.uuidv4();
+              }
+              figureIds.add(figure.id);
+          }
+      }
+      data.uniqueFigureIds = true;
+      return data;
+  }
+  /**
+   * sanity check: try to fix missing fields/corrupted state by providing
+   * sensible default values
+   */
+  function setDefaults(data) {
+      data = Object.assign(createEmptyWorkbookData(), data, { version: CURRENT_VERSION });
+      data.sheets = data.sheets
+          ? data.sheets.map((s, i) => Object.assign(createEmptySheet(`Sheet${i + 1}`, `Sheet${i + 1}`), s))
+          : [];
+      if (data.sheets.length === 0) {
+          data.sheets.push(createEmptySheet(INITIAL_SHEET_ID, "Sheet1"));
+      }
+      return data;
+  }
   /**
    * The goal of this function is to repair corrupted/wrong initial messages caused by
    * a bug.
@@ -27486,6 +27519,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           formats: {},
           borders: {},
           revisionId: DEFAULT_REVISION_ID,
+          uniqueFigureIds: true,
       };
       return data;
   }
@@ -28483,10 +28517,8 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           this.validateChartDefinition = (cmd) => validateChartDefinition(this, cmd.definition);
       }
       adaptRanges(applyChange) {
-          for (const sheetId of Object.keys(this.charts)) {
-              for (const [chartId, chart] of Object.entries(this.charts[sheetId] || {})) {
-                  this.history.update("charts", sheetId, chartId, chart === null || chart === void 0 ? void 0 : chart.updateRanges(applyChange));
-              }
+          for (const [chartId, chart] of Object.entries(this.charts)) {
+              this.history.update("charts", chartId, chart === null || chart === void 0 ? void 0 : chart.updateRanges(applyChange));
           }
       }
       // ---------------------------------------------------------------------------
@@ -28497,20 +28529,20 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               case "CREATE_CHART":
                   return this.checkValidations(cmd, this.chainValidations(this.validateChartDefinition, this.checkChartDuplicate));
               case "UPDATE_CHART":
-                  return this.validateChartDefinition(cmd);
+                  return this.checkValidations(cmd, this.chainValidations(this.validateChartDefinition, this.checkChartExists));
               default:
                   return 0 /* CommandResult.Success */;
           }
       }
       handle(cmd) {
-          var _a, _b;
+          var _a;
           switch (cmd.type) {
               case "CREATE_CHART":
                   this.addFigure(cmd.id, cmd.sheetId, cmd.position, cmd.size);
-                  this.addChart(cmd.id, cmd.sheetId, cmd.definition);
+                  this.addChart(cmd.id, cmd.definition);
                   break;
               case "UPDATE_CHART": {
-                  this.addChart(cmd.id, cmd.sheetId, cmd.definition);
+                  this.addChart(cmd.id, cmd.definition);
                   break;
               }
               case "DUPLICATE_SHEET": {
@@ -28519,7 +28551,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                       if (fig.tag === "chart") {
                           const figureIdBase = fig.id.split(FIGURE_ID_SPLITTER).pop();
                           const duplicatedFigureId = `${cmd.sheetIdTo}${FIGURE_ID_SPLITTER}${figureIdBase}`;
-                          const chart = (_b = (_a = this.charts[cmd.sheetId]) === null || _a === void 0 ? void 0 : _a[fig.id]) === null || _b === void 0 ? void 0 : _b.copyForSheetId(cmd.sheetIdTo);
+                          const chart = (_a = this.charts[fig.id]) === null || _a === void 0 ? void 0 : _a.copyForSheetId(cmd.sheetIdTo);
                           if (chart) {
                               this.dispatch("CREATE_CHART", {
                                   id: duplicatedFigureId,
@@ -28534,41 +28566,44 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   break;
               }
               case "DELETE_FIGURE":
-                  this.history.update("charts", cmd.sheetId, cmd.id, undefined);
+                  this.history.update("charts", cmd.id, undefined);
                   break;
               case "DELETE_SHEET":
-                  this.history.update("charts", cmd.sheetId, undefined);
+                  for (let id of this.getChartIds(cmd.sheetId)) {
+                      this.history.update("charts", id, undefined);
+                  }
                   break;
           }
       }
       // ---------------------------------------------------------------------------
       // Getters
       // ---------------------------------------------------------------------------
-      getContextCreationChart(sheetId, figureId) {
-          var _a, _b;
-          return (_b = (_a = this.charts[sheetId]) === null || _a === void 0 ? void 0 : _a[figureId]) === null || _b === void 0 ? void 0 : _b.getContextCreation();
-      }
-      getChart(sheetId, figureId) {
+      getContextCreationChart(figureId) {
           var _a;
-          return (_a = this.charts[sheetId]) === null || _a === void 0 ? void 0 : _a[figureId];
+          return (_a = this.charts[figureId]) === null || _a === void 0 ? void 0 : _a.getContextCreation();
       }
-      getChartType(sheetId, figureId) {
-          var _a, _b;
-          const type = (_b = (_a = this.charts[sheetId]) === null || _a === void 0 ? void 0 : _a[figureId]) === null || _b === void 0 ? void 0 : _b.type;
+      getChart(figureId) {
+          return this.charts[figureId];
+      }
+      getChartType(figureId) {
+          var _a;
+          const type = (_a = this.charts[figureId]) === null || _a === void 0 ? void 0 : _a.type;
           if (!type) {
               throw new Error("Chart not defined.");
           }
           return type;
       }
-      isChartDefined(sheetId, figureId) {
-          return figureId in (this.charts[sheetId] || {});
+      isChartDefined(figureId) {
+          return figureId in this.charts && this.charts !== undefined;
       }
       getChartIds(sheetId) {
-          return Object.keys(this.charts[sheetId] || {});
+          return Object.entries(this.charts)
+              .filter(([, chart]) => (chart === null || chart === void 0 ? void 0 : chart.sheetId) === sheetId)
+              .map(([id]) => id);
       }
-      getChartDefinition(sheetId, figureId) {
-          var _a, _b;
-          const definition = (_b = (_a = this.charts[sheetId]) === null || _a === void 0 ? void 0 : _a[figureId]) === null || _b === void 0 ? void 0 : _b.getDefinition();
+      getChartDefinition(figureId) {
+          var _a;
+          const definition = (_a = this.charts[figureId]) === null || _a === void 0 ? void 0 : _a.getDefinition();
           if (!definition) {
               throw new Error(`There is no chart with the given figureId: ${figureId}`);
           }
@@ -28580,28 +28615,35 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       import(data) {
           for (let sheet of data.sheets) {
               if (sheet.figures) {
-                  const charts = {};
                   for (let figure of sheet.figures) {
                       // TODO:
                       // figure data should be external IMO => chart should be in sheet.chart
                       // instead of in figure.data
                       if (figure.tag === "chart") {
-                          charts[figure.id] = this.createChart(figure.id, figure.data, sheet.id);
+                          this.charts[figure.id] = this.createChart(figure.id, figure.data, sheet.id);
                       }
                   }
-                  this.charts[sheet.id] = charts;
               }
           }
       }
       export(data) {
+          var _a;
           if (data.sheets) {
               for (let sheet of data.sheets) {
                   // TODO This code is false, if two plugins want ot insert figures on the sheet, it will crash !
                   const sheetFigures = this.getters.getFigures(sheet.id);
-                  const figures = sheetFigures;
-                  for (let figure of figures) {
+                  const figures = [];
+                  for (let sheetFigure of sheetFigures) {
+                      const figure = sheetFigure;
                       if (figure && figure.tag === "chart") {
-                          figure.data = this.getChartDefinition(sheet.id, figure.id);
+                          const data = (_a = this.charts[figure.id]) === null || _a === void 0 ? void 0 : _a.getDefinition();
+                          if (data) {
+                              figure.data = data;
+                              figures.push(figure);
+                          }
+                      }
+                      else {
+                          figures.push(figure);
                       }
                   }
                   sheet.figures = figures;
@@ -28609,13 +28651,13 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           }
       }
       exportForExcel(data) {
-          var _a, _b;
+          var _a;
           for (let sheet of data.sheets) {
               const sheetFigures = this.getters.getFigures(sheet.id);
               const figures = [];
               for (let figure of sheetFigures) {
                   if (figure && figure.tag === "chart") {
-                      const figureData = (_b = (_a = this.charts[sheet.id]) === null || _a === void 0 ? void 0 : _a[figure.id]) === null || _b === void 0 ? void 0 : _b.getDefinitionForExcel();
+                      const figureData = (_a = this.charts[figure.id]) === null || _a === void 0 ? void 0 : _a.getDefinitionForExcel();
                       if (figureData) {
                           figures.push({
                               ...figure,
@@ -28654,15 +28696,21 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
        * Add a chart in the local state. If a chart already exists, this chart is
        * replaced
        */
-      addChart(id, sheetId, definition) {
-          this.history.update("charts", sheetId, id, this.createChart(id, definition, sheetId));
+      addChart(id, definition) {
+          const sheetId = this.getters.getFigureSheetId(id);
+          if (sheetId) {
+              this.history.update("charts", id, this.createChart(id, definition, sheetId));
+          }
       }
       checkChartDuplicate(cmd) {
-          var _a;
-          if ((_a = this.charts[cmd.sheetId]) === null || _a === void 0 ? void 0 : _a[cmd.id]) {
-              return 84 /* CommandResult.DuplicatedChartId */;
-          }
-          return 0 /* CommandResult.Success */;
+          return this.getters.getFigureSheetId(cmd.id)
+              ? 84 /* CommandResult.DuplicatedChartId */
+              : 0 /* CommandResult.Success */;
+      }
+      checkChartExists(cmd) {
+          return this.getters.getFigureSheetId(cmd.id)
+              ? 0 /* CommandResult.Success */
+              : 85 /* CommandResult.ChartDoesNotExist */;
       }
   }
   ChartPlugin.getters = [
@@ -29043,7 +29091,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       allowDispatch(cmd) {
           switch (cmd.type) {
               case "CREATE_FIGURE":
-                  return this.checkFigureDuplicate(cmd.sheetId, cmd.figure.id);
+                  return this.checkFigureDuplicate(cmd.figure.id);
               case "UPDATE_FIGURE":
               case "DELETE_FIGURE":
                   return this.checkFigureExists(cmd.sheetId, cmd.id);
@@ -29109,9 +29157,8 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           }
           return 0 /* CommandResult.Success */;
       }
-      checkFigureDuplicate(sheetId, figureId) {
-          var _a;
-          if ((_a = this.figures[sheetId]) === null || _a === void 0 ? void 0 : _a[figureId]) {
+      checkFigureDuplicate(figureId) {
+          if (Object.values(this.figures).find((sheet) => sheet === null || sheet === void 0 ? void 0 : sheet[figureId])) {
               return 82 /* CommandResult.DuplicatedFigureId */;
           }
           return 0 /* CommandResult.Success */;
@@ -29126,6 +29173,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           var _a;
           return (_a = this.figures[sheetId]) === null || _a === void 0 ? void 0 : _a[figureId];
       }
+      getFigureSheetId(figureId) {
+          return Object.keys(this.figures).find((sheetId) => { var _a; return ((_a = this.figures[sheetId]) === null || _a === void 0 ? void 0 : _a[figureId]) !== undefined; });
+      }
       // ---------------------------------------------------------------------------
       // Import/Export
       // ---------------------------------------------------------------------------
@@ -29150,7 +29200,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           this.export(data);
       }
   }
-  FigurePlugin.getters = ["getFigures", "getFigure"];
+  FigurePlugin.getters = ["getFigures", "getFigure", "getFigureSheetId"];
 
   class FilterTable {
       constructor(zone) {
@@ -29877,7 +29927,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           switch (cmd.type) {
               case "CREATE_IMAGE":
                   if (this.getters.getFigure(cmd.sheetId, cmd.figureId)) {
-                      return 28 /* CommandResult.DuplicatedImageId */;
+                      return 28 /* CommandResult.InvalidFigureId */;
                   }
                   return 0 /* CommandResult.Success */;
               default:
@@ -29897,7 +29947,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                       if (fig.tag === "image") {
                           const figureIdBase = fig.id.split(FIGURE_ID_SPLITTER).pop();
                           const duplicatedFigureId = `${cmd.sheetIdTo}${FIGURE_ID_SPLITTER}${figureIdBase}`;
-                          const image = this.getImage(cmd.sheetId, fig.id);
+                          const image = this.getImage(fig.id);
                           if (image) {
                               const size = { width: fig.width, height: fig.height };
                               this.dispatch("CREATE_IMAGE", {
@@ -29935,13 +29985,19 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       // ---------------------------------------------------------------------------
       // Getters
       // ---------------------------------------------------------------------------
-      getImage(sheetId, figureId) {
-          var _a;
-          const image = (_a = this.images[sheetId]) === null || _a === void 0 ? void 0 : _a[figureId];
-          if (!image) {
-              throw new Error(`There is no image with the given figureId: ${figureId}`);
+      getImage(figureId) {
+          for (const sheet of Object.values(this.images)) {
+              if (sheet && sheet[figureId]) {
+                  return sheet[figureId];
+              }
           }
-          return image;
+          throw new Error(`There is no image with the given figureId: ${figureId}`);
+      }
+      getImagePath(figureId) {
+          return this.getImage(figureId).path;
+      }
+      getImageSize(figureId) {
+          return this.getImage(figureId).size;
       }
       // ---------------------------------------------------------------------------
       // Private
@@ -29983,7 +30039,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           return images;
       }
   }
-  ImagePlugin.getters = ["getImage"];
+  ImagePlugin.getters = ["getImage", "getImagePath", "getImageSize"];
 
   class MergePlugin extends CorePlugin {
       constructor() {
@@ -31784,9 +31840,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           return formatColors.filter(isDefined$1);
       }
       getChartColors(sheetId) {
-          const charts = this.getters
-              .getChartIds(sheetId)
-              .map((cid) => this.getters.getChart(sheetId, cid));
+          const charts = this.getters.getChartIds(sheetId).map((cid) => this.getters.getChart(cid));
           let chartsColors = new Set();
           for (let chart of charts) {
               if (chart === undefined) {
@@ -32142,37 +32196,34 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               invalidateCFEvaluationCommands.has(cmd.type) ||
               cmd.type === "EVALUATE_CELLS" ||
               cmd.type === "UPDATE_CELL") {
-              for (const sheetId in this.charts) {
-                  for (const chartId in this.charts[sheetId]) {
-                      this.charts[sheetId][chartId] = undefined;
-                  }
+              for (const chartId in this.charts) {
+                  this.charts[chartId] = undefined;
               }
           }
           switch (cmd.type) {
               case "UPDATE_CHART":
               case "CREATE_CHART":
               case "DELETE_FIGURE":
-                  if (this.charts[cmd.sheetId]) {
-                      this.charts[cmd.sheetId][cmd.id] = undefined;
-                  }
+                  this.charts[cmd.id] = undefined;
                   break;
               case "DELETE_SHEET":
-                  delete this.charts[cmd.sheetId];
+                  for (let chartId in this.charts) {
+                      if (!this.getters.isChartDefined(chartId)) {
+                          this.charts[chartId] = undefined;
+                      }
+                  }
                   break;
           }
       }
-      getChartRuntime(sheetId, figureId) {
-          var _a;
-          if (!((_a = this.charts[sheetId]) === null || _a === void 0 ? void 0 : _a[figureId])) {
-              const chart = this.getters.getChart(sheetId, figureId);
+      getChartRuntime(figureId) {
+          if (!this.charts[figureId]) {
+              const chart = this.getters.getChart(figureId);
               if (!chart) {
                   throw new Error(`No chart for the given id: ${figureId}`);
               }
-              if (!this.charts[sheetId])
-                  this.charts[sheetId] = {};
-              this.charts[sheetId][figureId] = this.createRuntimeChart(chart);
+              this.charts[figureId] = this.createRuntimeChart(chart);
           }
-          return this.charts[sheetId][figureId];
+          return this.charts[figureId];
       }
       /**
        * Get the background color of a chart based on the color of the first cell of the main range
@@ -37816,7 +37867,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       constructor(dispatch, getters, sheetId, copiedFigureId) {
           this.dispatch = dispatch;
           this.sheetId = sheetId;
-          const chart = getters.getChart(sheetId, copiedFigureId);
+          const chart = getters.getChart(copiedFigureId);
           if (!chart) {
               throw new Error(`No chart for the given id: ${copiedFigureId}`);
           }
@@ -37837,7 +37888,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
       constructor(dispatch, getters, sheetId, copiedFigureId) {
           this.dispatch = dispatch;
           this.sheetId = sheetId;
-          const image = getters.getImage(sheetId, copiedFigureId);
+          const image = getters.getImage(copiedFigureId);
           this.copiedImage = deepCopy(image);
       }
       paste(sheetId, figureId, position, size) {
@@ -43847,9 +43898,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
   Object.defineProperty(exports, '__esModule', { value: true });
 
 
-  __info__.version = '16.1.2';
-  __info__.date = '2023-03-01T14:43:45.667Z';
-  __info__.hash = '59bec2d';
+  __info__.version = '16.1.3';
+  __info__.date = '2023-03-06T07:18:06.669Z';
+  __info__.hash = '7e26a7c';
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);