brechtsanders / xlsxio

XLSX I/O - C library for reading and writing .xlsx files
MIT License
397 stars 113 forks source link

Still can't create Xlxs files compatible with Microsoft Office #97

Closed haephrati closed 3 years ago

haephrati commented 3 years ago

Checked the updated version but still get:

<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<recoveryLog xmlns="http://schemas.openxmlformats.org/spreadsheetml/2006/main"><logFileName>error123880_01.xml</logFileName><summary>Errors were detected in file 'https://d.docs.live.net/ea373ad9f9c16144/Documents/Billboards Project/test.xlsx'</summary><additionalInfo><info>Excel completed file level validation and repair. Some parts of this workbook may have been repaired or discarded.</info></additionalInfo></recoveryLog>

Test file attached. test2.xlsx

taozuhong commented 3 years ago

which zip library you are using? libzip or minizip?

haephrati commented 3 years ago

minizip

brechtsanders commented 3 years ago

Have you tried with lipzip? I looked at your test2.xlsx, and it's not generated by the current master. Can you try with the latest master sources? Which operating system are you using?

haephrati commented 3 years ago

I am using Windows 10 and VS2017

brechtsanders commented 3 years ago

And are you using XLSX I/O library built from master using MSVC?

haephrati commented 3 years ago

I have created my own static lib which is linked to my MSVC project. This lib contains your source files and other (for other purposes). I used the master branch.

brechtsanders commented 3 years ago

From what I see it looks like you were using code from before the recent changes. Can you rebuild your static lib from the latest master branch? If you send me a minimal code example on how to reproduce your issue that would also help to replicate your issue.

haephrati commented 3 years ago

I can send you some minimal code but prefare to send it just to you and not publish it here, if that's OK. Could you send me an email to michael [at] securedglobe [dot] net ?

brechtsanders commented 3 years ago

Issue resolved in master