ea74c96e98
* Name Clashes Between Parsed and Unparsed Drawings This is at least a partial fix for #2396 and #1767 (which has been around for a long time). PhpSpreadsheet renames drawing XML files when it reads them from a spreadsheet. However, when it writes unparsed drawing files, it uses the original names, which can result in a clash with the renamed files. The solution in this PR is to write the unparsed files using the same renaming convention as the the others. This is an incredibly simple fix, basically a one-line change, for such a long-lived problem. It is conceivable that this PR breaks a more sophisticated file than I have come across, e.g. with multiple unparsed files associated with a single worksheet. However, this PR does fix at least part of the problem for both issues, and causes no regression issues. The changed code was covered in only 2 tests - Reader/XlsxTest testLoadSaveWithEmptyDrawings and Writer/Xlsx/UnparsedDataTest testLoadSaveXlsxWithUnparsedData. 2396 is covered by a new test Unparsed2396Test. I had trouble figuring out what to test for 1767. Since it is a problem that becomes evident only when the output file is opened in Excel, I added a new sample to cover it. * Sloppy Errors I neglected to run php-cs-fixer and phpstan, and it bit me. * Scrutinizer It's not as good as Phpstan at recognizing problems that can't happen due to previous assertions. * Scrutinizer Again It can be really stupid sometimes. |
||
|---|---|---|
| .. | ||
| Autofilter | ||
| Basic | ||
| Calculations | ||
| Chart | ||
| DefinedNames | ||
| Reader | ||
| Reading_workbook_data | ||
| bootstrap | ||
| images | ||
| templates | ||
| Bootstrap.php | ||
| Header.php | ||
| favicon.ico | ||
| index.php | ||