Sync composer.lock (#3075)

When I cloned this morning, composer gave me a message that the lock file was not up to date with the latest changes in composer.json. I do not understand why, but it suggested to run `composer update`, which I did. This led to a handful of problems with php-cs-fixer, all fixed with changes to doc-blocks, and phpstan (only Writer/Xls/Worksheet required a change to code). We would presumably have had these problems at the start of next month when dependabot did its thing, so fix them now.

Author: oleibman

composer.lock

@@ -2912,7 +2912,7 @@ parameters:
 
 		-
 			message: "#^Cannot access offset 1 on array\\|false\\.$#"
-			count: 2
+			count: 1
 			path: src/PhpSpreadsheet/Writer/Xls/Worksheet.php
 
 		-

phpstan-baseline.neon

@@ -240,7 +240,7 @@ public static function isFormula($cellReference = '', ?Cell $cell = null)
      *
      * @param null|mixed $value The value you want converted
      *
-     * @return number N converts values listed in the following table
+     * @return number|string N converts values listed in the following table
      *        If value is or refers to N returns
      *        A number            That number value
      *        A date              The Excel serialized number of that date

src/PhpSpreadsheet/Calculation/Information/Value.php

@@ -313,8 +313,6 @@ public function getCalculatedValue(bool $resetLog = true)
      * Set old calculated value (cached).
      *
      * @param mixed $originalValue Value
-     *
-     * @return Cell
      */
     public function setCalculatedValue($originalValue): self
     {
@@ -352,8 +350,6 @@ public function getDataType(): string
      * Set cell data type.
      *
      * @param string $dataType see DataType::TYPE_*
-     *
-     * @return Cell
      */
     public function setDataType($dataType): self
     {
@@ -447,8 +443,6 @@ public function getHyperlink(): Hyperlink
 
     /**
      * Set Hyperlink.
-     *
-     * @return Cell
      */
     public function setHyperlink(?Hyperlink $hyperlink = null): self
     {
@@ -556,8 +550,6 @@ public function getAppliedStyle(): Style
 
     /**
      * Re-bind parent.
-     *
-     * @return Cell
      */
     public function rebindParent(Worksheet $parent): self
     {
@@ -650,8 +642,6 @@ public function getXfIndex(): int
 
     /**
      * Set index to cellXf.
-     *
-     * @return Cell
      */
     public function setXfIndex(int $indexValue): self
     {

src/PhpSpreadsheet/Cell/Cell.php

@@ -387,8 +387,6 @@ public static function changeBrightness($hexColourValue, $adjustPercentage)
      * @param int $colorIndex Index entry point into the colour array
      * @param bool $background Flag to indicate whether default background or foreground colour
      *                                            should be returned if the indexed colour doesn't exist
-     *
-     * @return Color
      */
     public static function indexedColor($colorIndex, $background = false, ?array $palette = null): self
     {

src/PhpSpreadsheet/Style/Color.php

@@ -8,6 +8,7 @@
 
 /**
  * @template TKey
+ *
  * @implements Iterator<TKey, Cell>
  */
 abstract class CellIterator implements Iterator

src/PhpSpreadsheet/Worksheet/CellIterator.php

@@ -732,7 +732,6 @@ private function writeDocumentSummaryInformation()
             } elseif ($dataProp['type']['data'] == 0x1E) { // null-terminated string prepended by dword string length
                 // Null-terminated string
                 $dataProp['data']['data'] .= chr(0);
-                // @phpstan-ignore-next-line
                 ++$dataProp['data']['length'];
                 // Complete the string with null string for being a %4
                 $dataProp['data']['length'] = $dataProp['data']['length'] + ((4 - $dataProp['data']['length'] % 4) == 4 ? 0 : (4 - $dataProp['data']['length'] % 4));
@@ -750,7 +749,6 @@ private function writeDocumentSummaryInformation()
             } else {
                 $dataSection_Content .= $dataProp['data']['data'];
 
-                // @phpstan-ignore-next-line
                 $dataSection_Content_Offset += 4 + $dataProp['data']['length'];
             }
         }

src/PhpSpreadsheet/Writer/Xls.php

@@ -2405,10 +2405,12 @@ public function processBitmapGd($image)
             for ($i = 0; $i < $width; ++$i) {
                 /** @phpstan-ignore-next-line */
                 $color = imagecolorsforindex($image, imagecolorat($image, $i, $j));
-                foreach (['red', 'green', 'blue'] as $key) {
-                    $color[$key] = $color[$key] + (int) round((255 - $color[$key]) * $color['alpha'] / 127);
+                if ($color !== false) {
+                    foreach (['red', 'green', 'blue'] as $key) {
+                        $color[$key] = $color[$key] + (int) round((255 - $color[$key]) * $color['alpha'] / 127);
+                    }
+                    $data .= chr($color['blue']) . chr($color['green']) . chr($color['red']);
                 }
-                $data .= chr($color['blue']) . chr($color['green']) . chr($color['red']);
             }
             if (3 * $width % 4) {
                 $data .= str_repeat("\x00", 4 - 3 * $width % 4);

src/PhpSpreadsheet/Writer/Xls/Worksheet.php

@@ -11,6 +11,7 @@ class FormulaTextTest extends AllSetupTeardown
 {
     /**
      * @param mixed $value
+     *
      * @dataProvider providerFormulaText
      */
     public function testFormulaText(string $expectedResult, $value): void

tests/PhpSpreadsheetTests/Calculation/Functions/LookupRef/FormulaTextTest.php

@@ -9,7 +9,9 @@ class SampleTest extends TestCase
 {
     /**
      * @runInSeparateProcess
+     *
      * @preserveGlobalState disabled
+     *
      * @dataProvider providerSample
      */
     public function testSample(string $sample): void