Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Unnecessary _xlfn prefix for SUMIFS #4182

Closed
BathoryPeter opened this issue Oct 3, 2024 · 0 comments · Fixed by #4186
Closed

Unnecessary _xlfn prefix for SUMIFS #4182

BathoryPeter opened this issue Oct 3, 2024 · 0 comments · Fixed by #4186

Comments

@BathoryPeter
Copy link

BathoryPeter commented Oct 3, 2024 •
edited
Loading

This is:

  • a bug report

What is the expected behavior?

Save SUMIFS function to XLSX without _xlfn prefix.

What is the current behavior?

PhpSpreadsheet adds a _xlfn prefix to SUMIFS, which breaks LO Calc compatibility.

What are the steps to reproduce?

I think SUMIFS function was mistakenly categorized as released in Excel 2019 in FunctionPrefix.php#L133
. As I can see it was added in Excel 2007.

What features do you think are causing the issue

  • Formula Calculations

Does an issue affect all spreadsheet file formats? If not, which formats are affected?

XLSX

Which versions of PhpSpreadsheet and PHP are affected?
oleibman added a commit to oleibman/PhpSpreadsheet that referenced this issue Oct 6, 2024
@oleibman
SUMIFS Does Not Require _xlfn
dab6ba8 
Fix PHPOffice#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.
@oleibman oleibman mentioned this issue Oct 6, 2024
Merged
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

SUMIFS Does Not Require _xlfn oleibman/PhpSpreadsheet
1 participant
@BathoryPeter