Saving spreadsheet with images result in errors - does not round-trip

[schungx]

Read a spreadsheet file with embedded images, then writing it back out as a new file.

The new file contains errors (Excel pops up warnings). Images are missing.

Attached is the test file test.xlsx and the output file output.xlsx.

• test.xlsx

• output.xlsx

[MathNya]

@schungx Thank you for your report. We will investigate the cause.

[schungx]

Anything I can do to help diagnose this problem? It seems to only happen when there are embedded images.

Most images ended up disappearing, but some are intact.

[MathNya]

@schungx Thank you very much. By analyzing the file you submitted, we were able to determine the cause of the error. We have corrected the program. Please obtain the latest version and check it.

[schungx]

Will do and revert.

[schungx]

I confirm that your fix solved the problems with the test file.

However, with a real file with hundreds of images, it comes back with errors:

with the log:

<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<recoveryLog
    xmlns="http://schemas.openxmlformats.org/spreadsheetml/2006/main">
    <logFileName>error545240_01.xml</logFileName>
    <summary>Errors were detected in file 'C:\Git\holycrown\xyz.xlsx'</summary>
    <repairedRecords>
        <repairedRecord>Repaired Records: Drawing from /xl/drawings/drawing1.xml part (Drawing shape)</repairedRecord>
        <repairedRecord>Repaired Records: Drawing from /xl/drawings/drawing2.xml part (Drawing shape)</repairedRecord>
    </repairedRecords>
</recoveryLog>

The most annoying thing is... it sometimes swap images with some other images, but not all the time.

I'll try to make another test file for you.

[schungx]

Here is another test file with the output. You'll find that:

1) There are errors when opening output.xlsx saying that some drawings have been fixed

2) Many images are swapped to something entirely different

3) Some images are not found

test2.xlsx

output.xlsx

[MathNya]

@schungx Hmmm, looks like it needs a little more renovation. We will continue to address this issue.