Formula openxml gets corrupted after opened and saved by NPOI 2.7.0

Question

Formula openxml gets corrupted after opened and saved by NPOI 2.7.0

Quark321 opened this issue 2 months ago · comments

Quark321 commented 2 months ago

NPOI Version

2.7.0

File Type

XLSX
XLS
DOCX
[x ] XLSM
OTHER

Upload the Excel File

I cannot upload file as it is my company IP

Reproduce Steps

After update to 2.7.0 from 2.6.2 the same file gives the corrupted file error after the file is opened and saved by NPOI.

error805920_01.xml

Errors were detected in file 'xxxxx.xlsm'

Excel completed file level validation and repair. Some parts of this workbook may have been repaired or discarded. Removed Records: Formula from /xl/calcChain.xml part (Calculation properties) Repaired Records: Cell information from /xl/worksheets/sheet3.xml part

Issue Description

After downgrade to 2.6.2 it works as expected.
I was not able to indentify cells or formulas affected apart the log file I have added

Andrey Bykiev · Answer 1 · Tue Apr 16 2024 03:18:40 GMT+0800 (China Standard Time)

Fixed in #1291

Tony Qu · Answer 2 · Tue Apr 16 2024 08:53:30 GMT+0800 (China Standard Time)

@Bykiev Why do you believe this issue is fixed by #1291? I don't see the clues.

Tony Qu · Answer 3 · Tue Apr 16 2024 08:54:20 GMT+0800 (China Standard Time)

@Quark321 Without the original Excel file, it's very hard to figure out what the issue is.

Andrey Bykiev · Answer 4 · Tue Apr 16 2024 13:06:55 GMT+0800 (China Standard Time)

@Bykiev Why do you believe this issue is fixed by #1291? I don't see the clues.

The issue is not reproduced in 2.6.2 and the same /xl/calcChain.xml corruption occured. 100% sure it's a shared formula bug, which was fixed. I believe we should release a patch version ASAP.

Quark321 · Answer 5 · Mon Apr 22 2024 15:43:34 GMT+0800 (China Standard Time)

@Quark321 Without the original Excel file, it's very hard to figure out what the issue is.

Hi Tony,

Unfortunately I can't share the file as macros and data are IP of the company. To be honest I am not a familiar with the OpenXML format but if you navigate me to the required xml nodes within the excel file, I can review those and share only affected fragments.

ahnochen1029 · Answer 6 · Tue Apr 23 2024 09:20:33 GMT+0800 (China Standard Time)

We are encountering the same issue.
We set "SUM(A1:A2)" at position A3, "SUM(B1:B2)" at position B3, and "SUM(C1:C2)" at position C3 in sheet 3, and use it as a template.
Then, we use NPOI to write data to sheet1 and sheet2, instead of sheet 3. The exported Excel file ends up having a corrupted /xl/calcChain.xml. Additionally, the formula in the C3 column of sheet 3 is lost.

Andrey Bykiev · Answer 7 · Tue Apr 23 2024 13:05:31 GMT+0800 (China Standard Time)

Can you guys reproduce this issue with 2.6.2?

ahnochen1029 · Answer 8 · Wed Apr 24 2024 16:07:42 GMT+0800 (China Standard Time)

@Bykiev
This issue does not occur in 2.6.2. It only occurs in 2.7.0.

Andrey Bykiev · Answer 9 · Wed Apr 24 2024 16:11:35 GMT+0800 (China Standard Time)

Can you please check with main branch?

Tony Qu · Answer 10 · Wed Apr 24 2024 16:54:21 GMT+0800 (China Standard Time)

I think it can be a regression bug. But I need an original Excel to help reproduce this and figure out what's changed.

ahnochen1029 · Answer 11 · Wed Apr 24 2024 16:59:56 GMT+0800 (China Standard Time)

template.xlsx
@tonyqus
Hi, maybe you can try this template, sheet3 have "SUM" formula.

Tony Qu · Answer 12 · Wed Apr 24 2024 19:41:07 GMT+0800 (China Standard Time)

This regression is caused by a change in #913 fix