-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding a 2nd sheet raise an exception #13
Comments
Yup, I can reproduce that here. I'll take a look. Thanks for the nice bug report. |
I've been playing with it abit, I think the problem may be something in LibreOffice, everything seems to work if the template was created by MS Excel 2007 |
That was going to be my first guess. This is interesting, because I think it's actually on the 2nd pass that it fails. That is, we read the template, write to an intermediate with the first result sheet and then try to read the intermediate back, and that's when it breaks (I think). |
The problem here is worse that reported. I cannot open any spreadsheet that's based on a LibreOffice template in either Excel or Libre Office. If I comment out the second sheet, the program completes without errors, but the resulting spreadsheet makes Excel throw errors on open and is empty in Libre Office. |
I'm unsure what the underlying bug with LibreOffice is, but FWIW I was able to create the templates with Google Sheets, exported as MS Excel 2011. Which is much more convenient then running a VM to get Excel setup. |
There's no bug in LibreOffice here, I don't think. I think that the problem is in POI and the way we use it. When we transform the workbook, we add a drawing where there was none before. I am going to start by resolving #26 and updating the program to use the new .getDrawingPatriarch method to see if that solves this problem. Therefore, this issue depends on #26. |
Using 0.3.0
Using this (simple) template
Created with LibreOffice Version: 4.4.2.2
As soon as I add a 2nd named sheet:
The following error is raised:
Possibly related to #4 and #7
The text was updated successfully, but these errors were encountered: