wip

Author: hokolomopo

src/model.ts

@@ -713,6 +713,7 @@ export class Model extends EventBus<any> implements CommandDispatcher {
     let data = createEmptyExcelWorkbookData();
     for (const handler of this.handlers) {
       if (handler instanceof BasePlugin) {
+        console.log(handler.constructor.name);
         handler.exportForExcel(data);
       }
     }

src/plugins/core/carousel.ts

@@ -1,9 +1,11 @@
 import { FIGURE_ID_SPLITTER } from "../../constants";
+import { UuidGenerator } from "../../helpers";
 import {
   Carousel,
   CarouselItem,
   CommandResult,
   CoreCommand,
+  ExcelWorkbookData,
   UID,
   UpdateCarouselCommand,
   WorkbookData,
@@ -139,4 +141,36 @@ export class CarouselPlugin extends CorePlugin<CarouselState> implements Carouse
       }
     }
   }
+
+  exportForExcel(data: ExcelWorkbookData): void {
+    // TODO ask around: should I create mock figure here, or at the xlsx_writer ? The problem of creating here is that the
+    // figures are only in the workbook, making calls to getters wrong inside the plugins following this
+    const uuidGenerator = new UuidGenerator();
+    for (const sheet of data.sheets) {
+      for (const carouselId in this.carousels[sheet.id] || {}) {
+        const carouselFigure = sheet.figures.find((fig) => fig.id === carouselId);
+        const carousel = this.carousels[sheet.id]?.[carouselId];
+        if (!carouselFigure || !carousel) {
+          continue;
+        }
+        sheet.figures = sheet.figures.filter((fig) => fig.id !== carouselId);
+
+        let offset = 0;
+        for (const item of carousel.items) {
+          if (item.type === "chart") {
+            const chartData = sheet.charts[item.chartId];
+            const newFigure = {
+              ...carouselFigure,
+              id: uuidGenerator.smallUuid(),
+              tag: "chart",
+              offset: { x: carouselFigure.offset.x + offset, y: carouselFigure.offset.y },
+            };
+            offset += 20;
+            chartData.figureId = newFigure.id;
+            sheet.figures.push(newFigure);
+          }
+        }
+      }
+    }
+  }
 }

src/plugins/core/chart.ts

@@ -12,6 +12,7 @@ import {
   CreateChartCommand,
   DeleteChartCommand,
   DOMDimension,
+  ExcelWorkbookData,
   HeaderIndex,
   PixelPosition,
   UID,
@@ -235,6 +236,10 @@ export class ChartPlugin extends CorePlugin<ChartState> implements ChartState {
     }
   }
 
+  exportForExcel(data: ExcelWorkbookData): void {
+    this.export(data);
+  }
+
   // ---------------------------------------------------------------------------
   // Private
   // ---------------------------------------------------------------------------

src/plugins/ui_core_views/evaluation_chart.ts

@@ -95,13 +95,13 @@ export class EvaluationChartPlugin extends CoreViewPlugin<EvaluationChartState>
 
   exportForExcel(data: ExcelWorkbookData) {
     for (const sheet of data.sheets) {
-      for (const chartId of this.getters.getChartIds(sheet.id)) {
+      for (const chartId of Object.keys(sheet.charts || {})) {
         const chart = this.getters.getChart(chartId);
         const excelDefinition = chart?.getDefinitionForExcel(this.getters);
-        const figureId = this.getters.getFigureIdFromChartId(chartId);
+        const figureId = sheet.charts[chartId].figureId;
 
         if (excelDefinition) {
-          sheet.charts[chartId] = { figureId, chart: excelDefinition };
+          sheet.charts[chartId].chart = excelDefinition;
         } else {
           const type = this.getters.getChartType(chartId);
           const runtime = this.getters.getChartRuntime(chartId);