Issue #533: Fix bug with phpspreadsheet creating default worksheet

bcgov · Feb 4, 2025 · 6a2455f · 6a2455f
1 parent 74f730a
commit 6a2455f
Showing 1 changed file with 5 additions and 10 deletions.
diff --git a/html/modules/custom/bc_dc/src/Controller/BcDcCreateFileController.php b/html/modules/custom/bc_dc/src/Controller/BcDcCreateFileController.php
@@ -10,7 +10,6 @@
 use Drupal\node\NodeInterface;
 use Drupal\path_alias\AliasManagerInterface;
 use PhpOffice\PhpSpreadsheet\Spreadsheet;
-use PhpOffice\PhpSpreadsheet\Worksheet\Worksheet;
 use PhpOffice\PhpSpreadsheet\Writer\Csv;
 use PhpOffice\PhpSpreadsheet\Writer\Xlsx;
 use Symfony\Component\DependencyInjection\ContainerInterface;
@@ -121,16 +120,12 @@ public function createFile(NodeInterface $node, string $format): BinaryFileRespo
     // with stream wrappers.
     $file_path = $this->fileSystem->getTempDirectory() . '/' . $filename;
 
-    // Generate the spreadsheet object and save to $file_path.
     $spreadsheet = new Spreadsheet();
-    $worksheets = new Worksheet($spreadsheet);
-    $spreadsheet->addSheet($worksheets);
-    $worksheets->fromArray($results);
-    // Remove first blank worksheet.
-    $sheetIndex = $spreadsheet->getIndex(
-      $spreadsheet->getSheetByName('Worksheet')
-    );
-    $spreadsheet->removeSheetByIndex($sheetIndex);
+    // Get default sheet.
+    $worksheet = $spreadsheet->getActiveSheet();
+    // Rename it to avoid conflicts.
+    $worksheet->setTitle('Data');
+    $worksheet->fromArray($results);
 
     switch ($format) {
       case 'xlsx':