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

[Chart] Empty values in chart creates an index overflow in data values array #747

Closed
guillaume-ro-fr opened this issue Oct 29, 2018 · 0 comments
Closed

[Chart] Empty values in chart creates an index overflow in data values array #747

guillaume-ro-fr opened this issue Oct 29, 2018 · 0 comments

Comments

@guillaume-ro-fr
Copy link
Contributor

This is:

- [x] a bug report

What is the expected behavior?

When a chart have the first of its values at null, the evaluation of the multi-level series must pass.

What is the current behavior?

Currently, detection is based on the 0 index, not the first index of the array.

What are the steps to reproduce?

I don't know if we can reproduce it without an existing spreadsheet from Excel : even with null values, PhpSpreadsheet fills the values, without leaving a gap in the indices in the XML (?).

Which versions of PhpSpreadsheet and PHP are affected?

PhpSpreadsheet 1.5.0
PHP 7.1
@guillaume-ro-fr guillaume-ro-fr changed the title [XSLX] Empty values in chart creates an index overflow in data values array [Chart] Empty values in chart creates an index overflow in data values array Oct 29, 2018
guillaume-ro-fr added a commit to guillaume-ro-fr/PhpSpreadsheet that referenced this issue Oct 29, 2018
@guillaume-ro-fr
Fix index overflow on data values array
d84837c 
Fix index overflow on data values array for multi level detection.

Fix PHPOffice#747
guillaume-ro-fr added a commit to guillaume-ro-fr/PhpSpreadsheet that referenced this issue Oct 29, 2018
@guillaume-ro-fr
Fix index overflow on data values array
1285daf 
Fix index overflow on data values array for multi level detection.

Fix PHPOffice#747
guillaume-ro-fr added a commit to guillaume-ro-fr/PhpSpreadsheet that referenced this issue Nov 6, 2018
@guillaume-ro-fr
Fix index overflow on data values array
5945761 
Fix index overflow on data values array for multi level detection.

Fix PHPOffice#747
guillaume-ro-fr added a commit to guillaume-ro-fr/PhpSpreadsheet that referenced this issue Jun 12, 2019
@guillaume-ro-fr
Fix index overflow on data values array
d0b7395 
Fix index overflow on data values array for multi level detection.

Fixes PHPOffice#747
Fixes PHPOffice#748
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

No branches or pull requests
1 participant
@guillaume-ro-fr