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.

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

fix php 8.1 error when autoFilter['ref'] is null in AutoFilter #2596

Merged
merged 1 commit into from
Feb 17, 2022
Merged

fix php 8.1 error when autoFilter['ref'] is null in AutoFilter #2596

merged 1 commit into from
Feb 17, 2022

Conversation

brainfoolong
Copy link
Contributor

This is:

- [x] a bugfix

Checklist:

Why this change is needed?

On PHP 8.1, when loading an existing file into PhpSpreadsheet, this error does happen sometimes. This change will fix this issue. I cannot provide a testfile, as this happen only on very specific files that i cannot post here, for data protection reasons. I tried creating a reproducible file for here, but it didn't worked out.

I don't know why autoFilter['ref'] is null, but it obviously is sometimes null. The key exists for sure, but it's null. Maybe it can cause other problems, but this fixed it for us.

image

@PowerKiKi PowerKiKi merged commit 1a4e4ed into PHPOffice:master Feb 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants