-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Cell coordinate can not be zero-length string #201
Comments
You can attach the file, and more description about it. |
@GreatHumorist http://osia6vire.bkt.clouddn.com/test_excel.xlsx this is excel PhpOffice\PhpSpreadsheet\Exception: Cell coordinate can not be zero-length string in /var/www/laravel/vendor/phpoffice/phpspreadsheet/src/PhpSpreadsheet/Cell.php:603 |
Hi, I tried to resume this bug, and find out that it's the bug with xml. As we know, a xlsx type file is a zip file in fact, so I rename and unzip your xlsx file and another xlsx file which can be correctly read by phpSpreadsheet, I found some difference in "./xl/worksheets/sheet1.xml":
In order to solve this problem, you can try another software to generate this file, or wait me to fix it. Maybe hours later. Just now I also found that when you merge even just two cell by software like excel, these 'r' attribute will automatic add to the xml file. |
@GreatHumorist Hi,I found that can fix files with Excel software,but I hope to fix them with code。 |
I have fixed this bug without strict test, you can update your code and try if it is correct. If the code is correct or not, pls tell me and give the sample data, I try to fix it again, or pull it. |
@GreatHumorist Hi,Thank you,This code is work fine,I hope composer install you fix |
Ok, I have pulled request, now is waiting merge. Could you pls give me a follow or star, thank you. |
@GreatHumorist My pleasure. |
I'm import excel ,load file throw this error。
I don't know what to do
The text was updated successfully, but these errors were encountered: