@odoo/o-spreadsheet 16.0.0 → 16.0.2

package/dist/o_spreadsheet.cjs.js

@@ -2803,7 +2803,7 @@ class ChartJsComponent extends owl.Component {
         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");
         }
@@ -3172,6 +3172,7 @@ exports.CommandResult = void 0;
     CommandResult[CommandResult["DuplicatedFigureId"] = 81] = "DuplicatedFigureId";
     CommandResult[CommandResult["InvalidSelectionStep"] = 82] = "InvalidSelectionStep";
     CommandResult[CommandResult["DuplicatedChartId"] = 83] = "DuplicatedChartId";
+    CommandResult[CommandResult["ChartDoesNotExist"] = 84] = "ChartDoesNotExist";
 })(exports.CommandResult || (exports.CommandResult = {}));
 
 var DIRECTION;
@@ -8640,7 +8641,7 @@ GaugeChartDesignPanel.components = { ColorPicker };
 
 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, this.env.model.getters);
         }
@@ -8833,7 +8834,7 @@ class ChartPanel extends owl.Component {
             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;
             }
@@ -8847,7 +8848,6 @@ class ChartPanel extends owl.Component {
             ...this.getChartDefinition(),
             ...updateDefinition,
         };
-        // TODORAR add a putain de test - c'est faux de nouveau ...
         return this.env.model.dispatch("UPDATE_CHART", {
             definition,
             id: this.figureId,
@@ -8855,12 +8855,11 @@ class ChartPanel extends owl.Component {
         });
     }
     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.");
         }
         const definition = getChartDefinitionFromContextCreation(context, type);
-        // TODORAR add a putain de test - c'est faux de nouveau ...
         this.env.model.dispatch("UPDATE_CHART", {
             definition,
             id: this.figureId,
@@ -8868,7 +8867,7 @@ class ChartPanel extends owl.Component {
         });
     }
     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.");
         }
@@ -8878,8 +8877,8 @@ class ChartPanel extends owl.Component {
         }
         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();
@@ -10015,7 +10014,7 @@ class ScorecardChart extends owl.Component {
         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;
@@ -10245,7 +10244,7 @@ class ChartFigure extends owl.Component {
         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 = {
@@ -26272,13 +26271,7 @@ function load(data, verboseImport) {
             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) {
@@ -26516,6 +26509,48 @@ const MIGRATIONS = [
         },
     },
 ];
+/**
+ * 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.
@@ -26654,6 +26689,7 @@ function createEmptyWorkbookData(sheetName = "Sheet1") {
         formats: {},
         borders: {},
         revisionId: DEFAULT_REVISION_ID,
+        uniqueFigureIds: true,
     };
     return data;
 }
@@ -27576,10 +27612,8 @@ class ChartPlugin extends CorePlugin {
         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));
         }
     }
     // ---------------------------------------------------------------------------
@@ -27590,20 +27624,20 @@ class ChartPlugin extends CorePlugin {
             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": {
@@ -27612,7 +27646,7 @@ class ChartPlugin extends CorePlugin {
                     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,
@@ -27627,44 +27661,44 @@ class ChartPlugin extends CorePlugin {
                 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) {
-        // TODORAR refaire ce shit truc
-        // on veut un  truc du genre Object.keys(this.charts[sheetId] || {} a mon avis
-        // return (Object.values(this.charts[sheetId] || {}).filter(isDefined) as AbstractChart[]).map((chart) => chart.id);
-        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}`);
         }
@@ -27676,28 +27710,35 @@ class ChartPlugin extends CorePlugin {
     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;
@@ -27705,13 +27746,13 @@ class ChartPlugin extends CorePlugin {
         }
     }
     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,
@@ -27750,15 +27791,21 @@ class ChartPlugin extends CorePlugin {
      * 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 83 /* CommandResult.DuplicatedChartId */;
-        }
-        return 0 /* CommandResult.Success */;
+        return this.getters.getFigureSheetId(cmd.id)
+            ? 83 /* CommandResult.DuplicatedChartId */
+            : 0 /* CommandResult.Success */;
+    }
+    checkChartExists(cmd) {
+        return this.getters.getFigureSheetId(cmd.id)
+            ? 0 /* CommandResult.Success */
+            : 84 /* CommandResult.ChartDoesNotExist */;
     }
 }
 ChartPlugin.getters = [
@@ -28139,7 +28186,7 @@ class FigurePlugin extends CorePlugin {
     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);
@@ -28205,9 +28252,8 @@ class FigurePlugin extends CorePlugin {
         }
         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 81 /* CommandResult.DuplicatedFigureId */;
         }
         return 0 /* CommandResult.Success */;
