SUMIFS sum values only once

Values were summed multiple times if it matched several conditions
whereas it should only be summed once.

Fixes #704
Fixes #710

CHANGELOG.md

@@ -9,7 +9,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Fixed
 
-- Support numeric condition in SUMIF, SUMIFS, AVERAGEIF, COUNTIF, MAXIF and MINIF [#683](https://github.com/PHPOffice/PhpSpreadsheet/issues/683)
+- Support numeric condition in SUMIF, SUMIFS, AVERAGEIF, COUNTIF, MAXIF and MINIF - [#683](https://github.com/PHPOffice/PhpSpreadsheet/issues/683)
+- SUMIFS containing multiple conditions - [#704](https://github.com/PHPOffice/PhpSpreadsheet/issues/704)
 
 ## [1.5.0] - 2018-10-21
 

src/PhpSpreadsheet/Calculation/MathTrig.php

@@ -1256,27 +1256,37 @@ class MathTrig
         $returnValue = 0;
 
         $sumArgs = Functions::flattenArray(array_shift($arrayList));
+        $aArgsArray = [];
+        $conditions = [];
 
         while (count($arrayList) > 0) {
             $aArgsArray[] = Functions::flattenArray(array_shift($arrayList));
             $conditions[] = Functions::ifCondition(array_shift($arrayList));
         }
 
-        // Loop through each set of arguments and conditions
+        // Loop through each sum and see if arguments and conditions are true
-        foreach ($conditions as $index => $condition) {
+        foreach ($sumArgs as $index => $value) {
-            $aArgs = $aArgsArray[$index];
+            $valid = true;
+
+            foreach ($conditions as $cidx => $condition) {
+                $arg = $aArgsArray[$cidx][$index];
 
                 // Loop through arguments
-            foreach ($aArgs as $key => $arg) {
                 if (!is_numeric($arg)) {
                     $arg = Calculation::wrapResult(strtoupper($arg));
                 }
                 $testCondition = '=' . $arg . $condition;
-                if (Calculation::getInstance()->_calculateFormulaValue($testCondition)) {
+                if (!Calculation::getInstance()->_calculateFormulaValue($testCondition)) {
-                    // Is it a value within our criteria
+                    // Is not a value within our criteria
-                    $returnValue += $sumArgs[$key];
+                    $valid = false;
+
+                    break; // if false found, don't need to check other conditions
                 }
             }
+
+            if ($valid) {
+                $returnValue += $value;
+            }
         }
 
         // Return

tests/PhpSpreadsheetTests/Calculation/MathTrigTest.php

@@ -568,6 +568,22 @@ class MathTrigTest extends TestCase
         return require 'data/Calculation/MathTrig/SUMIF.php';
     }
 
+    /**
+     * @dataProvider providerSUMIFS
+     *
+     * @param mixed $expectedResult
+     */
+    public function testSUMIFS($expectedResult, ...$args)
+    {
+        $result = MathTrig::SUMIFS(...$args);
+        self::assertEquals($expectedResult, $result, '', 1E-12);
+    }
+
+    public function providerSUMIFS()
+    {
+        return require 'data/Calculation/MathTrig/SUMIFS.php';
+    }
+
     /**
      * @dataProvider providerSUBTOTAL
      *

tests/data/Calculation/MathTrig/SUMIFS.php

@@ -0,0 +1,44 @@
+<?php
+
+return [
+    [
+        2,
+        [
+            [1],
+            [1],
+            [1],
+        ],
+        [
+            ['Y'],
+            ['Y'],
+            ['N'],
+        ],
+        '=Y',
+        [
+            ['H'],
+            ['H'],
+            ['H'],
+        ],
+        '=H',
+    ],
+    [
+        1,
+        [
+            [1],
+            [1],
+            [1],
+        ],
+        [
+            ['A'],
+            ['B'],
+            ['C'],
+        ],
+        '=B',
+        [
+            ['C'],
+            ['B'],
+            ['A'],
+        ],
+        '=B',
+    ],
+];