Skip to content

Commit c07675a

Browse files
authored
Merge pull request #4641 from oleibman/issue4638
Proper Output for BASE Function
2 parents 9b28b9e + dc11b8c commit c07675a

File tree

3 files changed

+4
-0
lines changed

3 files changed

+4
-0
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ and this project adheres to [Semantic Versioning](https://semver.org). Thia is a
3333
- Minor improvements to Calculation coverage. [PR #4624](https://github.com/PHPOffice/PhpSpreadsheet/pull/4624)
3434
- Conditional formatting in extLst. [Issue #4629](https://github.com/PHPOffice/PhpSpreadsheet/issues/4629) [PR #4633](https://github.com/PHPOffice/PhpSpreadsheet/pull/4633)
3535
- Php8.5 deprecates use of null as array index. [PR #4634](https://github.com/PHPOffice/PhpSpreadsheet/pull/4634)
36+
- Use prefix _xlfn for BASE function. [Issue #4638](https://github.com/PHPOffice/PhpSpreadsheet/issues/4638) [PR #4641](https://github.com/PHPOffice/PhpSpreadsheet/pull/4641)
3637

3738
## 2025-09-03 - 5.1.0
3839

src/PhpSpreadsheet/Writer/Xlsx/FunctionPrefix.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,8 @@ class FunctionPrefix
6666
. '|var[.]s'
6767
. '|weibull[.]dist'
6868
. '|z[.]test'
69+
// probably added with Excel 2010 but not properly documented
70+
. '|base'
6971
// functions added with Excel 2013
7072
. '|acot'
7173
. '|acoth'

tests/PhpSpreadsheetTests/Writer/Xlsx/FunctionPrefixTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ public static function functionPrefixProvider(): array
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()'],
4040
'SUMIFS reclassified as Legacy' => ['SUMIFS()', 'SUMIFS()'],
41+
'BASE improperly classified by MS' => ['_xlfn.BASE()', 'BASE()'],
4142
];
4243
}
4344
}

0 commit comments

Comments
 (0)