TypeError: Illegal offset type in file Xlsx.php on line xxx

[terrylinooo]

This is:

- [x] a bug report
- [ ] a feature request
- [ ] **not** a usage question (ask them on https://stackoverflow.com/questions/tagged/phpspreadsheet or https://gitter.im/PHPOffice/PhpSpreadsheet)

What is the expected behavior?

Load .xlsx file without error.

What is the current behavior?

TypeError: Illegal offset type in file xxxxx/vendor/phpoffice/phpspreadsheet/src/PhpSpreadsheet/Reader/Xlsx.php on line 962

What are the steps to reproduce?

Create a spreadsheet from Google Doc, then download it to local disk.
Load the .xlsx file.

Which versions of PhpSpreadsheet and PHP are affected?

1.18

I trace the bug and see this

Should check the blue part instead of red part.
I comment out the line and thing goes good now. But I think it should be fixed in here instead of modifying the vendor code.

[oleibman]

Can you supply a test file which demonstrates this problem?

[terrylinooo]

Can you supply a test file which demonstrates this problem?

Thank you for replying.

Here is the link: https://drive.google.com/file/d/1gP1gLFOjDOd8ccScAGYMyYJ-0zYEkMdZ/view?usp=sharing

I checked out the code and found that if I use setReadDataOnly(true) to skip the code block, and everything will work.
If I remove the $reader->setReadDataOnly(true); and then the error occurs.

$type = IOFactory::identify($filePath);
$reader = IOFactory::createReader($type);
$reader->setReadDataOnly(true);
$spreadsheet = $reader->load($filePath);

[oleibman]

Thank you for supplying the test file. I can confirm that your problem happens with the current release of PhpSpreadsheet. However, there have been changes to master, and I believe that it handles your spreadsheet correctly. Are you able to test against master to confirm?

[terrylinooo]

Thank you. I will test it again later.

[Michel-Cavaud]

Hello,
Do you have a date for correcting the problem on the master?
Thank you

[terrylinooo]

It's okay after upating to master branch.