PHPOffice / PhpSpreadsheet

tests/PhpSpreadsheetTests/Style/NumberFormat/Wizard/CurrencyTest.php

Spacing +8 added lines, -8 removed lines

@@ -67,12 +67,12 @@ 
             ["[\$€-fy-NL]\u{a0}#,##0.00;[\$€-fy-NL]\u{a0}#,##0.00-", '€', 'fy-NL'], // Trailing negative
             ["[\$€-nl-NL]\u{a0}#,##0.00;[\$€-nl-NL]\u{a0}-#,##0.00", '€', 'nl-NL'], // Sign between currency and value
             ["[\$€-nl-BE]\u{a0}#,##0.00;[\$€-nl-BE]\u{a0}-#,##0.00", '€', 'NL-BE'], // Sign between currency and value
-            ["#,##0.00\u{a0}[\$€-fr-BE]", '€', 'fr-be'],   // Trailing currency code
-            ["#,##0.00\u{a0}[\$€-el-GR]", '€', 'el-gr'],   // Trailing currency code
+            ["#,##0.00\u{a0}[\$€-fr-BE]", '€', 'fr-be'], // Trailing currency code
+            ["#,##0.00\u{a0}[\$€-el-GR]", '€', 'el-gr'], // Trailing currency code
             ['[$$-en-CA]#,##0.00', '$', 'en-ca'],
-            ["#,##0.00\u{a0}[\$\$-fr-CA]", '$', 'fr-ca'],   // Trailing currency code
+            ["#,##0.00\u{a0}[\$\$-fr-CA]", '$', 'fr-ca'], // Trailing currency code
             ['[$¥-ja-JP]#,##0', '¥', 'ja-JP'], // No decimals
-            ["#,##0.000\u{a0}[\$د.ب\u{200e}-ar-BH]", "د.ب\u{200e}", 'ar-BH', true],  // 3 decimals
+            ["#,##0.000\u{a0}[\$د.ب\u{200e}-ar-BH]", "د.ب\u{200e}", 'ar-BH', true], // 3 decimals
         ];
     }
 
@@ -130,12 +130,12 @@ 
             ["[\$€-fy-NL]\u{a0}#,##0;[\$€-fy-NL]\u{a0}#,##0-", '€', 'fy-NL'], // Trailing negative
             ["[\$€-nl-NL]\u{a0}#,##0;[\$€-nl-NL]\u{a0}-#,##0", '€', 'nl-NL'], // Sign between currency and value
             ["[\$€-nl-BE]\u{a0}#,##0;[\$€-nl-BE]\u{a0}-#,##0", '€', 'NL-BE'], // Sign between currency and value
-            ["#,##0\u{a0}[\$€-fr-BE]", '€', 'fr-be'],   // Trailing currency code
-            ["#,##0\u{a0}[\$€-el-GR]", '€', 'el-gr'],   // Trailing currency code
+            ["#,##0\u{a0}[\$€-fr-BE]", '€', 'fr-be'], // Trailing currency code
+            ["#,##0\u{a0}[\$€-el-GR]", '€', 'el-gr'], // Trailing currency code
             ['[$$-en-CA]#,##0', '$', 'en-ca'],
-            ["#,##0\u{a0}[\$\$-fr-CA]", '$', 'fr-ca'],   // Trailing currency code
+            ["#,##0\u{a0}[\$\$-fr-CA]", '$', 'fr-ca'], // Trailing currency code
             ['[$¥-ja-JP]#,##0', '¥', 'ja-JP'], // No decimals to truncate
-            ["#,##0\u{a0}[\$د.ب\u{200e}-ar-BH]", "د.ب\u{200e}", 'ar-BH', true],  // 3 decimals truncated to none
+            ["#,##0\u{a0}[\$د.ب\u{200e}-ar-BH]", "د.ب\u{200e}", 'ar-BH', true], // 3 decimals truncated to none
         ];
     }
 

tests/PhpSpreadsheetTests/Style/NumberFormat/Wizard/AccountingTest.php

Spacing +2 added lines, -2 removed lines

@@ -72,7 +72,7 @@ 
             ['[$$-en-CA]#,##0.00;([$$-en-CA]#,##0.00)', '$', 'en-ca'],
             ["#,##0.00\u{a0}[\$\$-fr-CA];(#,##0.00\u{a0}[\$\$-fr-CA])", '$', 'fr-ca'],
             ['[$¥-ja-JP]#,##0;([$¥-ja-JP]#,##0)', '¥', 'ja-JP'], // No decimals
-            ["#,##0.000\u{a0}[\$د.ب\u{200e}-ar-BH]", "د.ب\u{200e}", 'ar-BH', true],  // 3 decimals
+            ["#,##0.000\u{a0}[\$د.ب\u{200e}-ar-BH]", "د.ب\u{200e}", 'ar-BH', true], // 3 decimals
         ];
     }
 
@@ -135,7 +135,7 @@ 
             ['[$$-en-CA]#,##0;([$$-en-CA]#,##0)', '$', 'en-ca'],
             ["#,##0\u{a0}[\$\$-fr-CA];(#,##0\u{a0}[\$\$-fr-CA])", '$', 'fr-ca'],
             ['[$¥-ja-JP]#,##0;([$¥-ja-JP]#,##0)', '¥', 'ja-JP'], // No decimals to truncate
-            ["#,##0\u{a0}[\$د.ب\u{200e}-ar-BH]", "د.ب\u{200e}", 'ar-BH', true],  // 3 decimals truncated to none
+            ["#,##0\u{a0}[\$د.ب\u{200e}-ar-BH]", "د.ب\u{200e}", 'ar-BH', true], // 3 decimals truncated to none
         ];
     }
 

src/PhpSpreadsheet/Reader/Slk.php

Spacing +8 added lines, -8 removed lines

@@ -266,7 +266,7 @@ 
         $hasCalculatedValue = true;
     }
 
-    private function processCRecord(array $rowData, Spreadsheet &$spreadsheet, string &$row, string &$column): void
+    private function processCRecord(array $rowData, Spreadsheet & $spreadsheet, string &$row, string &$column): void
     {
         //    Read cell value data
         $hasCalculatedValue = false;
@@ -311,7 +311,7 @@ 
         $this->processCFinal($spreadsheet, $hasCalculatedValue, $cellDataFormula, $cellData, "$columnLetter$row", $tryNumeric);
     }
 
-    private function processCFinal(Spreadsheet &$spreadsheet, bool $hasCalculatedValue, string $cellDataFormula, string $cellData, string $coordinate, bool $tryNumeric): void
+    private function processCFinal(Spreadsheet & $spreadsheet, bool $hasCalculatedValue, string $cellDataFormula, string $cellData, string $coordinate, bool $tryNumeric): void
     {
         // Set cell value
         $spreadsheet->getActiveSheet()->getCell($coordinate)->setValue(($hasCalculatedValue) ? $cellDataFormula : $cellData);
@@ -321,7 +321,7 @@ 
         }
     }
 
-    private function processFRecord(array $rowData, Spreadsheet &$spreadsheet, string &$row, string &$column): void
+    private function processFRecord(array $rowData, Spreadsheet & $spreadsheet, string &$row, string &$column): void
     {
         //    Read cell formatting
         $formatStyle = $columnWidth = '';
@@ -389,7 +389,7 @@ 
         }
     }
 
-    private function addFormats(Spreadsheet &$spreadsheet, string $formatStyle, string $row, string $column): void
+    private function addFormats(Spreadsheet & $spreadsheet, string $formatStyle, string $row, string $column): void
     {
         if ($formatStyle && $column > '' && $row > '') {
             $columnLetter = Coordinate::stringFromColumnIndex((int) $column);
@@ -399,7 +399,7 @@ 
         }
     }
 
-    private function addFonts(Spreadsheet &$spreadsheet, string $fontStyle, string $row, string $column): void
+    private function addFonts(Spreadsheet & $spreadsheet, string $fontStyle, string $row, string $column): void
     {
         if ($fontStyle && $column > '' && $row > '') {
             $columnLetter = Coordinate::stringFromColumnIndex((int) $column);
@@ -409,7 +409,7 @@ 
         }
     }
 
-    private function addStyle(Spreadsheet &$spreadsheet, array $styleData, string $row, string $column): void
+    private function addStyle(Spreadsheet & $spreadsheet, array $styleData, string $row, string $column): void
     {
         if ((!empty($styleData)) && $column > '' && $row > '') {
             $columnLetter = Coordinate::stringFromColumnIndex((int) $column);
@@ -434,7 +434,7 @@ 
         }
     }
 
-    private function processPRecord(array $rowData, Spreadsheet &$spreadsheet): void
+    private function processPRecord(array $rowData, Spreadsheet & $spreadsheet): void
     {
         //    Read shared styles
         $formatArray = [];
@@ -487,7 +487,7 @@ 
         }
     }
 
-    private function processPFinal(Spreadsheet &$spreadsheet, array $formatArray): void
+    private function processPFinal(Spreadsheet & $spreadsheet, array $formatArray): void
     {
         if (array_key_exists('numberFormat', $formatArray)) {
             $this->formats['P' . $this->format] = $formatArray;

src/PhpSpreadsheet/Shared/StringHelper.php

Spacing +1 added lines, -1 removed lines

@@ -414,7 +414,7 @@
     {
         // characters
         $chars = self::convertEncoding($textValue, 'UTF-16LE', 'UTF-8');
-        $ln = (int) (strlen($chars) / 2);  // N.B. - strlen, not mb_strlen issue #642
+        $ln = (int) (strlen($chars) / 2); // N.B. - strlen, not mb_strlen issue #642
 
         return pack('vC', $ln, 0x0001) . $chars;
     }

src/PhpSpreadsheet/Chart/Legend.php

Spacing +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
      *
      * @return false|int
      */
-    public function getPositionXL(): int|bool
+    public function getPositionXL(): int | bool
     {
         // Scrutinizer thinks the following could return string. It is wrong.
         return array_search($this->position, self::POSITION_XLREF);

src/PhpSpreadsheet/Calculation/MathTrig/MatrixFunctions.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
      *
      * @return array|string The resulting array, or a string containing an error
      */
-    public static function sequence($rows = 1, $columns = 1, $start = 1, $step = 1): string|array
+    public static function sequence($rows = 1, $columns = 1, $start = 1, $step = 1): string | array
     {
         try {
             $rows = (int) Helpers::validateNumericNullSubstitution($rows, 1);

src/PhpSpreadsheet/HashTable.php

Spacing +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
      *
      * @return false|int Index
      */
-    public function getIndexForHashCode(string $hashCode): int|bool
+    public function getIndexForHashCode(string $hashCode): int | bool
     {
         // Scrutinizer thinks the following could return string. It is wrong.
         return array_search($hashCode, $this->keyMap, true);

tests/PhpSpreadsheetTests/Chart/Issue589Test.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
      *
      * @param string|string[] $color HEX color or array with HEX colors
      */
-    private function buildChartSpreadsheet(string|array $color): Spreadsheet
+    private function buildChartSpreadsheet(string | array $color): Spreadsheet
     {
         // Problem occurs when setting plot line color
         // The output chart xml file is missing the a:ln tag

tests/PhpSpreadsheetTests/Custom/ComplexAssert.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
         parent::__construct('complexAssert');
     }
 
-    private function testExpectedExceptions(string|float $expected, string|float $actual): bool
+    private function testExpectedExceptions(string | float $expected, string | float $actual): bool
     {
         //    Expecting an error, so we do a straight string comparison
         if ($expected === $actual) {