Skip to content

Commit dab6ba8

Browse files
committed
SUMIFS Does Not Require _xlfn
Fix #4182. It was on our list as "introduced in 2019". It was, in fact, available with Excel 2007 (https://support.microsoft.com/en-us/office/excel-functions-alphabetical-b3944572-255d-4efb-bb96-c6d90033e188#bm19), so does not require a prefix when writing it to a spreadsheet.
1 parent 87ddd21 commit dab6ba8

File tree

2 files changed

+1
-1
lines changed

2 files changed

+1
-1
lines changed

src/PhpSpreadsheet/Writer/Xlsx/FunctionPrefix.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,6 @@ class FunctionPrefix
130130
. '|ifs'
131131
. '|maxifs'
132132
. '|minifs'
133-
. '|sumifs'
134133
. '|textjoin'
135134
// functions added with Excel 365
136135
. '|anchorarray'

tests/PhpSpreadsheetTests/Writer/Xlsx/FunctionPrefixTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ public static function functionPrefixProvider(): array
3737
'DAYS/NETWORKDAYS 4' => ['ABS(_xlfn.DAYS(DATE(2023,1,1),TODAY()))', 'ABS(_xlfn.DAYS(DATE(2023,1,1),TODAY()))'],
3838
'DAYS/NETWORKDAYS 5' => ['NETWORKDAYS(DATE(2023,1,1),TODAY(), C:C)', 'NETWORKDAYS(DATE(2023,1,1),TODAY(), C:C)'],
3939
'COUNTIFS reclassified as Legacy' => ['COUNTIFS()', 'COUNTIFS()'],
40+
'SUMIFS reclassified as Legacy' => ['SUMIFS()', 'SUMIFS()'],
4041
];
4142
}
4243
}

0 commit comments

Comments
 (0)