Merge pull request #526 from hoogi91/dependabot/composer/phpoffice/ph…

…pspreadsheet-2.1.0 Bump phpoffice/phpspreadsheet from 1.29.0 to 2.1.0
hoogi91 · May 20, 2024 · 2839c2f · 2839c2f
2 parents 5bbd9ff + ba76a2a
commit 2839c2f
Show file tree

Hide file tree

Showing 7 changed files with 48 additions and 117 deletions.
diff --git a/Classes/Service/CellService.php b/Classes/Service/CellService.php
@@ -126,9 +126,10 @@ private function formatString(mixed $value, Cell $cell): string
                 setlocale(LC_NUMERIC, $currentLocale);
             }
 
-            return (string)$value;
+            return $value;
         }
 
-        return (string)NumberFormat::toFormattedString($value, NumberFormat::FORMAT_GENERAL);
+        /** @var bool|float|int|RichText|string|null $value */
+        return NumberFormat::toFormattedString($value, NumberFormat::FORMAT_GENERAL);
     }
 }
diff --git a/Classes/Service/SpanService.php b/Classes/Service/SpanService.php
@@ -150,7 +150,7 @@ public function getMergedCells(Worksheet $worksheet): array
     }
 
     /**
-     * @param array<array<int>|CellAddress|string> $references
+     * @param array<array{0: int, 1: int}|CellAddress|string> $references
      * @return array<int>
      */
     private function getCellStyleIndexesFromReferences(Worksheet $worksheet, array $references): array

diff --git a/Tests/Unit/DataProcessing/SpreadsheetProcessorTest.php b/Tests/Unit/DataProcessing/SpreadsheetProcessorTest.php
@@ -155,7 +155,7 @@ public static function processingDataProvider(): array
                         'firstColumnIsHeader' => false,
                     ],
                 ],
-                'alternativeExpectation' => self::validInputExpectationsOnlyWithBody(...),
+                'alternativeExpectations' => self::validInputExpectationsOnlyWithBody(...),
             ],
             'left head data is set because we do not have extracted head data' => [
                 'processConfig' => ['value' => 'file:123|2!A1:B2'],
@@ -173,7 +173,7 @@ public static function processingDataProvider(): array
                         'firstColumnIsHeader' => true,
                     ],
                 ],
-                'alternativeExpectation' => self::validInputExpectationsOnlyWithBody(...),
+                'alternativeExpectations' => self::validInputExpectationsOnlyWithBody(...),
             ],
             'head and foot data are not filled incorrectly when no data is given' => [
                 'processConfig' => ['value' => 'file:123|2!A1:B2'],
@@ -187,7 +187,7 @@ public static function processingDataProvider(): array
                         'firstColumnIsHeader' => false,
                     ],
                 ],
-                'alternativeExpectation' => self::validInputExpectationsWithEmptyBody(...),
+                'alternativeExpectations' => self::validInputExpectationsWithEmptyBody(...),
             ],
         ];
     }

diff --git a/Tests/Unit/DataProcessing/TabsProcessorTest.php b/Tests/Unit/DataProcessing/TabsProcessorTest.php
@@ -29,13 +29,7 @@ protected function getDataProcessor(): AbstractProcessor
     protected function validInputExpectations(MockObject $spreadsheetMock): void
     {
         // mock worksheet will be returned
-        $worksheetMock = $this->createConfiguredMock(
-            Worksheet::class,
-            [
-                'getTitle' => 'Worksheet #1',
-                'getHashCode' => '263df821f3760dc1ec4e',
-            ]
-        );
+        $worksheetMock = new Worksheet(null, 'Worksheet #1');
         $spreadsheetMock->expects(self::once())->method('getAllSheets')->willReturn([$worksheetMock]);
 
         // check if extract gets called
@@ -80,7 +74,7 @@ public static function processingDataProvider(): array
                 'expectedResult' => [
                     'someOtherVar' => [
                         // key is file uid and hash code
-                        '123263df821f3760dc1ec4e' => [
+                        '123b8367b83d9b7fa3124b32922f50ad2fc' => [
                             'sheetTitle' => 'Worksheet #1',
                             'bodyData' => ['body-data-mocked'],
                             'headData' => ['head-data-mocked'],
@@ -97,7 +91,7 @@ public static function processingDataProvider(): array
                 'expectedResult' => [
                     'spreadsheets' => [
                         // key is file uid and hash code
-                        '123263df821f3760dc1ec4e' => [
+                        '123b8367b83d9b7fa3124b32922f50ad2fc' => [
                             'sheetTitle' => 'Worksheet #1',
                             'bodyData' => ['body-data-mocked'],
                             'headData' => ['head-data-mocked'],

diff --git a/Tests/Unit/Service/CellServiceTest.php b/Tests/Unit/Service/CellServiceTest.php
@@ -112,9 +112,7 @@ public function testCatchingOfCalculatedCellValues(): void
         $mockedCell->method('getValue')->willReturn('MockValue');
 
         // build chain for style index search
-        $worksheetMock = $this->getMockBuilder(Worksheet::class)->disableOriginalConstructor()->getMock();
-        $worksheetMock->method('getParent')->willReturn($this->spreadsheet);
-
+        $worksheetMock = new Worksheet($this->spreadsheet, 'Worksheet');
         $mockedCell->method('getWorksheet')->willReturn($worksheetMock);
 
         self::assertEquals('MockValue', $this->cellService->getFormattedValue($mockedCell));

diff --git a/composer.json b/composer.json
@@ -21,7 +21,7 @@
     "php": "~8.1.0 || ~8.2.0",
     "ext-json": "*",
     "ext-mbstring": "*",
-    "phpoffice/phpspreadsheet": "^1.29",
+    "phpoffice/phpspreadsheet": "^1.29 || ^2.0",
     "typo3/cms-backend": "^11.5 || ^12.4",
     "typo3/cms-core": "^11.5 || ^12.4",
     "typo3/cms-extbase": "^11.5 || ^12.4",

diff --git a/composer.lock b/composer.lock