@@ -28222,6 +28268,9 @@ class FigurePlugin extends CorePlugin {
         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
     // ---------------------------------------------------------------------------
@@ -28246,7 +28295,7 @@ class FigurePlugin extends CorePlugin {
         this.export(data);
     }
 }
-FigurePlugin.getters = ["getFigures", "getFigure"];
+FigurePlugin.getters = ["getFigures", "getFigure", "getFigureSheetId"];
 
 class FilterTable {
     constructor(zone) {
@@ -31538,7 +31587,7 @@ class ClipboardFigureState {
             throw new Error(`No figure for the given id: ${copiedFigureId}`);
         }
         this.copiedFigure = { ...figure };
-        const chart = getters.getChart(this.sheetId, copiedFigureId);
+        const chart = getters.getChart(copiedFigureId);
         if (!chart) {
             throw new Error(`No chart for the given id: ${copiedFigureId}`);
         }
@@ -31990,9 +32039,7 @@ class CustomColorsPlugin extends UIPlugin {
         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) {
@@ -32259,37 +32306,34 @@ class EvaluationChartPlugin extends UIPlugin {
             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
@@ -42411,6 +42455,6 @@ exports.setTranslationMethod = setTranslationMethod;
 exports.tokenize = tokenize;
 
 
-__info__.version = '16.0.0';
-__info__.date = '2023-02-27T14:34:11.940Z';
-__info__.hash = '7081532';
+__info__.version = '16.0.2';
+__info__.date = '2023-03-06T07:17:51.313Z';
+__info__.hash = '6b4d816';

package/dist/o_spreadsheet.d.ts

@@ -810,6 +810,7 @@ interface WorkbookData {
         };
     };
     revisionId: UID;
+    uniqueFigureIds: boolean;
 }
 interface ExcelWorkbookData extends WorkbookData {
     sheets: ExcelSheetData[];
@@ -2175,7 +2176,8 @@ declare const enum CommandResult {
     NonContinuousTargets = 80,
     DuplicatedFigureId = 81,
     InvalidSelectionStep = 82,
-    DuplicatedChartId = 83
+    DuplicatedChartId = 83,
+    ChartDoesNotExist = 84
 }
 interface CommandHandler<T> {
     allowDispatch(command: T): CommandResult | CommandResult[];
@@ -3182,24 +3184,22 @@ declare abstract class AbstractChart {
  * This plugin manages charts
  * */
 interface ChartState {
-    readonly charts: {
-        [sheetId: UID]: Record<UID, AbstractChart | undefined> | undefined;
-    };
+    readonly charts: Record<UID, AbstractChart | undefined>;
 }
 declare class ChartPlugin extends CorePlugin<ChartState> implements ChartState {
     static getters: readonly ["isChartDefined", "getChartDefinition", "getChartType", "getChartIds", "getChart", "getContextCreationChart"];
-    readonly charts: ChartState["charts"];
+    readonly charts: Record<UID, AbstractChart | undefined>;
     private createChart;
     private validateChartDefinition;
     adaptRanges(applyChange: ApplyRangeChange): void;
     allowDispatch(cmd: Command): CommandResult | CommandResult[];
     handle(cmd: CoreCommand): void;
-    getContextCreationChart(sheetId: UID, figureId: UID): ChartCreationContext | undefined;
-    getChart(sheetId: UID, figureId: UID): AbstractChart | undefined;
-    getChartType(sheetId: UID, figureId: UID): ChartType;
-    isChartDefined(sheetId: UID, figureId: UID): boolean;
+    getContextCreationChart(figureId: UID): ChartCreationContext | undefined;
+    getChart(figureId: UID): AbstractChart | undefined;
+    getChartType(figureId: UID): ChartType;
+    isChartDefined(figureId: UID): boolean;
     getChartIds(sheetId: UID): string[];
-    getChartDefinition(sheetId: UID, figureId: UID): ChartDefinition;
+    getChartDefinition(figureId: UID): ChartDefinition;
     import(data: WorkbookData): void;
     export(data: WorkbookData): void;
     exportForExcel(data: ExcelWorkbookData): void;
@@ -3213,6 +3213,7 @@ declare class ChartPlugin extends CorePlugin<ChartState> implements ChartState {
      */
     private addChart;
     private checkChartDuplicate;
+    private checkChartExists;
 }
 
 interface ConditionalFormatState {
@@ -3263,12 +3264,14 @@ declare class ConditionalFormatPlugin extends CorePlugin<ConditionalFormatState>
 
 interface FigureState {
     readonly figures: {
-        [sheet: UID]: Record<UID, Figure | undefined> | undefined;
+        [sheet: string]: Record<UID, Figure | undefined> | undefined;
     };
 }
 declare class FigurePlugin extends CorePlugin<FigureState> implements FigureState {
-    static getters: readonly ["getFigures", "getFigure"];
-    readonly figures: FigureState["figures"];
+    static getters: readonly ["getFigures", "getFigure", "getFigureSheetId"];
+    readonly figures: {
+        [sheet: string]: Record<UID, Figure | undefined> | undefined;
+    };
     allowDispatch(cmd: CoreCommand): CommandResult;
     handle(cmd: CoreCommand): void;
     private updateFigure;
@@ -3279,6 +3282,7 @@ declare class FigurePlugin extends CorePlugin<FigureState> implements FigureStat
     private checkFigureDuplicate;
     getFigures(sheetId: UID): Figure[];
     getFigure(sheetId: string, figureId: string): Figure | undefined;
+    getFigureSheetId(figureId: string): UID | undefined;
     import(data: WorkbookData): void;
     export(data: WorkbookData): void;
     exportForExcel(data: ExcelWorkbookData): void;
@@ -3877,12 +3881,10 @@ declare class EvaluationPlugin extends UIPlugin {
 
 declare class EvaluationChartPlugin extends UIPlugin {
     static getters: readonly ["getChartRuntime", "getBackgroundOfSingleCellChart"];
-    readonly charts: {
-        [sheetId: UID]: Record<UID, ChartRuntime | undefined> | undefined;
-    };
+    readonly charts: Record<UID, ChartRuntime | undefined>;
     private createRuntimeChart;
     handle(cmd: Command): void;
-    getChartRuntime(sheetId: UID, figureId: UID): ChartRuntime;
+    getChartRuntime(figureId: UID): ChartRuntime;
     /**
      * Get the background color of a chart based on the color of the first cell of the main range
      * of the chart. In order of priority, it will return :
@@ -4850,7 +4852,6 @@ declare class SelectionInput extends Component<Props$y, SpreadsheetChildEnv> {
 }
 
 interface Props$x {
-    sheetId: UID;
     figureId: UID;
     definition: LineChartDefinition | BarChartDefinition | PieChartDefinition;
     updateChart: (definition: Partial<LineChartDefinition | BarChartDefinition | PieChartDefinition>) => DispatchResult;
@@ -5125,7 +5126,7 @@ declare class ChartJsComponent extends Component<Props$r, SpreadsheetChildEnv> {
     static template: string;
     private canvas;
     private chart?;
-    get background(): Color;
+    get background(): string;
     get canvasStyle(): string;
     get chartRuntime(): ChartJSRuntime;
     setup(): void;