fixed writeWorkbook

Author: Or-Mi

src/taskpane/components/App.tsx

@@ -280,8 +280,8 @@ const App = () => {
             cells: {},
           };
 
-          var rowsCount = lastRow.rowIndex;
-          var columnsCount = lastColumn.columnIndex;
+          var rowsCount = lastRow.rowIndex + 1; // zero-based index, so add 1
+          var columnsCount = lastColumn.columnIndex + 1; // zero-based index, so add 1
           usedRange.untrack();
           lastColumn.untrack();
           lastRow.untrack();
@@ -452,16 +452,7 @@ const App = () => {
       numberFormats
     };
   }
-  // Helper function to convert column number to Excel column letter
-  function getExcelColumn(colNum: number): string {
-    let columnName = '';
-    while (colNum > 0) {
-      const remainder = (colNum - 1) % 26;
-      columnName = String.fromCharCode(65 + remainder) + columnName;
-      colNum = Math.floor((colNum - 1) / 26);
-    }
-    return columnName;
-  }
+
   const writeWorkbook = async (jsonData: any) => {
     setWriteProcessedRows(0);
     let totalRows = 0;
@@ -476,12 +467,14 @@ const App = () => {
     setWriteTotalRows(totalRows);
     let writtenRows = 0;
     await Excel.run(async (context) => {
+      const worksheets = context.workbook.worksheets;
       try {
-        const ROW_CHUNK_SIZE = 2; // Adjust based on performance needs
-        const COL_CHUNK_SIZE = 50; // Adjust based on performance needs
-        const worksheets = context.workbook.worksheets;
+        await context.sync();
+        
+        // Suspend automatic calculation
+        context.workbook.application.calculationMode = Excel.CalculationMode.manual;
 
-        // Loop through each worksheet in the JSON data
+        // create worksheets
         for (const sheetData of jsonData.worksheets) {
           let worksheet = worksheets.getItemOrNullObject(sheetData.name);
           await context.sync();
@@ -490,7 +483,17 @@ const App = () => {
           if (worksheet.isNullObject) {
             worksheet = worksheets.add(sheetData.name);
             worksheet.activate();
+          } else {
+            // Clear existing content in the worksheet
+            worksheet.getUsedRange().clear(Excel.ClearApplyTo.contents);
           }
+        }
+
+
+        // Loop through each worksheet in the JSON data
+        for (const sheetData of jsonData.worksheets) {
+          let worksheet = worksheets.getItemOrNullObject(sheetData.name);
+          await context.sync();
 
           context.application.suspendScreenUpdatingUntilNextSync();
 
@@ -504,15 +507,18 @@ const App = () => {
           });
           sheet_cells.setCellProperties(properties);
 
-          // Simulate row-wise progress update
-          for (let row = 0; row < formulas.length; row++) {
-            writtenRows++;
-            setWriteProcessedRows(writtenRows);
-          }
+          writtenRows += sheetData.cells[Object.keys(sheetData.cells)[Object.keys(sheetData.cells).length - 1]].rowIndex + 1
+          setWriteProcessedRows(writtenRows);
 
           sheet_cells.untrack();
           await context.sync();
+// setWriteProcessedRows(sheetData.cells[Object.keys(sheetData.cells)[Object.keys(sheetData.cells).length - 1]].rowIndex + 1);
         }
+
+        // Rebuild the calculation engine to ensure all formulas are recalculated
+        context.workbook.application.calculate(Excel.CalculationType.fullRebuild);
+        await context.sync();
+
       } catch (error) {
         setWriteErrorMessage(error.message);
         throw error;