Incorrect SUMPRODUCT Calculation

Fix PHPOffice#3909. SUMPRODUCT is mishandling multi-row ranges. In Calculation/Calculation, `checkMatrixOperands` will often resize its operands. When it does so, it needs to recalculate the dimensions of each. This fixes the reported problem.

Likely cause was PR PHPOffice#3260. That ticket noted the poor coverage of the code being replaced. Tests of the problem in this ticket were absent and are now added. Despite this, I note that `resizeMatricesShrink` is virtually uncovered, and `resizeMatricesExpand` has substantial gaps in its coverage. I have covered some, but not all, of the Expand gaps. I am struggling to come up with examples to fill its remaining gaps and those for Shrink. However, I will merge this fix in about a week even if I don't succeed.

Author: oleibman

src/PhpSpreadsheet/Calculation/Calculation.php

@@ -3729,6 +3729,8 @@ private static function checkMatrixOperands(mixed &$operand1, mixed &$operand2,
             //    Given two matrices of (potentially) unequal size, convert the larger in each dimension to match the smaller
             self::resizeMatricesShrink($operand1, $operand2, $matrix1Rows, $matrix1Columns, $matrix2Rows, $matrix2Columns);
         }
+        [$matrix1Rows, $matrix1Columns] = self::getMatrixDimensions($operand1);
+        [$matrix2Rows, $matrix2Columns] = self::getMatrixDimensions($operand2);
 
         return [$matrix1Rows, $matrix1Columns, $matrix2Rows, $matrix2Columns];
     }

tests/PhpSpreadsheetTests/Calculation/Functions/MathTrig/SumProduct2Test.php

@@ -0,0 +1,34 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\MathTrig;
+
+use PhpOffice\PhpSpreadsheet\Reader\Xlsx as XlsxReader;
+
+class SumProduct2Test extends AllSetupTeardown
+{
+    public function testSUMPRODUCT(): void
+    {
+        $file = 'tests/data/Reader/XLSX/issue.3909b.xlsx';
+        $reader = new XlsxReader();
+        $spreadsheet = $reader->load($file);
+        $sheet = $spreadsheet->getActiveSheet();
+        self::assertSame('=SUMPRODUCT(((calNames=I3)*(calTiers=$K$2))*calHours)', $sheet->getCell('K3')->getValue());
+        self::assertSame(40, $sheet->getCell('K3')->getCalculatedValue());
+        self::assertSame(4, $sheet->getCell('L3')->getCalculatedValue());
+        self::assertSame(40, $sheet->getCell('M3')->getCalculatedValue());
+        self::assertSame(4, $sheet->getCell('N3')->getCalculatedValue());
+        self::assertSame(40, $sheet->getCell('K4')->getCalculatedValue());
+        self::assertSame(0, $sheet->getCell('L4')->getCalculatedValue());
+        self::assertSame(40, $sheet->getCell('M4')->getCalculatedValue());
+        self::assertSame(0, $sheet->getCell('N4')->getCalculatedValue());
+        self::assertSame(24, $sheet->getCell('K5')->getCalculatedValue());
+        self::assertSame(0, $sheet->getCell('L5')->getCalculatedValue());
+        self::assertSame(24, $sheet->getCell('M5')->getCalculatedValue());
+        self::assertSame(0, $sheet->getCell('N5')->getCalculatedValue());
+        self::assertSame('=SUMPRODUCT(calHours*((calNames=I3)*(calTiers=$K$2)))', $sheet->getCell('I14')->getValue());
+        self::assertSame(40, $sheet->getCell('I14')->getCalculatedValue());
+        $spreadsheet->disconnectWorksheets();
+    }
+}