export excel file with line chart error xl/drawings/drawing1.xml

[padi-dev-lucvt]

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?

Successfully save and open new .xlsx file with my data and chart

What is the current behavior?

The error that Component "xl/drawings/drawing1.xml" is deleted.
Don't understand why my chart became drawing.

What are the steps to reproduce?

I have a sample template and have corresponding chart data sheets
When I load them so when I fill them with data and export them, they give an error and the chart is not displayed



Sheet DATA_1 to display data for the chart "Comparison chart through experiments - phase A"
The data is still filled out but the chart is not displayed

Here is my code

[oleibman]

There have been a lot of recent changes to Charts. Can you test with release 1.24 to see if they resolve your problem? If it doesn't, is it possible to upload your spreadsheet?

[gotgot1995]

@padi-dev-lucvt I have the same error with version 1.19.0. @oleibman I'll try to upgrade to 1.24.1 to see if that solves it.

[gotgot1995]

It appears to me that version 1.24.1 does not solve the issue. My server is running PHP 7.3.33 with Apache for Debian. In my case, I'm trying to render a pie chart.

Here is a list of ideas for a new lead:

• I didn't have this issue before migrating from PhpExcel to PhpSpreadsheet. Maybe Rector didn't do the migration properly regarding graphs.
• What would be the best practice to render graphs using PhpSpreadsheet nowadays ? I'm still using the old JpGraph package. Could this be the root cause ?

Any help would be appreciated.

[oleibman]

We now use mitoteam/jpgraph rather than the old jpgraph package from Composer. Once that package is added to your project via Composer, the only change required is setting the renderer class:

Settings::setChartRenderer(\PhpOffice\PhpSpreadsheet\Chart\Renderer\MtJpGraphRenderer::class);

If this doesn't fix your problem, I might need to see your spreadsheet to investigate further. I will note that there are a handful of chart types for which the rendering doesn't seem to work properly "out of the box", and Pie Charts represent most of those, so I wouldn't be surprised if you still have a problem.

[SSI-johnnypops]

I've been having the same problem when reading and then writing out an Excel-generated .xlsx file.
After reading through the reader and writer code I was able to isolate the issue to 'Writer/Xlsx/Chart.php' in writePlotGroup(). I was able to successfully write out the file after removing $seriesIndex shenanigans.
Unfortunately, one of the samples: 'Chart/33_Chart_create_composite.php' was broken by the changes. After studying the reader code I was finally able to determine that the sample itself was generating invalid data structures and it looked suspiciously like the $seriesIndex hack was introduced just for this sample!

Attached are a pair of diffs: one for the writer code and one for the sample. The paths in the diffs need fixing but the actual changes are minimal.

I hope this helps someone.

fix-33_Chart_create_composite-sample.txt

fix-writer-xlsx-chart.txt

[oleibman]

Can you expand on why you think the data structures generated for the sample are invalid? Your solution looks like a change that would break existing code, as it did for the sample in question. I am not averse to making such a change, but I would need a better understanding of why the current sample code is wrong. For the record, $seriesIndex was introduced over 5 years ago; I cannot speak to the reasons for it, but that is a long time to have not shown up a problem.

[oleibman]

It would also be very helpful if you could upload a spreadsheet which demonstrates this problem.

[SSI-johnnypops]

I will try and strip out the problematic part. I don't think I can just upload the whole thing.

[SSI-johnnypops]

Here you go. I believe the data is just test data.
2019_AM_InventoryTemplate_Stripped.xlsx

My test script reads this in like this:
$reader = new \PhpOffice\PhpSpreadsheet\Reader\Xlsx();
$reader->setIncludeCharts(true);
$spreadsheet = $reader->load($template_filename);

and then writes it out like this:

$writer = new \PhpOffice\PhpSpreadsheet\Writer\Xlsx($spreadsheet);
$writer->setPreCalculateFormulas(false);
$writer->setIncludeCharts(true);
$writer->save($fileNameAndPath);

The resulting file incompletely loads in Excel with the 'recovery' popup and the message:
'Removed Part: /xl/drawings/drawing1.xml part. (Drawing shape)'

[SSI-johnnypops]

The connection with the sample script '33_Chart_create_composite.php' is that they both have a Chart with multiple DataSeries() in a PlotArea(). This is when $seriesIndex becomes involved in writing (it is 0 usually).

[oleibman]

Thank you for the sample file. I can duplicate your problem, so will take a look at the file, the existing code, and your suggested changes.

[SSI-johnnypops]

Thanks!

[SSI-johnnypops]

If it helps, my rough process was inspecting the output .xlsx files for differences and discovering that the 'c:idx' and 'c:order' fields in the 'chartN.xml' files were different and then finding the relevant code in 'Writer/Xlsx/Chart.php'.

The patch for that file that I attached causes the problematic spreadsheet I uploaded to work.

After getting the samples to work on my machine, I discovered only one that was broken by the change (I may have missed some).

After tearing my hair out trying to get the problematic spreadsheet AND the sample to both work, I eventually turned to the 'Reader' part of the code and discovered that the data-structures created by the Reader code and the sample differed and it seemed unlikely that a few tweaks to the Writer code could fix it (I did try various things though).

That's when I started to suspect that maybe the sample 'Chart/33_Chart_create_composite.php' was possibly faulty.

Good luck, Happy New Year!

[oleibman]

I believe we can get away without breaking existing usage by including the seriesIndex logic if and only if element 0 exists in plotSeriesOrder. This works with your sample file, with 33_Chart_create_composite unchanged, and with 33_Chart_create_composite as you modified it (which seems more sensible than the unchanged version). I need to think about whether it makes sense to grandfather the existing usage (I am strongly inclined to yes). I will probably have a PR ready in a day or two. None of our other samples or tests are affected by this change, since they all specify only element 0 in the plotSeries array which they pass to the PlotArea constructor.